Ajout de nouveaux lieux et noms pour les salons vocaux créés

[Kekwel]

Trouver d'autres noms et lieux pour un peu plus diversifier

via la méthode ci dessous déjà présente ou refactor pour passer les lieux et noms autre part en constantes ?

CDS-BOT/events/voiceStateUpdate.js

Lines 73 to 82 in 729b0ad

	function getChannelName() {
	// TODO a revoir
	const lieu = ['🪑 Bureau', '🛏️ Chambre', '🛋️ Salon', '🍽️ Cuisine', '🕸️ Grenier', '🚿 Salle de bain'];
	const names = ['Carmack', 'GabeN', 'Miyamoto', 'Kojima', 'Howard'];
	const lieuRandom = lieu[Math.floor(Math.random() * lieu.length)];
	const nameRandom = names[Math.floor(Math.random() * names.length)];
	return `${lieuRandom} de ${nameRandom}`
	}

[rick-gnous]

Peut-être mettre ça dans un fichier à part pour vraiment mettre n'importe quoi ? genre un json, juste un fichier texte avec une liste...

[0xN0x]

Comme ce sera utilisé par rien d'autre que cette fonction là, soit le laisser dans le fichier car c'est pas très grave, soit le déplacer dans data/<thing>.json je pense

[rick-gnous]

comme vous le sentez, j'ai tendance à mettre les datas à côté mais c'est vrai que ce n'est pas critique non plus

[0xN0x]

J'ai tendance a penser que c'set mieux de les séparer aussi, surtout si la liste va potentiellement continuer de grandir, mais j'avoue que comme c'est utilisé par uniquement cette fonction, je ne sais pas non plus ce qui est le mieux en terme de bonne pratique 🤷‍♂️

[rick-gnous]

bon bah, on laisse la décision finale à celui qui va faire ce ticket, et on va le pourir si on est pas d'accord avec 👀 /j