Discussion :

[simoxxx48]

Bonjour,

est-ce que c'est possible d'�crire ce code autrement pour que �a soit plus pro ? merci

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

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
if (nameTool.value === "" || nameTool.value == null) {
    nameTool.classList.add("is-danger");
    errorName.classList.add("is-danger");
    errorName.innerText = "Veuillez saisir le nom de l'outil";
    return;
  }
  else if (nameTool.value != null) {
    nameTool.classList.remove("is-danger");
    nameTool.classList.add("is-success");
    errorName.classList.remove("is-danger");
    errorName.innerText = "";
  }
  if (linkTool.value === "" || linkTool.value == null) {
    linkTool.classList.add("is-danger");
    errorLink.classList.add("is-danger");
    errorLink.innerText = "Veuillez saisir le lien de l'outil";
    return;
  }
  else if (linkTool.value != null) {
    linkTool.classList.remove("is-danger");
    linkTool.classList.add("is-success");
    errorLink.classList.remove("is-danger");
    errorLink.innerText = "";
  }
  if (valueCategory.value == "Default") {
    selectCategory.classList.add("is-danger");
    errorCategory.classList.add("is-danger");
    errorCategory.innerText = "Veuillez choisir une catégorie";
    return;
  }
  else if (valueCategory.value != "Default") {
    selectCategory.classList.add("is-success");
    selectCategory.classList.remove("is-danger");
    errorCategory.innerText = "";
  }
  if (checkboxRobot.checked == false) {
    errorRobot.innerText = "Veuillez cocher la case";
    errorRobot.classList.add("is-danger");
    return;
  }
  else {
    cancelModal();
    notificationSuccess();
  }

[nadfri2019]

Salut, il n'y a pas un erreur au niveau des "return;"???

Tes if ne retournent rien, ces return vont g�n�r�s des erreurs dans ton code, supprime les.

Je ne pense pas que les "if else(condition)" soient n�cessaires, un simple "elfe devrait suffire". Etant donn� que si la condition du if n'est pas respect�e, on est forcement dans le cas contraire (cad, xxx.value != null, ou != "Default")

Ensuite, tu peux utiliser classList.toggle() aulieu de classList.add()/remove()
Ou bien classList.replace(oldClass, newClass) qui remplace une class par une autre.

Effet du .toggle(maclass), si la class "maclass" n'est pas pr�sente alors elle est ajout�e, si elle est d�j� pr�sente alors elle sera supprim�e.

Tu devrais gagner quelques lignes.

[javatwister]

Bonjour,

ce serait bien aussi d'avoir une id�e de ton interface html, si elle est fonctionnelle;