Abend ich habe ein Problem mit meinen if schleifen (sie wachsen mir einfach über den Kopf), hatte auch schon mal die idee teile in listen zu schreiben weiß aber net wirklich mehr weiter.

zu den schleifen nun es werden immer in 4er Schritten andere koordinaten an label übergeben ich geb mal nur ein teil des codes da es insgesammt 224 label sind und in 4 schritten ist das etwa viel

Code :

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59

System.out.println(mitarbeiteranzeigen.length);         for (int i = 0; i < mitarbeiteranzeigen.length; ++i) {             mitarbeiteranzeigen[i] = new JLabel("Test Text " + i);             // für die Null             if (i < 28) {                 if (i < 4) {                     mitarbeiteranzeigen[i].setBounds(120, 65 + (i * 15), 150,                             25);                 } else if (i < 8) {                     mitarbeiteranzeigen[i].setBounds(120, 89 + (i * 15), 150,                             25);                 } else if (i < 12) {                     mitarbeiteranzeigen[i].setBounds(120, 113 + (i * 15), 150,                             25);                 } else if (i < 16) {                     mitarbeiteranzeigen[i].setBounds(120, 137 + (i * 15), 150,                             25);                 } else if (i < 20) {                     mitarbeiteranzeigen[i].setBounds(120, 161 + (i * 15), 150,                             25);                 } else if (i < 24) {                     mitarbeiteranzeigen[i].setBounds(120, 185 + (i * 15), 150,                             25);                 } else if (i < 28) {                     mitarbeiteranzeigen[i].setBounds(120, 205 + (i * 15), 150,                             25);                 }                 // i - 57             } else if (i < 56) {                 int d = 28;                 if (i < 32) {                     mitarbeiteranzeigen[i].setBounds(220, 65 + ((i-d) * 15), 150,                             25);                 } else if (i < 36) {                     mitarbeiteranzeigen[i].setBounds(220, 89 + ((i-d) * 15), 150,                             25);                 } else if (i < 40) {                     mitarbeiteranzeigen[i].setBounds(220, 113 + ((i-d) * 15), 150,                             25);                 } else if (i < 44) {                     mitarbeiteranzeigen[i].setBounds(220, 137 + ((i-d) * 15), 150,                             25);                 } else if (i < 48) {                     mitarbeiteranzeigen[i].setBounds(220, 161 + ((i-d) * 15), 150,                             25);                 } else if (i < 52) {                     mitarbeiteranzeigen[i].setBounds(220, 185 + ((i-d) * 15), 150,                             25);                 } else if (i < 56) {                     mitarbeiteranzeigen[i].setBounds(220, 205 + ((i-d) * 15), 150,                             25);                 }             } else if (i < 84) {                 int d = 56;                 if (i < 60) {                 mitarbeiteranzeigen[i].setBounds(340, 65 + ((i-d) * 15), 150,25);                           25);                                }                 ....             wnd.add(mitarbeiteranzeigen[i]);

Warum benutzt du denn kein Layout Und warum sind die Abstände so unregelmäßig dass sie sich nicht berechnen lassen?

Ich muss zeja zustimmen, du solltest ein Layout verwenden, es sein denn du möchtest selbst eines schreiben. Wenn dies so ist, dann überleg erstmal ob das sinvoll ist, mit vielen festen Werten zu arbeiten. Was passiert Beispielsweise, wenn die größe des Fensters verändert wird?

hab das nun aus ein paar for schleifen selbst rausbekommen fall es wenn interessiert hab ich alle abstände aus listen gezogen.

Code :

1 2 3 4 5 6 7 8 9 10 11 12 13 14

mitarbeiteranzeigen[i].setBounds(pos_x[l],                         ((pos_y[r]) + (x * 15)), 100, 20);                 ++x;                 if (x == 5) {                     x = 0;                     r++;                     if (r == 7) {                         r = 0;                         if (l == 8) {                             l = 0;                         }                         ++l;                     }                 }

Ich geb schon mal den Hinweis, dass hier ein Switch-Case-Konstrukt wirklich hilfreich wäre.