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 ocuments,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.
 -Permet de renvoyer le i eme document.
- Supprimer un document référencé par doc et renvoie true ,(false en cas d’échec).
 
 
Merci pour vos réponses.
 
CODE JAVA:
 

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;          
    
}
 
}

On n'a pas accès à la classe Document donc on ne sait pas ce qu'elle offre ni ce qu'est le champ numEnregistrement.
 
Hélas toutes les méthodes que je vois là sont fausses, il n'y a que le constructeur qui est correct et encore, il manque une initialisation: le nombre de documents actuellement insérés dans la bibliothèque doit être stocké dans la classe et initialisé à 0.
 
Pour supprimer un document passé en paramètre, il faut juste que tu parcoures le tableau de Document de 0 au nombre de documents actuellement insérés, en comparant à chaque fois le document en cours à celui passé en paramètre et en t'arrêtant dès que tu le trouves. Pour savoir comment comparer 2 objets Document, il faut connaître la définition de la classe Document. Si par exemple la méthode equals() est implémentée, il suffit de l'appeler. Sinon il faut comparer un champ faisant office de "clé primaire". Dans le cas particulier où un objet Document est toujours représenté par la même instance dans le système (plutôt rare), une simple comparaison d'instances (==) suffit.

idées rapides apres lecture rapide :
 
public class Biblio extends ArrayList{
}
 
je te suggere de regarder ArrayList (http://java.sun.com/j2se/1.4.2/docs/api/java/util/ArrayList.html) pour comprendre pourquoi
 
Ensuite on connait pas ta classe Document, mais peut etre que  
 
interface Document{  
    public String getAuthor();    
}
 
et  
 
public class Livre implements Document{
    private String author;
    public String getAuthor(){
        return this.author;
    }
}
 
ca pourrait t'aider.
 
[edit]
attention ca marchera pas forcement de suite. mais le principe c'est qu'en java ca sert a rien de reinventer la roue, le mieux c'est toujours de lire la doc.
d'ailleurs l'idéal ca serait que tu regardes du coté des "generics" et Collection<> ou List<>
 
[/edit]