array

[JAVAC1]

hallo leute,

kann mir von euch jemand sagen, was an meinem programm falsch ist:
ich will, dass am ende dasteht, wie oft die 2 in meinem array vorgekommen ist.
hier erstmal meine klasse mit der methode, anschließend eine klasse mit main()-methode und einem array, auf das die methode aus gucker angewendet werden soll.

class gucker2 {
	public int gucker2 (int[] array, int a) {
		int result=0;
		for(int i=0; i<array.length; i++) {
			if(array[i]==a) {
				result++;
			}
		}
		return result;
	}
}


class guckertester {					
	public static void main(String[]args) {
		int [] arr = {1,2,2,2,2,3,4,5};
		System.out.println(arr.gucker2(arr,2);
	}
}

der fehler ist irgendwo im methodenaufruf in der klasse guckertester.
vielen dank für eure hilfe!

 

[Gast2]

arr.gucker2(arr,2)

arr ist ein int[] Array, das kennt die Methode gucker2 nicht. Du brauchst dafür ein Objekt der Klasse gucker2.

BTW:
Die Namenswahl der Klasse und Methode ist ziemlich schlecht...
Klassen schreibt man groß.

 

[eRaaaa]

Du versuchst die Methode gucker2 auf einem Array-Objekt aufzurufen, das geht natürlich nicht. Entweder du brauchst ein Objekt deiner gucker2-Klasse (schlechte Namenswahl) oder du machst die Methode statisch, dann kannst du die Methode mittels gucker2.gucker2(...) aufrufen (jetzt siehst du wieso schlechte Namenswahl )

 

[JAVAC1]

tut mir leid,
ich habe jetzt das gucker2 objekt konstruiert.(die klassennamen hab ich erstmal kleine gelassen, änder ich dann noch)

class guckertester {  //cls heißt konsole leeren
	public static void main(String[]args) {
		gucker2 [] arr = new gucker2[4];
		int [] arr = {1,2,2,2};
		System.out.println(arr.gucker2(arr,2));
	}
}

aber es haut nicht hin.könntet ihr mir sagen, wie der richtig code lautet?ich möchte, dass es einmal funktioniert.

[EDIT]Bitte

-Tags benutzen!
L-ectron-X[/EDIT]

 

[eRaaaa]

new gucker2[4]; = array
new gucker2(); = gucker2-Objekt
und du kannst auch nicht zwei Variablen den gleichen Namen arr geben ...

 

[CR7]

du hast es auch viel zu kompliziert "gebastelt".

Ich würde es so machen:

1. erstmal abfragen wie viele Zahlen der Benutzer eingeben möchte
2. dann ein int[] deinArray = new int[dieZahlDenDerBenutzerEingegebenHat]
3. dann dein array durchgehen und nach einer 2 abfragen, wenn er etwas gefunden hat dann deinen counter einen hochzählen.

dann hast du auch schon, oder habe ich dich jetzt falsch verstanden!?

 

[Pippl]

du hast es auch viel zu kompliziert "gebastelt".

Ich würde es so machen:

1. erstmal abfragen wie viele Zahlen der Benutzer eingeben möchte
2. dann ein int[] deinArray = new int[dieZahlDenDerBenutzerEingegebenHat]
3. dann dein array durchgehen und nach einer 2 abfragen, wenn er etwas gefunden hat dann deinen counter einen hochzählen.

dann hast du auch schon, oder habe ich dich jetzt falsch verstanden!?

Falsch verstanden (zumindest etwas). Woher seine Daten kommen ist ihm nicht wichtig. Grad mal 3 trifft zu, aber er hat doch schon die Methode zum durchgehen wie oft eine gewünschte zahl (in seinem beispiel 2) vorkommt. nur wie er die methode aufruft ist falsch.

eRaaaa hat hier schon weitergeholfen.
Erstellen eine Instanz der Klasse gucker2 und dann kannst du die Methode gucker2 davon aufrufen und dein array + die gesuchte zahl übergeben.
Den Klassennamen oder Methodennamen würde ich noch ändern

 

[2AndAHalfBit]

Hmmm,

ich weiss ja nicht ob ich es richtig verstanden habe, aber Du willst doch einfach nur wissen, wie oft ein Element in einem Array vorkommt? Ich habe mal was gebastelt (aufwand 2 Minuten) und würde es zur Diskussion stellen.

Aus der Klasse sollte man noch ein Singleton machen, aber ansonsten wäre sie auch nicht beschränkt auf Integerwerte:

public class GuckerStart {
	
