Discussion :

[moimp]

Bonjour et meilleurs vœux à tous,
Je n'arrive pas à sortir de la boucle forEach en utilisant break;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
function saveDate() { // callBack for calendar
	tabForms.forEach(function(curForm){
		let idId = curForm.querySelector("[name*='id'");
		console.log(idId.value);
		if (idId.value == 0)
			break forEach;
		if (idId.value !=0) { // value=0 means a new element
			console.log(idId.value);
			curForm.submit();
		}
	}, false );
}

J'obtiens une erreur "Uncaught SyntaxError: Undefined label 'forEach'".
Si j'enlève l'étiquette, l'erreur est "Illegal break statement".
Je ne comprends pas comment étiqueter ma boucle pour faire fonctionner.

[ProgElecT]

Salut

Je suis étonné de cette syntaxe break forEach;, pourquoi forEach, a vérifier en regardant l'aide PHP qui ne donne pour paramètre un argument numérique.

[Doksuri]

break forEach;

pour utiliser un label sur un break, il faut avoir defini une region avant (ce qui n'est pas ton cas)
soit tu fais juste un break;, soit tu definis ta region "forEach" quelque part en amont

ps : ProgElecT, c'est pas du php

[NoSmoking]

Bonjour,
on ne quitte pas une boucle forEach, sauf à générer une erreur, dû au fait que c'est une fonction callback qui est exécutée à chaque itération.

Ressource :
• Array.prototype.forEach(), idem pour NodeList.forEach, il faut utiliser un autre type de boucle, voir la Note, en milieu de page, sur le lien.

[ProgElecT]

Désolé, j'ai cru être encor dans le forum PHP au moment de ma rédaction

[moimp]

Merci pour vos réponses qui m'ont éclairé.
J'ai complètement réécris mon code. Je teste la condition avant et je n'exécute forEach que si la condition est remplie.
@Doksuri
C'est ce que j'ai cherché à faire en lisant la doc MDN sans y parvenir.
@NoSmoking
En cherchant dans la doc, je ne me souviens pas avoir vu cette note. Soit je ne suis pas exactement tombé sur cette page, soit j'ai zappé la note.

[moimp]

Je reviens car compte tenu de vos remarques, j'ai encore un problème.
Avec ce nouveau code, je ne comprends pas pourquoi la condition (idId.value) n'est pas prise en compte et doit être remplacée par celle de la ligne 7

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
function saveDate() { // callBack for calendar
	console.log(tabForms);	// Un seul élément indexé 0
	let idId = tabForms[0].querySelector("[name*='id'");
	console.log(idId);		// input#id0, name=0[id], value=0
	console.log(idId.value); 	// 0
	//if (idId.value){ // value=0 means a new element => wait for complete saving
	if (idId.value != 0){ // value=0 means a new element => wait for complete saving
		console.log('boucle exécutée');		// 'boucle exécutée'
		tabForms.forEach(function(curForm){
			console.log('boucle exécutée');	// 'boucle exécutée'
			//curForm.submit();
		}, false );
	}
}

[moimp]

Désolé, je viens de comprendre mon erreur. J'ai confondu l'évaluation de la condition JavaScript avec celle de PHP. Explications ici:

Attention à ne pas confondre les valeurs booléennes « primitives » true et false avec les valeurs true et false d'un objet Boolean. Toute valeur qui n'est pas false, undefined, null, 0, -0, NaN ou la chaîne vide (""), et tout objet, y compris un objet Boolean dont la valeur est false, seront évalués à true lors d'une instruction conditionnelle if.