Wo ist der Fehler hier ??

Reply to Wo ist der Fehler hier ?? on Wed, 10 May 2006 23:27:51 GMT

--Neo-- — Wed, 10 May 2006 23:27:51 GMT

// Optimierung Landereihenfolge

#include
#include
#include
#include
#include
#include

int main(void)
{
  const max = 10;
  int flieger [max];
  int Bestfeld [max];
  int n, m, k;
  int j1, j2, j3, j4, j5, j6, j7, j8, j9, j10;
  int i, Summe, Bestzeit;

  cout << "Es wird jetzt eine Reihenfolge von Maschinen generiert und berechnet:  " << endl;
  cout << endl;
  cout << "1 = Light" << endl;
  cout << "2 = Medium" << endl;
  cout << "3 = Heavy" << endl;
  cout << endl;
  cout << endl;
  n=0;
  m=0;
  k=0;

    for (j1=1; j1 <=3; j1++)
        {
        flieger [1] = j1;
            if (j1=1) n++;
            if (j1=2) m++;
            if (j1=3) k++;
            for (j2=1; j2 <=3; j2++)
            {
            flieger [2] = j2;
                if (j2=1) n++;
                if (j2=2) m++;
                if (j3=3) k++;
                for (j3=1; j3 <=3; j3++)
                {
                flieger [3] = j3;
                    if (j3=1) n++;
                    if (j3=2) m++;
                    if (j3=3) k++;
                    for (j4=1; j4 <=3; j4++)
                    {
                    flieger [4] = j4;
                        if (j4=1) n++;
                        if (j4=2) m++;
                        if (j4=3) k++;
                        for (j5=1; j5 <=3; j5++)
                        {
                        flieger [5] = j5;
                            if (j5=1) n++;
                            if (j5=2) m++;
                            if (j5=3) k++;
                            for (j6=1; j6 <=3; j6++)
                            {
                            flieger [6] = j6;
                                if (j6=1) n++;
                                if (j6=2) m++;
                                if (j6=3) k++;
                                for (j7=1; j7 <=3; j7++)
                                {
                                flieger [7] = j7;
                                    if (j7=1) n++;
                                    if (j7=2) m++;
                                    if (j7=3) k++;
                                    for (j8=1; j8 <=3; j8++)
                                    {
                                    flieger [8] = j8;
                                        if (j8=1) n++;
                                        if (j8=2) m++;
                                        if (j8=3) k++;
                                        for (j9=1; j9 <=3; j9++)
                                        {
                                        flieger [9] = j9;
                                            if (j9=1) n++;
                                            if (j9=2) m++;
                                            if (j9=3) k++;
                                            for (j10=1; j10 <=3; j10++)
                                            {
                                            flieger [10] = j10;
                                                 if (j10=1) n++;
                                                 if (j10=2) m++;
                                                 if (j10=3) k++;
                                            if (((n==3) && (m==4)) && (k==3))
                                            {                                            {
                                            cout << "Test" << endl;
                                            Summe = 0;

                                            for (i=0; i <=9; i++)
                                                {
                                                if (flieger [i] == 1 && flieger [i+1] == 2)
                                                {
                                                    Summe = Summe + 82;
                                                }
                                                else if (flieger [i] == 2 && flieger [i+1] == 1)
                                                {
                                                     Summe = Summe + 146;
                                                }
                                                else if (flieger [i] == 3 && flieger [i+1] == 1)
                                                {
                                                      Summe = Summe + 167;
                                                }
                                                 else if (flieger [i] == 1 && flieger [i+1] == 3)
                                                  {
                                                      Summe = Summe + 78;
                                                  }
                                                  else if (flieger [i] == 3 && flieger [i+1] == 2)
                                                 {
                                                    Summe = Summe + 138;
                                                  }
                                                 else if (flieger [i] == 2 && flieger [i+1] == 3)
                                               {
                                                       Summe = Summe + 78;
                                                }
                                                  else if (flieger [i] == 1 && flieger [i+1] == 1)
                                                {
                                                        Summe = Summe + 82;
                                                   }
                                                    else if (flieger [i] == 2 && flieger [i+1] == 2)
                                                {
                                                       Summe = Summe + 82;
                                                 }
                                                 else if (flieger [i] == 3 && flieger [i+1] == 3)
                                                {
                                                         Summe = Summe + 167;
                                                 }
                                                }
                                                if (Summe < Bestzeit)
                                                {
                                                Bestzeit=Summe;
                                                Bestfeld [max] =flieger [max];
                                                cout << Bestzeit << endl;
                                                }

                   }
                                         }
                                     }
                                }
                            }
                         }
                    }
                 }
               }
             }

        }
  cout << "Reihenfolge: " << endl;
  cout << n << endl;
  cout << m << endl;
  cout << k << endl;

    for (i=1; i


Es muß wohl an den Klammern liegen, oder an dem if-Ausdruck in der Mitte...
Danke für eure Hilfe!



Reply to Wo ist der Fehler hier ?? on Thu, 11 May 2006 04:12:27 GMT
MFK — Thu, 11 May 2006 04:12:27 GMT
--Neo-- schrieb:

Es muß wohl an den Klammern liegen, oder an dem if-Ausdruck in der Mitte...

Und was bitte ist "Es"? Einfach nur den Code hinzuklatschen, ohne jede Fehlerbeschreibung, ist eine Frechheit.
Mir fällt nur auf, dass Bestzeit nicht initialisiert wird.



Reply to Wo ist der Fehler hier ?? on Thu, 11 May 2006 04:32:52 GMT
elise — Thu, 11 May 2006 04:32:52 GMT
*lach*
klammern zählen solltest du erstmal alleine.

eine ordentliche einrückung wirkt da wunder..
im übrigen sehe ich soviel "ähnlichen" code, dass da sicher was zu optimieren ist.
so long



Reply to Wo ist der Fehler hier ?? on Thu, 11 May 2006 05:06:27 GMT
David_pb — Thu, 11 May 2006 05:06:27 GMT
Hi!
 Ich würde sagen der Quellcode schreit nach einer Generalüberholung!!! :-|
grüße



Reply to Wo ist der Fehler hier ?? on Thu, 11 May 2006 12:44:40 GMT
cin — Thu, 11 May 2006 12:44:40 GMT
Das einzige, was mir spontan einfällt, ist, dass die letzte if bedingung vor dem grossen "for" zwei öffnende klammern enthält.

Bei solchen Problem hilft nur noch: Stift und Papier nehmen und Quellcode abschreiben. Dann jeder öffnenden Klammer eine andere Farbe geben und ruckzuck hat man das Problem gefunden.

Kürzer: Refactoring!!!!!!!!!!!!!1111111111111111111111111einhundertelf
edit: noch nen fehler: arrays fangen in c++ bei 0 an. Deine 2 Arrays besitzen 10 Elemente und zwar von 0-9. Du versuchst aber auf das 10. Element zuzugreifen, was im schlimmsten Fall zu einer Zugriffsverletzung führt.