Drop Down zusammenfassen - ESP8266

[Kolja L. (kolja82)]

Hallo

Gibt es eine Möglichkeit, diesen Codebatzen zusammenzufassen:

 webpage += "<select name='Line_01'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_02'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_03'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_04'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_05'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_06'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_07'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_08'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_09'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_10'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_11'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_12'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_13'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_14'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_15'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_16'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_17'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";
  webpage += "<select name='Line_18'><option value='000'>aus</option><option value='100'>rot</option><option value='010'>gruen</option><option value='001'>blau</option></select>\n";

also alle Parameter auslagern, sodass ich diese nur einmal ändern muss?

Und wäre das dann auch platzsparender?

Danke und Gruß

[Rufus Τ. F. (rufus)]

Ist das C++-Code? Da könntest Du mit einer Schleife und z.B. sprintf 
etwas erreichen.

[Kolja L. (kolja82)]

Rufus Τ. F. schrieb:
> Ist das C++-Code?

Jain, ich schreibe in der Arduino IDE

[justme (Gast)]

Kolja L. schrieb:
> also alle Parameter auslagern, sodass ich diese nur einmal ändern muss?

Die Codezeilen unterscheiden sich doch sowieso nur durch die beiden 
Ziffern hinter "Line_". Damit ist das doch prädestiniert für eine 
Schleife.

[Kolja L. (kolja82)]

Danke, so gehts:

  for (int j = 1; j <= 18; j++) {
    webpage += "<select name='Line_";
    webpage += j;
    webpage += "'><option value='000'>aus</option><option 
value='100'>rot</option><option value='010'>gruen</option><option 
value='001'>blau</option></select>\n";
  }

[Kolja L. (kolja82)]

Mist, hier funktioniert es nicht so wie oben:

      if (server.argName(i) == "Line_1") {
        int R = server.arg(i).substring(0, 1).toInt() * 400;
        int G = server.arg(i).substring(1, 2).toInt() * 400;
        int B = server.arg(i).substring(2, 3).toInt() * 400;
        setPWM(0, R, G, B);
      }
      if (server.argName(i) == "Line_2") {
        int R = server.arg(i).substring(0, 1).toInt() * 400;
        int G = server.arg(i).substring(1, 2).toInt() * 400;
        int B = server.arg(i).substring(2, 3).toInt() * 400;
        setPWM(1, R, G, B);
      }
      if (server.argName(i) == "Line_3") {
        int R = server.arg(i).substring(0, 1).toInt() * 400;
        int G = server.arg(i).substring(1, 2).toInt() * 400;
        int B = server.arg(i).substring(2, 3).toInt() * 400;
        setPWM(2, R, G, B);
      }

Noch n guter Hinweis?

[Michael (Gast)]

Die ersten drei Zeilen jedes if-blocks sind doch identisch, also pack 
sie oben drüber und mache in den if-blöcken nur den jeweiligen setPwm 
aufruf. oder setze dort nur eine variable auf den jeweiligen wert und 
setz die in den setpwm aufruf an erster stelle ein, dann ist nichts 
dupliziert.

Für dein erstes problem: schau dir trotzdem nochmal sprintf an, dein 
zeilenweises zusammenstöpseln ist nicht unbedingt schön