Problème appel de classe Java

**allan7070** · 04/01/2012, 13h56

Bonjour,

Je suis étudiant (Bachelier informatique) et je dois réaliser un projet

J'ai un problème lorsque je veux faire un appel de la classe Livre...
Je lance le programme en faisant un run,
Le Main est dans ma classe ma bilbiotheque, et je lui dis de créer un new Bibliotheque...
Le programe s'éxecute et me demande donc Combien de livre maximum voulez vous introduire dans le catalogue? jusque la tout va bien...
Mais après je lui demande d'instancier de nouveaux objet Livre mais il ne rentre pas dans la classe Livre

Voici ma classe Biblioteque:

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
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
public class Bibliotheque 
{

    private Livre[] catalogue;
    private int nLivre = 0; 
    
    
public Bibliotheque()
{
    System.out.println("Combien de livre maximum voulez vous introduire dans le catalogue?");
   Scanner sc=new Scanner(System.in);
    int maxl = sc.nextInt();
    //System.out.println(maxl);
    catalogue = new Livre[maxl];
    int i;
    for (i=0;i<maxl;i++)
    {
        System.out.println("Entre boucle for");
        catalogue[i] = new Livre();
    }
    
    System.out.println("Que souhaitez vous faire?");
    System.out.println("[1]Ajouter un livre");
    System.out.println("[2]Rechercher un titre");
    System.out.println("[3]Rechercher l'ISBN d'un livre");
    System.out.println("[4]Sortir");
    Scanner ss=new Scanner(System.in);
    int choix = ss.nextInt();
    while (choix > 4 )
    {
        System.out.println("Le numéro encodé est incorrect");
        System.out.println("Que souhaitez vous faire?");
        System.out.println("[1]Ajouter un livre");
        System.out.println("[2]Rechercher un titre");
        System.out.println("[3]Rechercher l'ISBN d'un livre");
        System.out.println("[4]Sortir");
        
    }
    switch (choix)
    {
        case 1: ajout();
            break;
        case 2: rechtitre();
            break;
        case 3: rechIsbn();
            break;
        case 4: System.exit(0) ;
            break;    
    }
    
}

private void ajout()
{
    System.out.println("Entre dans ajout");
    catalogue[nLivre] = new Livre();
    nLivre++;
    System.out.println("Livre ajouté");
            
}
private void rechtitre()
{
    System.out.println("Quel est le titre du livre recherché?");
    Scanner sc=new Scanner(System.in);
    String rech = sc.next();
    
    int i;
    for(i=0;i<nLivre;i++)
    {
        Livre l  = catalogue[i];
        
        if(l.equals(rech) )
        {
            System.out.println(l);
        }
         else System.out.println("Le livre n'est pas présent");
    }
}
private void rechIsbn()
{
    System.out.println("Quel est l'ISBN recherché?");
    Scanner sc=new Scanner(System.in);
    String isbn = sc.next();
    int i;
    for(i=0;i<nLivre;i++)
    {
        Livre l = catalogue[i];
        if(l.equals(isbn))
  
        {
            System.out.println(l);
        }
        else System.out.println("L'ISBN n'est pas présent");
    }
}

    public static void main(String[] args) 
    {
        Bibliotheque maBibli = new Bibliotheque();
    }
}

Et voici ma classe Livre

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
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
import java.util.*;
 
