Discussion :

[Bruno93]

Bonjour,

je suis débutant en php et en programmation en général. j'avais juste eu besoin de créer un espace personnel pour mon club de tir à l'arc pour la gestion des inscriptions aux concours.

j'apporte cette année une petit modification de rien du tout mais j'ai un peu de mal.

je fait appel à une BDD ou j'extrait les données pour les afficher dans un tableau, tout fonctionne correctement.

dans les données du tableau j'ai ajouté cette année un colonne "statut" avec en fonction des concours le statut OUVERT ou FERME afin que le compétiteur puisse voir s'il peut ou pas s'inscrire à un concours.

pour plus de visibilité je souhaitais mettre OUVERT en vert et FERME en rouge


voici une partie du 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
27
28
29
30
31
32
33
34
35
36
37
38
<?php
//On recupere les données des concours
$req = mysql_query("SELECT id, samedi, dimanche, date_limite, statut, organisateur, affichage, discipline FROM concours  order by id *1"); 
while($dnn = mysql_fetch_array($req))
{
	//changement de couleur pour les mots ouvert ou ferme
	$color="";
	if ($dnn['statut'] = 'OUVERT') 
	(
		$color='green';
	)
	else  
	(
		$color='red';
	)
?>
<script type="text/javascript">
	$(".jour").click(function(event){
		$(".id").attr("checked", false);
		$("input[name='samedi']").attr("checked", false);
		$("input[name='dimanche']").attr("checked", false);
		var $jour = $(this);
		$jour.attr("checked", true);
		$jour.parent().parent().find("td:first").find("input").attr("checked", true);
	})
</script>
	<tr bgcolor="#e9e9e9">
		<td><input type="radio" class="id" name="id" value="<?php echo $dnn['id']; ?>" required><?php echo $dnn['id']; ?></td>	
		<td><input type="radio" class="jour" name="samedi" value="<?php echo $dnn['samedi']; ?>" /><?php echo $dnn['samedi']; ?></td>
		<td><input type="radio" class="jour" name="dimanche" value="<?php echo $dnn['dimanche']; ?>" /><?php echo $dnn['dimanche']; ?></td>
    	<td style="color:red"><?php echo $dnn['date_limite']; ?></td>
		<td style="color:<?php echo$color ?>"><?php echo $dnn['statut']; ?></td>
		<td><?php echo $dnn['affichage']; ?></td>
    	<td style="color:blue"><?php echo $dnn['discipline']; ?></td>
    </tr>
<?php
}
?>

je ne vois trop où j'ai pu commettre une erreur (à mon avis ça doit être une erreur vilaine de débutant, en même temps j'en suis un....)

la partie qui pose problème est dans les lignes suivantes:

Code :

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

1
2
3
4
5
6
7
8
9
10
//changement de couleur pour les mots ouvert ou ferme
$color="";
if ($dnn['statut'] = 'OUVERT') 
(
	$color='green';
)
else  
(
	$color='red';
)

avec comme résultat un message d'erreur suivant:

Parse error: syntax error, unexpected ';' in /mnt/webm/d2/83/52028883/htdocs/espace_membre/inscriptions.php on line 173

la ligne 173 est $color='green';

j'ai essayé pas mal de choses depuis hier soir mais je ne trouve pas !

ce code marche pourtant dans une autre page de l'espace membres mais sous une autre forme pour afficher en vert ou rouge une somme d'argent que le compétiteur doit au club.

si vous pouvez me donner un coup de pouce ou un indice car à mon avis c'est peut être pas grand chose (mais je répète je suis mega débutant)

en tout cas je vous remercie d'avance

Bruno.

[ypcman]

dans le test du if == et pas =
les 2 choix doivent entre {} et pas entre ()

[Bruno93]

Merci vpcman.

de erreur de débutant en fait

par contre en modifiant j'avais un autre message d'erreur avec une boucle à l'infini et un message $end (je n'ai pas noté le détail...

alors j'ai essayé en modifiant le code en mettant un else if à la place comme suit:

Code :

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

1
2
3
4
5
6
7
8
9
10
//changement de couleur pour les mots ouvert ou ferme
$color="";
if ($dnn['statut']=='OUVERT') 
{
	$color='green';
}
else if ($dnn['statut']=='FERME')  
{
	$color='red';
}

et là plus de souci ça fonctionne sans problème.

j'espère que ça servira à d'autres débutants de passage

merci en tout cas pour la réponde rapide !

Bruno

[papajoker]

avec uniquement 2 etats, il y a plus compact (et plus lisible ?)

Code :

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

$color =  ($dnn['statut']=='OUVERT') ? 'green' : 'red';

[ypcman]

En élégant et lisible, je préfère

Code php :

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

1
2
$color = "red";
if($dnn['statut']=='OUVERT') $color= "green";

mais çà reste une affaire de goût