Probleme d'ArrayList

Version imprimable

Voir 40 message(s) de cette discussion en une page

autre petit probleme , j'essais de faire une class admin qui gere les commande d'admin , j'ai un petit probleme au niveau des permissions , voila le code :
Code:

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 package ircBot.commands.admin; import java.util.ArrayList; import ircBot.KBot; import ircBot.commands.admin.*; public class admin { public static void execute(KBot k, String sender, String login, String hostname, String[] command) { String pass = "***"; // Commande D'authentification ArrayList<String> users = new ArrayList(); if (users.contains(sender)) { // Commande d'arret du bot if (command[1].equalsIgnoreCase("quit")) { quit.execute(k, sender, login, hostname, command); } else if (command[1].equalsIgnoreCase("auth")) { k.sendMessage(sender, "Tentative d'authentification ..."); if (command[2].equals(pass)) { users.add(sender); k.sendMessage(sender, "OK!"); } else { k.sendMessage(sender, "Mot de passe Incorrect !"); } } } else { k.sendMessage(sender,"Accée refusé , veuillez vous authentifier !"); } } }
je voudrais que quand je tape :
!admin auth MotDePasse
il ajoute mon login (ici sender) a la liste des authentifié (ici users)
et que pour les commandes privé il detecte si le pseudo est bien dedans avec users.contains(sender) , mais j'ai l'impression que ca ne fonctionne pas ...
il me renvoie toujour: Accée refusé , veuillez vous authentifier ! (meme pour !admin auth)

autre probleme , a chaque fois que cette classe va etre appeler , les données vont etre ecraser ? parceque ca risque de poser probleme sur que users doit contenir les login authentifier , si ca le remet a zero a chaque commande !admin , c'est pas top :p

Si je comprend bien, dans ton code "sender" correspond à "loginId" et "login" correspond à "password". Tu mélanges un peu les noms de variables donc ca devient compliqué...

Ok voici le probleme, tu vérifies si "sender" est dans "users". Mais le probleme c'est que ta liste de users est vide, tu viens tout juste de la créer:

Code:

ArrayList<String> users = new ArrayList();

alors c'est normal que

Code:

users.contains(sender)

te retourne toujours false. et tu vas directement à "Accée refusé"

Ecris ton algorithme en pseudo-code avant:
Code:

1 2 3 4 5 6 SI loginId est dans la liste ALORS effectue les commandes SINON SI la commande est "auth" ALORS Ajoute le loginID SINON "Accée refusé"
Toi ton code en ce moment correspond a
Code:

1 2 3 4 5 6 7 SI loginId est dans la liste ALORS SI la commant est quitter ALORS effectue la commande quitter SINON SI la commande est "auth" ALORS Ajoute le loginID SINON "Accée refusé"
Tu vois que si le login est pas dans la liste alors c'est fini pour ta gueule. Je te laisse penser a ce que tu devrais faire comme changement dans ton code. Si jamais tu y arrive pas fais moi signe je te le ferai.. c'est pas un gros changement!

voila j'ai modifié le code comme il faut :

Code:

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
package ircBot.commands.admin;
 
import java.util.ArrayList;
 
import ircBot.KBot;
import ircBot.commands.admin.*;
 
public class admin {
 
	public static void execute(KBot k, String sender, String login,
			String hostname, String[] command) {
		String pass = "***";
		// Commande D'authentification
		ArrayList<String> users = new ArrayList();
		if (users.contains(sender)) {
			// Commande d'arret du bot
			if (command[1].equalsIgnoreCase("quit")) {
				quit.execute(k, sender, login, hostname, command);
			}
		}
		else if (command[1].equalsIgnoreCase("auth")) {
			k.sendMessage(sender, "Tentative d'authentification ...");
			if (command[2].equals(pass)) {
				users.add(sender);
				k.sendMessage(sender, "OK!");
			} else {
				k.sendMessage(sender, "Mot de passe Incorrect !");
			}
		}
		else
		{
			k.sendMessage(sender,"Accée refusé , veuillez vous authentifier !");
		}
 
	}
}

le probleme c'est qu'a chaque fois que mon bot appel cette classe , comme c'est une classe statique , mon ArrayList est remis a zero , donc les users authentifié effacer ... comment regler ce probleme ? :/

Pour que tous les utilisaterus se partage la meme ressource (ArrayList<String> users) il faut que celle-ci soit static.

Tu peux mettre une ressource static seulement à l'extérieur d'une méthode

Donc voici le code

Code:

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
package ircBot.commands.admin;
 
import java.util.ArrayList;
 
import ircBot.KBot;
import ircBot.commands.admin.*;
 
public class admin {
 
       private static ArrayList<String> users = new ArrayList();
 
	public static void execute(KBot k, String sender, String login,
			String hostname, String[] command) {
		String pass = "***";
		// Commande D'authentification
 
		if (users.contains(sender)) {
			// Commande d'arret du bot
			if (command[1].equalsIgnoreCase("quit")) {
				quit.execute(k, sender, login, hostname, command);
			}
		}
		else if (command[1].equalsIgnoreCase("auth")) {
			k.sendMessage(sender, "Tentative d'authentification ...");
			if (command[2].equals(pass)) {
				users.add(sender);
				k.sendMessage(sender, "OK!");
			} else {
				k.sendMessage(sender, "Mot de passe Incorrect !");
			}
		}
		else
		{
			k.sendMessage(sender,"Accée refusé , veuillez vous authentifier !");
		}
 
	}
}

01/09/2006, 15h19
kedare

merci ca fonctionne impec :)
si non comment faire pour faire l'inverse ?
je vais faire une fonction de-auth , mais je sait pas comment enlevé un champ de l'ArrayList
01/09/2006, 15h45
thibaut

Je voudrais faire une précision, il n'est pas recommandé (cf normes de codage sun par exmple) de faire de imports du syle

Code:

import java.toto.dibule.*;

pas parceque ça surcharge le code ou que ça inclu des choses inutiles, puisque dans tout les cas c'est un chemin qui est inclus, mais plutôt par soucis de lisibilité.

On préférera donc toujours:

Code:

import java.toto.dibule.SpaceCowboy*;

à

Code:

import java.toto.dibule.*;

A+
01/09/2006, 15h56
pmartin8

pour retirer un element d'une array list tu fais comme ca.

Code:

users.remove("leLoginAEnlever")
03/09/2006, 23h13
bouye

Citation:

Envoyé par thibaut

plutôt par soucis de lisibilité.

Tu noteras que c'est encore une de leurs propres regles qu'il ne suivent pas (cf leur code source - et ce n'est pas la seule regle de ce genre). Le point de vue de la lisibilite est tangent ... une liste de 3km de long non-triee n'est pas lisible tout simplement et la trier et la modifier pour suivre l'evolution du code (ajout/retrait de classes utilisees) prend du temps (et nuit donc a la maintenance). D'ou comme je l'ai indique : tu es libre de faire ce que tu veux sur ce sujet sans restriction (en suivant les regles de programmation de la ou tu bosses).
04/09/2006, 15h32
Glob

Concernant la question des imports... je préfère la notation "non étoilée", car il s'agit d'un moyen relativement rapide de jeter un oeil à ce qui est utilisé dans une classe.

Les IDE modernes trient très bien cette liste à la demande, avec suppresion des classes inutilisées avec un simple raccourci clavier. Lorsqu'elle s'allonge trop, c'est peut-être qu'il faut arrêter de taper du code et revoir les diagrammes de classe... :roll:

Mais il vrai que sous des éditeurs un peu plus passifs (genre notepad), le développeur gagnera un peu de temps en important (p.ex.) un javax.swing.* plutôt que de se taper un nouvel import à la main pour chaque nouveau composant utilisé.
04/09/2006, 17h24
Deadpool

Edit : fausse manip, à supprimer
04/09/2006, 23h18
bouye

Et, de la meme maniere, les IDE modernes sont capables, quand il y a trop d'imports fait a partir du meme package de les simplifier en une seule ligne, ce reduit la taille du fichier de code source et le rend plus concis. De meme un IDE moderne est capable de remonter au code d'une classe ou d'afficher son package lorsque le nom court est utilise au lieu du nom long.
Evidement il reste toujours la troisieme solution d'ecrire a chaque fois le nom long dans le code, evidement tout le monde (moi y-compris) s'ecrira que c'est stupide et une perte de temps (hors besoin de resoudre des conflits de nom courts), mais il se trouve que c'est encore plus lisible que les deux autres (mais c'est celle qui rallonge plus le code).
Alors :roll: de meme...

Donc, au lieu de continuer cet echange de ping-pong ou il semble visiblement que les deux cotes ont raison, l'important est de retenir comme cela a ete indique precedement que tu peux faire comme bon te semble (faut lire un peu) quand tu codes seul et que tu dois suivre les regles de codages de ton service/entreprise quand c'est pour le boulot. Les raisons initiales invoquee pour se restreindre a ce niveau etant non-valides car inspiree d'un mecanisme propre a un autre language (le C).
05/09/2006, 08h12
Glob

Citation:

Envoyé par bouye

Donc, au lieu de continuer cet echange de ping-pong ou il semble visiblement que les deux cotes ont raison

+1 :mouarf:

A+

Voir 40 message(s) de cette discussion en une page