search

matthieu637 / cpp-2a-crawler

2 stars 0 forks source link

Rudder #46

Open matthieu637 opened 7 years ago

matthieu637 commented 7 years ago

Je n'ai pas suivi à quoi servait la méthode addPriorityLink(List<PriorityLink> links) ?

Toutes les méthodes marquées par @Override n'ont pas forcément besoin d'être documentée (par exemple dans NaiveRudder), par contre ça serait bien que les méthodes de la classe abstraite principale (Rudder) le soient.

david540 commented 7 years ago

En faite c'était surtout pour les tests unitaires, savoir si les liens sont bien triés par ordre croissant de la valeur de priority, je ne pouvais pas faire cette vérification avec le rudder naif. Mais il n'apporte rien au programme en lui-même

matthieu637 commented 7 years ago

Ah d'accord. Je pense que déporter 

        for (PriorityLink link : links)
            LinkQueue.getInstance().addPriorityLink(link);

dans le tests unitaires serait mieux puisque c'est tout ce que fait la méthode.

david540 commented 7 years ago

D'accord, je ferais une nouvelle classe de test pour la LinkQueue par la même occasion.