Bin zu blöd um geteilt zu rechnen (Programmierung - Javascript)

**Blockkanone**

Hey

Ich arbeite gerade für DarkRP an einem Double or Nothing Calculator.

Und da will ich gerne den Durchschnitt berechnen:

Code

summe = 100;
durchschnitt = summe / 100;
// de facto output = 100
// gewünschter output = 1

Ich hab kein plan, warum das geteilt durch nicht funktioniert.

Hier nochmal den ganzen Code. Ich entschuldige mich jetzt schon für meine Variabelbennenung und den unoptimierten Code, ist halt 2 Uhr nachts.

Code

function main() {
            var i = -1;
            var v = 1;
            var r = 0;
            var z = [];
            var summe = 0;
            var durchschnitt;
            while (v = 10) {
                i = i + 1
                var min = 0;
                var max = 100;
                var x = (Math.random() * (max - min)) + min;

                if (x < 49) {
                    var min = 0;
                    var max = 100;
                    var x = (Math.random() * (max - min)) + min;
                }
                if (x < 49) {
                    var min = 0;
                    var max = 100;
                    var x = (Math.random() * (max - min)) + min;
                }
                if (x < 49) {
                    var min = 0;
                    var max = 100;
                    var x = (Math.random() * (max - min)) + min;
                }
                if (x < 49) {
                    var min = 0;
                    var max = 100;
                    var x = (Math.random() * (max - min)) + min;
                }
                if (x < 49) {
                    var min = 0;
                    var max = 100;
                    var x = (Math.random() * (max - min)) + min;
                }
                if (x < 49) {
                    var min = 0;
                    var max = 100;
                    var x = (Math.random() * (max - min)) + min;
                }
                if (x < 49) {
                    var min = 0;
                    var max = 100;
                    var x = (Math.random() * (max - min)) + min;
                }
                if (x < 49) {
                    var min = 0;
                    var max = 100;
                    var x = (Math.random() * (max - min)) + min;
                }
                if (x < 49) {
                    document.getElementById("nothing").innerHTML = "Du hast " + i +
                        " Versuche gebraucht. Das wären dann " + i * 500 +
                        " Dollar die du verbraten hast"
                        
                    var f = i * 500;
                    z.push(f);
                    r = r + 1;
                    // break;
                }
                if (r == 100) {
                    for (l = 0; l < z.length; l++) {
                        summe += z[l];
                    }

                    durchschnitt = summe / 100;
                    document.getElementById("nothing").innerHTML = "Durchschnittlich hast du " + durchschnitt + " Dollar ausgegeben"
                    break;
                }

            }
            document.getElementById("double").innerHTML =
                "Herzlichen Glückwunsch du hast ca. " + 500 * 2 * 2 * 2 * 2 * 2 * 2 * 2 * 2 * 2 + " gewonnen"
        }


        document.addEventListener("DOMContentLoaded", () => {
            main();
        });

Alles anzeigen

Ich hoffe einer von euch kann mir helfen. Ich verrek hier gleich

Freundliche Grüsse

Blockkanone

Alpha

1. Variablenbenennung ist echt scheiße...

2. Warum hast du da so oft den gleichen "if-Block"/Oder teilweise andere Inhalte?

3. Ich würde mal "summe / 100" in Klammern setzen.. Vielleicht hilft das.

**Blockkanone**

Zitat von Alpha

2. Warum hast du da so oft den gleichen "if-Block"/Oder teilweise andere Inhalte?

weil es double or nothing calculator ist und um den jackpot zu knacken muss man 10x hintereinander double or nothing schaffen

ich weiss, mit einer for schleife z.b. könnte man das auch machen, bin einfach zu faul dafür um 2 Uhr nachts

Ausserdem funktioniert es noch immer nicht, selbst mit klammern

Alpha

Wenn du in jedem if das Gleiche abfragst ist das nicht nur absolut Schwachsinnig, sondern wohlmöglich auch der Grund

Und da du "durchschnitt" nur dort unten benutzt, definiere es doch auch einfach erst da unten.

**Blockkanone**

Zitat von Alpha

Wenn du in jedem if das Gleiche abfragst ist das nicht nur absolut Schwachsinnig, sondern wohlmöglich auch der Grund

