Discussion :

[ETVigan]

Bonjour,

Ce n'est pas un problème de syntaxe ponctuel que je soumets mais + un souci de construction de mon module PHP qui est ponctué de TRY THROW CATCH... et à la limite, cela rend la lisibilité du code difficile...

Voici un extrait:

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
	 	
try
 {	// Verifier que le requête n'est pas vide ! 
   	// ----------------------------------------
       if( is_string($this->query) ) 
	   try
	  {	if( $this->result = mysql_query($this->query,$this->idConn[$this->id]) ) 
		{	// On teste le type de requête 'select','show','explain','describe')
							preg_match( '`SELECT|SHOW|DESCRIBE|EXPLAIN`i' , $this->query , $match );
							try
		 					{	switch( $match[0] )
								{	case 'SELECT' 	:	$i = 0;
				 										// On construit notre $resArray
				 										while( $row = mysql_fetch_assoc($this->result) ) 
														{	$resArray[$i++] = $row;
				 										}	//end while
									
				 										// On libere la ressource
														$this->mysqlFreeResult($this->result) ;
														return($resArray) ;
														break;
													
									case 'SHOW'		:	
									case 'EXPLAIN'	:
									case 'DESCRIBE'	:	// on informe juste que c'est bien passer
			      										return true;
									default			:	// On lance une exception  
		      											throw new Exception("SQL didn't find correct keyord to match MYsql, please check it !") ;
			  					}//end switch	
			 				}  catch(Exception $E) 
								{	$this->display_exception($E) ;
								}
						}				
					else 
						{	
		 				} 
			 	}  catchh(Exception $E) 
		    		{	$this->display_exception($E) ;
					}
			else 
				throw new Exception("SQL request string is empty and/or not valid, please check it !") ;
		}   catch(Exception $E) 
		    {	$this->display_exception($E) ;
			}

Code qui contient 3 X le trio (try, throw , catch).

Alors je me pose la question suivante, une construction du style suivant est'elle possible et accepté par PHP autrement si vous en avez une autre...

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
try
{	// Verifier que le requête n'est pas vide ! 
   	// ----------------------------------------
	if( is_string($this->query) ) 
		{	if( $this->result = mysql_query($this->query,$this->idConn[$this->id]) ) 
				{	// On teste le type de requête ('select','show','explain','describe')
					preg_match( '`SELECT|SHOW|DESCRIBE|EXPLAIN`i' , $this->query , $match );
		 			{	switch( $match[0] )
						{	case 'SELECT' 	:	$i = 0;
				 								// On construit notre $resArray
				 								while( $row = mysql_fetch_assoc($this->result) ) 
												{	$resArray[$i++] = $row;
				 								}	//end while
									
				 								// On libere la ressource
												$this->mysqlFreeResult($this->result) ;
												return($resArray) ;
												break;
													
							case 'SHOW'		:	
							case 'EXPLAIN'	:
							case 'DESCRIBE'	:	// on informe juste que c'est bien passer
			      								return true;
							default			:	// On lance une exception  
		      									throw new Exception("SQL didn't find correct keyord to match MYsql, please check it !") ;
			  			}//end switch	
				}				
			else 
				{	
		 		} 
		}  
	else 
		throw new Exception("SQL request string is empty and/or not valid, please check it !") ;
}   catch(Exception $E) 
	{	$this->display_exception($E) ;
	}

Construction dans laquelle je mets tout mon code entre une seul TRY - CATCH avec X THROW....

Qu'est ce cela donne au point de vue performance ?

Merci de votre feed-back

[grunk]

Une indentation correcte du code t'aiderais à y voir clair , là c'est un peu du n'importe quoi.

A un bloc try/catch correspond un test particulier et une exception particulière. Si tu as 3 try/catch c'est que tu à trois levée d'exception possible , tu ne devrais pas les remplacés par un seul. C'est cependant possible si les trois exception sont du même type.

De même je trouve ça étrange de levée des exceptions dans un try alors que son but est justement de tester leur absence (ou non).

Au nivea ude l'indentation il y'a deux standard soit :

Code :

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

1
2
3
4
5
if{
    //code
} else {
    //code
}

Soit

Code :

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

1
2
3
4
5
6
7
8
9
 
if
{
    //code
} 
else 
{
    //code
}

[ETVigan]

Au sujet de l'indentation, elle est due au copie/Collé sur le site et je en l'ai pas refaite.
Mais on a chacun ces petites habitudse et la mienne diffère de celles que tu proposes mais pas de bcp....
Au sujet du "Try Throw & Catch" je suis arrivé à une solution telle que je la proposais dans le forum, je ne sais si elle est valide pour un procédure ou un ensemble, moi j'ai l'ai faite pour une proc.

Mais je me pose toujours des questions quant au performance car je suppose que cela bouffe des ressources, qui peut me dire ?

A part cela tout va mieux sous le soleil, j'espère que tu en as aussi

[grunk]

Mais je me pose toujours des questions quant au performance car je suppose que cela bouffe des ressources

Un try/catch n'est pas consommateur de ressource. pas plus qu'une levée d'exception.
Dans un cas tu exécutes ton code classiquement dans l'autre tu lance une exception et termine prématurément le script donc pas de souçis à avoir coté performance avec les exceptions.