public class Livre
{
	/**
          * Variable privé qui stocke le titre du livre
          */
private String titre;
	/**
          * Stocke l'ISBNdu livre
          */
private String isbn;
	/**
          * Stocke le prixe du livre
          */
private float prix;
	/**
          * Stocke le nom de l'éditeur du livre
          */
private String editeurLivre;
	/**
          * Stocke le nom de l'auteur du livre
          */
private String auteurLivre;
 
/**
    * Constructeur par défaut
   */
public Livre()
{
this.titre="";
this.isbn="";
this.prix=0;
this.editeurLivre="";
this.auteurLivre="";
}
 
/**
    * Constructeur d'initialisation
    */
public void init()
{
Scanner sc=new Scanner(System.in);
System.out.print("titre du livre :");
this.titre=sc.nextLine();
System.out.print("isbn");
this.isbn=sc.next();
System.out.print("prix");
this.prix=sc.nextFloat();
sc.skip("\n");
System.out.print("éditeur");
this.editeurLivre=sc.nextLine();
System.out.print("auteur");
this.auteurLivre=sc.nextLine();
}
 
//***************************************************************************
//                                                                     Getters
//**************************************************************************
 
/**
    * Retourne l'auteur du livre
   * @return l'auteur du livre
   */
public String getAuteurLivre()
 {
return this.auteurLivre;
}
 
/**
    * Retourne l'editeur du livre
   * @return l'editeur du livre
   */
public String getEditeurLivre()
 {
return this.editeurLivre;
}
 
/**
    * Retourne l'ISBN du livre
   * @return l'ISBN du livre
   */
public String getIsbn()
 {
return this.isbn;
}
 
/**
    * Retourne le prix du livre
   * @return le prix du livre
   */
public float getPrix()
 {
return this.prix;
}
 
/**
    * Retourne le titre du livre
   * @return le titre du livre
   */
public String getTitre()
 {
return this.titre;
}
 
//***************************************************************************
//                                                               Setters
//***************************************************************************
 
/**
    * Définit l'auteur du livre
   * @param auteurLivre
   *                 l' auteur du livre
   */
public void setAuteurLivre(String auteurLivre)
 {
this.auteurLivre = auteurLivre;
}
 
/**
    * Définit l'editeur du livre
   * @param editeurLivre
   *                  l'editeur du livre
   */
public void setEditeurLivre(String editeurLivre)
 {
this.editeurLivre = editeurLivre;
}
 
/**
    * Définit le prix du livre
   * @param prix
   *                  le prix du livre
   */
public void setPrix(float prix)
 {
this.prix = prix;
}
 
}

Je ne vois vraiment pas pourquoi il ne veut pas rentrer dans ma classe Livre depuis ma classe Biblioteque...

**CheryBen** · 04/01/2012, 14h27

Salut, est-ce que le message "Entre boucle for" s'affiche?
Pour ce genre de problème, lancer l'application en mode debug dans un IDE (Eclipse, NetBeans ou IntelliJ) est très utile et très simple.

**professeur shadoko** · 04/01/2012, 14h44

plusieurs remarques:
- pour interagir avec l'utilisateur fait d'abord un objet Interaction pour lire/afficher depuis la console (ça t'évitera bien des surprises , en particulier le fait de recréer plusieurs Scanners

)

- du coup l'objet bibliotheque ne fera plus des choses étranges dans son constructeur (tu créera un bibliothèque de N livres quand tu auras saisi N)

- à (mon pas très humble) avis c'est une mauvaise idée de créer comme ça des livres "vides": crée le livre quand tu auras son titre, etc. (et pas de set: si ça s'appelle "bible" c'est pas "à l'ombre des jeunes filles en fleur"). (je sais que je vais entendre des commentaires à ce sujet mais je persévère à affirmer que les "set" intempestifs sont une ineptie de gestion)

**allan7070** · 04/01/2012, 16h05

Envoyé par CheryBen

Salut, est-ce que le message "Entre boucle for" s'affiche?
Pour ce genre de problème, lancer l'application en mode debug dans un IDE (Eclipse, NetBeans ou IntelliJ) est très utile et très simple.

Oui le message "boucle for" s'affiche bien, c'est pour cela que je ne comprend pas pourquoi il n'appelle pas la classe Livre...

**JoeChip** · 04/01/2012, 16h19

Que veux-tu dire par "appeller la classe Livre" ?

**Robin56** · 04/01/2012, 16h39

Envoyé par allan7070

Oui le message "boucle for" s'affiche bien, c'est pour cela que je ne comprend pas pourquoi il n'appelle pas la classe Livre...

Mets un point d'arrêt dans ton constructeur de Livre, là tu seras vraiment sur.

PS : Est-ce normal que ta méthode init() n'est appelée par aucune autre dans ta classe Livre ?

**thierryler** · 04/01/2012, 16h43

Déjà, au lieux d'utiliser des tableaux

Code :

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

1
2
 
private Livre[] catalogue;

Essaie avec des listes

Code :

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

1
2
 
List<Livre> livres = new ArrayList<Livre>();

En plus des réponses déjà données. J'ai le sentiment qu'un truc va se débloquer ;-)

En outre, dans ton code

Code :

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

1
2
3
4
5
 
 switch (choix)
    {
        case 1: ajout();
            break;

Tu appelles donc le bloc

Code :

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

1
2
3
4
5
6
7
8
9
 
private void ajout()
{
    System.out.println("Entre dans ajout");
    catalogue[nLivre] = new Livre();
    nLivre++;
    System.out.println("Livre ajouté");
 
}

Or ton constructeur

Code :

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

1
2
3
4
5
6
7
8
9
 
public Livre()
{
this.titre="";
this.isbn="";
this.prix=0;
this.editeurLivre="";
this.auteurLivre="";
}

Bah il ne fait rien...

Tu devrais peut être ajouter un appel à ta méthode init() quelque part...

**allan7070** · 04/01/2012, 19h31

Merci à tous pour vos réponses

Je pense que l'erreur devrait se situer au niveau de mon constructeur d'initialisation merci à vous Robin56 et Thierry.
Malheureusement je ne saurais tester ca maintenant (Boulot, boulot) mais je vais tester ca demain matin

Ce site est vraiment génial j'ai toujours recu beaucoup de réponses à mes problèmes et surtout de BONNES réponses constructive... Encore merci je vous tiens au courant dés demain

**thierryler** · 04/01/2012, 20h21

C'est surtout grâce à la qualité des réponses de robin et thierry

**allan7070** · 05/01/2012, 09h13

Voila merci le programe tourne nickel maintenant

J'ai juste ajouter la méthode Init dans mon constructeur de la classe Livre
Comme ceci:

public Livre()
{
this.titre="";
this.isbn="";
this.prix=0;
this.editeurLivre="";
this.auteurLivre="";
init();
}

**CheryBen** · 05/01/2012, 11h06

Envoyé par thierryler

C'est surtout grâce à la qualité des réponses de robin et thierry

Genre

Apprendre à debugger en mode pas à pas est primordial pour tout développeur.

A part ça Allan, je te conseille de séparer la classe Livre de sa méthode init(). (comme l'avait fait remarquer professeur shadoko) Cette méthode devrait être dans une autre classe gérant les intéractions clavier, le gros avantage est que tu pourras ainsi instancier des Livres autrement que via le scanner. Que se passera-t-il lorsque tu feras évoluer ton application et que tu voudras créer des Livres via une interface Swing? tu devras modifier ta classe Livre. Le scanner t'empêche également d'utiliser des Livres dans des tests unitaires.

Tu devrais faire par exemple :

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
public class Console {
    private static Scanner sc=new Scanner(System.in);
 
    public static Livre creerLivre() {        
        System.out.print("titre du livre :");
        String titre=sc.nextLine();
        System.out.print("isbn");
        String isbn=sc.next();
        System.out.print("prix");
        String prix=sc.nextFloat();
        sc.skip("\n");
        System.out.print("éditeur");
        String editeurLivre=sc.nextLine();
        System.out.print("auteur");
        String auteurLivre=sc.nextLine();
 
        return new Livre(titre, isbn, prix, editeurLivre, auteurLivre);
    }
}
 
public class Livre {
    public Livre(String titre, String isbn, String prix, String editeurLivre, String auteurLivre)
    {
        this.titre=titre;
        this.isbn=isbn;
        this.prix=prix;
        this.editeurLivre=editeurLivre;
        this.auteurLivre=auteurLivre;
    }
}

En faisant ainsi tu n'auras plus jamais besoin de modifier ta classe Livre, et elle peut être utilisée de diverses manières, peu importe si elle est intégrée à une console, une interface web, swing, dans des tests unitaires, etc...

Problème appel de classe Java

avec Java

Discussions similaires

Partager

Partager