Code Verbesserung, Array von Strings

Question

Ich bin völlig neu in der Programmierung. Kannst du mir ein paar Tipps geben, wie ich meinen Code verbessern kann?
Das Problem war:
Gegeben ein Array von Zeichenfolgen, ein neues Array ohne die Zeichenfolgen, die gleich der Zielzeichenfolge sind. Ein Ansatz besteht darin, die Vorkommen der Zielzeichenfolge zu zählen, ein neues Array mit der richtigen Länge zu erstellen und dann die korrekten Zeichenfolgen zu kopieren.
Und mein Code:

public String[] wordsWithout(String[] words, String target) { 
    int numberOfTargets = 0; 

    for (int i = 0; i < words.length; i++){ 
    if (words[i].equals(target)) numberOfTargets++; 
    } 

    String[] result = new String[words.length - numberOfTargets]; 

    for (int i = 0; i < words.length - numberOfTargets; i++){ // 1 
    result[i] = "0";          // 1 
    }               // 1 

    for (int i = 0; i < words.length; i++){ 
    if (!words[i].equals(target)){ 
     int j = 0;       // 2 
     while (!result[j].equals("0")){ // 2 
     j++;        // 2 
     }         // 2 
     result[j] = words[i]; 
    } 
    } 
    return result; 
} 

Beispiel dafür, wie Code funktioniert:

wordsWithout(["aa", "ab", "ac", "aa"], "aa") → ["ab", "ac"] 

Ich weiß, dass neue Array von ints von Nullen dy Standard gefüllt ist. Was ist mit neuen Arrays von Strings? Ich musste es künstlich mit Nullen füllen, die mit // 1 markiert waren, so dass ich zum rechten Element "scrollen" konnte, wenn ich meinem neuen Array Elemente hinzufügen musste, die mit // 2 markiert waren.
Mein Code scheint irgendwie peinlich zu sein. Gibt es Standardmethoden oder allgemeine Möglichkeiten, meinen Code zu verbessern?

Answer 1

Sie müssen nicht jedes Element auf "0" setzen.

tun Nur soviel:

public static String[] wordsWithout(String[] words, String target) { 
    int numberOfTargets = 0; 

    for (int i = 0; i < words.length; i++){ 
     if (words[i].equals(target)) numberOfTargets++; 
    } 

    String[] result = new String[words.length - numberOfTargets]; 
    int j =0; // for indices of result 
    for (int i = 0; i < words.length; i++){ 
     if (!words[i].equals(target)){   
      result[j++] = words[i]; 
     } 
    } 
    return result; 
} 

Answer 2

Sieht aus wie Ihr Code eine Menge von nur unter Verwendung eines ArrayList vereinfacht werden könnten.

public String[] wordsWithout(String[] words, String target) 
{ 
    ArrayList<String> list = new ArrayList<String>(); 
    for(int i = 0; i < words.length; ++i) 
    { 
     if(!words[i].equals(target)) 
     { 
      list.add(words[i]); 
     } 
    } 
    return list.toArray(new String[0]); 
} 

Grundsätzlich stattdessen die Größe des Zielarrays für die Berechnung und Initialisierung es, verwenden Sie eine Liste (die in der Größe variabel ist), setzen in allen Elementen Sie brauchen, und dann ein neues Array erstellen aus es.

Unabhängig davon, bitte erfinden Sie nicht Ihre eigenen Werte ("0"), um einen Nullwert zu beschreiben - dafür gibt es ein spezielles Schlüsselwort, null.

Answer 3

Verwenden

for (String s : words) { 

    if (s.equals(target)) 
    numberOfTargets++; 
}