Discussion :

[menoulette]

Bonsoir à vous tous

J'ai un petit tp à réaliser il s'agit d'établir un programme qui peut compter le nombre d'occurence d'un mot dans une chaine de caractère ..mais là je n'arrive pas à comprendre comment que la méthode indexOf() fonctionne c'est pour ca que j'ai un problème avec ce programme qui n'affiche pas le resultat juste ..quelqu'un pourra m'aider à trouver le problème et m'expliquer comment cette dernière fonctionne merci d'avance le voici le 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
 
 
 
 
import javax.swing.JOptionPane;
 
public class Occurence {
public static int NbrFois(String chaine,String mot){
	int cmpt=0;int i=0;
	while(i<chaine.length())
 
		{
		if (chaine.indexOf(mot, i)!=-1){
		cmpt++;
		}
		i=i + (mot.length());
 
		chaine=chaine.substring(i);
 
		}
 
	return cmpt;
}
	/**
         * @param args
         */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		 String chaine= JOptionPane.showInputDialog(null,"donner votre phrase");if (chaine==null) JOptionPane.showMessageDialog(null,"la chaine est vide");
		 String mot= JOptionPane.showInputDialog(null,"donner le mot à rechercher");if (mot==null)JOptionPane.showMessageDialog(null,"la chaine est vide");
		int nbre=NbrFois(chaine,mot);
		if(nbre!=0)
		JOptionPane.showMessageDialog(null,"Le mot {"+mot+"} apparait "+ nbre+" fois");else JOptionPane.showMessageDialog(null,"le mot n'apparait pas dans la phrase");
 
 
 
	}
 
}

Merci

[adiGuba]

Salut,

Code :

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

1
2
3
			i = i + (mot.length());
 
			chaine = chaine.substring(i);

D'un coté tu incrémentes l'index, de l'autre tu tronques la chaine. Forcément à un moment c'est plus bon...

Tu n'as pas besoin de tronquer la chaine puisque tu gère déjà proprement l'index...


a++

[menoulette]

Salut merci pour votre réponse mais le programme toujours ne calcule pas juste

[adiGuba]

Ah...

Je n'avais pas vu que tu ne récupérais pas l'index trouvé :

Code :

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

if (( i = chaine.indexOf(mot, i) )!= -1) {

a++

[menoulette]

Re :
Même avec ce test là et ca ne marche plus ...je sais pas ou se trouve bien le problème

[adiGuba]

Redonnes le code exact et les valeurs que tu utilises...

a++

[danimo]

Bonjour,

Ou alors:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
while(i<chaine.length()){
            i = chaine.indexOf(mot);
            if(i != -1)
            //if(chaine.indexOf(mot, i)!=-1)
            {
               cmpt++;
               chaine=chaine.substring(i + (mot.length()));
            }
            //i=i + (mot.length());
 
            //chaine=chaine.substring(i);
 
         }

Cordialement,

Dan

[menoulette]

Bonjour,

Ou alors:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
while(i<chaine.length()){
            i = chaine.indexOf(mot);
            if(i != -1)
            //if(chaine.indexOf(mot, i)!=-1)
            {
               cmpt++;
               chaine=chaine.substring(i + (mot.length()));
            }
            //i=i + (mot.length());
 
            //chaine=chaine.substring(i);
 
         }

Cordialement,

Dan

Ohhh merci beaucoup an ca marche super bien youpiiii merci beaucoup

[danimo]

Il y a quand même encore une petite modif (while...) à faire car il ne faut pas que i + la longueur de la sous-chaîne dépasse la longueur de la chaîne:

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
//while(i<chaine.length())
         while(i + mot.length() <= chaine.length())
         {
            i = chaine.indexOf(mot);
            if(i != -1)
            //if (( i = chaine.indexOf(mot, i) )!= -1) // ok
 
            //if(chaine.indexOf(mot, i)!=-1) // pas ok
            {
               cmpt++;
               chaine=chaine.substring(i + (mot.length()));
            }
            //i=i + (mot.length());
 
            //chaine=chaine.substring(i);
 
         }
 
         return cmpt;

Et il faut gérer les cas chaînes vides (try/catch) ...

Dan