char array-positionstausch

[canyakan95]

Hallo
ich habe ein char array, und wollte da 2 buchstaben miteinander vertauschen.
nur wird bei meinem tausch, jedesmal was weggelassen.
z.b bei {'n','a','b','e','r'} soll {'e','a','b','n','r'} rauskommen.
Mein code:

    public static String positionsTausch(char[]str,char x)
    {
        String s="";
        char temp;
        for(int i=0;i<str.length;i++)
        {
            if(str[i]==x)
            {
               
                temp=str[i];
                str[i]=str[0];
                str[0]=temp;
            }
            s+=str[i];
        }
        return s;
    }

 

[Bitfehler]

Beispiel:
Ein Zeichen aus der Mitte soll getauscht werden. Dann ist irgendwann die if-Bedingung erfüllt und das Zeichen landet nach dem Dreieckstausch im Index 0. Das durch den Tausch nach vorne geschobene Zeichen fügst du aber nicht deinem String s zu, stattdessen wird wieder das Zeichen der ehemaligen Position 0 in den String aufgenommen. Somit taucht ein Zeichen gar nicht auf und ein anderes doppelt. (Ich habe dein Programm nicht ausgeführt, aber das sollte dein Problem sein. Entspricht das beschriebene deinem Fehlerbild?)

 

[canyakan95]

Genau das ist es .könntest du mir vllt helfen und sagen, was ich an meinem code verbessern sollte.
Mfg

 

[Bitfehler]

Naja, du könntest aus deinem if ein if-else machen.
Innerhalb es if nimmst du dann das vorschobene Zeichen in den String auf und im else-Zweig, das was aktuell außerhalb des if steht.

Das erfordert nicht allzu viele Veränderungen an deinem Code. Musst mal ausprobieren.

 

[DrZoidberg]

Ändere die return Zeile so um

return new String(str);

Die Variable s kannst du dann ganz weg lassen.

 

[canyakan95]

Hey habe s ganz weggelassen und so hats auch geklappt mit return new String(str);
wie siehts denn aus ,wenn ich z.b sowas habe (public static int[] pos2(int[] arr,int x))
wie kann ich das hier machen ?? return new int[]{arr}??
Wäre auch meine letzte frage.
mfg

 

[DrZoidberg]

Du kannst ein Array kopieren mittels java.util.Arrays.copyOf(arr, arr.length);
Wobei es wahrscheinlich besser ist das Array am Anfang der Methode zu kopieren anstatt am Ende, da du sonst ja auch das Original Array veränderst.