Aide Tableau Java

Version imprimable

Aide Tableau Java

Bonjour,

Merci de me dire si ce code est correct ou pas.et comment l'optimiser.

Quelques détails sur le code :

-J'ai déjà créé les class suivantes :Documents,Livre,Manuels,Romans,Revues,Dictionnaire.(avec tout ce qu'il faut).
-je veux créer une class Biblio(qui gère un tableau de documents) avec les méthodes suivantes :
- Constructeur Tableau.
- Afficher les documents du tableau.
- Afficher les auteurs(sachant que Auteur est un attribut de la class livre).
- Ajout d'un document.
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 49 50 51 52 53 54 public class Biblio { Documents tab[]; public Biblio(int capacite) { tab = new Documents[capacite]; } public void afficherDocument() // { for (int i = 0; i<= tab.length;i++) { System.out.println(tab[i]); } } public void afficherAuteur() { int capacite=0; tab =new Documents[capacite]; for(int i =0 ; i<= tab.length; i++) { System.out.println(tab[i].getNum_enregistrement()); if (tab[i] instanceof Livre) System.out.println(" "+((Livre)tab[i]).getAuteur()); else System.out.println(" "); } } public boolean ajouter(Documents doc) { int i = 0; boolean trouvé = false; while(trouvé == false && i <= tab.length) { if(tab[i].numEnregistrement==(doc.numEnregistrement)) trouvé = true; else i=i+1; } if(trouvé == true) return false; else return true; } }
Merci
:P:P

Bonjour,

personnellement, je changerais 3 choses dans ta classe :

1. Ne pas utiliser un Array de documents mais une liste de documents :

Code:

List tab;

Tu l'initialises ensuite en ArrayList par exemple, cela te permet de ne pas bloquer ta liste de documents a un nombre définit mais de pouvoir en ajouter autant que tu le souhaites (via les méthodes de List, tu as donc les méthodes add, remove, size ou get, regarde la javadoc pour plus de détail).

2. Je n'ai pas testé, mais je pense très fortement que ta méthode afficherAuteur ne marche pas.
En effet, au début de la méthode, tu réinitialises tes documents avec une capacité de 0, normalement, tu ne rentres donc jamais dans la boucle for.
L'utilisation de la classe List pour tes documents, te permettra de résoudre le problème.

3. Dans la méthode ajouter, tu n'est pas obligé d'utiliser le boolean trouvé pour déterminer si tu retournes true ou false.
Plutôt que ca, il te faut renvoyer true par défaut et si tu trouve dans la boucle une correspondance, alors tu renvoie false.
Voici une version modifiée :
Code:

1 2 3 4 5 6 7 8 9 public boolean ajouter(Documents doc) { for (int i = 0 ; i < tab.length ; i++) { if(tab[i].numEnregistrement==(doc.numEnregistrement)) return false; } return true; }
Sans oublier que les identifiants d'un tableau commencent a base 0.
Ce qui veut dire qu'un tableau de taille 5 aura des identifiants de 0 a 4.
Par conséquent, ta ligne :

Code:

while(trouvé == false && i <= tab.length)

Devrait te donner une exception, il te faut arrêter a i < tab.length (ce problème se retrouvera aussi dans la classe List avec la méthode size)

09/02/2010, 09h25
Deaf
Bonjour,

je vais ajouter quelques conseils à ceux de henolivier.

Ton tableau de documents tab (ou ta List) devrait avoir une visibilité private.

Il est préférable d'utiliser les System.out.println() de manière exceptionnelle. Là, tu travailles avec une sortie console, mais tu peux très bien faire évoluer ton application pour avoir une interface graphique par exemple. Dans ce cas, tu pourrais vouloir afficher tes auteur dans un composant plutôt que sur la console.
La solution dans ce cas est de construire ta chaîne de caractères (via un StringBuilder) puis de renvoyer une String. Ainsi, c'est à la méthode appelante de choisir où afficher le résultat.
Cette remarque s'applique pour tes méthodes afficherAuteur et afficherDocument

En ce qui concerne le nom des variables, tu dois proscrire les accents et autres caractères spéciaux. Cela pourrait te poser des problèmes.

Et enfin, il est inutile de comparer un variable boolean avec true ou false.
A la place de :

Code:

if(toto == true || titi == false)

Tu peux écrire:

Code:

if(toto || !titi)

Tu peux même aller plus loin en remplaçant:
Code:

1 2 if(trouve == true) return false; else return true;
par

Code:

return !trouve;

Bonjour,
Merci pour les réponses.

voici le code après avoir appliqué vos conseils.j'y ai ajouté deux méthodes :
1- Permet de renvoyer le i eme document.
2- Supprimer un document référencé par doc et renvoie true ,(false en cas d’échec).

N.B :je suis obligé de travailler avec les tableaux.

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
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
import java.util.Scanner;
 
 
public class Biblio {
 
	Documents tab[];
 
 
	public Biblio(int capacite)
{
	tab = new Documents[capacite];
}
 
 
 
	public void afficherDocument()
{
 
	for (int i = 0; i< tab.length;i++)
	{
		System.out.println(tab[i]);
	}
}
 
 
public void afficherAuteur()
{
	int capacite=tab.length;
	tab =new Documents[capacite];
 
	for(int i =0 ; i< tab.length; i++)
	{
		System.out.println(tab[i].getNum_enregistrement());
		if (tab[i] instanceof Livre)
 
		System.out.println(" "+((Livre)tab[i]).getAuteur());	
 
		else
 
		System.out.println(" ");	
 
	}
}
 
 
public boolean ajouter(Documents doc)
{
for(int i=0; i < tab.length;i++)
{
	if(tab[i].numEnregistrement==(doc.numEnregistrement))
 
 
	    return true;	
 
	else  tab[i]=doc;	
 
}
return false;      
 
}
 
 
public Documents document(int i)
{
	Scanner sc = new Scanner(System.in);
	int var=sc.nextInt();
 
	for(int j=0; j < tab.length;j++)
	{
		if(tab[j].numEnregistrement==var)
			return document(i);
	}
	return null;			
}
 
 
public boolean supprimer(Documents doc)
{
	for(int i=0; i < tab.length;i++)
	{
		if(tab[i].numEnregistrement==(doc.numEnregistrement))
 
		    return true;	
 
		for(int j=i;j<tab.length;j++)
			tab[j]=tab[j-1];
 
	}
	       return false;      	
 
}
 
}

Merci
:P

Je renomerais déjà pas mal de choses:

Une classe doit être au singulier !
Documents -> Document
tab -> documents
...

* J'utiliserais une liste plutot qu'un tableau.

* A chaque classe une seule fonction, il ne faut pas et gérer l'écriture a l'écran et
traiter des informations ...

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
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
 
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
/**
 *
 * @author Toto
 */
public class Library {
 
    /**
     * Documents.
     */
    private List<Document> documents;
 
    /**
     * Create a library for a given capacity.
     * @param capacity maximal capacité
     */
    public Library(final int capacity) {
        documents = new ArrayList(capacity);
    }
 
    /**
     *
     * @return String representation to display
     */
    public String documentToString() {
        final StringBuilder builder = new StringBuilder();
        for (Document doc : documents) {
            if (doc instanceof Book) {
                builder.append(doc.getAuthor());
            }
        }
        return builder.toString();
    }
 
    /**
     *
     * @return all auteru of livre
     */
    public String getAuhtor() {
        final StringBuilder author = new StringBuilder();
        for (Document unDocument : documents) {
            if (unDocument instanceof Book) {
                author.append(unDocument.getAuthor());
            }
        }
        return author.toString();
    }
 
    /**
     * Vrai si le document est deja present.
     * @param doc doc
     * @return Vrai si le document est deja present.
     */
    public boolean isPresent(final Document doc) {
        final boolean trouve = false;
        final Iterator<Document> iterator = documents.iterator();
 
        while ((!trouve) && iterator.hasNext()) {
            if (iterator.next().getNumEnregistrement().equals(doc.getNumEnregistrement())) {
                return true;
            }
        }
        return false;
    }
}

12/02/2010, 04h31
ziliass

Bonjour,

Merci pour votre réponse, mais je suis obligé d'utiliser les tableaux.

Merci
:calim2: