Discussion :

[cinhic]

Bonjour,

cela fait un moment que je n'ai plus touché à PHP...
J'ai une instruction à traduire en PHP, pour creer un formulaire orienté objet( ce que je ne maitrise pas du tout , mais j'essaye ).

Code :

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

1
2
3
4
5
6
7
8
echo "<form name='f' action='index.php?page=".$numero_de_page+1."' method='post' >"; 
Pour chaque question $q de $this->pages[$numero_de_page] 
echo $q->texte; 
Si $q->type=="radio" 
pour chaque reponse $r de $q->reponses[] 
echo "<input type='radio' name='".$q->id."' value='".$r."' >"; 
fin de pour 
fin de si

je l'ai traduis ainsi :

Code :

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

1
2
3
4
5
6
7
function afficher($numero_de_page){ 
    echo("<form name='f' action='index.php?page=".$numero_de_page+1./"' method='post'>");   
     foreach ($this->page[$numero_de_page] as $q) { 
      echo $q->$texte; 
     if ($type=="radio"){ 
      foreach ($q->reponses[''] as $r) { 
      echo ("<input type='radio' name='".$q->id."' value='".$r."'>");}

bon ou pas bon?

Est ce que je suis plus censée dire

Code :

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

for $this->page[$numero_de_page]==$q

?
ou

Code :

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

$f=$q->afficher($this->pages[$numero_de_page])

Pourriez-vous m'aider ? merci d'avance.

[ThomasR]

Bonjour,

bon ou pas bon?

Ou la question : est-ce que ca fonctionne ?

La bonne synthaxe est bien :

Code :

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

foreach ($this->page[$numero_de_page] as $q)

Ici tu ne définies jamais $type. Tu voulais faire $q->type plutôt ?

Code :

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

if ($q->type === 'radio')

[cinhic]

Super Thomas, merci pour ta réponse.
Mon parseur avait du mal a passer aux noeuds suivants, du coup je ne pouvais pas trop verifier tous mes codes.
La page blanche n'étant plus blanche, je m'en vais de ce pas réctifier mes $q->type!

A...très bientôt je crois, et merci pour la réponse claire, rapide, et pas agressive ( ba oui, j'avais un peu peur des : "vas lire la FAQ et arrêtes de poster pour rien!")
donc merci merci merci merci

[BornBanane]

Quelques petites erreures je pense :

Code :

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

echo("<form name='f' action='index.php?page=".$numero_de_page+1./"' method='post'>");

=>

Code :

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

echo("<form name='f' action='index.php?page='".$numero_de_page+1."' method='post'>");

Le / est inutile et il manque un ' juste après le page=

Code :

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

foreach ($q->reponses[''] as $r) {

=>

Code :

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

foreach ($q->reponses as $r) {

Le foreach porte sur un tableau donc c'est plus logique comme ceci.

Tu as plein d'accolades non fermées. Mais la ça dépend de la logique donc je te laisse faire

[cinhic]

Super, petit à petit le code est plus propre. En faite j'ai pas mal d'autres attributs, radio, texte, time etc...donc je ne vous ai donné qu'un apperçu du début du code, et j'i un peu zappé la fermeture des balises...mais elles sont là en réalité.

Merci pour ton aide BornBanane

[ThomasR]

Quelques petites erreures je pense :
=>

Code :

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

echo("<form name='f' action='index.php?page='".$numero_de_page+1."' method='post'>");

Le / est inutile et il manque un ' juste après le page=

Attention, ceci est faux, en mettant un apostrophe après page, tu termines l'attribut action. Son code était donc correct (sauf le "/").

Les parenthèses autour de echo sont dépréciées, echo n'est pas une fonction.

Code :

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

echo '<form name="f" action="index.php?page='.($numero_de_page+1).'" method="post">';

[cinhic]

Exacte!