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;