Discussion :

[Istgas]

Bonjour,
je suis debutante en java , je cherche à créer une classe java qui me permet de supprimer des mots inutile (mot vide) comme : the , that, is, be, before, because, about, then, can,!,?,..... a partir d'un fichier txt .j'ai crée ce code :

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
public class MotVide {
	public static void main(String[] args){
		String fichier ="D:/application/entrer lemm/ambler.txt";
// exemple d'une liste
		  ArrayList<String> mots = new ArrayList<String>(Arrays.asList("to", "the", "a","or","on","is","in","be","are","is"));
		String  ch = "";
		boolean res ;
		int j;
 
		//lecture du fichier texte	
		try{
			InputStream ips=new FileInputStream(fichier); 
			InputStreamReader ipsr=new InputStreamReader(ips);
			BufferedReader br=new BufferedReader(ipsr);
			FileWriter fileWriter=new FileWriter("D:\\inputlemma.txt"); 
			BufferedWriter bufferedWriter=new    BufferedWriter(fileWriter); 
			String ligne;
 
			 while ((ligne=br.readLine())!=null ){	
				System.out.println(ligne);
				String str[]=ligne.split(" ");
 
				for (int i =0; i<str.length;i++ )
 
				{
				     ch = str[i];
				     j=0 ;
				     res =false ;
				while(j<mots.size() && res ==false)				
				{   
					if ( mots.get(j).equalsIgnoreCase(ch))
				         res= false ; 
				       else  
				         res = true ;
					  j++;
				}
				System.out.println(res);
						 if (res=true)
				 { bufferedWriter.write(str[i]); 
				   bufferedWriter.newLine(); }
 
			   } 
			 }
			br.close();	 	
		}		
		catch (Exception e){
			System.out.println(e.toString());
		}
	}
}

le résultat de code me donner un nouveaux fichier qui contient le même contenue que le fichier initiale, il ne me permet pas d’éliminer les mot vide je pense que la problème dans la fonction equals :
la variable res prend toujours la valeur true .
Svp aidez moi !!

[joel.drigo]

Bonjour,

Salut,

la variable res prend toujours la valeur true .

oui, parce que tu fais :

Code :

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

if (res=true)

or, le symbole de test d'égalité c'est == pas =, qui est le signe d'affectation

tu devrais faire :

Code :

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

if (res==true)

ou plus simplement

Code :

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

if (res)

Quand tu fais

Code :

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

if (res=true)

ça affecte true à res puis ça évalue l'expression res, donc qui vaut toujours true.

Par ailleurs tu peux remplacer :

Code :

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

1
2
3
4
5
6
7
8
while(j<mots.size() && res ==false)				
				{   
					if ( mots.get(j).equalsIgnoreCase(ch))
				         res= false ; 
				       else  
				         res = true ;
					  j++;
				}

par

Code :

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

res = !mots.contains(ch.toLowerCase())

(et ce sera un plus efficace d'utiliser un HashSet plutôt qu'une ArrayList pour mots)

[Istgas]

Mercii beacoup sa fonctionne mais lorsque j'ai termine la liste avec des autre mots il ne me donne pas une résulta

(et ce sera un plus efficace d'utiliser un HashSet plutôt qu'une ArrayList pour mots)

Est ce que tu me donnée un exemple c 'est quoi HasSet ??

[andry.aime]

Bonjour,

Pour éviter ce genre d'erreur lors d'un test sur l'égalité, j'ai pris l'habitude de mettre le constant à gauche, comme ça on aura une erreur de compilation au lieu du résultat. En java, l'opérateur "=" ne passe que pour les booléens dans un clause "if" mais l'astuce est valable pour les autres langages.

Code :

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

1
2
3
4
if ("abc".equals(a)) // permet de ne pas tester si a n'est pas null
if (true=a) // erreur de compilation
if (2==a) // au lieu de (a==2), dans le cas d'oublie d'un égal (=), on aura une erreur de compilation
...

A+.

[joel.drigo]

Mercii beacoup sa fonctionne mais lorsque j'ai termine la liste avec des autre mots il ne me donne pas une résulta

Pas de quoi Mais je ne comprends pas quand tu dis "j'ai termine la liste avec des autres mots"... mets ton code, ce sera plus parlant

Est ce que tu me donnée un exemple c 'est quoi HasSet ??

Ce n'est pas très important dans ton petit programme, mais quand il s'agit de vérifier l'existence de quelque chose dans un ensemble de valeurs, utiliser un HashSet est plus rapide qu'une ArrayList (c'est d'autant plus important que cette liste est grande).

Ca s'utilise pareil (si tu utilises contains(), pas si tu utilises ton while et get()) dans ton cas :

Code :

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

Set<String> mots = new HashSet<String>(Arrays.asList("to", "the", "a","or","on","is","in","be","are","is"));

Attention, un HashSet, comme tous les Set impose que les instances que l'on y met soient uniques (au sens de la méthode equals ) : mais c'est le cas d'une liste de stopwords (ça servirait à rien d'avoir 2 fois le même mot)

Je ne t'explique pas comment fonctionne un HashSet dans le détail, ça risque d'être un petit peu compliqué : sache juste que ça utilise une valeur int (un hash) pour accélérer le test d'égalité (et la recherche), avant de tester equals, d'ou le fait que ça soit plus rapide.

[AASProni]

Essai ce code

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
 
public class MotVide {
    public static void main(String[] args){
        String fichier ="D:/application/entrer lemm/ambler.txt";
        ArrayList<String> mots = new ArrayList<String>(Arrays.asList("to", "the", "a","or","on","is","in","be","are","is"));
        //lecture du fichier texte	
        try{
            InputStream ips=new FileInputStream(fichier); 
            InputStreamReader ipsr=new InputStreamReader(ips);
            BufferedReader br=new BufferedReader(ipsr);
            FileWriter fileWriter=new FileWriter("D:\\inputlemma.txt"); 
            BufferedWriter bufferedWriter=new    BufferedWriter(fileWriter); 
            String ligne;
 
             while ((ligne=br.readLine())!=null ){
                String resultat = "";
                System.out.println(ligne);
                String str[]=ligne.split(" ");
 
                for (int i =0; i<str.length;i++ ){
                     boolean res=false;          
                    for (int k = 0; k < mots.size(); k++) {                                    
                        if ( !mots.get(k).equalsIgnoreCase(str[i])){  
                            res=true; break;       
                        }                                   
                    }
                    if(!res){                   
                            bufferedWriter.write(str[i]+" ");
                            resultat += str[i]+" ";
                    }
                }
                System.out.println(resultat);
                bufferedWriter.newLine();
             }
            br.close();	 	
        }		
        catch (Exception e){
                System.out.println(e.toString());
        }
    }
}

tes erreurs:
1-n'emploi pas while si tu peut employer for
2-tu emploie beucoup trop de variable inutile : ch, j ...

[andry.aime]

@AASProni
Il faut éviter de manipuler un String pour des concaténations parce que c'est immuable et gardé en mémoire pendant certain temps, il faut utiliser un StringBuilder.

A+.