while schleife innerhalb endloser while schleife

[Beuth (Gast)]

Ich habe ein Problem mit while-Schleife innerhalb eine endlose 
while-Schleife

while (1) {
 ...
 sharp = distance(AD1);
 i=0;
 while (sharp > 3.6 && sharp < 12.1) {
      
   P83 = 1;
   wait_10ms(i);

   i+=5;
   sharp = distance(AD1);
   if (i<20) {
     P80=0; P81=0; P81=0; P82=0; 
   } else {
     if (i < 40) {
        P80 = 1; P81 = 0; P82 = 0; P83 = 0;
     } else {
  if (i < 55) {
    P80 = 0; P81 = 1; P82 = 0; P83 = 0; 
  } else {
    P80 = 0; P81 = 0; P82 = 1; P83 = 0;
  }
     }
   } 
 }
 ...
}

Ich möchte alle if-Anweisung ausserhalb der while-Schleife verschieben, 
aber dieser Code gibt keine Ausgabe an Ports.
Wo liegt der Fehler?

Danke für die Hilfe.

[Robert K. (molch)]

Hi,
warum machst du nach deinem else eine geschweifte Klammer?

if (i<20) {
      P80=0; P81=0; P81=0; P82=0;
    } else {
      if (i < 40) {
         P80 = 1; P81 = 0; P82 = 0; P83 = 0;
      } else {
   if (i < 55) {
     P80 = 0; P81 = 1; P82 = 0; P83 = 0;
   } else {
     P80 = 0; P81 = 0; P82 = 1; P83 = 0;
   }

Ändert in diesem Beispiel an der Logik des Ausdrucks nichts aber ein 
else if finde ich besser lesbar.

Ansonsten: Kompletten Code posten!
Hast du Pxx auch auf Ausgang gesetzt?

Grüße

[Karl H. (kbuchegg)]

Beuth schrieb:

Ich geh mal davon aus, dass die Zuweisungen an P80, P81, P82 und P83 die 
Portausgaben sind.

> Ich möchte alle if-Anweisung ausserhalb der while-Schleife verschieben,

Wozu?
Der ganze if-else if Block dient nur dazu, den Messvorgang an den 
Portpins P80, P81 und P82 nach aussen zu transportieren. Wenn du den aus 
der Schleife rausziehst, kannst du ihn auch gleich löschen, weil er dann 
keine Funktion mehr erfüllt.

Edit:
OK, keine Funktion ist übertrieben. Er kann dann immer noch die Funktion 
erfüllen, über die Portpins zu dokumentieren wieviele 
Schleifendurchgänge notwendig waren, bis die Schleife verlassen wurde, 
sofern sie überhaupt mal verlassen wird

  while (1) {
    ...
    sharp = distance(AD1);
    i = 0;
    while (sharp > 3.6 && sharp < 12.1) {
      P83 = 1;
      wait_10ms(i);

      i += 5;
      sharp = distance( AD1 );

      if( i >= 20 )
         P83 = 0;
    }

    if (i < 20) {
      P80 = 0;
      P81 = 0;
      P82 = 0; 
    }
    else if (i < 40) {
      P80 = 1;
      P81 = 0;
      P82 = 0;
    }
    else if (i < 55) {
      P80 = 0;
      P81 = 1;
      P82 = 0; 
    }
    else {
      P80 = 0;
      P81 = 0;
      P82 = 1;
    }
  }

[Johann L. (gjlayde)]

Der Fehler liegt fast immer an der Stelle:

Beuth schrieb:
>  ...

[Beuth (Gast)]

Hallo,

@Robert K.
es war einen Schreibfehler.

      } else {
         if (i < 55) {
           P80 = 0; P81 = 1; P82 = 0; P83 = 0;
         } else {
           P80 = 0; P81 = 0; P82 = 1; P83 = 0;
         }
      }

So sollte es sein.

Wenn diese If-Anweisung innerhalb in while-Schleife liegt, läuft das 
ganze Programm eigentlich fehlerfrei.
Aber diese Ausgänge sind mit einem Roboterarm angeschlossen, und möchte 
nur den letzten Wert von i überprüfen.
z.B: wenn i = 45 ist, möchte ich direkt P81=1 am Ausgang ausgeben, nicht 
erstmal P80=1, dann P81=1.
Versteht Ihr, was ich meinte?
Durch die Kodierung ist 3 unterschiedliche Greifpositionen programmiert:
P80=1 od. P81=1 od. P82=1

@Karl heinz Buchegger
Wenn P83=1 ist, beginnt das Hauptprogramm von dem Roboter.
Danke für die Korrektur.

Ich würde die Code mit else if verbessern, wie Ihr vorgeschlagen hast