[HELP] Programm fertig leider viele Bugs.

SideWinder

Ich glaub das Problem liegt an deinen vielen Switches, das sollte so aussehen:

switch(Menue1)
{
    case 0:
    break;

    case 1:
    break;

    ...
}

und nicht:

// FALSCH :warning:

switch(Menue1)
{
    case 0:
    break;
}

switch(Menue1)
{
    case 1:
    break;
}

Aber leider ist die Klammerung derart schlecht, dass ich nichtmal die Struktur dahinter erkennen kann

Und bitte ändere die Klammerung ist ja ekelhaft!

MfG SideWinder

mose

Ahso sry wegen den Klammern. Wie sollte iche s den besser machen? danke schonmahl

otze

dir fehlt sicher ne Klammer, das kann aber niemand erkennen, da du a) die Klammern absolut grottig positionierst(sodass man nicht erkennen kann, dass eine fehlt) und b) da du eine absolut häßliche schlangenfunktion geschrieben hast, sodass ich keine lust hab, die Klammern händisch richtig zu setzen.

ok: klammerkurs für anfänger:

if(a)
{
    if(b)
    {
        switch(c)
        {
        }
    }
}

wenn dus so machst, musst du nur die augen nach oben und unten bewegen und kannst sofort fehlende/falsche Klammern erkennen. toll oder?

mose

Ahja. Die klammern müssen sein. Sonst läuft nichts.
Sagt mir wie man sowas besser machen kann. Ich blicke da durch. Sry wenn ihr das nicht macht.

SideWinder

mose schrieb:

Ahso sry wegen den Klammern. Wie sollte iche s den besser machen? danke schonmahl

Man rückt die Klammern ein:

if(...)
{
    while(...)
    {
    } // immer schön unter die zugehörige start-anweisung
}

MfG SideWinder

miller_m

breaks fehlen imho auch.

otze

da ist so einiges komisch wenn man sich das mal anschaut

shapeless

du solltest vll auch beachten das man switches eher benutzen soll wenns mehr als 2 fälle sind. bei der menue1 sache sinds 12 das ist ok, aber bei 2en zB deiner "beenden" abfrage ists nur unübersichtlich.
Wie SideWinder schon sagte sind deine switches eh falsch.
Dazu kommt das du dir vll gedanken machen solltest eine schleife zu benutzen, in der du immer wieder die berechnungen und eingaben machst mis jemand bei der abfrage obs weiter gehen soll "nein" eingibt.
dazu solltest du dir überlegen ob du die einzelnen berechnungen nicht in funktionen packen solltest.Nur mal so als vorschlag

SideWinder

Jo, hier liegt ganz klar scherwer Fall von Spaghetticode vor (und das ganz ohne goto)

MfG SideWinder

mose

JO danke ! Aber ka wie ich es machen soll. In der Schule sind wir erst bei Hello World. Hab mir versucht zuhause weitere Sachen bei zu bringen. Naja kommt nur scheisse bei rum.

mose was muss man rauchen um solch kranken code zu schreiben?

SideWinder

@mose:

http://tutorial.schornboeck.net
http://www.volkard.de/vcppkold/inhalt.html

MfG SideWinder

mose

Wieso ist der code krank? der ist logisch aufgebaut ! Wenn manw eiss wie kann man den verstehen. Ahja sowas finde ich irgentwie feige. Die Person anzugreifen weil er so einen Code geschrieben hat. Besonders wenn er versucht sich das grad an zu eignen.

SideWinder

@mose: Nimms doch nicht persönlich, hier bist du im Internet. Der kann dich vollkommen anonym und zum Spaß angreifen und du regst dich auf obwohls gar nicht nötig ist. Lass ihn reden

MfG SideWiner

mose

Mich regt es auf das so welche Leute nicht normal reden können. Müssen sofort die anderen angreifen ! Die sind im RL genau so drauf!

otze

mose schrieb:

Wieso ist der code krank? der ist logisch aufgebaut ! Wenn manw eiss wie kann man den verstehen.

dass ist das problem: man kann nicht selber drauf kommen, sondern man muss den code selber geschrieben haben

gute Klammerung trägt viel zur lesbarkeit bei, da man genau sieht wo etwas anfängt und wo etwas aufhört.

und...das ganze ist viel zulang. das ist einfach nicht wartbar. stell dir vor: bei deinem copy&paste ist dir an einer stelle ein fehler unterlaufen, der jetzt natürlich überall vorkommt. anstatt an einer stelle korrigieren zu müssen, musst du es jetzt an x stellen tun. Und du wirst wohl zugeben, dass dies nicht sinn der sache sein kann.

mose lerne es kleine funktionen zu schreiben. dann siehts nicht mehr so krank aus.

mose

JO da hasse recht. leider kann ich noch nichts anderes.

ZuK

otze schrieb:

stell dir vor: bei deinem copy&paste ist dir an einer stelle ein fehler unterlaufen, der jetzt natürlich überall vorkommt. anstatt an einer stelle korrigieren zu müssen, musst du es jetzt an x stellen tun. Und du wirst wohl zugeben, dass dies nicht sinn der sache sein kann.

Schon geschehen die konstante Pi ist falsch. Kommt im programm min 7x vor.
Sowas solltest du wirklich vermeiden.
Kurt

LOL