Discussion :

[laurentSc]

Le titre semble relever du B-A BA; et pourtant, je suis coincé :

Code php :

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

1
2
3
4
5
6
include("message.php");
//foreach ($messages as $key => $value){
foreach ($messages as $value){
	echo " val=".$value."<br>";
	//echo(getMessage($key));echo "<br>";
}

et message.php :

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
22
23
24
25
26
27
28
29
30
<?php
header('Content-type: text/html; charset=UTF-8');
 
$messages=null;
 
function loadMessages()
{
	global $messages;
	$messages=array();
 
	$path="./messages.txt";
 
	if (file_exists($path)) $txt=file_get_contents($path, FILE_USE_INCLUDE_PATH); else exit("file doesn't exist");
	$tab=explode("\n",$txt);
	foreach($tab as $line){
		$s=explode("=",$line);
		$messages[$s[0]]=$s[1];
	}
}
 
function getMessage($key)  {
	global $messages;
	if ($messages==null)
		loadMessages();
 
	$msg=$messages[$key];
 
	return $msg;
}
?>

et j'ai

Warning: Invalid argument supplied for foreach() in C:\wamp\www\classe17\getmessage.php on line 5

[sabotage]

Si tu n'appelles pas tes fonctions, $messages est NULL.

[laurentSc]

Merci pour l'explication (très rapide) ; ça tombe bien que je connaisse le contenu de "message.txt" pour appeler "getMessage" mais en supposant que je ne connaisse pas "message.txt" , y avait-il un moyen de s'en sortir ?

[sabotage]

T'en sortir pour obtenir quoi ?

[Invité]

Bonjour,
- explique exactement ce que ce code est censé faire
- donne-nous un extrait du fichier message.txt (qu'on puisse tester)
- et quand tu as un message d'erreur "....on line 5" : indique-nous quelle est la ligne en question

En l'occurence, ici :

Code :

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

1
2
3
4
5
if(is_array($messages)){
  foreach ($messages as $key => $value){
    // .......
  }
}

Ca ne résoud pas ton problème de conception.
Ca permet juste d'éviter le message d'erreur.

Je t'invite même à relire ta propre "signature" :

Il vaut mieux viser la perfection et la manquer que viser l'imperfection et l'atteindre. - Bertrand Russell

J'ai notamment du mal à comprendre l'intérêt d'écrire des fonctions ici, et ta logique.

J'ajouterai pour finir que ta méthode pour parcourir les lignes d'un fichier (ici avec file_get_contents(...) et explode(..... '\n') n'est peut-être pas la meilleure.

[laurentSc]

Désolé de répondre si tardivement, mais avant, j'ai pas eu de temps.

Un peu de précision : je viens de commencer un bouquin : http://www.amazon.fr/PHP-MySQL-Entra.../dp/2746040573 qu'on m'avait recommandé y a un bout de temps (en janvier dernier !), le but étant de réviser des notions que j'avais vues lors d'une formation (sur le MVC). Ce bouquin a un format curieux : en gros, une série d'exercices, et ils précisent qu'il est mieux de les faire tous...Donc le code n'est pas de moi (à part la ligne pour le mettre en utf-8 et le rajout du "if file-exist" et du coup l'utilisation d'un chemin relatif) ; c'est la solution d'un exercice, juste le fichier "message.php" et "message.txt" :

Code :

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

1
2
titre=bienvenue
invite=Quel est votre nom ?

En gros, qu'est-ce que fait ce code : 2 fonctions :
- loadMessages() qui lit le fichier "message.txt" (qui contient une liste clé/valeur (clé=valeur)) et fabrique un tableau PHP associatif $messages dans lequel on retrouve ces associations (clé/valeur )

- getMessage($key) qui doit retourner la valeur asssociée à la clé passée en argument (dans le tableau $messages).

Et ce que je voudrais obtenir, c'est d'abord de créer le tableau $messages puis de l'afficher (mais en supposant ne pas connaître "message.txt").

Et la ligne de l'erreur (ligne 5) correspond à

Code :

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

foreach ($messages as $value){

[sabotage]

c'est d'abord de créer le tableau $messages

Je repete : étant donné que c'est la fonction loadMessages() qui créé ce tableau, il faut qu'elle soit appellée quelque part avant le foreach.

[laurentSc]

Ca semble évident :

Code :

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

1
2
3
4
5
6
7
<?php
include("message.php");
loadMessages();
foreach ($messages as $key => $value){
	echo(getMessage($key));echo "<br>";
}
?>

Merci Sabotage