Discussion :

[OrangeAmps]

Bonsoir,

Je cherche à enlever d'une Arraylist de String (mots) les mots présent dans un fichier .txt

Voici le code que j'ai écris pour le moment

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
public static String removeStopWords(String str,File file) throws IOException{
    	BufferedReader br = new BufferedReader(new FileReader(file));
    	try{
 
    	String[] strTab = str.split(",");
    	ArrayList<String> list = new ArrayList<String>(Arrays.asList(strTab)); // conversion de la string en arraylist
    	Iterator<String> itr = list.iterator();
    	String line = br.readLine();
        //parcours du texte arraylist
    	while(itr.hasNext()){
    		//parcours du fichier txt de stopwords
    		while((line != null)){
    			if(itr.equals(line))
    				itr.remove(); // on enlève le stopwords présent dans l'arraylist
    		}
    		itr.next();
    	 }
    	//conversion Arraylist en string
    	String text = "";
    	for(String s : list)
    		text += s + " ";
    	System.out.println(text);
    	return text;
   } finally{
	   br.close();
   }
    }

Seulement ça ne m'affiche rien dans la console, j'ai pourtant fermé mes deux boucles et fermé le bufferedreader.
J'ai l'impression que la méthode boucle à l'infini.

[thelvin]

Alors pour rappel :
- Les collections ont une méthode remove() qui sert à enlever un objet qu'elles contiennent. Ce sera bien plus simple de t'en servir, que de chercher toi-même l'objet à enlever.
- readLine() ne lit qu'une seule ligne. En toute logique il faudrait l'appeler plusieurs fois, pas une.
- D'où vient itr déjà ? En quoi itérer là-dessus pourrait-il avoir le moindre effet sur line ? (Sans compter qu'il n'y a de toute façon pas d'itération sans itr.next()...)

[OrangeAmps]

Par "Collection" vous entendez les Hashset ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
public static String removeStopWords(String str) throws IOException{
 
    	String line;
    	String[] strTab = str.split(",");
    	HashSet<String> hashset = new HashSet<String>(Arrays.asList(strTab)); // conversion de la string en arraylist
    	Iterator<String> itr = hashset.iterator();
 
    	br = new BufferedReader(new FileReader("StopWords.txt"));
        //parcours du texte arraylist
    	while(itr.hasNext()){
    		String n = itr.next();
    		//parcours du fichier txt de stopwords
    		while((line = br.readLine()) != null){
    			System.out.println(line);
    			if(n.equals(line))
    				itr.remove(); // on enlève le stopword présent dans l'arraylist
    		}
    	 }
    	//conversion Arraylist en string
    	String text = "";
    	for(String s : hashset)
    		text += s + " ";
    	System.out.println(text);
    	return text;
   }

J'ai modifié deux trois trucs, notamment changer l'Arraylist en hashet
Bref maintenant la console m'affiche la String passé en argument telle quelle, aucun mot n'a été enlevé, pourtant je pensais que les iterator.remove() supprimerait les String en questions.

[thelvin]

Par "Collection" vous entendez les Hashset ?

J'entends absolument toutes les Collections.
Donc notamment ArrayList, et aussi LinkedList, et TreeSet, LinkedHashSet et oui, HashSet aussi. Ainsi qu'absolument toutes les autres. Elles ont toutes une méthode remove().
L'idée de cette méthode, c'est de faire comme si elle était là et s'en servir. Pas de faire comme si elle était pas là et ne pas s'en servir.

Puisqu'il existe une méthode pour enlever des trucs de la liste, tu n'as pas besoin d'un itérateur pour le faire, évidemment -_-° ! Ta variable itr est parfaitement inutile et n'a pas de raison d'être. Ne parlons même pas de la boucle dessus.

Bref maintenant la console m'affiche la String passé en argument telle quelle, aucun mot n'a été enlevé, pourtant je pensais que les iterator.remove() supprimerait les String en questions.

Ça ne marche pas parce que tu lis toutes les lignes de ton fichier, lors de ta toute première boucle, au tout premier appel de next(). Donc aux boucles suivantes, le fichier a été entièrement lu et il n'y a plus de ligne, donc aucun mot à enlever.
Ton programme est fait n'importe comment, il est clair que tu ne comprends rien à ce que tu fais.
Mais la première chose à faire, c'est d'arrêter d'utiliser un itérateur, puisqu'il ne sert à rien. Ça simplifiera un peu les choses. Notamment, tu n'auras plus de boucle imbriquée.