elegantere Lösung zu else if.

[Chris D. (chris_gleichda)]

Moin
Gibt es eine elegantere Lösung um Anweisungen wie if und elwe if 
auszuführen. Ich will nämlich lins am arduino auslesen und falls eine 
Spannung anliegt soll das was in der if Anweisung steht ausgeführt 
werden. Das hab ich so so gemacht.

While (diesdas > 1)
If(digitalRead(pin1 == HIGH)){
//tu etwas
}
Else if(digitalRead (pin2 == HIGH)){
//tu etwas anderes
}

Und so geht das bei mir mit 20 anderen pins weiter. Es sollen so lange 
diese pins anlusgelesen werden bis die Bedingung in der while schleife 
nicht mehr stimmt.
Gibt es dafür elegantere Lösungen oder kann es hierbei zu komplikationen 
kommen?

[osterhase (Gast)]

Chris D. schrieb:
> Ich will nämlich lins am arduino auslesen

Was sind lins? Ist das ein spezielles Arduino Shield?

Sagt dir "switch" und "case" etwas?

[Chris D. (chris_gleichda)]

Meinte pins haha

[Chris D. (chris_gleichda)]

Und das sagt mir etwas.
Wie läuft das mit dem case und so ab? Läuft das so ähnlich wie diese 
else ifs dauernd bis man etwas drückt?

[port (Gast)]

Wenn man alle portpins nacheinander durchgehen will, nimmt man ein for 
Schleife. In der Schleife braucht man dann nur eine if Abfrage.

[CC (Gast)]

switch/case wird hierbei nicht helfen, damit kann man nur eine Variable 
auf verschiedene Werte abfragen:

int x = f();

switch(x)
{
  case 1:
    do_something();
    break;

  case 2:
    do_something_else();
    break;

  default:
    do_nothing();
}

Dies entspricht in etwa:

if(x == 1)
  do_something();
else if(x == 2)
  do_something_else();
else
  do_nothing();

[BattMan (Gast)]

Warum liest Du nicht erst einmal die Ports komplett ein und prüfst 
danach
welche Pins gesetzt sind oder nicht.
Das funktioniert allerdings nur dann, wenn die Ports komplett für Deine 
Anwendung reserviert sind.

Bspw. so:

if ( a=PINx ) { /* Prüft ob irgend ein Pin gesetzt ist, also NICHT 0 ist*/
 
            for(i=0; i < 8; i++ ){
                if ( a & ( 1 << i)){ /* Einzelne Pins abfragen*/

[Chris D. (chris_gleichda)]

Port schrieb:
Wenn man alle portpins nacheinander durchgehen will, nimmt man ein for 
Schleife. In der Schleife braucht man dann nur eine if Abfrage.

Wichtig is aber das kein pin ausversehen durch zu kurzen tastendruck 
übersehen wird. Wie würde diese for schleife dann aussehen??
Und wie kommt man in die auszuführenden befehle??

[port (Gast)]

Chris D. schrieb:
> Wichtig is aber das kein pin ausversehen durch zu kurzen tastendruck
> übersehen wird. Wie würde diese for schleife dann aussehen??
> Und wie kommt man in die auszuführenden befehle??

Dann nimmt man Interrupts.

[Mike (Gast)]

Chris D. schrieb:
> Wichtig is aber das kein pin ausversehen durch zu kurzen tastendruck
> übersehen wird.

Ich fürchte, du überschätzt deine Drückgeschwindigkeit.
Wie lang ist denn dein kürzester Tastendruck?

[BattMan (Gast)]

BattMan schrieb:
> if ( a & ( 1 << i)){ /* Einzelne Pins abfragen*/

Zur weiteren Veranschaulichung:

         if ( a & ( 1 << i)){ /* Einzelne Pins abfragen*/

        switch(i){

        case: 0: Tue_etwas(); /* PIN Nr. 0 ist gesetzt */
                    break;
        usw...

[BattMan (Gast)]

BattMan schrieb:
> case: 0: Tue_etwas(); /* PIN Nr. 0 ist gesetzt */

Schreibfehler von mir:

Richtig ist:

         case 0: Tue_etwas(); /* PIN Nr. 0 ist gesetzt */

Der Doppelpunkt nach dem case war falsch!

[Kein Name (Gast)]

Warum willst du überhaupt eine andere Lösung?

Ein Programm soll funktionieren und es soll einfach und verständlich 
sein. Damit man bei einem Fehler sofort sieht, was man ändern muss.

So gesehen ist das Programm ideal.

Natürlich kann sich heraus stellen, das es nicht so einfach geht, dass 
du z.B. einen Tastenentprellung in einer Interruptroutine brauchst.

Aber warum anders machen, wenn es funktioniert?

P.S. Beispiele für elegante Lösungen findest du recht einfach. Suche 
nach umfangreichen Open-Source-Projekten, an denen nur 1 oder 2 Leute 
arbeiten.