OOP Übergeben von Arrays per Konstruktor

[Vampy]

Hallo,

ich versuche eine Matrix per Konstruktor an die Klasse zu übergeben. (Hoffe, ich habe mich korrekt ausgedrückt.)

Zu meinen Problemen:

Geht das, was ich gemacht habe? Ist das 'logisch'?
Wenn ja, wie kann ich es machen?
Ist der Konstruktor sinnvoll? Wenn ja, was genau sollte darein?

Im Moment habe ich als Ausgabe die Referenzen des Arrays als Matrix! :/

[JAVA=42] public class testdriver {

public static void main(String[] args) {

int[][] test = new int[][] {

{ 4, 0, 0, 0, 0, 0, 0, 0, 0 },
{ 5, 0, 0, 0, 0, 0, 0, 0, 0 },
{ 0, 0, 0, 0, 0, 0, 0, 0, 0 },
{ 0, 0, 0, 0, 1, 0, 0, 0, 0 },
{ 0, 0, 0, 0, 1, 0, 0, 0, 0 },
{ 0, 0, 0, 0, 1, 0, 0, 0, 0 },
{ 0, 0, 0, 0, 0, 0, 0, 0, 0 },
{ 0, 0, 0, 0, 0, 0, 0, 0, 0 },
{ 0, 0, 0, 0, 0, 0, 0, 0, 0 },

};

Solve sudoku = new Solve(test);

sudoku.print();
}

}
[/code]


Hier die Klasse dazu:

[JAVA=42]public class Solve {

private static int[][] sudoku;

public Solve(int[][] matrix) {

sudoku = matrix.clone();

}

public void print() {
print(sudoku);
}

public static void print(int[][] matrix) {

for (int zeile = 0; zeile < matrix.length; zeile++) {
for (int spalte = 0; spalte < matrix[zeile].length; spalte++) {

System.out.print(matrix[spalte] + " ");

}
System.out.println();
}

}
}[/code]


Sieht noch sehr anfänglich aus, ich will einfach erstmal hinkriegen, dass ich die AnfangsMatrix übergebe und damit arbeiten kann.

 

[XHelp]

Ich verstehe nicht so ganz die Frage. Es ist natürlich legitim Werte, ohne die eine Instanz keinen Sinn macht, im Konstruktor zu übergeben.
Zu der Clone-Methode: das wird nicht reichen, da es ein 2D Array ist. Es sollte eher (ungetestet) sowas wie:

sudoku = new int[matrix.length][];
for (int i=0;i<sudoku.length;i++) {
  sudoku[i] = matrix[i].clone();
}

sein. Oder statt dem innerem

clone

System.arraycopy benutzen

 

[Vampy]

Erstmal vielen Dank für deine Antwort.

Zu meiner Frage:

Ich habe im Testdriver die gegebene Matrix, die möchte ich per Konstruktor an die Klasse übergeben und dann damit arbeiten. Ist sowas sinnvoll? Wenn nein, wie könnte es anders gehen?

 

[pro2]

Kann man so machen, aber ich frage mich, warum alles static ist?^^ Wenn du das static willst, dann kannst du dir auch direkt eine Methode zum Ausgeben eines Arrays schreiben und die dann static machen. Dann brauchst du nicht extra ein Objekt anlegen!

 

[Vampy]

Hm - ist doch nur die 'Print-Methode' static, die wird dann an durch die 'Nicht-static-Methode' print aufgerufen.

Ist es sinnvoller ohne Objekte zu arbeiten? Ich glaube nicht, oder täusch ich mich? :/

 

[pro2]

class Hilfsmittel{
    public static void printArray(int[][] array){
        //mach was
    }
}

Und in der anderen Klasse dann einfach:

Hilfsmittel.printArray(test);

 

[Vampy]

Hm, das funktioniert immernoch nicht so richtig.

Ich steh total auf dem Schlauch, um mal ehrlich zu sein.


Nochmal zur Verständnis:


Objekt erstellen, Matrix per Konstruktor an die Klasse übergeben, Matrix per 'Print-Methode' ausgeben und die Methode in der TestdriverKlasse aufrufen.

Problem:

Es werden nur die Adressen z.B. [I@71e2b67c sowas ausgegben (Das als Matrix!) :/

 

[TiJay]

Hallo,

Du schreibst:

public static void print(int[][] matrix) {
 
        for (int zeile = 0; zeile < matrix.length; zeile++) {
            for (int spalte = 0; spalte < matrix[zeile].length; spalte++) {
                
                System.out.print(matrix[spalte] + " ");
                
            }
            System.out.println();
        }
 
    }

Logisch, dass es nicht klappt, denn mit

System.out.print(matrix[spalte] + " ");

gibst du ja ein Array aus und nicht das Element selbst.

Es müsste vielmehr heißen

System.out.print(matrix[zeile][spalte] + " ");

oder nicht?

Diese komischen Zeichen erhältst du, weil die Klasse Array (jedes Array, auch ein int-Array erbt von Array) die Object.toString()-Methode nicht überschreibt und beim Ausgeben eines Arrays immer der Wert ausgegeben wird, den die toString()-Methode zurückgibt.

Gruß,
Timo

 

[Vampy]

Oh mein Gott!! Ich hab zwar nicht viel Ahnung von Java (Glaub ich), aber auf sowas hätte ich selber kommen müssen. Man wird halt immer blinder, wenn man einfach drauf los programmiert und dann denkt, die 'einfachen' Sachen sind richtig.

Vielen, vielen Dank!!


PS: Wie close ich das hier?

 

[TiJay]

Oben und unten ist ein Button "Thema als 'erledigt' markieren".

Bitteschön ;-)

 

