Du vector à la NullPointerException, pourquoi ?

Marsh Posté le 02-11-2005 à 21:20:51

Je suis en train de faire un petit programme d'échange de fichiers, pour
l'instant ce n'est que je début mais je rencontre déja un problème.

J'essai de transférer des objets Réponse (voir le code ci-dessous) entre
un serveur et un client. Je rencontre un problème au niveau du serveur
mais je n'arrive pas du tout à comprendre quelle en est la raison. Je
poste ici car j'ai passé mon aprés-midi à essayer de trouver la
solution :sweat:

Selon les traces d'exécution le client arrive bien a créer un objet
réponse, à l'instancier et à l'utiliser. Tout le contraire du serveur
qui créer l'objet, l'instancie mais ne peut l'utiliser... Avez vous une
idée ?

Merci de votre aide.

L'exécution du serveur :

---------Demarrage Server---------

Ouverture du serveur : 2024
>> Connection du client : /127.0.0.1
chiffre : 1
Exception in thread "main" java.lang.NullPointerException
at Reponse.setCode(Reponse.java:15)
at FTPRequest.<init>(FTPRequest.java:34)
at Server.<init>(Server.java:38)
at Server.main(Server.java:71)

L'exécution du client :

---------Demarrage Client---------

chiffre : 1

!!!! Erreur de connection !!!!

Impossible de se connecter au serveur.
Verifiez que le logiciel serveur est bien lance sur l'hote
distant.

Le code de la classe réponse :

Code :

import java.util.*;
import java.lang.*;
public class Reponse implements java.io.Serializable{
protected Vector <String> answer;
protected boolean codeOk = false;
public void Reponse(){
answer = new Vector <String> ();
}
public void setCode(int code){
System.out.println("chiffre : " +String.valueOf(code));
System.err.println("size = "+answer.toString());
answer.add(String.valueOf(code));
codeOk = true;
}
public void setData(String m){
if(codeOk && answer.size()!=0)
answer.add(m);
}
public int getCode(){
return Integer.parseInt(answer.elementAt(0).toString());
}
public int getNbElement(){
return answer.size();
}
public String getFirstElement(){
return getDataAt(1);
}
public String[] getData(){
String[] rep = new String[answer.size()-1];
if(codeOk && answer.size()>1){
for(int i = 1; i < answer.size() ; i++)
rep[i-1]=answer.elementAt(i).toString();
return rep;
}
return null;
}
public void afficherData(){
if(codeOk && answer.size()>2){
System.out.println("> "+answer.elementAt(0).toString()+" :
"+answer.elementAt(1));
for(int i = 1; i < answer.size() ; i++)
System.out.println(" "+answer.elementAt(i));
}
}
public String getDataAt(int i){
if(codeOk && answer.size()>1 && i < answer.size() && i > 0)
return answer.elementAt(i).toString();
else
return null;
}
public String getCmd(){
String request = this.getDataAt(1);
int index = request.indexOf(' ');
if (index != -1)
return request.substring(0, index).toUpperCase();
else
return request.toUpperCase();
}
public String getArg(){
String arg = this.getDataAt(1);
int index = arg.indexOf(' ');
if(index != -1)
return this.getDataAt(1).substring(index + 1);
else
return null;
}
}

Le code du client :

Code :

import java.io.*;
import java.net.*;
import java.util.*;
import java.lang.*;
import java.lang.Thread;
import java.lang.Object;
public class Client extends Thread {
/** Socket du client**/
private Socket sock;
/** Buffer d'entree des informations serveur (messages)*/
private ObjectInputStream in;
/** Messages envoyes par le serveur */
private Reponse reponse;
/** Port utilise par defaut par le client*/
public static final int PORT = 2024;
/** Port utilise par le client pour se connecter au serveur*/
public static int port;
/** Port utilise par le client pour se connecter au serveur*/
public static InetAddress adrServ ;
/** Constructeur de l'objet Client
@param sock socket du serveur*/
public Client(Socket sock) throws Exception {
this.sock = sock;
in = new ObjectInputStream(this.sock.getInputStream());
}
/** Thread de reception et de traitement des messages et donnees
serveur */
public void run() {
while (true) {
try {
reponse = (Reponse) in.readObject() ;
} catch (Exception e) {}
if(reponse.getCode()==0){
System.err.println("Connection Terminee" );
System.exit(0);
}
System.out.println("> "+reponse.getCode()+" :
"+reponse.getDataAt(1));
for(int i = 2; i < reponse.getNbElement() ; i++)
System.out.println(" "+reponse.getDataAt(i));
}
}
/** Methode main : lance le Thread de reception et
envoie les commandes entrees par l'utilisateur vers le serveur.
@param args parametres du client */
public static void main(String args[]) {
String sentence;
System.out.println("---------Demarrage Client---------" + '\n');
if (args.length > 2) {
System.err.println("Usage: Client ou Client [ port ] ou Client
[ port ] [ ip serveur] " );
System.exit(1);
}
port = PORT;
if (args.length >= 1) {
port = Integer.parseInt(args[0]);
}
try{
adrServ=InetAddress.getLocalHost();
if (args.length == 2) {
adrServ = InetAddress.getByName(args[2].toString());
}
}
catch(Exception e){
System.err.println("!!! Erreur : Le cerveur cible n'existe pas.\n" );
}
try {
Socket clientSocket = new Socket(adrServ, port);
ObjectOutputStream out = new
ObjectOutputStream(clientSocket.getOutputStream());
out.flush();
Client c = new Client(clientSocket);
BufferedReader inFromUser = new BufferedReader(new
InputStreamReader(System.in));
Reponse commande = new Reponse();
commande.setCode(1);
c.start();
while (true) {
sentence = inFromUser.readLine();
commande.setData(sentence);
out.writeObject(commande);
}
} catch (Exception e) {
System.err.println("\n\n!!!! Erreur de connection !!!!\n" );
System.err.println("\nImpossible de se connecter au serveur." );
System.err.println("Verifiez que le logiciel serveur est bien lance
sur l'hote distant.\n\n" );
}
}
}

Le code du serveur (une partie) :

Code :

import java.net.*;
import java.io.*;
import java.util.*;
import java.lang.*;
import java.lang.reflect.*;
public class FTPRequest extends Thread{
/** Tampon d'entree des commandes client */
protected ObjectInputStream in;
/** Flux de sortie vers le client */
protected ObjectOutputStream out;
/** Socket attache au client */
private Socket sock;
protected Reponse com;
protected Reponse reponse;
/** Constructeur du constructeur du gestionnaire de requete FTP.
@param sock socket attache au client.
@param f dossier de demarrage du serveur. */
public FTPRequest(Socket sock, String f) throws Exception {
this.sock = sock;
out = new ObjectOutputStream(this.sock.getOutputStream());
out.flush();
in = new ObjectInputStream(this.sock.getInputStream());
reponse = new Reponse();
com = new Reponse();
com.setCode(1);
reponse.setCode(1);
this.start();
}
/** Constructeur thread serveur.
*/
public void run() {
try{
reponse.setData("220 localhost FTP server ready.\n" );
out.writeObject(reponse);
this.processRequest();
}
catch(Exception e){}
}
/** Methode principale du gestionnaire de requete FTP.*/
private void processRequest() throws Exception {
int i = 2;
while (com.getCode()!=90) {
com = (Reponse) in.readObject();
reponse.setCode(i);
i++;
out.writeObject(reponse);
}
sock.close();
}
}

Reply

Marsh Posté le 02-11-2005 à 21:20:51

Reply

Marsh Posté le 02-11-2005 à 21:35:12

il me semble que le NPE vient du fait que ton Vector est vide

je sais pas trop pourquoi, mais j'ai eu a peu près la même chose avec une LinkedList<String>

avant et après l'instanciation par new LinkedList<String>(), je faisais un test par un autre thread
sur (maListe!=null) et qui me répondait toujours true

donc ta ligne 15 dans Reponse.java peut faire la même erreur
essaie de faire un test sur answer.size()>0 (qui marche mieux que (answer!=null) du coup)
et d'afficher answer.toString que si le résultat est vrai

mais alors pourquoi il dit que answer et mon truc à moi sont null, je sais pas pourquoi et même, je comprends pas :heink:

---------------
The Rom's, à votre service

Reply

Marsh Posté le 02-11-2005 à 21:39:55

public void Response() ?

skoi ce constructeur avec un type de retour ? c'est legal, ca, en java ?

Reply

Marsh Posté le 02-11-2005 à 21:51:59

ah ben ouais ca viens de là
public void Reponse() c'est une méthode
et public Reponse() il instancie pas ton Vector !
(genre le constructeur de base)

mais ca trouve pas plus d'explications à mon truc ca, dommage

---------------
The Rom's, à votre service

Reply

Marsh Posté le 02-11-2005 à 21:53:40

Mais quel con je suis :fou:

Bon désolé... d'un coté je ne ferais plus l'erreur.

Merci à tous deux pour votre aide et à chrisbk pour la solution :hello:

Reply

Marsh Posté le 02-11-2005 à 21:56:12

mais c'est la méthode avec le même nom que la classe qui me semble bizarre
ou alors c'est vraiment un constructeur qui renvoie un void donc un null,
mais apparement non, sinon l'exec aurait bloqué à "com.setCode(1);"
dans FTPRequest.<init>(FTPRequest.java:34)

---------------
The Rom's, à votre service

Reply

Marsh Posté le 03-11-2005 à 16:53:53

En fait c'était juste le constructeur... Programmant avec vim je n'ai pas de création automatique de fichier .java comme avec un ide plus développé :whistle: