Array statt String

[Tina87]

Hallo bin neu im Forum und Java-Anfänger.
Habe eine Hausaufgabe bekommen, die ich zwar glöst habe, aber nicht wirklich mit zufrieden bin.

Aufgabe war es alle Vokale in einem char durch "a" zuerstetzen.
Die Initialisierung war schon gegeben:

public class Array
{
public static void main(String args [ ])
{
char[] eingabe = {'D','r','e','i',' ','C','h','i','n','e','s','e','n'};
char ersatz = 'a';
char[] ausgabe = new char[eingabe.length];

hab das jetzt so gelöst:

 for (int i=0; i<=eingabe.length; i=i+1)
      {
          
         if (eingabe[2] == 'e')
         {
             eingabe[2] = ersatz;
            }
         if (eingabe[3] == 'i')
         {
             eingabe[3] = ersatz;
            }
         if (eingabe[7] == 'i')
         {
             eingabe[7] = ersatz;
            }
         if (eingabe[9] == 'e')
         {
             eingabe[9] = ersatz;
            }  
         if (eingabe[11] == 'e')
         {
             eingabe[11] = ersatz;
            }
              
        }
              
      
      System.out.println(eingabe);
      
    }
}

Da ich mit arrays nicht wirklich gut umgehen kann, fällt es mir schwer die Lösung einfacher zu machen. Problem ist auch, dass ich nicht das mit "char ausgabe = new char ..." benutzt habe, weil ich gar nicht weiß was ich damit anfangen soll.
Hoffe mir kann jemand helfen und das auch auf verständlicher Weise.
lg

 

[Spin]

huhu, es gibt viele möglichkeiten deine Hausaufgaben zu lösen ,)

Aber ich denke es soll ziemlich einfach sein und euren Kenntnissen genügen.
Daher die Frage ob du schonmal mit :

1. Wrapper klassen gearbeitet hast und Autoboxing kennst?

Bsp.:

Character a;

if(a.equals(....));

2. Strings gearbeitet hast?. Du kannst deine Elemente auch als Strings betrachten.

3. binären Operatoren gearbeitet hast , wie || oder sowas


grüße

 

[faetzminator]

Ich denke, es sollten keine Strings und auch nicht die Klasse Arrays verwendeet werden. Darum hier mein simpler Lösungsvorschlag:

public static void main(String[] args) {
    char[] input = {'D', 'r', 'e', 'i', ' ', 'C', 'h', 'i', 'n', 'e', 's', 'e', 'n'};
    System.out.println(input);
    replace(input);
    System.out.println(input);
}

private final static char[] TO_REPLACE = {'e', 'i', 'o', 'u', 'A', 'E', 'I', 'O', 'U'};

private final static char TO_INSERT  = 'a';

public static void replace(char[] chars) {
    for (int i = 0; i < chars.length; i++) {
        for (int j = 0; j < TO_REPLACE.length; j++) {
            if (chars[i] == TO_REPLACE[j]) {
                chars[i] = TO_INSERT;
                break;
            }
        }
    }
}

 

[Tina87]

erstmal danke für eure schnellen antworten.
also von wrapper und dem anderen hab ich noch nie was gehört. wir hatten bis jetzt wirklich nur die geringsten grundlagen.
aber auf jeden fall war der sinn der aufgabe keine strings zu benutzen, sondern char-array.
deshalb sollte die initialisierung auch genau so verwendet werden, um die aufgabe zu lösen.
deshalb wär es gut, wenn ihr mir dabei hilfe gibt.
eure lösungsvorschläge könnt ich zwar übernehmen, aber ich versteh sie nicht wirklich und sie sind über meinen wissen. da würde sich mein lehrer auch wundern und das ist ja auch nicht der sinn der hausaufgabe.

 

[SlaterB]

> Problem ist auch, dass ich nicht das mit "char ausgabe = new char ..." benutzt habe, weil ich gar nicht weiß was ich damit anfangen soll.

du ersetzt die Zeichen in Eingabe direkt, das ist nicht gut,
mit einem zweiten Array könntest du da die Änderungen reinschreiben und das Original so lassen wie es ist

-----

if (eingabe[2] == 'e')
         {
             eingabe[2] = ersatz;
            }
         if (eingabe[3] == 'i')
         {
             eingabe[3] = ersatz;
            }
         if (eingabe[7] == 'i')
         {
             eingabe[7] = ersatz;
            }
         if (eingabe[9] == 'e')
         {
             eingabe[9] = ersatz;
            }  
         if (eingabe[11] == 'e')
         {
             eingabe[11] = ersatz;
            }

