Gittermuster mit Stringbuilder und einfacher Schleife erstellen

[masul]

Hallo!

Ich muss gerade eine Aufgabe zum Thema Schleifen lösen. Dabei geht es darum, ein Gitter mit Hilfe des StringBuilders zu erstellen.
Die Aufgabe besteht darin, nur eine einfache Schleife zu schreiben, in den Aufgaben zuvor haben wir immer geschachtelte Schleifen benutzt. Negative Parameter sollen durch 0 ersetzt werden (habe ich mit if gelöst). Die Parameter sollen jeweils vor der Ausgabe beliebige Werte annehmen können und das Gitter abhängig von diesen Werten ausgeben.
Mit der for-Schleife lassen sich erstmal die Spalten darstellen.
Ich habe damit angefangen, erstmal StringBuilder-Objekte (für Zeile, Spalte, und ein leeres Objekt, um später alles zusammenzufügen) zu erzeugen:

  static public String gitter (int spa, int zei)

   {  StringBuilder sp = new StringBuilder("+-");
      StringBuilder ze = new StringBuilder("|");
      StringBuilder zs = new StringBuilder();

      if (spa<0)  spa=0;
      if (zei<0)  zei=0;

      for (int i=1, j=1; i<=spa ; i++)
      {  zs.append (sp);

      }
      zs.append ("+\n");

      return zs.toString();
   } //gitter()

Das Muster soll so aussehen, wie auf dem Bild.

 

[nrg]

und du darfst wirklich nur eine einzige schleife verwenden?

 

[masul]

Ja, als Hinweis wurde auch genannt, dass man sich genauer anschauen soll, aus was für Zeilen ein Gitter besteht. Dann soll man eine solche Zeile in einem StringBuilder-Objekt zusammenbauen. Vielleicht hilft das ja weiter..

 

[tfa]

Man könnte den Schleifenzähler bis spa*zei laufen lassen und sich dann die Spalten- und Zeilennummern mit / und % ausrechnen.

 

[Der Müde Joe]

so ein schnellschuss:

StringBuilder builder = new StringBuilder(200);

final int X = 10;
final int Y = 5;

int x = 0;
int y = 0;

for(int i =  0 ;  i < (X * Y) ; i++)
{
   if (x <= y) {
      if (i % X == 0) {
         x++;
         builder.append ('\n');
      } else {
         builder.append ("+-");
      }
   } else {
      if (i % Y == 0) {
         y++;
         builder.append ('\n');
      } else {
         builder.append ("| ");
      }
   }
}
System.out.println (builder.toString ());

 

[masul]

Ich verstehe nicht genau, was du meinst..Zunächst würde ich außerdem gerne mal wissen, ob meine StringBuilder sinnvoll gewählt sind.
Also ich habe mir gedacht, dass ich erst die (+-)-Zeile, dann die (|)-Zeile ausgeben lassen möchte, und zwar so oft wie der Wert, der im 2. Parameter angegeben ist. Zum Schluss soll noch einmal die (+-)-Zeile ausgegeben werden.

 

[masul]

Ich habe deinen Code gerade eingefügt, aber irgendwie gibt er mir immer dasselbe aus, unabhängig von den Parametern.

 

[Der Müde Joe]

Die grossen X und Y sind die Parameter. Mehr verrat ich nicht ;-)

(und hat noch eine kleinen Fehler drin sehe ich grad)

 

[nrg]

nur einfügen und knopf drücken reicht halt manchmal nicht. kuck doch mal wo er die spalten und zeilen definiert hat und wie du das an deine methode angleichen kannst

 

[masul]

Ok, dankeschön! Ich werde mir den Code jetzt genauer anschauen und versuchen, ihn nachzuvollziehen
So habe ich auf jeden Fall mehr davon, statt einfach nur kopieren und einzufügen.

 

[Der Müde Joe]

oder noch besser (und auch richtig)

StringBuilder builder = new StringBuilder(200);

