Funktion soll ein String in Gitterform liefern

[Guest]

Hallo zusammen,
ich heiße Jan und bin Student der Medieninformatik an der TFH-Berlin.
Wir lernen gerade die Java programmierung und dazu habe ich mal ein paar fragen
Wir sollten eine Funktion programmieren die ein Schachbrett muster liefert....
Das habe ich auch noch hinbekommen...
Quelltext:

  static public String schach05(String eingabe1, String eingabe2){
     String schach5 = "";

     for(int e=0; e < eingabe1.length(); e++){
        for(int i=0; i < eingabe2.length(); i++){
           schach5 = schach5 + sortChar(eingabe2.charAt(i), eingabe1.charAt(e)) + " ";
        }
        schach5 = schach5 + "\n";
     }
     return schach5;
  }

d.h. Wenn ich schach05(123,AB) aufrufe dann gibt die Funktion mir einen String wieder der folgendermaßen aussieht:
A1 B1
A2 B2
A3 B3

Jetzt soll ich eine weitere Funktion programmieren die mir ein Gitter-Muster liefert...
gitter(1,1) soll folgenden String liefern:
+-+
| |
+-+

und gitter(2,3) soll foldengen String liefern:
+-+-+
| | |
+-+-+
| | |
+-+-+
| | |
+-+-+

Kann mir einer einen Tip geben wie ich diese gitter funktion realisieren kann?
Sie muss ja ähnlich sein wie diese schachfunktion, weil normalareweise bauen die Augaben aufeinander auf..

Gruß
Jan

 

[siriuswhite]

also das system soll so funktionieren:
Die erste Zahl sagt wieviele Verknüpfungen an den + dranhängen
Die zweite Zahl sagt wieviele ||| in dem Gitter sind
also wäre Gitter(4,3)

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

oder sehe ich das falsch??

 

[Gast]

Das ist leider bischen blöd zu erkennen hier.. wenn man eine andere schriftart wählt dann sind die "|" direkt unter dem "+" sodass sich ein richtiges gitter ergibt.....
gitter(5,2) ergibt z.b.
+-+-+-+-+-+
| | | | | |
+-+-+-+-+-+
| | | | | |
+-+-+-+-+-+

also die erste zahl gibt an, wieviele "quadrate" das gitter hat.. also ein quadrat ist:
+-+
| |
+-+
und die zweite zahl gibt an, viele zeilen das gitter hat...

 

[SlaterB]

poste deine gitter in
[ code] ...... [/ code] (ohne Leerzeichen) dann sieht man sie besser

tja, du musst dabei arbeiten, Schritt für Schritt das Bild aufmalen, hängt ja alles von den Parametern x und y ab,

pürfe evtl. ob einer der Parameter 0 ist und wirf ne Exception oder überlege was dann auszugeben ist,
gehen wir mal von > 0 aus:

als erstes kommt logischerweise die oberste Zeile, die braucht man immer, x ist egal, Anzahl Spalten ist dagegen wichtig,
schreibe zuerst ein '+' und dann pro y einmal '-+', also mit einer Schleife über y,
was war daran nun schwer? nix
genau mit solchen einfachen Schritten kommst du nun Zeile um Zeile zum Ziel,

 

[Guest]

Danke für die Antwort...
ich habe es jetzt hin bekommen:

 static public String gitter(int eingabe1, int eingabe2){
      String gitterString = "+";
      for(int i=0; i < eingabe1; i++){
	  		  gitterString = gitterString + "-+";
		  }
      for(int e=0; e < eingabe2; e++){
		  gitterString = gitterString + "\n";
		  gitterString = gitterString + "| ";
		  for(int i=0; i < eingabe1; i++){
			  gitterString = gitterString + "| ";
		  }
		  gitterString = gitterString + "\n";
		  gitterString = gitterString + "+";
		  for(int i=0; i < eingabe1; i++){
			  gitterString = gitterString + "-+";
		  }
	 }
      return gitterString;
  }

ist die funktion gut geschrieben, oder habe ich das was verkompliziert mit den vier for schleifen?
Gruß

 

[Guest]

Habe noch vergessen die negativen zahlen auszugrenzen mit:

eingabe1 = Math.max(eingabe1, 0);
eingabe2 = Math.max(eingabe2, 0);

 

[SlaterB]

ich würde gitterString in solchen übersichtlichen Operation g nennen,
die sich daraus ergebende Kürze bringt bestimmt mehr Vorteile als der Nachteil der einen schlecht benannten Variable

den Block

gitterString = gitterString + "+";
for(int i=0; i < eingabe1; i++){
gitterString = gitterString + "-+";
}

hast du zweimal im Code, den könntest du durchaus in eine Operation auslagern (getHauptlinie(y))

außerdem nur einmal in einer Variablen speichern und dann diesen fertigen String wiederholt anbauen, statt jedesmal Buchstabe für Buchstabe neuaufzubauen

wenn du dann noch die andere Zeile auch in eine Unteroperation versteckst, dann hast du einen griffigen Hauptcode der Form:

String zeileA = ..;
String zeileB = ..;
g = zeileA;
for (...) {
g += zeileB;
g += zeileA;
}

und die Übersicht gewinnt


oder gar

String zeileA = ..;
String zeileB = ..;
String zeileAundB = zeileB + zeileA;
g = zeileA;
for (...) {
g += zeileAundB;
}


sind aber nur Details, im Grunde tuts deine Version auch