den Inhalt eines Arrays umdrehen

[1207]

Hallo,
ich habe bereits eine Methode implementiert, die mir ein Array von char zurückliefert:

static char[] createCharArrayWithNames() {
	 
	    char[] name = new char[] {'C','i','n','j','a',';','S','a','v','a','n','a','h'};
	    return name;
	    }

nun soll ich in einer neuen Methode den Inhalt dieses Arrays umdrehen, ich habe da auch schon was zu geschrieben, aber leider zeigt mir meine Konsole immer 2 Fehler an und ich weiss nicht warum das Fehler sein sollen
Aber das hier ist meine Methode zum umdrehen:

static char[] reverse(char[] chars) {
	 
	     int m = chars.length-1;
		 char test = chars[0];
		 
		 for( int i=0; i < chars.length/2; i++)
		 
		    test = chars[i];
            chars[i] = chars[m];            //das ist meine Zeile 111
            chars[m] = test;
            m--;  

     return chars;
}

und das steht in meiner main-Methode:

public static void main(String[] args) {
      System.out.println(createCharArrayWithNames());
	  
	  System.out.println("reverse:" + reverse( name ));
	  
	  }

und das sind die Fehler die mir gesagt werden:
C:\Users\C_I_N_J_A\Desktop\blatt5.java:111: error: cannot find symbol
chars = chars[m];
^
symbol: variable i
location: class blatt5
C:\Users\C_I_N_J_A\Desktop\blatt5.java:155: error: cannot find symbol
System.out.println("reverse:" + reverse( name ));
^
symbol: variable name
location: class blatt5
2 errors
<<< Process finished. (Exit code 1)
================ READY ================

Also wenn ich das richtig verstehe, sagt der, dass ich keine variable i in der Zeile 111 habe, aber ich habe die doch erst in dem for-Schleifen Kopf bestimmt? Das verstehe ich nicht

 

[rme]

Hallo

Du bist ja heute spezial-fleißig, so viele Aufgaben auf einmal zu machen.. Hoffentlich hast du Spaß daran und quälst dich nicht nur hindurch.

Das Problem ist schnell behoben: So Anweisungen, die den Kontrollfluss des Programms regeln (if, while, for...), können nur eine Anweisung ausführen. Die Zeile, über die sich der Compiler beschwert, ist für den Compiler eine neue Anweisung, weshalb er "i" nicht mehr kennt, da das nur in der Anweisung der Schleife gültig war.

Abhilfe: Mit geschweiften Klammern kann man aus mehreren Anweisungen einen Anweisungsblock machen, der mehrere Anweisungen zu einer gruppiert. Also so:

for(...) {
  anweisung_1;
  ...
  anweisung_n;
}

Man sollte sich angewöhnen, diese Klammern immer zu setzen. Also auch, wenn man tatsächlich nur eine Anweisung braucht. Denn wenn man später doch mal mehrere benötigt und sie ergänzt, vergisst man die Klammern und es gibt Fehler.

 

[1207]

Ja, die Zettelabgabe rückt immer näher...

Also ich habe jetzt die Klammern um die Schleife gesetzt und juhu er kennt das i !
Aber das Array umdrehen will er immernoch nicht

static char[] reverse(char[] chars) dieses char[] chars sagt mir doch, dass ich dem ein Array von chars geben muss, was ich zumindest glaube zu tun, mache ich das doch in meiner main-Methode, da name ein char Array ist und bsp ja auch, aber warum gibt der mir dann immernoch den Fehler aus?

public static void main(String[] args) {
      System.out.println(createCharArrayWithNames());
	  
	  char[] bsp = new char[] {'H', 'a', 'l', 'l', 'o'};
	  System.out.println("reverse:" + reverse( bsp ));
	  System.out.println("reverse:" + reverse( name ));
	  
	  }

Das ist der Fehler:
C:\Users\C_I_N_J_A\Desktop\blatt5.java:150: error: cannot find symbol
System.out.println("reverse:" + reverse( name ));
^
symbol: variable name
location: class blatt5
1 error
<<< Process finished. (Exit code 1)

 

[rme]

Name ist kein char-Array, bsp schon. Wo legst du Name denn an? Das, was du innerhalb von createCharArrayWithNames anlegst, ist nach außen nicht sichtbar. Das hat man so gemacht, weil es sonst ja ein großes Chaos wäre, wenn alle Variablen überall sichtbar wären

Aber da die Methode das angelegte Array zurückgibst, könntest du es dir ja abspeichern, statt es nur auszugeben. So, wie du es bei der Aufgabe mit dem float-Array auch gemacht hast. Probier das mal und wenn es nicht klappt, melde dich einfach nochmal..

 

[1207]

So das hab ich jetzt gemacht aber es gibt nur so komische willkürliche Zeichen aus :'(