final int X = 4;
final int Y = 7;

String row = "+-";
String column = "| ";
String actual = row;

for (int i = 0; i < (X * Y); i++) {
   if (i % X == 0 && i != 0) {
      if (actual.equals (row)) {
         actual = column;
      } else {
         actual = row;
      }
      builder.append ('\n');
   }
   builder.append (actual);
}
System.out.println (builder.toString ());

 

[masul]

Das Problem ist doch aber, wenn X = 0 ist, dann kommt eine Programmausnahme, weil man ja nicht durch 0 teilen darf..

 

[jgh]

+-+-+-+-
| | | |
+-+-+-+-
| | | |
+-+-+-+-
| | | |
+-+-+-+-

ich bezweifel, dass dieses die richtige Lösung ist (bei deinem oben geposteten Code)
mal abgesehen davon, dass deine Zeilen teilweise mit einem "-" enden, stimmen weder die Anzahl an Zeilen, noch die Anzahl an Spalten.

oder bei 1,1 kommt sowas raus...:
+-

edit:
den Problemfall 0 muss man imho irgendwie so abdecken:

if (spa <=0||zei<=0){
			return "+";
		}

 

[masul]

Also ich habe jetzt erstmal komplett neu angefangen und das ganze erstmal mit verschachtelten
Schleifen gemacht. Es funktioniert. Jetzt muss ich nur noch wissen, wie ich das umschreiben kann..

  static public String gitter (int spalte, int zeile)
   {  String plus  = "+-";
	  String strich= "| ";
	  StringBuilder erg = new StringBuilder ("");

      if (spalte<0)  spalte=0;
      if (zeile<0)  zeile=0;
      for (int j=1; j<=zeile;j++)
      {  for (int i=1; i<=spalte; i++)
         { erg.append(plus);
         }
         erg.append("+\n");
         for (int i=1; i<=spalte;i++)
         { erg.append(strich);
         }
         erg.append("|\n");
       }
       for (int i=1; i<=spalte; i++)
	   { erg.append(plus);
	   }
       erg.append("+\n");

      return erg.toString();
   } //gitter()

 

[tfa]

Du könntest erstmal für "plus" und "strich" einfache String-Variablen verwenden statt StringBuilder. Siehe Beispiel vom müden Joe.

 

[masul]

Ok, ich habs abgeändert. Aber wie kann ich es auch ohne Verschachtelung schreiben??

 

[nrg]

..

Siehe Beispiel vom müden Joe.

 

[Der Müde Joe]

StringBuilder builder = new StringBuilder(200);

final int X = 5;
final int Y = 1;

String row = "+-";
String column = "| ";
String actual = row;
// anzhal di wir brauchen..
int max  = X * ((Y*2)+1);

for (int i = 0; i <= max; i++) {
   if (i % X == 0 && i != 0) {
      if (actual.equals (row)) {
         actual = column;
         builder.append ('+');
      } else {
         builder.append ('|');
         actual = row;
      }
      builder.append ('\n');
   }
   if (i < max) {
      builder.append (actual);
   }
}
System.out.println (builder.toString ());

 

[masul]

Ich verstehe diese Lösung nicht ganz, denn bei der Initialisierung erhält doch actual direkt den Wert von row:

String row = "+-";
String actual = row;

Und später wird in der If-Bedingung geprüft, ob actual gleich row ist:

 if (actual.equals (row)) {

Diese Bedingung müsste doch dann immer erfüllt sein, oder mache ich da einen Denkfehler??

 

[Der Müde Joe]

Der Code

      if (actual.equals (row)) {
         actual = column;
  //       builder.append ('+');
      } else {
       //  builder.append ('|');
         actual = row;
      }

mach einen Swap. Wenn actual row ist, dann setzt er ihn auf column, sonst umgekehrt.
Immer wenn man X-mal actual gezeichnet hat,kommt ein ende (+ oder |) dann kommt ne neue Linie und es wird geswappt.