Discussion :

[nicolas2603]

Bonjour, j'aurais besoin de vos lumières pour comprendre pourquoi un code javascript est lu même lorsqu'il est dans une condition qui n'est pas validée. Attention, je dis lu, pas exécuté ! Je sais qu'il est lu car mon outil Web Developer me retourne une erreur sur une des lignes dans la condition.

Je m'explique : sur une page j'ai un formulaire; lorsqu'il est envoyé, la page est rechargée et m'affiche une div. Sur cette page j'ai un variable javascript qui prend pour valeur la valeur PHP envoyée par la méthode POST.

Le problème c'est qu'à la première arrivée sur la page, puisqu'aucune valeur POST n'est encore envoyée, je n'ai pas de valeur PHP et donc ma variable javascript retourne une erreur. Logique. Donc j'ai pensé mettre une condition qui vérifiait si la div était visible (POST passé) ou pas, mais malgré ça la ligne est quand même lue.

Code :

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

1
2
3
4
5
if ($('#tabs').is(':visible')) {
	var mc=<?php echo $_POST['mot']; ?>;
	$('#child').val(mot);
	alert('detect');
}

Au premier chargement de la page, pas d'alert donc, mais une erreur à la ligne 2 "syntax error : var mc=;". Une explication ? Merci d'avance.

[SpaceFrog]

il est ou le test isset sur le post ?

[Invité]

salut,

je sais pas ce que te renvois ta page au premier chargement, mais si elle te renvoit du code html avec un

Code :

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

<div id="tabs"></div>

, a fortiori, le div n'a pas de raison detre caché et le if est validé.

L'erreur comme lindique web developper c'est que ton code avec le truc php équivaut à écrire :

Code :

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

var mc=;

ce qui est syntaxiquement invalide.
(pour ten convaincre, clique droit, code source sur ta page php générée dans ton browser).

Donc le plus simple c que tu écrives

Code :

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

var mc='<?php echo $lol;?>';

et là pu de probleme.

[SpaceFrog]

Donc le plus simple c que tu écrives
Code :
Sélectionner tout - Visualiser dans une fenêtre à part

var mc='<?php echo $lol;?>';

et là pu de probleme.

???
heu le plus simple est de faire un test isset ...

[Invité]

Bonjour,
on peut mixer les 2 solutions :

var mc='<?php if(isset($_POST['mot'])) echo $_POST['mot']; ?>';

Ce qui donnera :

var mc='';
ou
var mc='lemot';

Ce qui règle en même temps les problèmes d'erreur PHP et JavaScript.

[nicolas2603]

J'ai comme le sentiment que vous allez me dire que j'ai super mal construit ma page !! J'ai en premier mon formulaire, ensuite j'ai le isset, puis j'ai le javascript :

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
<div>
    <form>...</form>
    <?php if (isset($_POST['mot'])) { ?>
        <div id="tabs">...</div>
    <?php } ?>
</div>
 
<script>
$(function() {
if ($('#tabs').is(':visible')) {
	var mc=<?php echo $_POST['mot']; ?>;
	$('#child').val(mot);
	alert('detect');
}
});
</script>

Je viens de tester en mettant le javascript DANS le isset, et c'est pareil !

[nicolas2603]

En revanche, la solution de rajouter les ' ' semble résoudre le problème, mon erreur n'apparait plus ! Merci à tous.

[Eric2a]

Salut,

Si tu ne veux ni la balise (div="tabs") ni la balise script quand la variable n'est pas transmise, tu peux aussi faire...

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
<?php
$mot=(isset($_POST['mot']))?$_POST['mot']:'';
?>
 
<div>
	<form>...</form>
<?php if($mot!='') { ?>
		<div id="tabs">...</div>
<?php } ?>
</div>
 
<?php if($mot!=''){ ?>
<script type="text/javascript">
	$(function(){
		if ($('#tabs').is(':visible')){
			var mc='<?php echo $mot; ?>';
			$('#child').val(mot);
			alert('detect');
		}
	});
</script>
<?php } ?>

[nicolas2603]

Oui c'est une solution également. Merci !

[Invité]

???
heu le plus simple est de faire un test isset ...

le prends pas mal si mon avis diffère. Moi cque je vois c'est que quelquesoit la valeur de mot, qui sémantiquement est un string, on va se taper une erreur en ayant var mc=leMotAvecRienAutour;

Effectivement, lexécution de la présence de tabs ou pas, n'a rien a voir avec le probleme du mot. Mais la question était double.
Pourquoi le if est exécuté, et pourquoi ca merde a la ligne var mc.

edit:jreaux62 avait déjà précisé le probleme.