Discussion :

[TheBuzz]

Bonjour à Toutes et Tous,

Voici une question des plus simples pour les non-amateurs comme moi :

Après un appel à une requête MySQL prenant l'ensemble des données de ma table et suivant une boucle while, j'affiche correctement mes données selon le modèle suivant :

valeur1 valeur2 valeur3 valeur4

Maintenant je désire la chose suivante :
Si la valeur de valeur2 contient 0, alors n'affiche pas valeur2

Au final, j'aurais donc :
valeur1 valeur3 valeur4

Merci pour vos commentaires

[Sebastien14]

Salut,

Et pourquoi tu ne fais pas simplement un if sur ta valeur 2??!

[TheBuzz]

salut Sebastien14,

Je fais:

Code :

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

if ($valeur['valeur2'] = 0 ? '' : $valeur['valeur2']);

mais rien, l'affichage se fait toujours

[Sebastien14]

Il faut faire :

if ($valeur['valeur2'] == 0)

Il faut bien m'être deux fois le signe égal.

[TheBuzz]

Ai bien fait la modification:

Code :

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

if ($valeur['valeur2'] == 0 ? '' : $valeur['valeur2']);

mais toujours identique à l'affichage

[kstou2001]

pourquoi tu as un ?
et au lieu de faire

if ($valeur['valeur2'] == 0)

fais plutot if ($valeur['valeur2'] != 0)
vu que si c'est egal a 0 tu ne fais rien

[TheBuzz]

Merci de vos réponses,

Je joins mon code pour plus de clareté :

Code :

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

1
2
3
4
5
$cadre_recherche = tep_db_query("SELECT * FROM products_cadre");
	while( $cadre = tep_db_fetch_array($cadre_recherche) ) {
	if ($cadre['cadre_RAL'] == 0 ? '' : $cadre['cadre_RAL']);
		$tmp_html .= '<p class="Item" onmouseover="MouseOverItem(this)" onmouseout="MouseOutItem(this)" onclick="CacheItem(\'liste1\',\'' . $cadre['products_cadre_id'] . '\', \'' . ' ' . $cadre['cadre_texte'] . ' ' . $cadre['cadre_RAL'] . '\'); MouseOutItem(this)"><img class="images" id="' . $cadre['products_cadre_id'] . '" src="http://www.living-stores.com/catalogue/images/' . $cadre['cadre_image'] . '" alt="' . $cadre['cadre_texte'] . '" />' . '   ' . $cadre['cadre_texte'] . ' ' . $cadre['cadre_NCS'] . ' - ' . $cadre['cadre_RAL'] . ' - ' . $cadre['cadre_VSR'] . ' - ' . $cadre['cadre_STOBAG'] . '</p>';
	}

[Sebastien14]

Oui c'est vrai que viens faire ce ?, tu veux faire quoi exactement comme condition?

[kstou2001]

en gros faudrait faire un truc du style:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
$tmp_html .= "<p class=\"Item\" onmouseover=\"MouseOverItem(this)\" onmouseout=\"MouseOutItem(this)\" onclick=\"CacheItem('liste1','".$cadre['products_cadre_id']."', ". $cadre['cadre_texte']."
 
if($cadre['cadre_RAL'] !="0")
{
    $tmp_html .= $cadre['cadre_RAL']
}
$tmp_html .= "'); MouseOutItem(this)\"><img class=\"images\" id=\"".$cadre['products_cadre_id']."\" src=\"http://www.living-stores.com/catalogue/images/".$cadre['cadre_image']."\" alt=\"".$cadre['cadre_texte']."\" />"."   ".$cadre['cadre_texte']." ".$cadre['cadre_NCS']." - "
 
if($cadre['cadre_RAL'] !="0")
{
    $tmp_html .= $cadre['cadre_RAL']
}
$tmp_html .= " - ".$cadre['cadre_VSR']." - ".$cadre['cadre_STOBAG']."</p>";

[ChRiiS]

TheBuzz,
En gros, si compris, tu veux afficher ton echo de toute facon, mais si $cadre['cadre_RAL'] vaut 0, tu voudrais ne pas l'afficher, c'est ça ?

Si oui, en gardant ton if façon ternaire ça donnerais:

Code :

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

1
2
$cadre['cadre_RAL']=($cadre['cadre_RAL']==0?'':$cadre['cadre_RAL']);
echo '<p class="Item" onmouseover="MouseOverItem(this)" onmouseout="MouseOutItem(this)" onclick="CacheItem(\'liste1\',\'' . $cadre['products_cadre_id'] . '\', \'' . ' ' . $cadre['cadre_texte'] . ' ' . $cadre['cadre_RAL'] . '\'); MouseOutItem(this)"><img class="images" id="' . $cadre['products_cadre_id'] . '" src="http://www.living-stores.com/catalogue/images/' . $cadre['cadre_image'] . '" alt="' . $cadre['cadre_texte'] . '" />' . '   ' . $cadre['cadre_texte'] . ' ' . $cadre['cadre_NCS'] . ' - ' . $cadre['cadre_RAL'] . ' - ' . $cadre['cadre_VSR'] . ' - ' . $cadre['cadre_STOBAG'] . '</p>';

Mais sinon au format "habituel" ça donnerais plutôt:

Code :

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

1
2
if($cadre['cadre_RAL']==0) $cadre['cadre_RAL']='';
echo '<p class="Item" onmouseover="MouseOverItem(this)" onmouseout="MouseOutItem(this)" onclick="CacheItem(\'liste1\',\'' . $cadre['products_cadre_id'] . '\', \'' . ' ' . $cadre['cadre_texte'] . ' ' . $cadre['cadre_RAL'] . '\'); MouseOutItem(this)"><img class="images" id="' . $cadre['products_cadre_id'] . '" src="http://www.living-stores.com/catalogue/images/' . $cadre['cadre_image'] . '" alt="' . $cadre['cadre_texte'] . '" />' . '   ' . $cadre['cadre_texte'] . ' ' . $cadre['cadre_NCS'] . ' - ' . $cadre['cadre_RAL'] . ' - ' . $cadre['cadre_VSR'] . ' - ' . $cadre['cadre_STOBAG'] . '</p>';

Tiens nous au courant si on est passé à côté de ce que tu voulais.

[TheBuzz]

Merci à tous de vos réponses.

J'ai en fait effectué ce code ci-dessous qui est absolument fonctionnel et inclus dans mon while :

Code :

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

1
2
3
4
$RAL = ' - ' . $cadre['cadre_RAL'];
		if ( $RAL == " - 0" ) {
			$RAL = '';
		}

et rajouter la valeur $RAL au bon endroit.



Merci encore et A+