Discussion :

[lucest]

Bonjour à tous,

Je fais appel à vous pour la première fois directement car je sèche vraiment et que je dois trouver une solution très rapidement.

Voici mon pb :

Je réalise une application flash avec une passerelle PHP pour accéder à ma BDD nommé AMFPHP. J'ai donc une classe PHP qui fonctionne trés bien avec plusieurs fonctions dedans sans pb.

J'ai dû rajouter une fonction récursive en plus qui a pour but de générer un login et un pwd automatiquement et d'en faire un nouveau dans le cas où il existe déjà. Le pb est que dès que la récursivité s'enclenche ($this->generateLog($log2) ça me retourne null.

Voici le code qui va plus vous parler :

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
function generateLogPwd($nom, $prenom, $date) {
		$log=strtolower(substr($prenom, 0, 1).$nom);
		$pwd=strtolower($prenom.$date);
		$logGenerate=$this->generateLog($log);
		$tab=array(login=>$logGenerate, pwd=>$pwd);
		return $tab;
	}
	function generateLog($log2) {
		static $nb=0;
		$sql=mysql_query('SELECT login FROM member WHERE login="'.$log2.'"');
		if(!mysql_num_rows($sql)){
			return $log2;
		}else{
			$nb2++;
			$log2=$log2.$nb2;
			$this->generateLog($log2);
		}
	}

Merci d'avance pour vos réponses.

Stéphane.

[binouzzz19]

Salut,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
function generateLog($log2) {
		static $nb=0;
		$sql=mysql_query('SELECT login FROM member WHERE login="'.$log2.'"');
		if(!mysql_num_rows($sql)){
			return $log2;
		}else{
			$nb2++;
			$log2=$log2.$nb2;
			$this->generateLog($log2);
		}
	}

Est-ce normal que tu aies $nb2 au lieu de $nb ?

Cordialement.

[Seb33300]

Bonjour,


Tout d'abord il vaudrait mieux faire

Code :

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

1
2
3
$sql=mysql_query('SELECT COUNT(login) as nb FROM member WHERE login="'.$log2.'"');
$result = mysql_fetch_array($sql);
	if($result['nb'] == 0){

Ensuite, tu incrémente $nb2 dans ton else alors que c'est $nb que tu as déclaré plus haut. :/

[lucest]

Tout d'abord merci pour vos réponses.

J'ai fait les modifs selon vos dires mais rien de mieux.

Je pense que le pb vient plutôt de la définition de la récursivité dans le else de generateLog.

Voici le nouveau code :

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
function generateLogPwd($nom, $prenom, $date) {
		$date=str_replace("/", "", $date);
		$log=strtolower(substr($prenom, 0, 1).$nom);
		$pwd=strtolower($prenom.$date);
		$logGenerate=$this->generateLog($log);
		$tab=array(login=>$logGenerate, pwd=>$pwd);
		return $tab;
	}
	function generateLog($log2) {
		static $nb2=0;
		$sql=mysql_query('SELECT COUNT(login) as nb FROM member WHERE login="'.$log2.'"');
		$result = mysql_fetch_array($sql);
		if($result['nb'] == 0){
			return $log2;
		}else{
			$nb2++;
			$log2=$log2.$nb2;
			$this->generateLog($log2);
		}
	}

[guigo]

Salut,
Il faudrait que tu nous donne le résultat de la fonction pour mieux déterminer le pb...
Mais essaye ca :

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
 
 
function generateLogPwd($nom, $prenom, $date) {
		$date=str_replace("/", "", $date);
		$log=strtolower(substr($prenom, 0, 1).$nom);
 		$pwd=strtolower($prenom.$date);
		$logGenerate=$this->generateLog($log);
		$tab=array(login=>$logGenerate, pwd=>$pwd);
		return $tab;
	}
function generateLog($log2,$nb2=0) {
 
		$sql=mysql_query('SELECT COUNT(login) as nb FROM member WHERE login="'.$log2.'"');
		//$result = mysql_fetch_array($sql);
		$result=mysql_count_rows($sql);
                if($result != 0){
			return $log2;
		}else{
			$nb2++;
			$log2=$log2.$nb2;
			$this->generateLog($log2, $nb2);
		}
	}

A voir, car sans avoir le resultat c pas simple...

Guigo

[lucest]

Merci pour la réponse.

J'ai mis le résultat que cela me renvoyé.

Lorsque dans la fonction récursive je passe dans le if il n'y a pas de pb mais lorsque je passe dans le else (que le login existe déjà) là ça me renvoie null au lieu de rajouter le chiffre $nb au bout du login déjà existant et de réessayer voir si il existe également.

N.B. : mysql_count_rows n'existe pas !

[lucest]

Même résultat avec ça :

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
 
<?
class Tuteur {
function Tuteur(){
		include ("../includes/config.php");
		$logAll="";
	}
function generateLog($log2,$nb2=0) {
		$sql=mysql_query('SELECT COUNT(login) as nb FROM member WHERE login="'.$log2.'"');
		$result = mysql_fetch_array($sql);
		if($result['nb'] == 0){
			$logAll=$log2;
			return $logAll; //retourne (null) si il est passé une fois dans le else avant, et renvoi bien $log2 si il n'y passe pas
		}else{
			$nb2++;
			$log2=$log2.$nb2;
			$logAll=$this->generateLog($log2, $nb2);
		}
	}
}