Zusammenfassen/Vereinfachen von if-Abfragen

[Timo (Gast)]

Hallo,

ich habe das Problem dass mein Code langsam total uübersichtlich wird. 
Der Grund sind viele zusammenhängende if-Abfragen. Hat jemand ne Idee, 
wie folgenes (nur Beispiel) zu vereinfachen bzw. zusammenzufassen wäre:

if(count <= 15)
{
  button2 = 0x02;
}

else if(count <= 30)
{
  button2 = 0x00;
}

else if(count <= 45)
{
  button2 = 0x02;
}

else if(count <= 60)
{
  button2 = 0x00;
}

else if(count <= 75)
{
  button2 = 0x02;
}

else if(count <= 90)
{
  button2 = 0x00;
}

else if(count <= 105)
{
  button2 = 0x02;
}

Krieg es einfach nicht hin......

[fjgh (Gast)]

Such mal nach "case"

[Дуссель дукъ (Gast)]

Nee "case" kann keine Bereiche.
Etwas eleganter :

found = false

if (not found) and (x<..) { found = true; .. }

[Zittermann (Gast)]

Дуссель дукъ schrieb:

>Nee "case" kann keine Bereiche.

Dann nimm Bascom. Dessen "select case" kann Bereiche auswerten.

gez. Zittermann

[Peter II (Gast)]

fjgh schrieb:
> Such mal nach "case"

und was soll das bringen?

auf die schnelle würde ich sagen

button2 = (((count+1)/15)&1) ? 0x02 : 0x00;

[Marcel (Gast)]

x = count / 15;

ist x gerade button2 = 0
sonst button2 = 2

[Uwe (de0508)]

Hallo  Timo

man die IF-THEN-ELSE Schachtelung auch als Macro ausführen, dann sieht 
es etwas kompakter aus.

#define COMP(arg, statement) \
if(count <= (arg)) \
do { statement } while(0) \
else

Beispiel

COMP(15,code1)
COMP(30,code2)
{ }

[Micha (Gast)]

Дуссель дукъ schrieb:
> Nee "case" kann keine Bereiche.
> Etwas eleganter :

Hm, ich glaube der GCC kann das auch in C (als specifische Erweiterung)
http://gcc.gnu.org/onlinedocs/gcc-4.1.2/gcc/Case-Ranges.html

[Karl H. (kbuchegg)]

In diesem Fall liegt der Schlüssel für einen einfacheren Code in der 
Erkenntnis, dass die Vergleichswerte alle Vielfache von 15 sind. In 
solchen Fällen geht mit ein bischen Rumrechnerei auf Grundschulniveau 
immer was (wurde ja schon gezeigt)

Wer die Division scheut, kann auch so vorgehen

  while( count > 30 )
    count -= 30;

  if( count <= 15 )
    button2 = 0x02;
  else
    button2 = 0x00;

[oldmax (Gast)]

Hi
Wie wär es mit
Button2 =((Trunc(Count/15) And $01)-1)*-1*2
Kann es leider grad nicht ausprobieren aber das Ergebnis von Count / 15 
ist entweder Grade oder ungerade, d. H. Bit 0 ist bei Ungerade gesetzt. 
Das Ergebnios von And 01 ist also 0 oder 1 und das mal 2 ist entweder 0 
oder 2. Da aber bei Werten <15 in Button 2 soll, einfach vom Ergebnis 
Trunc(Count/15) And $01) 1 abziehen und anschließend aus -1 +1 machen..
Einfach mal probieren. Es gibt Compiler, die können das. C kenne ich 
allerdings nicht und zur Zeit hab ich noch nicht mal VB zur Verfügung.
Gruß oldmax

[Peter II (Gast)]

oldmax schrieb:
> C kenne ich
> allerdings nicht und zur Zeit hab ich noch nicht mal VB zur Verfügung.

die C Lösung steht schon weiter oben.

[Patrick (Gast)]

Micha schrieb:
> Hm, ich glaube der GCC kann das auch in C (als specifische Erweiterung)

Ja - mit dem Nachteil, dass der Code dann eben nicht mehr "generisch" 
bzw. übertragbar ist - was bei Mikrocontroller-Code ja ohnehin eher 
selten der Fall ist, daher spricht hier wohl nichts dagegen, diese 
Erweiterung zu verwenden.

Warum jedoch nicht einfach:

if(count <= 15
  || (count > 30 && count <= 45)
  || (count > 60 && count <= 75)
  || (count > 90 && count <= 105))
{
  button2 = 0x02;
}
else
{
  button2 = 0x00;
}

Oder sequentiell:

button2 = 0x00;
if(count <= 15)
{
  button2 = 0x02;
}
if(count > 30 && count <= 45)
{
  button2 = 0x02;
}
if(count > 60 && count <= 75)
{
  button2 = 0x02;
}
if(count > 90 && count <= 105)
{
  button2 = 0x02;
}

[Tom (Gast)]

Timo schrieb:
> Code langsam total unübersichtlich
Schritt 1: Mit einen Kommentar beschreiben, was der lange Teil tut:

// decode count-dings to button-dings. 
if(count <= 15)
....

Schritt 2:  Den langen Teil in eine eigene Funktion auslagern, die so 
heißt wie der eben geschriebene Kommentar:

uint8_t decode_count_to_button(uint8_t x)
{
    switch(x)
    {
        case  0...15: return 0x02;
        case 16...15: return 0x00;
        // oder rechnen
    }
}
...
...
...
//decode_count_to_button
button2 = decode_count_to_button(count);

Schon ist der Code an der ursprünglichen Stelle lesbar und der Kommentar 
überflüssig, da im Code fast wörtlich wiederholt => weg mit dem 
Kommentar.


Uwe S. schrieb:

#define COMP(arg, statement) \

Sehr intuitiv und ohne nachlesen der Definition von COMP verständlich. 
Eine selbstzusammen#definete Programmiersprache ist immer gut, C kann 
einfach zu wenig.