Mehrfache if.Verkleinern

[Sepplhuber (Gast)]

Hallo meine Frage frage.

Habe 200 Led´s und wollte nicht 200 mal eine if abfrage machen.

Kann ich sowas auch verkleiner oder zusammenfügen?

Zb

if (incoming == "ledaus") 
    {
      leds[0] = CRGB::Black;
      FastLED.show();
      Serial.println("Led´s ausgeschaltet"); 
      }
      if (incoming == "start") 
      {
      leds[0] = CRGB::Green;
      FastLED.show();
      Serial.println("you started the demo routine"); 
      }
      if (incoming == "ledaus") 
      {
      leds[1] = CRGB::Black;
      FastLED.show();
      Serial.println("Led´s ausgeschaltet"); 
      }
      if (incoming == "1") 
      {
      leds[1] = CRGB::Red;
      FastLED.show();
      Serial.println("you started the demo routine");
      }
      if (incoming == "ledaus") 
      {
      leds[2] = CRGB::Black;
      FastLED.show();
      Serial.println("Led´s ausgeschaltet");
      }
      if (incoming == "2") 
      {
      leds[2] = CRGB::Red;
      FastLED.show();
      Serial.println("you started the demo routine");
      }
      if (incoming == "ledaus") 
      {
      leds[3] = CRGB::Black;
      FastLED.show();
      Serial.println("Led´s ausgeschaltet");
      }
      if (incoming == "3") 
      {
      leds[3] = CRGB::Red;
      FastLED.show();
      Serial.println("you started the demo routine");
      }

Danke für die super klotage und mein Hirn

[Philipp G. (geiserp01)]

Warum fragst Du denn 400mal nach 'ledaus'?

[Sepplhuber (Gast)]

jede LED wird einzeln angesteuert

[Christian B. (casandro)]

Abhängig von der Sprache und Laufzeitbedingungen könntest Du zum 
Beispiel ein Array von Structs machen welche jeweils 3 Elemente haben. 
Das erste ist Dein "incoming"-Wert, das zweite der Wert für leds[3], der 
dritte Dein Debugtext.

Du schreibst dann daraum eine Schleife die alle 200 Einträge dieser 
Liste durchgeht und dann beim richtigen die entsprechende Aktion mach 
und dann raus springt.

Wenn Incoming auch nur eine Zahl sein darf, dann kannst Du das nach 
einer Bereichtsüberprüfung direkt als Index in die Tabelle verwenden. 
Alternativ kannst Du auch mit Vergleichen und einer sortierten Liste 
arbeiten, dann reduzierst Du die Durchläufe von n auf log(n)... was aber 
bei 200 Einträgen noch nicht so viel ausmacht.

Es ist aber gut, dass Du siehst, dass solche Strukturen unschön sind. 
Ein gängiger Weg solche Probleme zu lösen ist über eine Datenstruktur zu 
gehen.

[Stefan F. (Gast)]

Sieht nach C++ aus.

In Java und C# könntest du diese if-Kette durch switch-case ersetzen. 
leider geht das in C++ nicht.

Du könntest allerdings das Schlüsselword else benutzen, um die 
Ausführung nach einem Treffer abzukürzen. Leider sieht das aber nicht 
schöner aus, als der jetzige Quelltext.

Ehrlich gesagt finde ich das auch nicht schlimm. Es ist klar erkennbar, 
was der Code tun soll. Das wichtigste Kriterium für guten Quelltext ist 
daher schon erfüllt.

An der Einrückung würde noch arbeiten.

[Sepplhuber (Gast)]

Genau da liegt mein problem es sollen mal über 1000Leds damit gesteuert 
werden. dann müsste ich alles 1000 mal schreiben %rolleyes%

[Peter D. (peda)]

Sepplhuber schrieb:
> if (incoming == "ledaus")

Schön, wenn man Code aus dem Zusamenhang reißt und dann der Leser 
rumrätseln darf.
Wie ist denn incoming definiert?

Wenn man freundlich ist, dann postet man Codeschnipsel immer so, daß sie 
auch compilieren, d.h. mit allen nötigen Deklarationen.

Sepplhuber schrieb:
> Habe 200 Led´s und wollte nicht 200 mal eine if abfrage machen.

Dann beschreibe einfach mal, was Du überhaupt machen willst.

[Joachim B. (jar)]