public static void main(String[] args) {
     char[] name = createCharArrayWithNames();
	     for(int i=0; i < name.length; i++) {
		 System.out.println(name[i]);
		 }
	  
	  char[] bsp = new char[] {'H', 'a', 'l', 'l', 'o'};
	  System.out.println("reverse:" + reverse( bsp ));
	  System.out.println("reverse:" + reverse( name ));
	  
	  }

Was ist denn jetzt schon wieder falsch, dass nur das raus kommt:

reverse:[C@723d7c
reverse:[C@22c95b

 

[rme]

Es kompiliert und gibt keine Exception zur Laufzeit, das ist doch schonmal ein gutes Zeichen Das Problem ist genau wie bei dem float-Array gestern: Du kannst Arrays nicht ohne Weiteres mit System.out.println() anzeigen, sondern musst (da ihr noch keine imports verwenden dürft) eine Schleife erstellen, die den Inhalt ausgibt. Also den Rückgabewert von reverse() abspeichern und dann ausgeben, genau wie bei den floats

Du könntest übrigens print statt println verwenden, damit die Zeichen dann alle in der gleichen Zeile landen.

 

[1207]

och das mit den schleifen in der main-Methode ist komisch

char[] name = createCharArrayWithNames();
	     for(int i=0; i < name.length; i++) {
		 System.out.print(name[i]);
		 }
	  
	  char[] bsp = new char[] {'H', 'a', 'l', 'l', 'o'};
	  
	  char[] wert = reverse( bsp );
	     for(int i=0; i < bsp.length; i++) {
		 System.out.print("reverse:" + reverse( bsp ));}
	  
	  char[] namen = reverse( bsp );
	     for(int i=0; i < name.length; i++) {
		 System.out.print("reverse:" + reverse( name ));}

und dann kommt sowas bei raus
Cinja;Savanahreverse:[C@19efb05reverse:[C@19efb05reverse:[C@19efb05reverse:[C@19efb05reverse:[C@19efb05reverse:[C@723d7creverse:[C@723d7creverse:[C@723d7creverse:[C@723d7creverse:[C@723d7creverse:[C@723d7creverse:[C@723d7creverse:[C@723d7crev

 

[rme]

Ich glaube, du hast heute schon ein bisschen zu viel Java gemacht Das ist normal, dass man irgendwann übersättigt ist und es immer schwieriger wird, dann Konzentration zu finden.

Deine Schleife besteht aus der gleichen Anweisung, die du vorher hattest: Der Versuch, das ganze Array auszugeben. Schau dir nochmal deine Lösung für die Aufgabe mit dem Anlegen des float-Arrays an. Die Schleife darf reverse gar nicht mehr aufrufen, das hast du vor der Schleife ja schon getan Du hast beim 1. Teil das Ergebnis in "wert" abgelegt, beim 2. Teil in "namen". Deshalb muss die Schleife einfach wert bzw. namen ausgeben.

 

[DrZoidberg]

Wieso gibt reverse das Array zurück? Das ist unnötig, da du ja kein neues Array erstellst.

 

[eMmiE]

Hi,
wenn dir das Programm bei Sysout´s sowas ausgibt, wie [@986149786, dann ist das der Pfad, den der PC nutzt um mithilfe von Java an die Datei, die man ja irgendwo auf der Festplatte oder dem RAM speichern muss, zu finden.

Das passiert meistens, wenn du so was wie Felder hast.
Bei primitiven Datentypen, wie int, long, u.Ä. hat er eine Methode toString() in der Klasse drin, wodurch dann der Wert ausgegeben wird und nicht der Pfad.

In deinem Fall ist es sicher hilfreich, sich eine Methode toString(char[] in) zu schreiben (siehe Spoiler)

Spoiler: toString()

public String toString(char[] in) {
  String rück = "[";
  for (int i = 0;i < in.length;i += 1) {
    rück += in[i]+",";
  }
  rück += "]";
  return rück;
}

Diese Methode gibt dir jetzt einen String zurück,der den Array repräsentiert.

 

[DrZoidberg]

In deinem Fall ist es sicher hilfreich, sich eine Methode toString(char[] in) zu schreiben (siehe Spoiler)

Oder er verwendet einfach java.util.Arrays.toString

 

[1207]

Juhu, ich habe es endlich auch mal geschafft

char[] name = createCharArrayWithNames();
	     for(int i=0; i < name.length; i++) {
		 System.out.print(name[i]);
		 }
	 
	 char[] namen = reverse( name );
	     for(int i=0; i < name.length; i++) {
		 System.out.print( namen[i] );}
	  
	  char[] bsp = new char[] {'H', 'a', 'l', 'l', 'o'};
	  
	  char[] wert = reverse( bsp );
	     for(int i=0; i < bsp.length; i++) {
		 System.out.print( wert[i] );}

Vielen Dank für die Hilfe!!!