Problem mit Arrays ....

[LuluIndependent]

Ich habe ein ganz doofes Problem, dass mich mittlerweile seit Stunden quält ... Ich hoffe ihr könnt mir helfen .___.

package ub06;
public class hauptprogramm {

	
	public String[][] ausgeben(){
		String plaetze[][] = new String[9][6];
		plaetze[3][3] = "[X]";
		System.out.println(plaetze[3][3]);
		return plaetze;
	}
	
	public static void main(String[] args) {
		// TODO Auto-generated method stub 
		String plaetze[][] = new String[9][6];
		int i, j;
		for ( i = 0; i < plaetze.length; ++i) {
			for( j = 0;j < plaetze[i].length; ++j){
				if (j == 0){
					System.out.print("Reihe:" + i + "  ");
				}
				plaetze[i][j] = "[]";
				System.out.print("" + plaetze[i][j]);
			}
			plaetze[3][3]="X";
			System.out.println();
	}
}
}

Die Hauptintension war es eine Sitzkarte zu erstellen, daher die [] für jeden Sitzplatz ... nun wollte ich besetzte Plätze mit [X] markieren, aber warum wird der Platz [3][3] nicht überschrieben, egal wo ich es hinstecke....?

 

[rme]

Hallo

Keine Panik, das schaffen wir schon In Zeile 5 definierst du eine Methode "ausgeben", die du allerdings nie aufrufst. Alles, was du dort hineinschreibst, wird in deinem Programm also nicht passieren.

In Zeile 24 belegst du erstmals einen Platz - aber deine Schleife überschreibt in Zeile 21 jeden Platz mit "[]", bevor er ausgegeben wird.

Was war denn deine Intention? Was soll die Schleife in main machen?

 

[LuluIndependent]

Erst einmal großes Danke !

Ich weiß ... ursprünglich war die Methode dafür gedacht beliebige Plätze zu markieren, nachdem der Benutzer eine Eingabe getätigt hat, aber nachdem das nicht geklappt, hab ich das erst einmal rausgenommen

Ziel war es mittels der Schleifen erst einmal den kompletten Array auszugeben und jeden Stringslot mit [] zu belegen. Danach sollten die [] per ausgeben()-MEthode nach und nach durch [x] ersetzt werden, bis alle Plätze belegt sind. Die Ausgabe der [] funktioniert auch geordnet ganz gut, aber das "Überschreiben" nimmt er absolut nicht an.

 

[rme]

Doch doch, das Überschreiben in Zeile 24 nimmt er an - aber nach der Zeile, in der du überschreibst, geht die Schleife ja noch weiter.. und sie gelangt dann in Zeile 21, die das eben mit "X" überschriebene nun wieder mit "[]" überschreibt, bevor es ausgegeben wird. Deshalb siehst du von außen gar nicht, dass es zwischendurch mal "X" war

Vorschlag: Lass die Schleife erstmal in Ruhe das Array mit "[]" füllen.
Danach eine Anweisung, die einen Platz befüllt.
Dann eine weitere Schleife, die das Array ausgibt.

Wenn du bei deinem Ansatz bleiben willst, die gleiche Schleife zum Ausgeben und zum Initialisieren zu verwenden, dann solltest du mit einer if-Anweisung in Zeile 21 prüfen, ob i und j gerade 3 sind - wenn ja, wird "X" geschrieben und sonst wird (so wie jetzt) "[]" geschrieben.

 

[LuluIndependent]

Der Ansatz ist leider von der Uni so vorgegeben @.@

So, mittlerweile habe ich es mal an dem konkreten Beispiel von [³][³] hinbekommen! Danke erst einmal
Jetzt zwingt mich die Aufgabe noch, das Reservieren der Plätze in einer öffentlichen MEthode zu machen ... Folgenden Code besitze ich jetzt:

