Discussion :

[mimihh]

Bonsoir,
j'ai un fichier contenant dans chaque ligne plusieurs caractères et un nombre inconnu de "1" je voulais calculer le nombre de 1 dans chaque ligne. Pour cette raison, j'ai mis le contenu de mon fichier dans une liste contenant une ligne dans chaque élément. en comptant le nombre de 1 dans chaque ligne, le résultat obtenus est incorrecte j'obtiens en fait, des nombres très élèves pour chaque ligne. Pouvez vous m'aidez pour détecter l'erreur?
voici mon 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
 
public static void main(String[] args) throws Exception {
     List<String> list1 = new ArrayList<String>();
     List<String> list2 = new ArrayList<String>();
     int nb=0;
 
        File f = new File("file.txt");
        FileReader fr = new FileReader(f);
        BufferedReader br= new BufferedReader(fr);
        String ligne = "";
 
        ligne=br.readLine();
        int i=0;
        while(ligne!=null)
        {
            list1.add(ligne.toString());
 
            ligne=br.readLine();
            i++;
        }
        nb=0;
        br.close();
        for(i=0;i< list1.size();i++){
          if (list1.get(i).contains("1")){
              nb++;
        System.out.println(nb); }
        }
}

[joel.drigo]

Salut,

Tu ne comptes pas le nombre de "1" dans chaque ligne, tu comptes le nombre de lignes qui contient "1", ce que signifie exactement ton test :

Code :

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

if (list1.get(i).contains("1"))

Tu n'es pas obligé de stocker tes lignes dans une liste : tu peux analyser tes lignes directement au fur et à mesure que tu les lis.

Pour compter le nombre de "1", tu peux rechercher les "1", avec indexOf() par exemple :

Code :

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

1
2
3
4
5
6
7
8
int nb=0;
String ligne;
while( (ligne=br.readLine()) != null ) {
   int pos=-1;  
   while( (pos=ligne.indexOf("1",pos+1)) >=0 ) {
       nb++;
   }
}

[oliv37]

Tu peux également utiliser une expression régulière qui pour chaque ligne remplace les caractères non égal à 1 par le caractère vide. La longueur de la chaîne retournée te donnera le nombre de 1 que contient la ligne :

Code :

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

1
2
3
4
5
 
String ligne;
while( (ligne=br.readLine()) != null ) {
   int numberOfOne = ligne.replaceAll("[^1]*", "").length();
}

[thelvin]

Tu peux également utiliser une expression régulière qui pour chaque ligne remplace les caractères non égal à 1 par le caractère vide. La longueur de la chaîne retournée te donnera le nombre de 1 que contient la ligne :

...... -_-°.
Impressionnant pour la facilité d'écriture.

Mais à proscrire pour la complexité algorithmique et en mémoire de la chose. Cela dit, quand le temps de calcul reste suffisamment bas, je suppose que ça passe...

[hwoarang]

Tu peux également utiliser une expression régulière qui pour chaque ligne remplace les caractères non égal à 1 par le caractère vide. La longueur de la chaîne retournée te donnera le nombre de 1 que contient la ligne

les regexp, c'est un peu le bazooka pour tuer la mouche dans ce cadre la
Sans compter que tu proposes les regexp à quelqu'un qui ne fait pas la différence entre compter le nombre de 1 par ligne et compter le nombre de lignes qui contiennent un 1.
Bref, une approche comme celle de joel.drigo, bien que plus longue, me parait plus adaptée (meme s'il manque le fait de devoir stocker le nombre de 1 par ligne et pas le nombre total)

[mimihh]

Merci pour vos réponses, mais pourriez vous me dire à quel niveau exactement puise je faire l’affichage car il m'affiche pour chaque ligne un nombre très élevé alors qu'il ne doit pas dépasser le 50

[thelvin]

Le nombre en question ne commencerait pas par "12345678910111213" par hasard ?