->
nur das aktuelle Zeichen eingabe prüfen

char c = eingabe[i];
if (c == 'a' ||c == 'i' || ..) {
 c = ersatz;
}
ausgabe[i] = c;

 

[KrokoDiehl]

--> EDIT
Holla, überlesen das es ja ins ausgabe-Array soll, dann ist mein Hinweis natürlich nicht korrekt
<-- EDIT

Deine Lösung sieht im Ansatz ja schon gut aus:
Du gehst in einer For-Schleife durch das ganze Array und schaust an jeder Stelle, ob das Zeichen dort ein Vokal ist. Wenn ja, wird es ersetzt.
Nur der Zugriff auf die Arrayposition ist dir etwas missglückt ...

char[] eingabe = ...;

for (int i = 0; i < eingabe.length; i++)
{
    // 'i' durchläuft hier alle Stellen im Array, also:
    // wenn 'eingabe' an der Position 'i' ein Vokal ist
    // dann setze in 'ausgabe' das Zeichen 'ersatz'
    // sonst übernehme das Zeichen aus 'eingabe'
    if (eingabe[i] == ...)
        ...
}

Oder Merke: In einer For-Schleife sind absolute Indexangaben (wie

[2]

) eher selten korrekt, sondern die Regel ist der Zugriff über die Laufvariable á la

[i]

.

 

[Spin]

Naja Sinn der Hausaufgabe wird sein: dass du dir was überlegst.

Was verstehst du denn noch nicht an @faz programm?

 

[Tina87]

also ich hab das programm jetzt so geändert:

public class DreiChinesen
{
  public static void main(String args [ ])
  {
       char[] eingabe    = {'D','r','e','i',' ','C','h','i','n','e','s','e','n'};
      char ersatz       = 'a'; 
      char[] ausgabe    = new char[eingabe.length];
      
     
      for (int i=0; i<=eingabe.length; i=i+1)
      {
         char c = eingabe[i];
         
        if (c == 'a' ||c == 'i' || c == 'e') 
        {
          c = ersatz;
        }
        ausgabe[i] = c;
    }
    System.out.println(ausgabe);
      }
}

er findet zwar keine syntax-fehler, aber nach dem compilen kommt:
"ArrayIndexOutOfBoundsException:
13"
weiß nicht was das bedeutet. und was ist "||" für ein zeichen?bedeutet das "und"?
man hats auch nicht einfach mit mir...

 

[faetzminator]

[c]for (int i=0; i<=eingabe.length; i=i+1)[/c] --> [c]for (int i=0; i<eingabe.length; i=i+1)[/c]
|| ist OR / oder
&& ist AND / und

 

[KrokoDiehl]

In der Informatik zählt man von 0 bis n-1 ... also wenn ein Informatiker bis 10 zählen soll sieht das so aus:

0, 1, 2, 3, 4, 5, 6, 7, 8, 9

Das macht ihn zwar nicht gesellschaftsfähig, aber dafür hat er mit Arrays weniger Probleme, weil diese nämlich genauso "denken".
Also: Schau einmal auf deine For-Schleife. Bei welchem Index beginnt sie, wo hört sie auf und wie groß ist dein Array?

 

[T_Wolf]

Der Fehler bezeichnet, dass Du auf ein Arrayelement zugreifen willst, welches nicht existiert (meisstens eines zu weit). Der Fehler passiert mir auch häufiger, einfach die Schleife nocheinmal genau ansehen und überlegen was passiert, in deinem Fall läuft die Schleife wegen dem <= ein Element weiter, also einfach das "=" weglassen und es klappt

 

[Tina87]

Juhu es funktioniert...
klar.hatte wieder vergessen, dass man ja bei 0 anfängt zu zählen.
also hab ich jetzt in der for-schleife "i<=eingabe.length-1" eingefügt und dann ging.
echt lieben dank.

 

[Painii]

also hab ich jetzt in der for-schleife "i<=eingabe.length-1" eingefügt und dann ging.

Ist das gleiche wie "i<eingabe.length", persönliche Präferenz wäre aber nicht noch irgendwie was abziehen zu müssen... Aber ist genauso korrekt.