Java Array Basic

[spiritofaustria]

Hallo!

Ich muss eine Array Klasse mit ein paar Methoden schreiben und es happert ziemlich stark beim Code.
Hab natürlich schon selber versucht, es zu lösen, aber es kommen immer nur kryptische ZahlenBuchstaben - Kombinationen nach dem Erzeugen eines Random Arrays heraus. Auch die anderen Methoden scheinen nicht zu funktionieren...

Kann mir da wer weiterhelfen?

Ich poste mal die Aufgabenstellung und meinen Code:

(Auskommentierte Passagen, haben zu Problemen geführt)


Definieren Sie mit Hilfe einer Klasse einen abstrakten Datentyp IntArray,
der intern als Array von int-Werten implementiert ist, und neben den
Standardmethoden (Konstruktoren, toString, equals, clone, setXxx, getXxx, etc.)
auch jeweils eine Methode für die Bestimmung

a2) des groessten Wertes in a
b2) des zweitgroessten Wertes in a (ist definiert als der groesste Wert,
der kleiner als das Maximum ist)
c2) des Wertes, der am haeufigsten vorkommt

enthaelt.



1. Die Klasse Array2

import java.util.Random;


public class Array2 {

	int i = 0;
	int [] a;
	private Array b;
	
	
	public Array2(int i){
	
		int[] a = new int [i];
		Random generator = new Random();
		
		while (i < a.length){
			a [i] = generator.nextInt (100);
			i++;	
		}
	}
	
	public String toString (int [] a ){
		for (int i = 0; i < a.length; i++) {
			int Element = a [i];
			return "Element: " + Element;
		}
		return "Der derzeitig gespeicherte Array ist: ";
	}
	
	
	public boolean equals (int [] a, int [] b){
				int bo = 0;
				for (int i = 0; i < a.length; i++) {
				if (a [i] == b [i]) bo = 1; 
				} 
			return (bo==1);
		}
	
	
	public int[] clone (int [] a){
		int i = a.length;
		int [] b = new int [i];
		while (i < a.length){
			a[i]=b[i];
			i ++;
		}
		return b;
	}
	
		
	public void setArray (int[] c){
		 int [] a = c;
	}
	
/*	public int getArray (){
		if (a.length == 1) return a [0];
		for (int i = 0; i < a.length; i ++) {
			return a [i];
		}
	}
*/	
	
	public int max (int [] a){
		int i = 0;
		int max = 0;
		while (i < a.length){
			if (a[i] < a[i+1]) max = a[i+1];	
			if (a[i] > a [i+1]) max = a[i];
			if (a[i] == a[i+1]) max = a[i];
			i++;
		}
		return max;
	}
		
		public int max2 (int [] a){
			int i = 0;
			int max = 0;
			int max2 = 0;
			while (i < a.length){
				/*if (a [i + 1] > a [i]) max = a[i + 1];
				if (a[i] > a [i+1]) max = a[i];
				if (a[i] == a[i+1]) max = a[i];
				if (max > a [i+1] & a [i+1] > a [i]) max2 = a[i+1];*/
				if(a[i]>max2 && a[i]!=max) {a[i]=max2;}
				i++;
			}
			return max2; //return von zweitgroeßtem Wert;	/* 

			
	}
	
/*		public int most (int []a){
			int z = 0;
			for (int i = 0; i < a.length; i ++){
				if (a[i] == a [i+1]) z++; //benachbarte Elemente werden verglichen
				
				int l = 1;
				while (l < a.length){
					if (a [i] == a [l]) z++; // erstes wird mit allen Elementen verglichen
					l ++;
				}
				
				/*for (int i,l = 1; l < a.length;  l ++)
				if (a [i] == a [l]) z++; // erstes wird mit allen Elementen verglichen
			}
			return z;
		}*/
}

2. Das Testprogramm (wsl. komplett chaotisch, weil ich verschiedenste Wege ausprobiert habe)

public class Test {

	
	public static void main(String[] args) {
		int[] a = new int [5]; // Array mit 5 Elementen
		
		a [0] = 5;
		a [1] = 7;
		a [2] = 1;
		a [3] = 20;
		a [4] = 15;
		
		System.out.println (a[0]);
		System.out.println (a[1]);
		System.out.println (a[2]);
		System.out.println (a[3]);
		System.out.println (a[4]);
				
		Array2 d = new Array2 (10);
		System.out.println (d);
		
		int [] c = new int [7];
		System.out.println (c);
		
		int[] b = a.clone();
		System.out.println (b [3]);
		
		
		
//		int max = d.max(c);
		int max2 = d.max2(c);
		System.out.println (max2);
//		System.out.println (max);
		
	//	a.setArray (d);
		System.out.println (d);
		
	//	d.getArray ();
		System.out.println (a);
		
		
		a.toString ();
		System.out.println (a[2]);
		
		c.toString ();
		System.out.println (c);
		
		
	//	int z = d.most (c);
	//	System.out.println (z);
		
		
		d.equals (a, b);
		
	}

	
}

