Discussion :

[Yuki_Hime]

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 ?????

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
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é ????

[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!!

[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

[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...

[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

[Celira]

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

Code :

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

if  ( ($cmpt <= 2) && (!empty($id)) )

[Yuki_Hime]

Effectivement............................................................
Now ça marche !!!!!
thx

[micetf]

Proposition pour coder différemment cette fonction :

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
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