[Vampy]

Kann den Button nicht anklicken, trotzdem hab ich eine weitere Frage bezüglich OOP!

Ist es sinnvoll, was ich da geschrieben habe?

Solve sudoku = new Solve(test);

		sudoku.print(test);

Wenn ich jetzt die Matrix verändern würde, würde sich die anfängliche Matrix ändern? (So sollte es gedacht sein!)

Weil beim 'print' übergebe ich ja die anfängliche Matrix...

 

[TiJay]

Naja, so ist die erste Zeile quasi überflüssig.

Versuchs mal so:

Solve.print(test);

oder so:

Solve sudoku = new Solve(test);
sudoku.print();

Bei deiner Variante legst du ja zuerst ein neues Objekt an, was später aber gar nicht mehr verwendet wird. Du brauchst aber auch keine Instanz, um print(test) aufzurufen, weil das eine static-Methode ist und auch mit dem Klassennamen davor funktioniert.

Bei deiner Variante und bei meiner ersten Variante kannst du die Matrix ändern und sie wird verändert ausgegeben.

Bei der zweiten Variante kannst du die Matrix von außerhalb nicht mehr ändern, das müsste dann in der Klasse Solve geschehen.

Gruß,
Timo

 

[Vampy]

Bei der zweiten Variante meckert er, weil nichts übergeben wird. Wenn ich 'test' übergebe, passt alles.

Die Methode 'print' ist nicht auf static gesetzt.

Ich will die Matrix auch NUR in der Klasse Solve ändern, sonst nichts.

 

[xehpuk]

Bei der zweiten Variante meckert er, weil nichts übergeben wird.

Entfern den Parameter von

print()

und nutze in ihr die Instanzvariable.

 

[Vampy]

Sorry, aber kannst du mir den zweiten Teil nochmal genauer erklären? Was genau meinst du damit?

Ich glaub ich sollte mir OOP nochmal genauer angucken. Besonders die Begrifflichkeiten und wie sie zusammenhängen. :/

 

[xehpuk]

Pro Solve-Instanz hat man ein Sudoku-Array. Dieses ist also immer an eine Instanz gebunden, also ist es deren Aufgabe, dieses auszugeben. Deswegen braucht man hier kein

static

.
Möchte man eine individuelle textuelle Repräsentation haben, so überschreibt man normalerweise die

toString()

, die jede Klasse von

Object

erbt.

Letztendlich könnte deine Klasse (fürs Erste) so aussehen:

import java.util.Arrays;

public class Solve {
	private final int[][] sudoku;
	
	public Solve(final int[][] matrix) {
		final int length = matrix.length;
		sudoku = new int[length][];
		for (int i = 0; i < length; i++) {
			sudoku[i] = matrix[i].clone();
		}
	}
	
	@Override
	public String toString() {
		final StringBuilder sb = new StringBuilder();
		final int length = sudoku.length;
		if (length != 0) {
			sb.append(Arrays.toString(sudoku[0]));
			for (int i = 1; i < length; i++) {
				sb.append('\n');
				sb.append(Arrays.toString(sudoku[i]));
			}
		}
		return sb.toString();
	}
	
	public void print() {
		System.out.println(this); // in dieser Methode wird this.toString() aufgerufen
		// System.out.println(this.toString()) geht auch alternativ
	}
	
	public static void main(final String[] args) {
		final int[][] test = new int[][] {
			{ 4, 0, 0, 0, 0, 0, 0, 0, 0 },
			{ 5, 0, 0, 0, 0, 0, 0, 0, 0 },
			{ 0, 0, 0, 0, 0, 0, 0, 0, 0 }, 
			{ 0, 0, 0, 0, 1, 0, 0, 0, 0 },
			{ 0, 0, 0, 0, 1, 0, 0, 0, 0 }, 
			{ 0, 0, 0, 0, 1, 0, 0, 0, 0 },
			{ 0, 0, 0, 0, 0, 0, 0, 0, 0 }, 
			{ 0, 0, 0, 0, 0, 0, 0, 0, 0 },
			{ 0, 0, 0, 0, 0, 0, 0, 0, 0 },
		};
		final Solve solve = new Solve(test);
		solve.print();
	}
}

 

[Vampy]

Hat sich erledigt, ich werde es ohne Objekte machen. Sonst ist mir das zu anstrengend. Bis jetzt funktioniert es wunderbar.

Vielen Dank für alle Antworten.

Kann man das hier löschen/closen? Ich kann's nicht als erledigt markieren.