package ub06;
import javax.swing.JOptionPane;
public class hauptprogramm {

	
	public static String[][] eingeben(){
		int i, j;
		String plaetze[][] = new String[9][6];
		String wahl1 = JOptionPane.showInputDialog("Reihe?");
		String wahl2 = JOptionPane.showInputDialog("Platz?");
		int k = Integer.parseInt(wahl1);
		int l = Integer.parseInt(wahl2);
		if (k == i && l == j){
			plaetze[i][j] = "[X]";
			}
		return plaetze;
		}
	
	
	public static void main(String[] args) {
		// TODO Auto-generated method stub 
		String plaetze[][] = new String[9][6];
		int i, j;
		for ( i = 0; i < plaetze.length; ++i) {
			for( j = 0;j < plaetze[i].length; ++j){
				if (j == 0){
					System.out.print("Reihe:" + i + "  ");
				}
				plaetze[i][j] = "[]";
				plaetze[3][3]="[X]";
				eingabe();
				System.out.print("" + plaetze[i][j]);
			}
			System.out.println();
	}
}
}

Problem 1: Momentane Werte von i & j übertragen?
Problem 2: The method() is undefined for the type hauptprogramm

 

[rme]

Deine Methode hast du in Zeile 6 "eingeben" getauft, in Zeile 31 verlangst du aber mit "eingabe" nach ihren Diensten

Damit du die aktuellen Werte von i und j in der Methode kennst, kannst du Parameter beim Definieren der Methode in Zeile 6 verwenden:

    public static String[][] eingeben(int i, int j)

Das "int i, j;" am Anfang kannst du dann entfernen, da i und j nun von "außen" kommen. Damit sie das auch wirklich tun, musst du in Zeile 31 aber dafür sorgen, dass das auch passiert:

  eingeben(i, j);

Dadurch haben i und j dann innerhalb der Methode genau die Werte, die sie innerhab der Schleife gerade hatten.

Ich bin mir trotzdem nicht sicher, ob du die Aufgabe richtig interpretierst - denn der Rückgabetyp von eingeben deutet darauf hin, dass es eher so sein sollte, dass eingeben() mit einer Schleife das ganze Array füllt und in main dann nur eingeben() aufgerufen wird. Vielleicht gibt es dann noch eine Methode ausgeben(), die das ganze wieder ausgibt. Wenn du uns die Aufgabenstellung zeigst, können wir da besser helfen

 

[LuluIndependent]

Sowas in der Richtung ist es in der Tat, aber dies dann in die Methoden zu überführen, ist für mich schaffbar ... nur das akute Einschreiben quält so x_X Ich entschuldige mich auch für doofe Fehler, aber irgendwann will der Kopf einfach nit mehr

package ub06;
import javax.swing.JOptionPane;
public class hauptprogramm {

	
	public static String[][] eingeben(int i, int j){
		String plaetze[][] = new String[9][6];
			plaetze[i][j] = "[X]";
		return plaetze;
		}
	
	
	public static void main(String[] args) {
		// TODO Auto-generated method stub 
		String plaetze[][] = new String[9][6];
		int i,k,l, j;
		String wahl1 = JOptionPane.showInputDialog("Reihe?");
		String wahl2 = JOptionPane.showInputDialog("Platz?");
		k = Integer.parseInt(wahl1);
		l = Integer.parseInt(wahl2);
		for ( i = 0; i < plaetze.length; ++i) {
			for( j = 0;j < plaetze[i].length; ++j){
				if (j == 0){
					System.out.print("Reihe:" + i + "  ");
				}
				plaetze[i][j] = "[]";
				if (k == i && l ==j){
					eingeben(i, j);
				}
				plaetze[i][j] = "[]";
				System.out.print("" + plaetze[i][j]);
			}
			System.out.println();
	}
}
}

 

[rme]

Achso - das Problem ist jetzt, dass "eingeben" sich ein eigenes Array anlegt, das mit dem aus dem Hauptprogramm nichts zu tun hat. Deshalb werden da auch insbesondere keine Plätze reserviert, sondern in dem anderen, das nach dem Verlassen der Methode stirbt :/

