Discussion :

[yoann92]

Bonjour à tous ,

j'ai cette erreur : Notice: Undefined offset: 1
sur la ligne

Code :

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

1
2
 
if (($tab[0] === $username) && ($tab[1] === $md5pass)){

Je sais pas trop quoi faire.
Pourriez vous m'aider s'il vous plaît ?
Voici le 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
21
22
23
24
25
26
 
$username = $this->_request->getPost('username');	
$password = $this->_request->getPost('password');
$auth = false; 
$md5pass = md5($password);
 
$annuaire = fopen('../application/configs/confsecours.txt', 'r'); //ouverture d'un fichier en lecture
 
if (isset($_POST['valider'])){
	if($annuaire){
              while (!feof($annuaire)){
              $ligne=fgets($annuaire,42);
	      $tab=explode(":",$ligne);
                  if (($tab[0] === $username) && ($tab[1] === $md5pass)){
		  //echo "$tab[0]";
		 //echo "$tab[1]";	
	         $auth =true;
	         echo "authentification r&eacuteussie";
		}
		else{
		echo "authentification &eacutechou&eacutee, veuillez recommencer";
		$auth =false;
		}
         }
    }
}

merci d'avance pour votre aide.

[Benjamin Delespierre]

Tu devrais vérifier que la ligne n'est pas vide avant de l'utiliser, tu devrais aussi valider que les éléments sont présents dans ton tableau avant de l'utiliser.
Enfin, tu as des structures de contrôles inutiles.

Voici ton morceau de script corrigé:

Code PHP :

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
// code igniter ??
$username = $this->_request->getPost('username');   
$password = $this->_request->getPost('password');
$auth     = false; 
$md5pass  = md5($password);
$annuaire = fopen('../application/configs/confsecours.txt', 'r');
 
if (isset($_POST['valider']) && $annuaire){
  while (!feof($annuaire)){
    $data = fgetcsv($annuaire,42,':');
    if (!array_filter($data))
      continue; // sauter les lignes vides
    if ($auth = isset($data[0],$data[1]) && $data[0] == $username && $data[1] == $md5pass)
      break; // plus besoin de continuer
  }
 
  if ($auth)
    echo "authentification r&eacuteussie";
  else
    echo "authentification &eacutechou&eacutee, veuillez recommencer";
}

[yoann92]

Bonjour Benjamin Delespierre,
Tout d'abord merci de ton intérêt.
J'ai mis en commentaire mon code pour mettre celui que t'as modifié , cela me fait un écran blanc

Merci d'avance

[Benjamin Delespierre]

Merci d'avance

de quoi ? de deviner l'erreur ?

[yoann92]

non , merci de l'aide que tu m'apportes .

[yoann92]

Le code fonctionne, merci

juste un truc quand il ya un mauvais mot de passe ou quoi
erreur => Warning: array_filter() expects parameter 1 to be array, boolean given in


merci d'avance pour votre aide.

[Benjamin Delespierre]

Ah oui, j'avais oublié ce cas là. Tant mieux, on peut compresser le code un peu plus:

Code PHP :

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

1
2
3
4
5
6
7
8
9
10
11
12
if (isset($_POST['valider']) && $annuaire){
  while (!feof($annuaire) || !$auth){
    if (!$data = fgetcsv($annuaire,42,':'))
      continue;
    $auth = isset($data[0],$data[1]) && $data[0] == $username && $data[1] == $md5pass)
  }
 
  if ($auth)
    echo "authentification r&eacuteussie";
  else
    echo "authentification &eacutechou&eacutee, veuillez recommencer";
}

[yoann92]

Merci beaucoup pour ton aide,

il y a une erreur:
Parse error: syntax error, unexpected '}'
sur l'accolade en dessous de cette ligne

Code :

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

1
2
3
 
 
$auth = isset($data[0],$data[1]) && ($data[0] == $username && $data[1] == $md5pass)

je regarde également pourquoi

merci d'avance

[Benjamin Delespierre]

Il manque un ; à la ligne 5 du script que je t'ai donné.

[yoann92]

Merci Benjamin,

il y a encore erreur :
=> Parse error: syntax error, unexpected ')' in

sur la ligne :

Code :

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

1
2
 
$auth = isset($data[0],$data[1]) && $data[0] == $username && $data[1] == $md5pass);

je suis en train de regarder aussi.

merci d'avance,

[yoann92]

si tu réponds au message, si t'as pas de réponse tout de suite , c'est que je pars du bureau , j'essaie de revenir tout à l'heure.
Sinon je suis là demain à 9h.

Au cas ou, bonne soirée.
Et un grand merci.