	public static void main(String args[])
	{
		Integer[] someValues = {1,2,2,2,2,3,4,5};
		
		ArrayUtilities<Integer> arrayUtilities = new ArrayUtilities<Integer>(someValues);
		System.out.println(arrayUtilities.getElementQuantity(2));
		System.out.println(arrayUtilities.getElementQuantity(4));
	}
}

public class ArrayUtilities<T extends Comparable<T>> {
	private T[] array;
	
	public ArrayUtilities(T[] array)
	{
		this.array = array;
	}
	
	public int getElementQuantity(T elementToCompare)
	{
		int foundElements = 0;
		
		for(T currentElement : this.array)
			if(currentElement.compareTo(elementToCompare) == 0)
				foundElements++;
				
		return foundElements;
	}
}

Eigentlich löst das die Aufgabe schon oder?

 

[HimBromBeere]

Aus der Klasse sollte man noch ein Singleton machen

Warum? Wo bitte ist der Sinn, wenn man nur eine einzige ArrayList bauen könnte?
Wie auch immer. Ob man unbedingt das Comparable-Interface verwenden muss, wag ich mal zu bezweifeln, da man ja eigtl. nur auf Gleichheit prüft (was man auch selbst über equals machen kann), aber wenn man das Ganze natürlich erweitern will auf "Suche alle Elemente kleiner 4" wäre das widerum eine Idee...

 

[2AndAHalfBit]

Hallo zusammen,

ja HimBromBeere hat recht. So wie ich die Klasse geschrieben habe, macht es wirklich keinen Sinn daraus ein Singleton zu machen, also alles gut.

Zum IComparable-Interface: Meine Lösung ist generisch, also nicht beschränkt auf Integerwerte. Man kann die Lösung ohne Änderung auch auf eigene Datentypen anwenden, sofern sie IComparable implementieren. Deshalb hier die Verwendung der compareTo-Methode.

An die Möglichkeit mit Equals zu arbeiten habe ich noch gar nicht gedacht - Aber ist eigentlich auch wayne...

@Threadersteller: Ist Dir damit geholfen?

Grüße

 

[snm]

Hallo JAVAC1,

ich glaube die meisten Beiträge hier haben dich eher verwirrt, als weitergebracht
Damit du dein Programm zum Laufen bringst, solltest du die Methode gucker2 im Ausgangsprogramm als statisch deklarieren.

public static int gucker2 (int[] array, int a) {
        int result=0;
        for(int i=0; i<array.length; i++) {
            if(array[i]==a) {
                result++;
            }
        }
        return result;
    }

Der Rest dieser Methode ist in Ordnung. Jetzt solltest du noch über die Bennenung der Klassen und Methoden nachdenken. Klassennamen sollten generell groß geschrieben werden, um Verwechslungen zu vermeiden. Man sollte sich solche Details angewöhnen, sonst kommt man früher oder später durcheinander.

class Gucker {
    public static int gucker2 (int[] array, int a) {
        int result=0;
        for(int i=0; i<array.length; i++) {
            if(array[i]==a) {
                result++;
            }
        }
        return result;
    }
}
 
 
class Guckertester {                    
    public static void main(String[]args) {
        int [] arr = {1,2,2,2,2,3,4,5};
        System.out.println(Gucker.gucker2(arr,2));
    }
}

Nun kannst du die Methode gucker2 aufrufen und dein Programm müsste funktionieren.

 

[jgh]

omg, der eine erzählt was von Comparable und Generics, der andere sagt mach die Methode statisch...unglaublich! Mal abgesehen von der sehr merkwürdigen Benamung, langt es einfach ein Objekt der Klasse [c]gucker2[/c] zu erstellen und auf diesem die Methode [c]gucker2(int[] aray, int a)[/c] aufzurufen...alles andere ist, na ja....

public static void main(String[] args) {
		int[] arr = { 1, 2, 2, 2, 2, 3, 4, 5 };
		gucker2 gucker2 = new gucker2();
		System.out.println(gucker2.gucker2(arr, 2));
	}