Sepplhuber schrieb:
> Genau da liegt mein problem es sollen mal über 1000Leds damit gesteuert
> werden. dann müsste ich alles 1000 mal schreiben %rolleyes%

das schreit doch geradezu nach einer Schleife:
sieht nach c# oder Arduino aus das deutet darauf hin:
      FastLED.show();
      Serial.println("Led´s ausgeschaltet");

for(uint16_t i = 0; i<1000; i++)
{ 
   if(!strcmp(incoming, "ledaus"))
   {
      leds[i] = CRGB::Black;
      FastLED.show();
   }
   // alternativ FastLED.show(); am ENDE der Schleife
}

man sollte aber incoming noch erweitern WELCHE LED an oder aus sein 
soll, also das Kommando noch zerlegen.

[Sepplhuber (Gast)]

Bin bei einem Pick to Light System funktioniert alles Super , Programm 
wurde um den Arduino Uno anzusteuern mit C# geschrieben.

[c]

String incoming = "";
void loop(){

 if (Serial.available())
  {
    incoming = Serial.readStringUntil('\n');
    Serial.println(incoming);
    {
    if (incoming == "ledaus")
    {
      leds[0] = CRGB::Black;
      FastLED.show();
      Serial.println("Led´s ausgeschaltet");
      }
      if (incoming == "start")
      {
      leds[0] = CRGB::Green;
      FastLED.show();
      Serial.println("you started the demo routine");

[Falk B. (falk)]

Sepplhuber schrieb:
> Hallo meine Frage frage.
>
> Habe 200 Led´s

Und einen Deppenapostroph.

>und wollte nicht 200 mal eine if abfrage machen.

Wer will das schon?

> Kann ich sowas auch verkleiner oder zusammenfügen?

Sicher.

>
> Zb
>
> <c/>

Formatierungen brauchen hier eckige Klammern [ ]

> Danke für die super klotage und mein Hirn

Wie sieht denn deine Kommandostruktur aus?
Logischerweise muss man das Kommando dekodieren und daraus einen Index 
einer Variable berechnen und eben NICHT 200 Vergleiche machen. Außerdem 
erscheint mir deine Steuerlogik mehr als fragwürdig.

Die Formatierung deines Quelltextes ist sehr schlecht, man sieht die 
Struktur nicht!

[Andreas S. (Firma: Schweigstill IT) (schweigstill)]

Wenn es sich um ein Programm in C handeln sollte, lässt es sich wie 
folgt vereinfachen:

{}

incoming wird nämlich sicherlich nicht die Adresse der nur lokal 
gültigen Stringliterals "ledaus", "start", usw. beinhalten, so dass der 
Vergleich in jedem Fall scheitern wird. Ob der Compiler dies auch selbst 
erkennen kann, hängt von der Vorgeschichte von incoming ab. Falls er 
dazu in der Lage sein sollte, wird der Compiler ein sehr aussagekräftige 
Warnung ausgeben.

[Falk B. (falk)]

So als Ansatz

if (incoming == "ledaus") {
  leds[lednr] = CRGB::Black;
  FastLED.show();
  Serial.println("Led´s ausgeschaltet");
}

if (incoming == "start") {
  leds[lednr] = CRGB::Green;
  FastLED.show();
  Serial.println("you started the demo routine");
}

int tmp = atoi(incomming);
if (tmp >0 && tmp<200 ) {
  lednr = tmp;
  Serial.println("you started the demo routine");
}

[Irgend W. (Firma: egal) (irgendwer)]

Joachim B. schrieb:
> Serial.println("Led´s ausgeschaltet");
> for(uint16_t i = 0; i<1000; i++)
> {
>    if(!strcmp(incoming, "ledaus"))
>    {
>       leds[i] = CRGB::Black;
>       FastLED.show();
>    }
>    // alternativ FastLED.show(); am ENDE der Schleife
> }

Wohl eher sorum:

if(!strcmp(incoming, "ledaus"))
{
  for(uint16_t i = 0; i<1000; i++)
  {
    leds[i] = CRGB::Black;
  }
  FastLED.show();
  Serial.println("Led´s ausgeschaltet");
}

1000 ma die selbe if macht nicht wirklich Sinn und die print würde bei 
dir immer aufgerufen auch wenn der Befahl garnicht "ledaus" lautet.

[Sepplhuber (Gast)]

lednr ist nicht declariert :(

[Udo S. (urschmitt)]

Sepplhuber schrieb:
> lednr ist nicht declariert :(

Das ist jetzt dein Teil der Arbeit

[Joachim B. (jar)]

Falk B. schrieb:
> Und einen Deppenapostroph.

Falk B. schrieb:
> So als Ansatz
> if (incoming == "ledaus") {
>   leds[lednr] = CRGB::Black;
>   FastLED.show();
>   Serial.println("Led´s ausgeschaltet");
> }

hattest du doch gerade moniert?

Irgend W. schrieb:
> Wohl eher sorum:

hast Recht war ja nur grob fertig, klar kommt erst mal:

1. Befehl (was zu tun)
2. wer (bitteschön -> was wie wo -> welche LED eine Spezielle oder 
alle?)

je nach Geschmacksrichtung und Belieben ausbaufähig.

[Sepplhuber (Gast)]

Vielen Dank allen hat super Funktioniert, mein Arduino ist gerade 
abgeraucht.

Toll

[Random .. (thorstendb)]

Die strings sollten im Vorfeld geparst und durch enums ersetzt werden. 
Damit lassen sich dann ggf. switches realisieren unnd unnötige string 
compares vermeiden.

[Peter D. (peda)]

Sepplhuber schrieb:
> if (incoming == "ledaus")
>     {
>       leds[0] = CRGB::Black;
>       FastLED.show();
>       Serial.println("Led´s ausgeschaltet");
>       }
>       if (incoming == "start")

Warum sollte sich plötzlich incoming auf magische Weise geändert haben?
Zwischen den beiden ifs ist kein Schreibzugriff auf incoming.
Da steht immer noch "ledaus", d.h. der Vergleich mit "start" ergibt 
immer false.

Typischer Weise parst man eine komplette Zeile, d.h. zerlegt sie in 
Befehl, Argument 1, Argument 2 usw. Sind Argumente Zahlen, weist man sie 
einer Variablen zu, z.B. LED-Nummer, LED-Farbe usw).
Hier mal ein Beispiel in C:

void execute(char* buf)
{
  if (*buf == '\0')                                     // do nothing on empty commands
    return;
  cmd_data.answer = buf;                                // answer overwrite command
  cmd_data.cmd_tok = strtok(buf, " ");                  // command token
  cmd_data.par_tok = strtok(NULL, " ");                 // parameter token
  cmd_data.val_tok = strtok(NULL, " ");                 // value token
  for (command_t* cmd_p = cmd_table; pgm_read_byte(cmd_p->cmd) != '\0'; cmd_p++)
  {
    if (cmd_strcmp_P(cmd_data.cmd_tok, cmd_p->cmd) == 0)
    {
      func f_ptr = (func)pgm_read_word(&cmd_p->func);
      f_ptr();
      return;
    }
  }
  answer_err(CMDERR_UNKNOWN_CMD);
}

Die Befehle mit Funktionspointer stehen in einem Array.

[macgyver (Gast)]

Sepplhuber schrieb:
> Vielen Dank allen hat super Funktioniert, mein Arduino ist gerade
> abgeraucht.

so richtig mit "magischer Rauch kommt raus" oder einfach nur "er tut 
nicht was ich gerne hätte"?
den magischen Rauch frei zu lassen muss man mit einer Programmänderung 
auch mal schaffen.

[Sepplhuber (Gast)]

Habe statts deklariert , explodier eingeben das war wohl ein 
schreibfehler.

[Osszilierer (Gast)]

Sepplhuber schrieb:
> Vielen Dank allen hat super Funktioniert, mein Arduino ist gerade
> abgeraucht.

hoffe der Rauch entspricht auch den LED Farben

[Sepplhuber (Gast)]

Nein bei Black sind sie aus :(

Also sehe überhaupt nichts mehr

[leo (Gast)]

Peter D. schrieb:
> Warum sollte sich plötzlich incoming auf magische Weise geändert haben?

Lass dich nicht von der grausigen Einrueckung ablenken.

leo

[Sepplhuber (Gast)]

Ich sag mal Danke , bin jetzt verwirrt :)

[Random .. (thorstendb)]

Bischen rumgespielt, hoffentlich hab ich alles richtig gelesen. Da jeder 
if block gleich wieder geschlossen wird, kann man da einiges verkürzen. 
Und wenn eh C++ verwendet wird ...

#include <map>
#include <string>
using namespace std;

enum func_e {
  LED_OFF   = 0,
  LED_START,
  LED_1,
  LED_2,
  LED_3,
};

enum color_e {
  Black,
  Red,
  Green,
};


map<string, func_e> funcs = {
  { "ledaus", LED_OFF   },
  { "start" , LED_START },
  { "1"     , LED_1     },
  { "2"     , LED_2     },
  { "3"     , LED_3     },
};


int leds[4] = { 0 };

void show()
{
}

void LedFunc(const string& incoming)
{
  auto it = funcs.find(incoming);
  if(it == funcs.end())
    return;

  func_e func = it->second;
  
  switch(func) {
    case LED_OFF:
      leds[0] = Black;
      leds[1] = Black;
      leds[2] = Black;
      leds[3] = Black;
      break;
    case LED_START:
      leds[0] = Green;
      break;
    default: {
      int idx = (func - LED_1) +1;
      if(idx > 3)
        return;
      leds[idx] = Red;
    } break;
  }

  show();

  if(!func)
    printf("Leds ausgeschaltet");
  else
    printf("you started the demo routine");
}

[Dirk B. (dirkb2)]

Peter D. schrieb:
> strtok

strtok ist doof, da nicht reentrant.

sscanf kann man da auch einsetzen.

[Peter D. (peda)]

leo schrieb:
> Lass dich nicht von der grausigen Einrueckung ablenken.

Ja, darauf bin ich hereingefallen.
Ist aber trotzdem Mist.

[Nick M. (Gast)]

Der ganze code ist so wirr, da sollte man damit anfangen sich zu 
überlegen, was man überhaupt will.
Und dann noch alles per String übergeben. Oder Strings dazu verwenden 
...

... da wäre es schon eine geeignete Strafarbeit alle 800 Möglichkeiten 
einzeln per if und string hinzuschreiben.

Think!

[Peter D. (peda)]

Dirk B. schrieb:
> strtok ist doof, da nicht reentrant.

Wozu sollte jemand sowas brauchen?
Sepplhuber wird bestimmt kein Multitasking auf seinem Arduino Uno laufen 
haben und damit auch keine 2 UARTs gleichzeitig parsen wollen.
Und selbst wenn, dann kann man einfach die 3 Zeilen atomar kapseln.
Man muß nicht Probleme an die Wand malen, wo keine sind.

Dirk B. schrieb:
> sscanf kann man da auch einsetzen.

Dann muß man wieder entsprechend lange Puffer reservieren, was auch 
fehleranfällig sein kann.
Und der Formatstring bei scanf ist auch eine Wissenschaft für sich. Was 
passiert bei "%5s %5s", wenn der erste String 6 Zeichen lang ist?
Muß man nicht gelesene Strings vorher nullen?

[Sepplhuber (Gast)]

War scho super, leider blicke ich jetzt überhaupt nicht mehr durch , 
also weiter mit Google Danke euch....

[Random .. (thorstendb)]

Schau dir mal den Ansatz von mir an, ggf. kommst damit weiter.
Vielleicht solltest du aber nochmal einen Schritt zurücktreten, und dir 
die Anforderungen einmal neu anschauen - schreib doch mal hier auf, was 
überhaupt erreicht werden soll.
Deine "200 LEDs" musst du auf jeden Fall rechnerisch ansteuern, nicht 
mit 200 ifs.

Siehe dazu mein Beispiel (aus deinem Code umgebaut):

int idx = (func - LED_1) +1;
if(idx > 3)
  return;
leds[idx] = Red;

[Brummbär (Gast)]

Stefanus F. schrieb:
> In Java und C# könntest du diese if-Kette durch switch-case ersetzen.
> leider geht das in C++ nicht.

Quatsch: In C++ kannst Du alles machen, was in C geht.

[Dirk B. (dirkb2)]

Peter D. schrieb:
> Dirk B. schrieb:
>> strtok ist doof, da nicht reentrant.
>
> Wozu sollte jemand sowas brauchen?

Z.B. wenn du einen (Sub-)String scannen möchtest.

Peter D. schrieb:
> Und der Formatstring bei scanf ist auch eine Wissenschaft für sich. Was
> passiert bei "%5s %5s", wenn der erste String 6 Zeichen lang ist?

Es werden nur 5 Zeichen gelesen.

> Muß man nicht gelesene Strings vorher nullen?

Nein.
Das macht sscanf, wenn gültige Zeichen gefunden worden - andernfalls 
wird garnichts geschrieben.

[Stefan F. (Gast)]

Brummbär schrieb:
>> In Java und C# könntest du diese if-Kette durch switch-case ersetzen.
>> leider geht das in C++ nicht.
> Quatsch: In C++ kannst Du alles machen, was in C geht.

Switch-case mit String können C und C++ nicht. Java und C# können das 
aber.

[Dirk B. (dirkb2)]

Brummbär schrieb:
> Stefanus F. schrieb:
>> In Java und C# könntest du diese if-Kette durch switch-case ersetzen.
>> leider geht das in C++ nicht.
>
> Quatsch: In C++ kannst Du alles machen, was in C geht.

In C geht das auch nicht (wegen der Strings, das mag Switch nicht)

[Brummbär (Gast)]

Sepplhuber schrieb:
> if (incoming == "ledaus")

Ist ein direkter "==" bei einem String möglich?
"ledaus" ist ein Zeiger auf das erste Zeichen - in diesem Fall "l".

incoming ist zwar auch ein Zeiger, aber sicherlich nie auf genau dieses 
"ledaus".

Das geht zwar in einigen Skriptsprachen, aber nicht in C/C++.

[Dirk B. (dirkb2)]

Brummbär schrieb:
> Ist ein direkter "==" bei einem String möglich?

Bei std::string ja, bei C-Strings nicht (in dem Sinn als ersatz für 
strcmp())

> Das geht zwar in einigen Skriptsprachen, aber nicht in C/C++.
Bei C geht es definitiv nicht.

[Stefan F. (Gast)]

Brummbär schrieb:
> Ist ein direkter "==" bei einem String möglich?

Bei Arduino: ja

https://www.arduino.cc/reference/en/language/variables/data-types/string/operators/comparison/

[Sepplhuber (Gast)]

Brummbär schrieb:

Das geht zwar in einigen Skriptsprachen, aber nicht in C/C++.
Das geht zwar in einigen Skriptsprachen, aber nicht in C/C++.

Mein Code läuft einwandfrei nur halt mit den vielen IF abfragen, ich 
denke Arduino ist C++

[NichtWichtig (Gast)]

Bei solcher Menge an Möglichkeiten wäre ein binärere Baumstruktur auch 
cool.

1. Buchstabe checken, links oder rechts weiter
2. Buchstabe checken: links oder rechts weiter
3. Buchstabe checken: links oder rechts weiter
4. Buchstabe checken: links oder rechts weiter
5. ....
6. ...
7. ..
....

RatzFatz ist man bei dem richtigen Eintrag, jedesmal egal was empfangen 
wurde.

[NichtWichtig (Gast)]

https://de.wikipedia.org/wiki/Bin%C3%A4rer_Suchbaum

[Brummbär (Gast)]

Dirk B. schrieb:
> Bei std::string ja, bei C-Strings nicht (in dem Sinn als ersatz für
> strcmp())

Stimmt, da war ja was: Operatorüberladung

[Andreas S. (Firma: Schweigstill IT) (schweigstill)]

Brummbär schrieb:
> "ledaus" ist ein Zeiger auf das erste Zeichen - in diesem Fall "l".

Nein, das ist nicht korrekt! Man kann lediglich sagen, dass sich ein als 
Stringliteral verwendetes character-Array bei Vergleichen wie ein 
entsprechender const-Zeiger verhält.

Hier zum Unterschied zwischen String bzw. als String verwendetem Array:

#include <stdio.h>
#include <string.h>

int main(int argc, char ** argv) {
  int len, size;
  char * ptr     = "ledaus";
  char   str1[]  = "ledaus";
  char   str2[7] = "ledaus";
  
  len   = strlen(ptr);
  size  = sizeof(ptr);
  printf("%s %d %d\n", ptr   , len, size);

  len   = strlen(str1);
  size  = sizeof(str1);
  printf("%s %d %d\n", str1  , len, size);

  len   = strlen(str2);
  size  = sizeof(str2);
  printf("%s %d %d\n", str2  , len, size);

  return 0;
} // main()

Die entsprechenden Ausgaben lauten auf einem x86-64 mit 64Bit-Linux, 
einmal als 64 Bit-Anwendung und einmal als 32 Bit-Anwendung kompiliert:

aschweig@bouvier:~/cpp> gcc -m32 -o string string.c
aschweig@bouvier:~/cpp> ./string
ledaus 6 4
ledaus 6 7
ledaus 6 7
aschweig@bouvier:~/cpp> gcc -o string string.c
aschweig@bouvier:~/cpp> ./string
ledaus 6 8
ledaus 6 7
ledaus 6 7

[Sven K. (quotschmacher)]

Sepplhuber schrieb:
> Nein bei Black sind sie aus :(
>
> Also sehe überhaupt nichts mehr

ja natürlich. wie sollte eine led aussehen, die schwarz leuchtet?

Sepplhuber schrieb:
> if (incoming == "ledaus")
>     {
>       leds[0] = CRGB::Black;
>       FastLED.show();
>       Serial.println("Led´s ausgeschaltet");
>       }
>       if (incoming == "start")
>       {
>       leds[0] = CRGB::Green;
>       FastLED.show();
>       Serial.println("you started the demo routine");
>       }

gehe das mal durch und denke nach. wir gehen in die schleife rein, wenn 
incoming == ledaus. wenn dem so ist, kommt der vergleich, ob incoming 
denn == start wäre. geht das überhaupt?

und zur optimeirung: setze erst alle leds[0] bis leds[1000] und mache 
danach ein FastLED.show().

[Peter D. (peda)]

Dirk B. schrieb:
> Es werden nur 5 Zeichen gelesen.

Meine Frage war eher, landet dann das 6. Zeichen im 2. String oder wird 
es übersprungen?

Dirk B. schrieb:
> andernfalls
> wird garnichts geschrieben.

Also muß ich vorher das erste Byte nullen, sonst steht noch der alte 
String drin.

[Jens G. (jensig)]

Sven A. (quotschmacher)

>gehe das mal durch und denke nach. wir gehen in die schleife rein, wenn
>incoming == ledaus. wenn dem so ist, kommt der vergleich, ob incoming
>denn == start wäre. geht das überhaupt?

Doch, es kommt zu dem vergleich. Noch einer, der sich durch die 
Scheißformatierung verwirren läßt. War doch schon kurz vorher geklärt.

[Dirk B. (dirkb2)]

Peter D. schrieb:
> Dirk B. schrieb:
>> Es werden nur 5 Zeichen gelesen.
>
> Meine Frage war eher, landet dann das 6. Zeichen im 2. String oder wird
> es übersprungen?

Ja. Denn es ist ja ein gültiges Zeichen für %5s


> Dirk B. schrieb:
>> andernfalls
>> wird garnichts geschrieben.
>
> Also muß ich vorher das erste Byte nullen, sonst steht noch der alte
> String drin.

Nein, du mußt den Rückgabewert von sscanf auswerten.
Der gibt an, wieviel Formate gelesen wurden. Bei deinem Formatstring 
wäre das 2.

[Sven K. (quotschmacher)]

Jens G. schrieb:

>
> Doch, es kommt zu dem vergleich. Noch einer, der sich durch die
> Scheißformatierung verwirren läßt. War doch schon kurz vorher geklärt.

das der vergleich kommt wollte ich auch nicht anzweifeln, nur, dass der 
nie zutreffen kann.

[Samuel C. (neoexacun)]

Sven A. schrieb:
> das der vergleich kommt wollte ich auch nicht anzweifeln, nur, dass der
> nie zutreffen kann.

Und damit liegst du immernoch falsch. Schau dir die fehlerhafte 
Formatierung deines zitierten Codeschnippsels nochmal an.

[Sven K. (quotschmacher)]

jepp. gesehen. da fehlt die eine öffnende geschweifte klammer, die ich 
mir dank der einrückung gedacht habe.

[Eric B. (beric)]

Sven A. schrieb:
> Sepplhuber schrieb:
>> if (incoming == "ledaus")
...
>
> gehe das mal durch und denke nach. wir gehen in die schleife rein,

http://if-schleife.de/
(sorry, ist Freitag ;-))

[Stefan F. (Gast)]

"Es gibt keine if-Schleifen, sondern nur if-Abfragen!"

Der Satz ist aber auch quatsch. Mit "if" wird nichts abgefragt, sondern 
es wird eine Bedingung geprüft.

Unter Abfragen verstehe ich z.B. so etwas:  taster = PIND && (1<<3);
Oder: spannung = analogRead(A0);

[M. K. (sylaina)]

Stefanus F. schrieb:
> "Es gibt keine if-Schleifen, sondern nur if-Abfragen!"
>
> Der Satz ist aber auch quatsch. Mit "if" wird nichts abgefragt, sondern
> es wird eine Bedingung geprüft.
>
> Unter Abfragen verstehe ich z.B. so etwas:  taster = PIND && (1<<3);
> Oder: spannung = analogRead(A0);

Öhm, doch. If fragt schon ab. Heißt ja „Wenn“.
Deine Abfragen sind keine Abfragen sondern Zuweisungen ;)

[Stefan F. (Gast)]

M. K. schrieb:
>>taster = PIND && (1<<3);
> Deine Abfragen sind keine Abfragen sondern Zuweisungen ;)

Der linke Teil ja, der rechte Teil ist die Abfrage.

If macht für mich nur einen Test auf etwas, das vorher abgefragt oder 
berechnet wurde. In Assembler heisst der entsprechende Befehl bei 
einigen Mikrocontrollern sogar "tst" (siehe 
http://www.keil.com/support/man/docs/armasm/armasm_dom1361289913099.htm), 
nicht "query".

Herrlich, wie uneindeutig unsere Sprache ist. Den Deutschlehrern wird 
nie langweilig. Stell Dir mal vor, wir würden uns im echten Leben (im 
Beruf) ebenso an einzelnen Worten festbeißen... nee, besser nicht.

Ja, ich weiß, dass ich damit angefangen habe.

[Peter D. (peda)]

Dirk B. schrieb:
> Ja. Denn es ist ja ein gültiges Zeichen für %5s

So, ich hab es jetzt auf sscanf umgestellt. Wenn der erste String in den 
2. Puffer überläuft, ist das egal, da dann ja der erste schon falsch 
ist.
Beachten muß man, wenn der String nur Whitespaces beinhaltet, wird nicht 
0 zurück gegeben, sondern EOF. Man prüft daher besser, ob die Anzahl 
stimmt, z.B. 3 ist.
Tricky ist außerdem, einen Pufferüberlauf zu verhindern.
Hier hab ich die Lösung gefunden:
https://medium.com/@hauyang/convert-int-into-string-with-c-macro-125eeaa71600
Der Code:

#define CMD_SIZE        19              // for sscanf, decimal number only !
#define PAR_SIZE        19

typedef char cmd_str_t[PAR_SIZE+1];
typedef char par_str_t[PAR_SIZE+1];

#define STR_INDIR(x) #x
#define STR(x) STR_INDIR(x)
#define CMD_FMT                 "%" STR(CMD_SIZE) "s"
#define PAR_FMT                 "%" STR(PAR_SIZE) "s"
// ..
  cmd_data.args = sscanf_P(buf, PSTR(CMD_FMT PAR_FMT "%g"), cmd_data.cmd, cmd_data.par, &cmd_data.val);

sscanf sieht zwar eleganter aus gegenüber strtok, erfordert aber 
deutlich mehr Gehirnschmalz, um es auch sicher zu machen.

[M. K. (sylaina)]

Stefanus F. schrieb:
> Der linke Teil ja, der rechte Teil ist die Abfrage.

Öhm...auch nicht. Der rechte Teil ist einfach nur eine Verknüpfung. 
Abgefragt wird da nix. Du hast hier sicher die Pin-Abfrage im Kopf nach 
dem Schema

if(PINB && (1 << PB0))

if fragt ja nur, ob der Ausdruck/die Bedingung wahr ist. Ist die 
Bedingung wahr wird der folgende Befehl oder Block abgearbeitet, ist die 
Bedingung nicht war wird der folgende Befehl oder Block übersprungen.

If prüft also nur, ob die Bedingung wahr ist oder nicht und prüfen kann 
man auch durch das nette Wort Abfragen ersetzen. Und in C wird jeder 
Wert, der nicht 0 ist, als wahr betrachtet bzw. jeder Wert, der 0 
entspricht, wird als nicht wahr betrachtet.

Wie gesagt, das Schlüsselwort if ist ja nicht vom Himmel gefallen, es 
ist eine Abfrage. ;)

[Stefan F. (Gast)]

M. K. schrieb:
> Wie gesagt, das Schlüsselwort if ist ja nicht vom Himmel gefallen, es
> ist eine Abfrage. ;)

Wenn du meinst. Solange wir beide das Schlüsseldorf sinnvoll benutzen 
können (woran ich nicht zweifle) ist mir egal, wie wir es bezeichnen.