Zufälliges Alphabet erzeugen

[StrikeTom]

Hi Leute,
ich stehe momentan echt auf dem Schlauch:bahnhof::
Mein Ziel ist es einen String zu erzeugen, welcher jeden Buchstaben des Alphabets einmal
in einer zufälligen reinfolge enthällt.
Das klapt auch fast, nur enthällt der String, welchen meine Funktione zurückgibt kein "z"
Mein Code:

public String randAlpha(){
		char[] letter = {'a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z'};
		ArrayList<Character> letterlist = new ArrayList<Character>();
		for(int i = 0; i < letter.length; i++){
			letterlist.add(letter[i]);
		}
		String alphas = "";
		for(int i = 0, j = letter.length; i < letter.length; i++, j--){
			int rnd = (int)(Math.random()*j);
			alphas += letterlist.get(rnd);
			letterlist.remove(rnd);
		}
		return alphas;
	}

Weiß jemand, wo der Fehler liegt?
Danke im Vorraus😉

 

[Deros]

hab es mal getestet und bei mir ist immer ein z drin, aber der code ist weder effektiv noch ist das Ergebnis wirklich zufällig

 

[Thorakas]

Abend.

Ich habe den Code mehrfach durchlaufen lassen und konnte immer alle Buchstaben sehen. Du wirst ihn wohl einfach nur übersehen haben 😉

 

[pl4gu33]

nur nen kleiner Tipp du kannst das auch so mixen:

Collections.shuffle(letterlist);
String rueck="";
for(int i=0;i<letterlist.size();i++){
      rueck+=letterlist.get(i);
}

 

[AquaBall]

Was machst du mit deinem 'i' ?

for(int j = letter.length; j>0; j--){
	int rnd = (int)(Math.random()*j);
	alphas += letterlist.get(rnd);
	letterlist.remove(rnd);
}

 

[StrikeTom]

Was machst du mit deinem 'i' ?

Danke für den Hinweis 🙄
Ich hab mich wohl wirklich vertan. Trotzdem danke 😀
Gesendet von meinem Liquid

 

[faetzminator]

Etwas einfacheres als folgendes Beispiel fällt mir auf die Schnelle auch nicht ein.

public String randAlpha() {
    List<Character> chars = new ArrayList<Character>();
    for (char c = 'a'; c <= 'z'; c++) {
        chars.add(c);
    }
    Collections.shuffle(chars);
    StringBuilder sb = new StringBuilder();
    for (char c : chars) {
        sb.append(c);
    }
    return sb.toString();
}

Falls du das Array beibehalten willst, funktioniert auch:

public String randAlpha() {
    List<Character> chars = Arrays.asList('a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l',
            'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z');
    // ...
}