problème de conditions

Version imprimable

Bonjour,
j'ai un souci dans cette fonction et j'avoue que je ne vois pas où est le hic
J'ouvre un fichier contenant une liste d'identifiants, je ne travaille qu'avec les 3 premiers.
Le problème est que quand je n'en met qu'un, il ne rentre pas dans la boucle (le test preint ne marche pas), les autres conditions par contre fonctionnent ?????:calim2::aie:

Code:

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
function get_scenario($file){

$open = fopen($file, "r") or die("<strong>problème d'ouverture fichier</strong>");
		
	if ($open) {
	$cmpt=0;
	$id="";
		while (!feof($open)) {
		$cmpt++;
			$id=$id.fgets($open, 4096);
		}	
	$id=rtrim($id);
	$ID = str_replace("\n"," ",$id);
	$IDS = str_replace("\n","_",$id);
	
	if ( ($cmpt<2) && (!empty($id)) ) {//ne respecte pas la condition
	$requete_python = ('python createNewickTags.py '.$ID.'');
	print ("    test    ");print($requete_python) or die;
	exec($requete_python);	
	
	$create_page = (' perl ../bin/tree_viewer/view-tree.pl query='.$ID.' db=HOVERGEN type=species > ../tmp/scenario.html');
	exec($create_page);
	}
		elseif ( ((($cmpt>2) && ($cmpt<=4))) && (!empty($id)) ){
		$requete_python2 = ('python createNewickTags.py "'.$ID.'"');
		exec($requete_python2);	
		
		$create_page2 = (' perl ../bin/tree_viewer/view-tree.pl query='.$IDS.' db=HOVERGEN type=species > ../tmp/scenario.html');
		exec($create_page2);
		}	
			elseif ($cmpt>4){
			$_SESSION['ERROR'] = ('Cette requete ne prend que 3 IDs en argument<br> veuillez resaisir une requete');
			header("location:../pages/acceuil2.php") or die("Ne retourne pas à la page d'acceuil");	
			}			
	}
	fclose($open);
		
	header("location:../tmp/scenario.html");

}

Où me suis-je donc planté ????:(:(:(:(:(:(

20/07/2009, 14h26
fucce

Résultat normal à cause de ta boucle while : tu vas boucler en récupérant toutes tes lignes jusqu'à la fin de ton fichier. Mets tes tests dans ta boucle, cela fonctionnera mieux.
PS : il est pas très clean ton code!! :roll:
20/07/2009, 14h28
Yuki_Hime

tu veux bien préciser un peu plus stp !!! je met les if directement dans le while alors?

ben oui je suis une newbie en php, sorry si c'est pas joli
20/07/2009, 14h35
Celira

Personnellement je laisserais les tests où ils sont et je commencerai par faire un var_dump de $cmpt et $id avant le if... histoire de voir pourquoi la condition n'est pas validée...
20/07/2009, 14h40
Yuki_Hime

En mettant le if dans le while, seule la première condition fonctionne................

En faisant le var_dump j'obtiens
var_dump($cmpt); var_dump($id);
int(2) string(9) "HBG000010"

j'avoue que je suis perdue
20/07/2009, 14h54
Celira

Aux dernières nouvelles, 2 n'est pas strictement inférieur à 2, donc la condition $cmpt <2 ne peut pas être validée :roll:
Je pense donc que ta condition devrait plutôt être un inférieur ou égal

Code:

if ( ($cmpt <= 2) && (!empty($id)) )
20/07/2009, 14h56
Yuki_Hime

Effectivement............................................................:cry:
Now ça marche !!!!!:ccool:
thx

Proposition pour coder différemment cette fonction :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
function get_scenario($file){
 
	$arguments = file($file) or die("<strong>problème d'ouverture fichier</strong>");
 
	$cmpt=count($arguments);
	if ($cmpt < 4) {
		$IDpython = implode(' ',$arguments);
		$requete_python = ('python createNewickTags.py '.$IDpython.'');
		exec($requete_python);	
 
		$IDperl = ($cmpt<3) ? $IDpython : implode('_',$arguments);
		$create_page =(' perl ../bin/tree_viewer/view-tree.pl query='.$IDperl.' db=HOVERGEN type=species > ../tmp/scenario.html');
		exec($create_page);
	}	
	else {
		$_SESSION['ERROR'] = ('Cette requete ne prend que 3 IDs en argument<br> veuillez resaisir une requete');
		header("location:../pages/acceuil2.php") or die("Ne retourne pas à la page d'acceuil");	
	}			
}

Fred