Methoden in einer array werte vergleichen

[toblo]

wie kann ich in einer array alle werte auf gemeinsamkeit überprüfen?
also z.b array {1,2,3,4,3}
vergleiche
1 mit 2
1 mit 3
1 mit 4
1 mit 3
2 mit 3
2 mit 4
2 mit 3
...
falls die ziffern gleich sind, soll er eine ändern

    for (int i = 0; i<reihe.length;i++){
      for(int j = 0; j<reihe.length;j++){
        if(reihe[i]==reihe[j]){
          reihe[i]= (int) (Math.random()*50);
        }
      }
    }

iwie kommen immer noch gleiche zahlen

 

[pro2]

Gleiche Zahlen kommen immer noch vor, weil es ja passieren kann, dass Math.Random() zwei mal die gleiche Zahl setzt!

 

[jgh]

so mal ein schnellschuss...funktioniert nicht das?

 for (int i = 0; i<reihe.length;i++){
	      for(int j = 0; j<reihe.length;j++){
	        while(reihe[i]==reihe[j]){
	          reihe[i]= (int) (Math.random()*50);
	        }
	      }
	    }

[edit]wobei das mit der doppelten for-schleife muss man doch auch eleganter lösen können...mmmg, fussball geht los[/edit]

 

[pro2]

so mal ein schnellschuss...funktioniert nicht das?

 for (int i = 0; i<reihe.length;i++){
	      for(int j = 0; j<reihe.length;j++){
	        while(reihe[i]==reihe[j]){
	          reihe[i]= (int) (Math.random()*50);
	        }
	      }
	    }

[edit]wobei das mit der doppelten for-schleife muss man doch auch eleganter lösen können...mmmg, fussball geht los[/edit]

Das ist eine glatte Endlosschleife. Da er ja mit j und i = 0 anfängt, wird die erste Überprüfung auf jeden Fall gleich sein. Und dementsprechend auch alle folgenden Zufallszahlen

@TO Such mal im Forum, es gab die Frage schon des öfteren. Finde leider auf Anhieb nichts.
Du könntest http://www.java-forum.org/java-basics-anfaenger-themen/105767-java-random.html benutzen, da dein Code hier auch eh eine Zufallszahl für jedes Feld generiert, warum s.o.

 

[jgh]

ok, als Schnellschuss vorbei, aber es fehlt eigentlich nur ein

if (i == j)continue;

		for (int i = 0; i < reihe.length; i++) {
			for (int j = 0; j < reihe.length; j++) {
				if (i == j)
					continue;
				while (reihe[i] == reihe[j]) {
					reihe[i] = (int) (Math.random() * 50);
				}
			}
		}

dann klappt das auch...

 

[TKausL]

ok, als Schnellschuss vorbei, aber es fehlt eigentlich nur ein

if (i == j)continue;

		for (int i = 0; i < reihe.length; i++) {
			for (int j = 0; j < reihe.length; j++) {
				if (i == j)
					continue;
				while (reihe[i] == reihe[j]) {
					reihe[i] = (int) (Math.random() * 50);
				}
			}
		}

dann klappt das auch...

Wofür alles doppelt durchgehn?

		for (int i = 0; i < reihe.length; i++) {
			for (int j = i + 1; j < reihe.length; j++) {
				while (reihe[i] == reihe[j]) {
					reihe[j] = (int) (Math.random() * 50);
				}
			}
		}

 

[toblo]

alles klar vielen dank!!