Probleme d'ArrayList

Version imprimable

Salut !
j'ai un probleme , je fait un petit Bot IRC en java avec la lic PircBot , tout va bien , mais quand j'essais de diviser le code en plusieurs fichier , comme ceci :
ici le fichier de mon objet bot :

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
import java.io.File;
import org.jibble.pircbot.*;
import commands.req;
 
public class KBot extends PircBot {
 
        public KBot() {
                this.setName("KRuB0t");
        }
 
        public void onConnect() 
        {
                sendMessage("NickServ","IDENTIFY ***");
        }
 
        public void onMessage(String channel, String sender, String login,
                        String hostname, String message) {
                        String[] command = message.split(" ");
 
                if (command[0].equalsIgnoreCase("!req")) {
                        req.execute(channel,sender,login,hostname,command);
                }
        }
}

et ici ma classe req (appeler quand la commande commence par !req donc)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
package commands;
import java.io.File;
import org.jibble.pircbot.DccFileTransfer;
 
public class req {
 
        public static void execute(String channel, String sender, String login,
                        String hostname, String[] command)
        {
                sendMessage(sender,"Sending you Source File ...");
                File file = new File("KBot.java");
                DccFileTransfer t = dccSendFile(file,
                sender, 120000);
        }
 
}

le problem ese situe a sendMesage et dccSendFile , se sont des methodes de l'objet PircBot (hérité par KBot) , je voudrais savoir comment faire pour utiliser ces methodes dans un fichier different de la classe elle meme ?
comment faire ?
merci !

31/08/2006, 05h32
bouye

Si ce sont des methodes statique : <nom de classe>.<nom de methode>...
Si ce sont des methodes d'instance : <nom de variable>.<nom de methode> en supposant que <nom de variable> soit de type <nom de class>...

Donc soit : KBot.sendMessage() ou PircBot.sendMsesage()...
Soit il te faut une variable maVariable de type KBot ou PircBot et faire maVariable.sendMessage()...

Resp avec l'autre nom de methode.

j'ai fait comme ca :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
package ircBot.commands;
import java.io.File;
import org.jibble.pircbot.DccFileTransfer;
 
public class req {
 
        public static void execute(KBot k , String channel, String sender, String login,
                        String hostname, String[] command)
        {
                k.sendMessage(sender,"Sending you Source File ...");
                File file = new File("KBot.java");
                k.DccFileTransfer t = dccSendFile(file,
                sender, 120000);
        }
 
}

mais j'ai cette erreur :
KBot cannot be resolved to a type

31/08/2006, 14h31
pmartin8

Tu N'aurais pas oublié ton import par hasard?
31/08/2006, 14h44
kedare

oups en effet , ca marche maintenant , merci :)
31/08/2006, 15h04
Scorpyosis

Pense au tag Résolu alors ;)

pas encore :)
j'ai un autre probleme du meme type
j'ai stocker la liste des fichiers dans un fichier xml , je veut faire le "lecteur xml" dans un fichier different , mais il me trouve une erreur aux {} de la classe ,c'est normal ?

Syntax error on token "{", { expected after this token

(je voudrais pouvoir l'utiliser comme ceci : FilesListReader fichiers = new FilesListReader ....)
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 package ircBot.commands.req; import java.io.File; import javax.xml.parsers.*; import org.w3c.dom.Document; public class FilesListReader { try { File files = new File("files.xml"); DocumentBuilderFactory factory = DocumentBuilderFactory .newInstance(); DocumentBuilder builder = factory.newDocumentBuilder(); Document doc = builder.parse(files); } catch (Exception e) { e.printStackTrace(); } }
(c'est juste le debut de code ca , j'ai pas tout ecrit a cause du probleme qui me casse la tete :))

Ok

Premierement tes imports sont pas tres bons. Import jamais avec *. Si tu utilises Eclipse tu peux arreter de te casser la tete avec ca en apuyant sur Ctrl-Shift-O

Mais ton erreur cette fois-ci viens du fais que tu es en train decrire des instructions a l'extérieur d'une méthode. Les instructions doivent etre a lintéreurs d'une méthode (en général)...

Essaye ca (remarque les imports)
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 package ircBot.commands.req; import java.io.File; import javax.xml.parsers.DocumentBuilder; import javax.xml.parsers.DocumentBuilderFactory; import org.w3c.dom.Document; public class FilesListReader { public void maMethode() { try { File files = new File("files.xml"); DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance(); DocumentBuilder builder = factory.newDocumentBuilder(); Document doc = builder.parse(files); } catch (Exception e) { e.printStackTrace(); } } }

31/08/2006, 16h25
kedare

pourquoi il faut jamais utiliser * ? :)
31/08/2006, 16h33
pmartin8

