Générateur de citations aléatoires en JavaScript

Question

Bonjour Coach, voici la résolution de mon travail
même si j'ai prix presque 2 jours pour pouvoir y arriver 😂
let citationPlace = document.querySelector('#citation');
let auteur = document.querySelector('#auteur');
let nouveau = document.querySelector('#nouveau');
// Tableau de citations
let citations = [
["La vie est un mystère qu'il faut vivre, et non un problème à résoudre.", "Gandhi"],
["Le plus grand risque est de ne prendre aucun risque.", "Mark Zuckerberg"],
["Méritez votre statut de leader chaque jour.", "Mickael Jordan"],
["Soyez le changement que vous voulez voir dans le monde.", "Gandhi"],
["A chaque fois que vous vous retrouvez du même côté que la majorité, il est temps de prendre du recul, et de réfléchir.", "Mark Twain"],
["Seulement ceux qui prendront le risque d’aller trop loin découvriront jusqu’où on peut aller.", "T.S Elliot"],
["Le succès c’est tomber sept fois, se relever huit.", "Proverbe japonais"],
["Dans vingt ans vous serez plus déçus par les choses que vous n’avez pas faites que par celles que vous avez faites. Alors sortez des sentiers battus. Mettez les voiles. Explorez. Rêvez. Découvrez.", "Mark Twain"],
["Si vous attendez pour agir, tout ce que vous gagnerez, avec le temps, c’est de l’âge.", "Brian Tracy"],
["Quand on concentre son attention sur un seul projet, l’esprit suggère constamment des idées et des améliorations qui lui échapperaient s’il était occupé avec plusieurs projets en même temps.", "P.T. Barnum"],
["Se dédier à faire tout ce que l’on peut pour aider les autres à obtenir ce qu’ils veulent, c’est la clé du succès.", "Brian Sher"],
["Si vous pensez que vous êtes trop petit pour avoir de l’impact, essayez d’aller au lit avec un moustique.", "Anita Roddick"],
["Ne jugez pas chaque jour sur ce que vous récoltez, mais sur les graines que vous semez.", "Robert Louis Stevenson"],
["L’action est la clé fondamentale de tout succès.", "Pablo Picasso"],
["Le succès, c’est se promener d’échecs en échecs tout en restant motivé.", "Winston Churchill"],
["Votre avenir est créé par ce que vous faîtes aujourd’hui, pas demain.", "Robert T. Kiyosaki"],
["Ne vous découragez pas, c’est souvent la dernière clef du trousseau qui ouvre la porte.", "Zig Ziglar"],
["Pour gagner votre vie, apprenez à l’école. Pour gagner une fortune, apprenez par vous-même.", "Brian Tracy"],
["Les gagnants trouvent des moyens, les perdants des excuses…", "F. D. Roosevelt"],
["Vous n’êtes jamais trop vieux pour vous fixer de nouveaux buts, ou rendre vos rêves réalité.", "C.S. Lewis"],
["Un pessimiste voit la difficulté dans chaque opportunité. Un optimiste voit une opportunité dans chaque difficulté.", "Winston Churchill"]
];
const max = citations.length;
let dernier = 0;
let nombreAleatoire = 0;
function genererNombreEntier(max) {
return Math.floor(Math.random() * Math.floor(max));
}
nouveau.addEventListener('click', () => {
do {
nombreAleatoire = genererNombreEntier(max);
} while (nombreAleatoire === dernier);
dernier = nombreAleatoire;
citationPlace.textContent = citations[nombreAleatoire][0];
auteur.textContent = citations[nombreAleatoire][1];
});

Louis-Nicolas Leuillet · Accepted Answer

Deux jours, c'est rien ! L'important c'est que tu y es arrivé, et le résultat est vraiment propre.
Le code est bien structuré : la séparation des données (le tableau citations ), la logique (la fonction genererNombreEntier ) et la gestion des événements. La boucle do...while pour éviter d'afficher deux fois la même citation à la suite, c'est exactement ce qu'il fallait faire, bien joué.
Une petite chose à noter : tu peux éviter de déclarer nombreAleatoire en dehors de la fonction click , puisqu'elle n'est utilisée qu'à l'intérieur. Ça donnerait quelque chose comme ça :
nouveau.addEventListener('click', () => {
let nombreAleatoire;
do {
nombreAleatoire = genererNombreEntier(max);
} while (nombreAleatoire === dernier);
dernier = nombreAleatoire;
citationPlace.textContent = citations[nombreAleatoire][0];
auteur.textContent = citations[nombreAleatoire][1];
});
C'est un détail, mais c'est une bonne habitude de limiter la portée des variables au strict nécessaire. Dans l'ensemble, c'est du bon travail, continue comme ça !

Générateur de citations aléatoires en JavaScript

Formation JavaScript (legacy)