Elemente aus zwei verschiedenen Arrays miteinander vergleichen und gegeben falls entfernen

[Bademeister007]

public class Satz2
{
public ArrayList<Wort> satz = new ArrayList<Wort>();
public Wort w;

public Satz2(ArrayList<Wort> satz)
{
this.satz = satz;
System.out.println("\n---Satz mit vereinzelten Woertern geaddet---");

}


public void entferneStoppWoerter(ArrayList<Wort> stoppwoerter) throws FileNotFoundException
{
System.out.println("---Stoppwoerter entfernen---");


String [] s = new String[satz.size()];
String[] stopp = new String[stoppwoerter.size()];
//maybe my loops are wrong?
for( int i =0; i <s.length; i++)
{

s = satz.get(i).getWoerter();

for( int j =0; j < stopp.length; j++)
{

stopp[j] = stoppwoerter.get(j).getWoerter();


}
if(s.equals(stopp))
{
System.out.println("---gefunden-------------------");
satz.remove(satz.get(i)); //i thought remove is the right command
setWoerterListe(satz);
}
else
i++;

}
}


}

Hallo Leute ich hoffe ihr könnt mir bei mein Problem weiterhelfen.
Ich versuche ein bestimmtes Element zu entfernen. Hierbei geht es um ein Wort z. B "dabei" und wenn dieses Wort in beiden Arrays vorhanden ist, soll es entfernt werden.
Ich dachte Der Remove Befehl wäre hier optimal aber anscheinend gibt es Schwierigkeiten.

ich freue mich auf Antworten.

 

[Barista]

Das übliche hier:

1. Code-Tags benutzen,

2. eine richtige Frage stellen

 

[Barista]

Für die Stop-Wörter bietet sich die Benutzung eines Set an.

 

[Barista]

Bei der remove-Methode musst Du aufpassen.

Es gibt leider eine Überladung:

remove(int) (hier wäre removeAt() besser gewesen, aber Autoboxing kam erst mit Java5, die remove-Methoden gab es schon vorher)

remove(Object)

Ich vermute, Du rufst versehentlich remove(Object) auf

 

[Barista]

Es gibt auch noch

removeAll()

Damit reduziert sich alles auf ein Statement.

Aber da wäre der Lerneffekt sehr gering.

 

[Oneixee5]

Das Array-gewurstel ist völlig unnötig. Direkt mit Listen arbeiten ist viel komfortabler:

        List<String> stopwords = Arrays.asList("red", "blue");
        List<String> otherList = new ArrayList(Arrays.asList("red", "green", "green", "yellow"));

        System.out.println(otherList);

        Iterator<String> iter = otherList.iterator();
        String s;
        while (iter.hasNext()) {
            s = iter.next();
            if (stopwords.contains(s)) {
                iter.remove();
            }
        }

        System.out.println(otherList);

Wobei stopwords auch ein Set sein könnte. Im Prinzip musst du ja nur die Liste durchgehen und alle Elemente entfernen, welche dir nicht passen.
Da du eine Klasse Wort verwendest, muss diese ggf. equals implementieren damit die contains-Methode funktioniert.

 

[Bademeister007]

Das Array-gewurstel ist völlig unnötig. Direkt mit Listen arbeiten ist viel komfortabler:

        List<String> stopwords = Arrays.asList("red", "blue");
        List<String> otherList = new ArrayList(Arrays.asList("red", "green", "green", "yellow"));

        System.out.println(otherList);

        Iterator<String> iter = otherList.iterator();
        String s;
        while (iter.hasNext()) {
            s = iter.next();
            if (stopwords.contains(s)) {
                iter.remove();
            }
        }

        System.out.println(otherList);

Wobei stopwords auch ein Set sein könnte. Im Prinzip musst du ja nur die Liste durchgehen und alle Elemente entfernen, welche dir nicht passen.
Da du eine Klasse Wort verwendest, muss diese ggf. equals implementieren damit die contains-Methode funktioniert.

