Discussion :

[gpierre13]

Bonjour à tous,

voici un bout de code en Javascript tiré d'un programme sur lequel je travail.

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
 
function submit_simulate(docType) {
 
          if (simulatePressed || docType == 'nothing') {
              return true;
          }
 
          if (isBasketEmpty()) {
              return false ;
          }
 
          if ( checkDeliveryDate() == false ){
          	alert("<isa:translate key="b2b.order.required.delivryDate" />");
          	return false;
          }
 
          if ( checkingDate == false ) {
          	if ( checkReqDateChange(true) == false ){
	          	// -- prevent user to send wrong date.
	          	return false;
	          }
 
          }else{
          	//if orderTool.js is running, don't go to next step !
          	return false;
          }
 
 
 
 
 
          // ----------------------------------
          //UPDATED ( 24-04-06 )
          //          
          if ( docType == 'order' || docType == 'ordertemplate'){
          	//MUST take a look about all selected product.
          	var wrongList = isBasketContainsMultipleSameProduct() 
 
          	if ( wrongList.length > 0 ){
          		if ( wrongList.length == 1 ){
          			alert("<isa:translate key="b2b.ordr.alr.basket.multiple.product"/>" + '\n' + "<isa:translate key="b2b.order.display.productno"/> : " + wrongList[0]);
 
 
          		}else{	
          			var fullList = '';
          			for(var i = 0; i < wrongList.length; i++){
          				//
          				fullList += "<isa:translate key="b2b.order.display.productno"/> : " + wrongList[i];
          				if ( i < wrongList.length - 1 )
          					fullList += '\n';
          			}
          			alert("<isa:translate key="b2b.ordr.alr.basket.multiple.products"/>" + '\n' + fullList);
          		}
          		return false;
 
          	}
          }
          // ----------------------------------
 
          parent.positions.document.forms["order_positions"].simulatepressed.value=docType;
          parent.positions.document.forms["order_positions"].submit();
          simulatePressed = true;
          return true;
 
      }

D'après mes connaissances, toute la deuxième partie de ce code devrait ne devrait jamais être exécuté car après un

Code :

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

1
2
3
4
5
 
else{
          	//if orderTool.js is running, don't go to next step !
          	return false;
          }

la fonction devrait renvoyé FALSE et se terminer.

Pourtant, après de nombreux test, cette fonction arrive jusqu'à la fin de son code...


Quelqu'un aurait-il une explication à me fournir ? Car je ne comprends vraiment pas.

Merci d'avance et bon travail à tous !!


Greg.

[tusssss]

Parce que tu ne rentre jamais dans ce else ?

As tu essayé d'y placer un alert dans ce else pour vérifier?

[gpierre13]

Mais avant le ELSE, il n'y a que de IF avec à chaque fois des RETURN dedans, donc théoriquement, quelles que soient les conditions, le programme rentra dans un IF ou finira dans le ELSE et donc passera pas un RETURN.

Non ?


Mais tu as raison, je vais de ce pas mettre une alerte dans le ELSE.

++

[tusssss]

Code :

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

1
2
3
4
5
6
7
8
9
10
if ( checkingDate == false ) {
          	if ( checkReqDateChange(true) == false ){
	          	// -- prevent user to send wrong date.
	          	return false;
	          }
 
          }else{
          	//if orderTool.js is running, don't go to next step !
          	return false;
          }

En regardant de plus près, tu verras que si on rentre dans le premier if et pas dans le second, on se retrouve à passer sans rencontrer de return puisque le else est accroché au premier if....

[gpierre13]

Ha ouiiiiiiiiiiii...


Bien vu !!

Merci beaucoup pour ton aide !!!

Bye.