En utlisant javax.xml.parsers.*;
Ca te donne acces a toutes les classes et les méthodes statiques du package "javax.xml.parsers". En tulisant * le compilateur prend tout le code contenu dans le package parsers et l'inclus a ton projet!! Imagine le code inutile qui est inclus!

Toi tu veux utiliser seulement DocumentBuilder et DocumentBuilderFactory. En important seulement les classes que tu veux, tu t'assures de toujours utiliser les objets et méthodes que tu veux.
Par exemple, tu t'assures de ne pas utiliser des methodes du SAXparser qui est aussi inclus dans le package javax.xml.parsers, mais que tu n'utilises pas dans ton projet. Tu vois que c'est une question de sécurité aussi...

Maintenant un probleme sur le XML ...

voila ma classe pour lire le fichier :

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.req;
 
import java.io.File;
 
import javax.xml.parsers.DocumentBuilder;
import javax.xml.parsers.DocumentBuilderFactory;
 
import org.w3c.dom.Document;
import org.w3c.dom.Element;
import org.w3c.dom.NamedNodeMap;
import org.w3c.dom.NodeList;
 
 
public class FilesListReader {
 
  public void readList() {
    try {
      File files = new File("ircBot/commands/req/files.xml");
      DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
      DocumentBuilder builder = factory.newDocumentBuilder();
 
      Document doc = builder.parse(files);
      Element root = doc.getDocumentElement();
      NodeList list = root.getChildNodes();
      for(int i = 0; i < list.getLength(); i++)
      {
    	 NamedNodeMap fichiers = list.item(i).getAttributes();
    	 String filename = fichiers.getNamedItem("name").getNodeValue();
    	 System.out.println(filename);
      }
    }
    catch (Exception e) {
      e.printStackTrace();
    }
  }
}

voila mon fichier xml:

Code:

1
2
3
4
<?xml version="1.0"?>
<files>
	<file id="1" name="req.java" description="Fichier java source"/>
</files>

j'ai fait une classe executable pour pas avoir a lancer le bot pour lire le fichier xml :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
package ircBot.commands.req;
 
public class TestFile {
 
	/**
         * @param args
         */
	public static void main(String[] args) {
		FilesListReader files = new FilesListReader();
		files.readList();
 
	}
 
}

le probleme c'est que j'ai cette erreur :

Code:

1
2
3
java.lang.NullPointerException
	at ircBot.commands.req.FilesListReader.readList(FilesListReader.java:28)
	at ircBot.commands.req.TestFile.main(TestFile.java:10)

d'ou ca peut venir ?:/
merci

Ok ton ficher XML est un peu mal formé. XML est super intéressant quand il est bien utilisé

regarde ca: je te reviens avec du code pour lire ton fichier!
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 <?xml version="1.0"?> <files> <file> <id>1</id> <name>req.java</name> <description>Fichier java source</description> </file> <file> <id>2</id> <name>req2.java</name> <description>Fichier java source2</description> </file> </files>

Voici ton try-catch refait rapidement

