Discussion :

[Imer2008]

Bonjour à tous,

Je débute totalement dans la programmation Java alors j'espère que vous pourrez me conseiller car là je tourne en rond !

J'ai un fichier contenant des noms par ligne, exemple :
durant
dupond
thierry
bernard

Je souhaite ouvrir ce fichier texte, le lire et comparer ces données au login Windows (System.getProperty("user.name")).

Si le login windows se trouve dans le fichier texte, on ouvre l'application sinon on la ferme.

Je pensais faire de cette manière, mais c'est bien là le problème je n'y arrive pas !

1- ouverture du fichier
2- on compte le nombre de ligne(nbLigne) du fichier
3- on créé un tableau de String de longueur nbLigne. (mon problème actuel se situe là)
3- on parcourt le fichier et on insère chaque donnée de ligne dans le tableau créé
4- on reparcourt ce tableau créé en comparant cette fois avec le login Windows.
5- on ouvre l'application (ou pas) si on trouve le login.

Il y a surement plus simple à faire et dans ce cas, je suis tout ouï !

Merci à ceux qui m'aiguilleront !

[wax78]

Est-ce nécessaire toutes ses étapes ? (garder en mémoire la liste du fichier etc...)

Pourquoi ne pas faire tout simplement :

1- ouverture du fichier
2- on parcourt le fichier et on vérifie que la ligne lue contient ou non le nom du user. (on arrête directement la boucle si trouvé).
3- on ouvre l'application (ou pas) si on trouve le login.

Sinon pour le genre de probleme lire le nbr de ligne puis creer un tableau du nombre de ligne c'est un peu long. Crée un liste (arraylist ou autre) dans laquelle tu rajouteras ce que tu ligne (donc tu n'as plus besoin de compter le nbr de ligne a l'avance...)

[Imer2008]

Re,

J'ai trouvé une solution pour mon problème (avec intégration d'un bouléen).
Ce code peut sûrement être perfectionné, donc n'hésitez pas à me le dire!

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
package application;
 
import java.io.IOException;
import java.io.FileReader;
import java.io.BufferedReader;
import java.io.FileWriter;
import java.io.BufferedWriter;
 
import javax.swing.ImageIcon;
import javax.swing.JOptionPane;
 
public class Fichier
{
	// Déclaration des variables
	private String fichier = "agent/agent.txt";
	JOptionPane jop1, jop2;
	private String line = "";
	private int nbLigne = 0;
	private String userName = System.getProperty("user.name");
	boolean trouve = false;
 
	public Fichier()
	{
		try
		{
			// On ouvre le fichier en lecture
			BufferedReader buff = new BufferedReader(new FileReader(fichier));
			try 
			{
				// on parcourt le fichier tant qu'une ligne existe et que le booléen est faux
				while ((line = buff.readLine()) != null && trouve == false)
				{
					// si on trouve le userName
					if (line.equals(userName))
						trouve = true; // booleen devient "vrai"
					nbLigne++;
				}		
 
				// booléen vrai = user trouvé
				if (trouve == true)
				{
					jop1 = new JOptionPane();
					ImageIcon img = new ImageIcon("Icones/Info.png");
					jop1.showMessageDialog(null, "Bienvenue dans l'application !\n Vous êtes logués sous : "+ userName, "Bienvenue !", JOptionPane.INFORMATION_MESSAGE, img);
				}
				// sinon message de non autorisation
				else
				{
					jop2 = new JOptionPane();
					ImageIcon img = new ImageIcon("Icones/erreur.png");
					jop2.showMessageDialog(null, "Vous n'êtes pas autorisé à utiliser cette application !!", "Erreur !", JOptionPane.WARNING_MESSAGE, img);
					System.exit(0);
				}
			} 
			finally 
			{
				// on ferme notre fichier
				buff.close();
			}
		}
		catch (IOException ioe)
		{
			System.out.println(ioe.toString());
		}
	}
}

[wax78]

Ca a ma l'air mieux a premiere vue. Tu px éventuellement utiliser aussi cette forme pour sortir de la boucle mais ce que tu as fait est juste aussi.

Code :

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

1
2
3
4
5
6
7
8
9
while ((line = buff.readLine()) != null)
				{
					// si on trouve le userName
					if (line.equals(userName))
{
						trouve = true; // booleen devient "vrai"
break;
}
				}

[fastdeath124]

C'est une bonne solution et ça devrait marcher normalement, mais tu peux l'améliorer un peu:

1- La variable booléenne n'est pas utile, il suffit d'utiliser "break;" ce qui permettra de sortir de la boucle sans faire un test à chaque itération pour connaitre sa valeur.
2- Au lieu de compter le nombre de lignes, je te conseille d'utiliser une structure dynamique (par exemple ArrayList dans lequel il est possible d'ajouter à la fin sans même savoir l'indice d'ajout)

[Imer2008]

Merci à vous 2 pour vos commentaires.

1- La variable booléenne n'est pas utile, il suffit d'utiliser "break;" ce qui permettra de sortir de la boucle sans faire un test à chaque itération pour connaitre sa valeur.

Je ne vois pas comment ne pas utiliser le booléen malheureusement, car sans le booléen, comment dire au programme d'utiliser le 1er ou le 2eme message ?


On initialise le booléen à faux.
Début de boucle.
Si le string = userName, booléen devient vrai et on arrête la boucle (break);
sinon on continue
Fin de boucle

Si le booléen est vrai, alors message d'accueil
sinon message de refus.

[fastdeath124]

A la fin de la boucle "while", au lieu de tester sur la variable booléenne (if trouve==true), tu peux faire le test suivant:
- if(line ==null){...} : c'est le cas équivalent à trouve==false car si aucun résultat n'a été trouvé, line contiendra la valeur "null" retournée par readLine()
- if(line !=null){...} : c'est le cas équivalent à trouve==true

[wax78]

Oui c'est une solution aussi ca.

[Imer2008]

OK, effectivement ca fonctionne aussi ! Merci beaucoup !