Du könntest es so machen, dass "plaetze" auch an die Methode übergeben wird, so wie du es bei i und j gemacht hast. Dann bekommt die Methode 3 Parameter und besteht nur noch aus der momentanen Zeilen 8 und 9.

 

[LuluIndependent]

Na eeeeeeendlich. Großes Danke! Hier mal als Zwischenergebnis:

package ub06;
import javax.swing.JOptionPane;
public class hauptprogramm {

	public static int einlese1(){
		int k;
		String wahl1;
		wahl1 = JOptionPane.showInputDialog("Reihe?");
		k = Integer.parseInt(wahl1);
		return k;
		}
	
	public static int einlese2(){
		int l;
		String wahl2;
		wahl2 = JOptionPane.showInputDialog("Reihe?");
		l = Integer.parseInt(wahl2);
		return l;
		}
	
	public static String[][] eingeben(int i, int j, String plaetze[][]){
			plaetze[i][j] = "[X]";
		return plaetze;
		}
	
	public static String[][] feld(){
		
		String plaetze[][] = new String[9][6];
		int i, j;
		einlese1();
		einlese2();
		for ( i = 0; i < plaetze.length; ++i) {
			for( j = 0;j < plaetze[i].length; ++j){
				if (j == 0){
					System.out.print("Reihe:" + i + "  ");
				}
				if (k == i && l ==j){
					eingeben(i, j, plaetze);
				}
				else{
				plaetze[i][j] = "[ ]";
				}
				System.out.print("" + plaetze[i][j]);
			}
			System.out.println();
	}
		return plaetze;
	}
	
	
	public static void main(String[] args) {
		// TODO Auto-generated method stub 
		feld();
}
}

Sollte es dennoch Verbesserungsvorschläge geben, kannst du die gerne äußern!
Und ich melde mich, wenn es nochmal Probleme gibt! >_<

 

[rme]

Na also, das sieht doch schon viel besser aus.. gut gemacht

Was mir noch auffällt: Deine Methoden geben alle Werte zurück, aber diese verwendest du nie. Also einlese1 fragt dich z.B. "Reihe?" und speichert das in einer lokalen Variable k, die zurückgegeben wird. Aber dieses k benutzt du nie, dadurch hat das k gar keinen Effekt. Wenn du das speichern möchtest, könntest du das z.B. so tun:

  int wahl = einlese1();

Danach landet das, was einlese1 eingelesen hat, in der Variable wahl, sodass du das Ergebnis verwenden kannst.

Weiterhin scheinen einlese1 und einlese2 genau das gleiche zu machen. Ich nehme an, dass deine Idee war: Ich brauche zwei verschiedene Reihen beim Einlesen, also brauche ich auch zwei Methoden. Das tolle beim Programmieren ist aber, dass du Dinge magisch wiederverwenden kannst Nämlich zum Beispiel so:

  int wahl = einlese1();
  int wahl2 = einlese1();

Dadurch wird der Code von einlese1 zweimal ausgeführt - fragt den Benutzer nach 2 Werten und speichert sie in verschiedenen Variablen im Hauptprogramm.

 

[rme]

Ich habe es mal angepasst .... Mir fehlt da ehrlich gesagt noch die Routine für ^^"

Das ist doch ganz normal, wir haben alle mal klein angefangen und fanden diese neue Welt seltsam und sadistisch

Gibt es eine Möglichkeit einen Befehlblock auszuführen, wenn ALLE Felder [j] = "[X]" sind?

Hm. In einer Schleife zählen, wie oft [X] vorkam und das nach der Schleife mit der Anzahl der Gesamtplätze vergleichen?

 

[LuluIndependent]

Wah, da bin ich vorhin echt im falschen Topic gelandet >_<

Danke für die Hilfe - Mir fehlt da ehrlich gesagt noch ein wenig die Routine ^^"

Gibt es eine Möglichkeit gewisse BEfehle auszuführen, wenn alle Slots == [X] sind?

 

[rme]

Siehe oben, ich hab trotz des falschen Topics frech hier geantwortet