Nein es geht darum, ein bestimmtes Element zu entfernen. Als in deinem Beispiel könnte man red entfernen. Dh if(stopwords.equals(otherList). Es ist auch nicht mein vollständiger Code sondern nur ein Teil Code von einem ganzen, deswegen sieht das so durcheinander aus. Mir gehts ja nur um das Prinzip dahinter.

 

[Barista]

Nein es geht darum, ein bestimmtes Element zu entfernen.

Dann filterst Du eben deine Stopwords auf dieses Element.

Es ist auch nicht mein vollständiger Code sondern nur ein Teil Code von einem ganzen, deswegen sieht das so durcheinander aus.

Code baut man am besten aus klar definierten Bausteinen.

Am besten Dinge, die in der Informatik einen Namen haben.

Es ist bereits alles erfunden.

"Bei mir" heisst meist "falsch".

 

[Barista]

Nein es geht darum, ein bestimmtes Element zu entfernen.

Dieser Ton nervt mich.

Niemand ist verpflichtet, Dir hier zu helfen.

Und Dir steht nicht zu, Fehler bei Leuten zu finden, die versuchen, Dir anhand Deiner lieblos und unklar gestellten Frage zu helfen.

 

[Bademeister007]

Dieser Ton nervt mich.

Niemand ist verpflichtet, Dir hier zu helfen.

Und Dir steht nicht zu, Fehler bei Leuten zu finden, die versuchen, Dir anhand Deiner lieblos und unklar gestellten Frage zu helfen.

 

[Oneixee5]

OK, noch mal das Prinzip dahinter. Gegeben ist folgender Code:

public class App {

    private static class Wort {

        private String content;

        public Wort(String content) {
            this.content = content;
        }

        public String getContent() {
            return content;
        }

        @Override
        public String toString() {
            return "Wort [content=" + content + "]";
        }

    }

    public static void main(String[] args) {
        Wort stopword = new Wort("red");
        List<String> otherList = new ArrayList(
                Arrays.asList(new Wort("red"), new Wort("green"), new Wort("yellow"), new Wort("green")));

        System.out.println("\u001b[31m" + otherList);

        while (otherList.remove(stopword));

        System.out.println("\u001b[32m" + otherList);
    }

}
Ergebnis:
[Wort [content=red], Wort [content=green], Wort [content=yellow], Wort [content=green]]
[Wort [content=red], Wort [content=green], Wort [content=yellow], Wort [content=green]]

Jetzt sollte man meinen, das stopword müsste aus der Liste entfernt werden. Das funktioniert aber nicht weil die Methode equals von Wort nicht überschrieben wurde. Die Standardimplementierung vergleicht irgend einen Kauderwelsch, welcher uns hier nicht interessieren muss.

Wenn wir aber die Klasse Wort ändern, indem equals (und wann immer equals überschrieben wird, sollte man auch hashcode überschreiben) überschreiben, dann sieht die Sache anders aus:

    private static class Wort {

        private String content;

        public Wort(String content) {
            this.content = content;
        }

        public String getContent() {
            return content;
        }

        @Override
        public String toString() {
            return "Wort [content=" + content + "]";
        }

        @Override
        public int hashCode() {
            final int prime = 31;
            int result = 1;
            result = prime * result + ((content == null) ? 0 : content.hashCode());
            return result;
        }

        @Override
        public boolean equals(Object obj) {
            if (this == obj)
                return true;
            if (obj == null)
                return false;
            if (getClass() != obj.getClass())
                return false;
            Wort other = (Wort) obj;
            if (content == null) {
                if (other.content != null)
                    return false;
            } else if (!content.equals(other.content))
                return false;
            return true;
        }

    }
Ergebnis:
[Wort [content=red], Wort [content=green], Wort [content=yellow], Wort [content=green]]
[Wort [content=green], Wort [content=yellow], Wort [content=green]]

Jetzt wird in den Vergleich der konkrete Inhalt einbezogen. Jetzt kann das Stoppwort entfernt werden.

 

[Bademeister007]

OK, noch mal das Prinzip dahinter. Gegeben ist folgender Code:

public class App {

    private static class Wort {

        private String content;

        public Wort(String content) {
            this.content = content;
        }

        public String getContent() {
            return content;
        }

        @Override
        public String toString() {
            return "Wort [content=" + content + "]";
        }

    }

    public static void main(String[] args) {
        Wort stopword = new Wort("red");
        List<String> otherList = new ArrayList(
                Arrays.asList(new Wort("red"), new Wort("green"), new Wort("yellow"), new Wort("green")));

        System.out.println("\u001b[31m" + otherList);

        while (otherList.remove(stopword));

        System.out.println("\u001b[32m" + otherList);
    }

}
Ergebnis:
[Wort [content=red], Wort [content=green], Wort [content=yellow], Wort [content=green]]
[Wort [content=red], Wort [content=green], Wort [content=yellow], Wort [content=green]]

Jetzt sollte man meinen, das stopword müsste aus der Liste entfernt werden. Das funktioniert aber nicht weil die Methode equals von Wort nicht überschrieben wurde. Die Standardimplementierung vergleicht irgend einen Kauderwelsch, welcher uns hier nicht interessieren muss.

Wenn wir aber die Klasse Wort ändern, indem equals (und wann immer equals überschrieben wird, sollte man auch hashcode überschreiben) überschreiben, dann sieht die Sache anders aus:

    private static class Wort {

        private String content;

        public Wort(String content) {
            this.content = content;
        }

        public String getContent() {
            return content;
        }

        @Override
        public String toString() {
            return "Wort [content=" + content + "]";
        }

        @Override
        public int hashCode() {
            final int prime = 31;
            int result = 1;
            result = prime * result + ((content == null) ? 0 : content.hashCode());
            return result;
        }

        @Override
        public boolean equals(Object obj) {
            if (this == obj)
                return true;
            if (obj == null)
                return false;
            if (getClass() != obj.getClass())
                return false;
            Wort other = (Wort) obj;
            if (content == null) {
                if (other.content != null)
                    return false;
            } else if (!content.equals(other.content))
                return false;
            return true;
        }

    }
Ergebnis:
[Wort [content=red], Wort [content=green], Wort [content=yellow], Wort [content=green]]
[Wort [content=green], Wort [content=yellow], Wort [content=green]]

Jetzt wird in den Vergleich der konkrete Inhalt einbezogen. Jetzt kann das Stoppwort entfernt werden.

Hey vielen vielen Dank für deine Mühe und Hilfestellungen. Leider bringt mich das ganze doch nicht weiter. Mir fehlt nun mal noch ein gewisses Verständnis. Trotzdem Danke nochmal!

 

[mihe7]

Um mal auf Deinen ursprünglichen Code zurückzukommen: da stimmt zwar so gut wie nix, aber man sieht ein Problem, nämlich dass Du mittels eines Index stur von vorne nach hinten durch die Liste iterierst und dabei Elemente entfernst. Das verschlimmbesserst Du noch, in dem Du i ein weiteres mal inkrementierst, wenn kein Stopwort gefunden wurde.

Du musst Dir klar machen, was da passiert: die äußere for-Schleife inkrementiert nach jedem Durchlauf i, so dass i normalerweise Werte 0,1,2,3,.. annimmt. Nehmen wir mal an, es gelte i == 2 und Du löscht das Element an dieser Position. Dann rutschen die Folgelemente nach vorne, so dass das Element, das zuvor an Index 3 stand nun an Index 2 steht. Die for-Schleife inkrementiert nun i, so dass i == 3 gilt. Folglich hast Du das Element, das ursprünglich an Inde 3 stand, übersprungen.

Beispiel: Du willst c und d löschen

0 a
1 b
2 c  <- i
3 d
4 e

Vergleich: ah, c ist das zu löschende Element, also löschen wir Element 2
0 a
1 b
2 d  <- i
3 e

Jetzt inkrementiert die Schleife das i
0 a
1 b
2 d 
3 e <- i

Das d wird niemals verglichen. Weil du zusätzlich i nochmal inkrementierst, wenn das Wort kein Stoppwort ist, passiert noch schlimmeres:

0 a <- i
1 b
2 c  
3 d
4 e

Ah, a kein Stopwort -> inkrementiere i. Ende der for-Schleife: inkrementiere i nochmal
0 a
1 b
2 c <- i  
3 d
4 e

OK, c Stopwort -> entferne Element 2, inkrementiere i
0 a
1 b
2 d
3 e  <-  i

Kurz: jedes zweite Wort fällt aus dem Vergleich heraus, denn entweder ist es kein Stopwort, dann wird i zweimal inkrementiert oder es ist ein Stopwort, dann wird es gelöscht und i inkrementiert.

Um das zu umgehen, lässt Du den else-Zweig weg und dekrementierst das i, wenn ein Element gelöscht wurde (das Dekrementieren hebt das Inkrementieren der for-Schleife auf). Andere Möglichkeit: Du gehst einfach von hinten nach vorne durch die Liste. Dann muss der Schleifenzähler innerhalb der Schleife nicht angepasst werden.

 

[Trjavnamen]

ein anderer Ansartz alle elemente die gelöscht werden sollen mit Null überschreiben , dann sortieren nochmals die schleife rückwärts durchlaufen und alle Null löschen-

 

[mihe7]

ein anderer Ansartz alle elemente die gelöscht werden sollen mit Null überschreiben , dann sortieren nochmals die schleife rückwärts durchlaufen und alle Null löschen-

Aufwand mindestens O(n*log(n)) -> nix gut.