arrays vergleichen

[java_anfänger(=]

Hallo, ich bin totaler Java Anfänger,
ich komme grad nicht weiter und zwar soll ich zwei arrays miteinander vergleichen (den Inahalt)

public class GrosseZahl {


byte[] ziffern;											// Objekvariable deklariert


public void setZiffern(byte[] ziffern){					//Setter-Methode
	this.ziffern = ziffern;
	}

byte[] getZiffern(){									//Getter-Methode
	return ziffern;
	}


public static boolean equals(byte[] ziffern, byte[] andereZahl)				
{		
		  if (ziffern == null && andereZahl == null){  
		    return true;  
		  }  
		  if (ziffern == null || andereZahl == null){  
		    return false;  
		  }  
		  if (ziffern.length != andereZahl.length){  
		    return false;  
		  }  
		  for (int i=0; i<ziffern.length; i++){  
		    if (ziffern[i] != andereZahl[i]){  
		      return false;  
		    }  
		  }  
		  return true;  
		}   
   
}



blic class GroesseZahlTester {

	public static void main(String[] args) {
		

		GrosseZahl zahl2 = new GrosseZahl("000003450004");
		GrosseZahl zahl3 = new GrosseZahl("000003450004");
		
		System.out.println(zahl2.equals(zahl3));

Ich bin mir ziemlich sicher, dass der Fehler in der Übergabe liegt, aber ich kriege es nicht richtig hin, hat jmd vielleicht eine idee?
Vielen Dank im Voraus für eure Hilfe

 

[ursteiner]

das sollte funktionieren:

public class GrosseZahl {

	private byte[] ziffern;                                         // Objekvariable deklariert

	public void setZiffern(byte[] ziffern) {                 // Setter-Methode
		this.ziffern = ziffern;
	}

	public GrosseZahl(String input) {
		this.ziffern = input.getBytes();
	}

	public byte[] getZiffern() {                                    // Getter-Methode
		return ziffern;
	}

	public boolean equals(GrosseZahl andereZahl) {
		if (ziffern == null && andereZahl.ziffern == null) {
			return true;
		}
		if (ziffern == null || andereZahl == null) {
			return false;
		}
		if (ziffern.length != andereZahl.ziffern.length) {
			return false;
		}
		for (int i = 0; i < ziffern.length; i++) {
			if (ziffern[i] != andereZahl.ziffern[i]) {
				return false;
			}
		}
		return true;
	}

	public static void main(String[] args) {

		GrosseZahl zahl2 = new GrosseZahl("000003450004");
		GrosseZahl zahl3 = new GrosseZahl("000003450004");

		System.out.println(zahl2.equals(zahl3));
	}
}

 

[java_anfänger(=]

Vielen Dank, es funktioniert auch alleine, aber wenn ich es in mein komplettes Programm schreibe funktioniert es nicht...

public class GrosseZahl {

byte[] ziffern;											// Objekvariable deklariert

public void setZiffern(byte[] ziffern){					//Setter-Methode
	this.ziffern = ziffern;
	}

public GrosseZahl(String input) {
	this.ziffern = input.getBytes();
	}

byte[] getZiffern(){									//Getter-Methode
	return ziffern;
	}
	   
	 

public GrosseZahl(String zahl) {
	zahl = zahl.replaceFirst("0*", "");						 // entfernt führende Nullen
	byte[] ziffern = new byte[zahl.length()];				
    for(int i = 0; i < zahl.length(); i++){ 					// Schleife zum druchlaufen der Array Felder
    	char c = zahl.charAt(i);
    	ziffern[i] = (byte) (c - '0'); 						//schreibt die einzelnen Zahlen jeweils in ein array Feld
    	this.ziffern=ziffern;
    	
    	}	      
    
 }


public String toString(){
StringBuilder s = new StringBuilder();
for(byte value: ziffern){
    s.append(value);										 // übergibt die Zahlen des Array in den String
}
return s.toString();								// übergibt den String "s" an die Methode "to.String" zurück
}


public static boolean equals(byte[] ziffern, byte[] andereZahl)				
{		
		  if (ziffern == null && andereZahl == null){  
		    return true;  
		  }  
		  if (ziffern == null || andereZahl == null){  
		    return false;  
		  }  
		  if (ziffern.length != andereZahl.length){  
		    return false;  
		  }  
		  for (int i=0; i<ziffern.length; i++){  
		    if (ziffern[i] != andereZahl[i]){  
		      return false;  
		    }  
	  }  
 return true;  
		}    
}





public class GroesseZahlTester {

	public static void main(String[] args) {
		
		GrosseZahl zahl1 = new GrosseZahl("00000345000467");
		System.out.println(zahl1);					
		
		
		//equals-Methode testen
		GrosseZahl zahl2 = new GrosseZahl("000003450004");
		GrosseZahl zahl3 = new GrosseZahl("000003450004");
		
		System.out.println(zahl2.equals(zahl3));

Wäre für Hilfe sehr dankbar =)

 

[knilch]

Hi,
Erst mal eine Anmerkung:
Du hast in der Klasse GrosseZahl den Konstruktor: public GrosseZahl(String zahl) {...} und public GrosseZahl(String input){...}.
Das geht so nicht. Beim Konstruktor (gilt auch für Methoden) dürfen die Argument Typen und Anzahl nie gleich sein:

public GrosseZahl(String input){...}

und

public GrosseZahl(String zahl) {...}

das darf nicht vorkommen!

public GrosseZahl(String input){...}

und

public GrosseZahl(String zahl, String zahl2) {...}

das geht.

Dann bei der main- Methode:
mit dem was du geschrieben hast, vergleichst du die Instanzen zahl2 und zahl3.

System.out.println(zahl2.equals(zahl3));

Dies ist nicht das selbe, als das was du wahrscheinlich möchtest. Wenn du einen Vergleich mit der equals Methode machten willst, du du geschrieben hast, dann musst du das so machen:

System.out.println(equals(zahl2.getZiffern(), zahl3.getZiffern()));

Denn zahl2.equals(zahl3) macht einen Vergleich der instanzen. Und dieser darf im Normalfall nicht true zurückgeben, ausser du überschreibst die equals Methode in der Klasse GrosseZahl (und auch hashCode)
Du hast in deiner Klasse GrosseZahl die Methode

public static boolean equals(byte[] ziffern, byte[] andereZahl) {...}

geschrieben. Diese vergleicht die Bytes von den Zahlen.
Diese ist aber nicht für den Vergleich von Instanzen anwendbar.
Dazu muss die equals- Methode ähnlich wie die von Ursteiner implemenitiert werden.

 

[java_anfänger(=]

okay danke für die Hilfe
Gibt es auch eine Möglichkeit sie zu vergleichen mit dem Aufruf den ich verwendet habe?

System.out.println(zahl2.equals(zahl3));

Wenn ja dann müsste ich meine Methode bestimmt auch veränderen oder?

 

[knilch]

Ja, das ist richtig,
Dann müsstest du equals überschreiben (so wie ursteiner es gezeigt hat)

	public boolean equals(GrosseZahl andereZahl) {
		if (ziffern == null && andereZahl.ziffern == null) {
			return true;
		}
		if (ziffern == null || andereZahl == null) {
			return false;
		}
		if (ziffern.length != andereZahl.ziffern.length) {
			return false;
		}
		for (int i = 0; i < ziffern.length; i++) {
			if (ziffern[i] != andereZahl.ziffern[i]) {
				return false;
			}
		}
		return true;
	}

(ich habs aber nicht überprüft...)
Damit funktioniert der Vergleich: zahl2.equals(zahl3)

 

[java_anfänger(=]

Juhuuu es klappt
Tausend Dank!!