Der Grund warum ich nicht geteilt durch rechnen kann ist weil ich zu viele ifs habe? What. Ich benutz ja die variablen durchschnitt um summe in den common ifs nicht

Alpha

Zitat von Blockkanone

Der Grund warum ich nicht geteilt durch rechnen kann ist weil ich zu viele ifs habe? What. Ich benutz ja die variablen durchschnitt um summe in den common ifs nicht

Das nicht, aber sowas trägt nicht unbedingt zu Fehlerfreien Ausführungen bei.

**Blockkanone**

Zitat von Alpha

Das nicht, aber sowas trägt nicht unbedingt zu Fehlerfreien Ausführungen bei.

Ich weiss, ich weiss. Aber meine Konsole ist leer und ich habe noch immer keine ahnung warum das nicht funktioniert

Alpha

Dann würde ich vermuten das der Wert für "summe" falsch gesetzt wird. Das macht halt irgendwie sonst echt nicht viel sinn

**Blockkanone**

Zitat von Alpha

Dann würde ich vermuten das der Wert für "summe" falsch gesetzt wird. Das macht halt irgendwie sonst echt nicht viel sinn

Also die Summe ist das Array "z" zusammengezählt. Und der Durchschnitt übernimmt einfach den wert der Summe

Felix | Not_Fex

Dude, du definierst 8 mal das gleiche - solang du nicht notepad benutzt sollte dein Editor dir das doch fett unterstreichen.

Mach ne schleife...

Alpha

Es ist halt schwer dir bei so einem Code zu helfen, der halt absolut null Sinn macht..

Felix | Not_Fex

Code

var i = 0;
do {
  i += 1;
  console.log(i);
} while (i < 10);

Das führt 10 mal aus, solange bis i < 10 ist.

Sowas bitte anwenden.

oder machst halt ne while schleife

(das wär dann sowas hier:)

Code

var n = 0;
var x = 0;
while (n < 10) {
  n++;
  x += n;
  console.log(n);
}

oder auch mit einer foreach:

Code

[1,2,3].forEach(function(i) {
  something();
});

Ravish

Sorry aber ich kann da echt nicht helfen weil ich da nicht durchsehe.

Der Code ist sehr unschön geschrieben sehr unübersichtlich.

Da merke ich das erste mal was mir Mekphen immer sagen wollte mit:

Zeilenabstand

Denk dran andere wollen den vielleicht auch sehen.

**Mandelcar**

Ich frage mich gerade nur was du damit erreichen willst ?

**P4sca1**

while (v=10) führt nie aus. Das ist dein Problem.

Außerdem kannst du deinen Code mal hier reinhauen: https://prettier.io/playground/

**Blockkanone**

Zitat von Ravish

Sorry aber ich kann da echt nicht helfen weil ich da nicht durchsehe.

Der Code ist sehr unschön geschrieben sehr unübersichtlich.

Ich werd ihn nachher umschreiben, damit er mehr leserlich ist

Zitat von Wcar

Ich frage mich gerade nur was du damit erreichen willst ?

Es soll eine Wahrscheinlichkeitsberechnung für das DarkRP Casino Spiel "Double or Nothing" werden

Zitat von P4sca1

while (v=10) führt nie aus. Das ist dein Problem.

Doch, die führt aus. Es ist nur durchschnitt = summe / 100 das nicht funktioniert. Denn dort übernimmt die Variable durchschnitt den wert von summe

**Blockkanone**

Wenn ich es ausführe kommt das raus:

Code

Herzlichen Glückwunsch du hast ca. 256000 gewonnen

Durchschnittlich hast du 15544750 Dollar ausgegeben

Aber eigentlich sollte es mit der aktuellen Durchschnittsberechnung das raus kommen:

Code

Herzlichen Glückwunsch du hast ca. 256000 gewonnen

Durchschnittlich hast du 155447.50 Dollar ausgegeben

**Mattzimann**

Dude, bisschen viel Aufwand für die Wahrscheinlichkeit nh.

2^10=1/1024 Gewinnchance

tobias

Ist dieser Beitrag noch aktuell?

**Blockkanone**

Zitat von tobias.

Ist dieser Beitrag noch aktuell?

Eigentlich ja

Legende