Je ne l'ai pas trop testé mais avec la version du fichier XML que je tai envoyé ca devrait etre #1!

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
39
40
41
42
43
44
45
46
47
48
    try {
      File files = new File("files.xml");
      DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
      DocumentBuilder builder = factory.newDocumentBuilder();
 
      Document doc = builder.parse(files);
      Element root = doc.getDocumentElement();
      NodeList list = root.getElementsByTagName("file");
 
      //Pour tous les fichiers
      for (int i = 0; i < list.getLength(); i++) {
        Element file = (Element) list.item(i);
 
        //Va chercher le ID
        NodeList lId = file.getElementsByTagName("id");
        if(lId.getLength() > 0){
          Element idNode = (Element) lId.item(0);
          NodeList ids = idNode.getChildNodes();
          if (ids.getLength() == 1) {
            String fileId = ids.item(0).toString();
          }
        }
 
        //Va chercher le name
        NodeList lName = file.getElementsByTagName("name");
        if(lName.getLength() > 0){
          Element nameNode = (Element) lName.item(0);
          NodeList names = nameNode.getChildNodes();
          if (names.getLength() == 1) {
            String fileName = names.item(0).toString();
          }
        }
 
        //Va chercher la description
        NodeList lDesc = file.getElementsByTagName("description");
        if(lDesc.getLength() > 0){
          Element descNode = (Element) lDesc.item(0);
          NodeList descs = descNode.getChildNodes();
          if (descs.getLength() == 1) {
            String fileDescription = descs.item(0).toString();
          }
        }
      }
 
    }
    catch (Exception e) {
      e.printStackTrace();
    }

31/08/2006, 18h00
kedare

merci ca fonctionne :)
mais c'est normal que les string renvoyé ai cette forme ?
[#text: req.java]
31/08/2006, 19h01
pmartin8

Normalement non,

Est-ce que tu as bien le meme fichié XML?
De mon coté ca retourne vraiment "req.java"

Sinon vérifie que tu as vraiment le meme code que moi...
31/08/2006, 19h22
kedare

si non comment je peut faire pour acceder au variable (les string que le xml ressort donc) depuis un autre fichier (donc une autre classe ?)
merci
31/08/2006, 19h57
pmartin8

Explique-moi un peu ce que tu tentes de faire et je vais te trouver une bonne facon de le faire... J'ai un peu de temps ;)
31/08/2006, 19h58
kedare

hmm , en faite je pense que je vais pas utiliser le XML pour ca, un simple systeme qui scanne les fichier d'un repertoire sera plus simple a mon avis :p
31/08/2006, 20h02
pmartin8

Oui Effectivement !

Fais moi signe si tu as d'autres questions
31/08/2006, 23h05
bouye

Citation:

Envoyé par pmartin8

En utlisant javax.xml.parsers.*;
Ca te donne acces a toutes les classes et les méthodes statiques du package "javax.xml.parsers". En tulisant * le compilateur prend tout le code contenu dans le package parsers et l'inclus a ton projet!! Imagine le code inutile qui est inclus!

non, Non, NON ET NON !

Le import est l'equivalent du using namespace en C++, pas du #include en C/C++ !!!!!

Il 'sagit d'une racourcit syntaxique destine a la compilation pour la resolution du nom complet d'une classe a partir de son nom court. En utilisant cette directive, IL N'Y A AUCUNE INCLUSION SUPPLEMENTAIRE DE CODE VENANT D'UNE AUTRE CLASSE APRES LA COMPILATION. Juste un remplacement des noms courts par les noms longs dans le code. Java est dynamique, pas statique !

Dans la majorite des cas un .* est largement suffisant. Il peut exister effectivement des cas ou il faut utiliser le nom de classe seule pour eviter des conflits (perso ca ne m'est jamais arrive mais dans une discussion precedente qqun a indique qu'il avait un conflit dans un SDK de Oracle ou je ne sais plus quoi qui n'etait resolu qu'en specifiant le nom de la classe exacte).

Apres il y a effectivement des gens qui preferent mettre juste le nom des classes qu'ils utilisent pour des raisons a eux, principalement parcequ'ils pensent qu'une liste d'inclusion de 3Km de long est plus simple a lire (j'imagine qu'ils la trient eux-meme)... et on peut supposer que la compilation doit etre 1~2 ms alors plus rapide (car la resolution du nom court en nom long est plus rapide mais ce ne sont la que des suppositions). Perso je n'utilise que des .*;

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+

Fuseau horaire GMT +2. Il est actuellement 10h59.