Wäre wirklich super wenn mir jemand helfen könnte, ich weiß echt nicht mehr was ich noch probieren soll und bin schon total verwirrt.

Danke schon im Vorhinein,

Tamara

 

[fastjack]

* toString geht nicht, weil Du immer das erste Element zurückgibst und dann die Methode beendest (return ...)

* equals geht nicht, bo ist immer eins, wenn zwei Elemente gleich sind und kann sich nicht mehr ändern. Wenn Du einmal auf ungleiche Elemente stößt, kannst Du schon mit false beenden.

* clone geht nicht, weil du Elemente des vorhandenen Arrays mit denen des neuen (leeren) Arrays füllen willst und dann das neue leere Array b zurückgibst. Es wird schon Fehler bei der Zuweisung a=b hageln.

* max Das Maximum kannst Du viel leichter suchen. Mach eine Methodenvariable, die als maximum wirkt. Geh das Feld durch und vergleiche die Elemente mit diesem maximum. Ist das Element größer, setzt Du das maximum auf dieses Element. Am Ende gibst du das maximum zurück.

Mach für jeden Test eine eigene Methode. Ein return bricht die Methode ab, also aufpassen! Statt der nervenden while-Schleifen würde ich for-Schleifen nehmen.

 

[spiritofaustria]

Ok das mit dem Maximum hab ich verstanden, das geht wirklich so viel einfacher! Dankeschön =)

Aber bei der toString Methode muss ich das doch so machen, dass immer ein Wert pro "Durchlauf" ausgegeben wird, oder wie kann ich sonst das ganze Array ausgeben lassen? Geht das ohne for-Schleife? *noob*

Meinst du das mit clone und equals etwa so?

public boolean equals (int [] a, int [] b){
				int bo = 0;
				for (int i = 0; i < a.length; i++) {
				if (a [i] != b [i]) return (bo == 0); 
				} 
			return (bo==1);
		}
	
	
	public int[] clone (int [] a){
		int i = a.length;
		int [] b = new int [i];
		Random generator = new Random();
		
		while (i < a.length){
			b [i] = generator.nextInt (100);
			i++;	
		}	
		while (i < a.length){
			a[i] = b[i];
			i ++;
		}
		return b;
	}

Und weißt du was mein Konstruktor hat, weil der erzeugt ja immer nur so kryptische Dinge?

Lg =)

 

[Volvagia]

Mit return wird gar nichts ausgegeben, sondern 1 Wert zurückgegeben, und danach wird die Methode beendet. Hat er doch schon geschrieben.
System.out.println(Obj.) ruft einfach nur die toString-Methode der Klasse auf, die in der Standartversion den Hashwert und den Klassennamen ausgibt.

 

[fastjack]

Der Konstruktor erzeugt Zufallswerte, mehr nicht.

toString kannst Du so machen:

public String toString(int[] a ) {
    StringBuilder s = new StringBuilder();
    for (int i = 0; i < a.length; i++) {
        s.append("Element: ");
        s.append(a[i]);
        if (i < a.length - 1) {
            s.append(", ");
        }
    }
    return s.toString();
}

// noch einfacher

public String toString(int[] a ) {
    return Arrays.asList(a).toString();
}

// equals

public boolean equals (int[] a, int[] b) {
    for (int i = 0; i < a.length; i++) {
        if (a[i] != b[i]) {
            return false; // wenn ein Element falsch ist, kannst Du abbrechen
        } 
    }
    return true; // wenn alles gleich ist, gibst Du true zurück
}
    
// clone

public int[] clone (int[] a) {
    int[] b = new int[a.length];
    for (int i = 0; i < this.a.length; i++) {
        b[i] = a[i];
    }
    return b;
}

Ungefähr s, Du mußt eventuell auch abchecken, wenn jemand ungültige Eingabeparameter in die Methoden gibt, z.B. null.

edit: Tamara ist ein sehr schöner Name.

 

[spiritofaustria]

Dankeschön an euch beide =)
Ich werd mal weiterbasteln und schaun obs dann funktioniert

@fastjack: Danke für das Kompliment :applaus: