Discussion :

[diblasio]

Bonjour,

Je suis étudiant et on apprend a lire dans les fichier en java et utilise netbeans

mon problème est le suivant j'ai une classe nommée Personnes avec comme attribut numeros, nom, prenom, age, salaire, un constructeur vide et un constructeur paramétré avec comme paramètre tous les attribut.
J'ai les getter et setter, et une méthode read qui lit dans un fichier binaire 1 personne a la fois et qui la retourne.
Moi je pensais dans ma méthode read de retourner l'objet en cours avec this au lieu de retourner un objet personne et d'instancier une nouvelle personne à l'intérieur de la methode .
Seulement netbeans me donne une erreur et me dis

illegal start of type
invalid method declaration; return type required

voici ma classe

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
67
68
69
70
71
72
73
74
 
public class Personnes {
 
    private int numero;
    private String nom;
    private String prenom;
    private int age;
    private int salaire;
 
    public Personnes(){
 
    }
    public Personnes(int numero, String nom, String prenom, int age, int salaire) {
        this.numero = numero;
        this.nom = nom;
        this.prenom = prenom;
        this.age = age;
        this.salaire = salaire;
    }
 
    public int getSalaire() {
        return salaire;
    }
 
    public int getAge() {
        return age;
    }
 
    public String getNom() {
        return nom;
    }
 
    public int getNumero() {
        return numero;
    }
 
    public String getPrenom() {
        return prenom;
    }
 
    public void setAge(int age) {
        this.age = age;
    }
 
    public void setNom(String nom) {
        this.nom = nom;
    }
 
    public void setNumero(int numero) {
        this.numero = numero;
    }
 
    public void setPrenom(String prenom) {
        this.prenom = prenom;
    }
 
    public void setSalaire(int salaire) {
        this.salaire = salaire;
    }
 
        public this read(DataInputStream f) throws IOException {
 
        try {
            this.setNumero(f.readInt());
            this.setNom(f.readUTF()) ;
            this.setPrenom(f.readUTF()) ;
            this.setAge(f.readInt());
            this.setSalaire(f.readInt());
        } catch (EOFException e) {
            this = null;
        }
        return this;
    }
}

Et ce que il n'est pas possible de retourner l'objet en cours ou c'est moi qui fais une erreur.
merci d'avance pour votre aide

Di Blasio Michael

[super_toinou]

Hello,

ne pas confondre class et instance.
this est une instance de ta classe Personne.

Ta signature doit retourner des class et pas des instances.
Ca devrait etre donc :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
  public Personnes read(DataInputStream f) throws IOException {
 
        try {
            this.setNumero(f.readInt());
            this.setNom(f.readUTF()) ;
            this.setPrenom(f.readUTF()) ;
            this.setAge(f.readInt());
            this.setSalaire(f.readInt());
        } catch (EOFException e) {
            this = null;
        }
        return this;
    }

Attention à la conception de ton truc, c'est quand même bien faux.
this = null à éviter (je sais mm pas ce que ca donne à l'exécution)!!

De plus ta class devrait s'appeler Personne (sans s à la fin)

Enfin tu devrait créer une méthode en charge de construire et retrouver tes personnes (on appelle ça un DAO pour data access object).

++

[thelvin]

this = null à éviter (je sais mm pas ce que ca donne à l'exécution)!!

'Suffit d'essayer. Ça ne compile pas, donc ça ne s'exécute pas non plus.
On ne peut pas affecter this.

C'est pas tant à éviter, qu'une syntaxe incorrecte qui ne compile pas.

[diblasio]

merci pour la réponse aussi rapide
C'est bien juste this = null ne fonctionne pas
donc si on ne prend pas en compte la DAO car pour l'instant je ne connais. pas la méthode "juste" serrait de créer un nouvelle objet Personne dans ma méthode comme ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
    public Personne read(DataInputStream f) throws IOException {
        Personne pers = null;
        try {
            pers.setNumero(f.readInt());
            pers.setNom(f.readUTF());
            pers.setPrenom(f.readUTF());
            pers.setAge(f.readInt());
            pers.setSalaire(f.readInt());
        } catch (EOFException e) {
            pers = null;
        }
        return pers;
    }

[diblasio]

En lisant un article sur wiki sur DAO
je devrais avoir une classe qui me récupère les personne dans mon fichier
et les stocker dans un Array de personne dans ma classe Personne
et quand je veux récupérer une personne la méthode la récupère dans mon array de personne.

J'ai lu en vitesse sa pourrais être cela le principe

[super_toinou]

Yep c'est ça !!

par contre ton code va te donner un NullPointer
il faut faire une new avant

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public Personne read(DataInputStream f) throws IOException {
        Personne pers = new Personne();
        try {
            pers.setNumero(f.readInt());
            pers.setNom(f.readUTF());
            pers.setPrenom(f.readUTF());
            pers.setAge(f.readInt());
            pers.setSalaire(f.readInt());
        } catch (EOFException e) {
            pers = null;
        }
        return pers;
    }

[diblasio]

Un grand merci pour toute ces info et cette rapidité

mais juste encore un petit détail avant tag en resolu le post

le yep étais pour le DAO ou pour la méthode

[super_toinou]

pour les 2 !

[diblasio]

ok merci