[Résolu] Javascript qui plante

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [Résolu] Javascript qui plante

bepimaco

Festina lente

Je n'arrive pas à comprendre ce qui ne marche pas ?

C'est un script de vérification de formualaire qui permet de choisir les colonnes à imprimer d'une liste d'adhérents. Les 2 premiers tests se font sans rpoblèmes, mais les autres ne passent pas ?

Code :

function valider(){
if(document.imprimerFormulaire.nom.checked == false ){
alert("Vous ne pouvez supprimer le nom" );
return false;
}
if(document.imprimerFormulaire.prenom.checked == false ){
alert("Vous ne pouvez supprimer le prénom" );
return false;
}
var compteur = 54;
if(document.imprimerFormulaire.unite.checked == true ){
compteur = compteur + 35;
}
if(document.imprimerFormulaire.numad.checked == true ){
compteur = compteur + 13;
}
if(document.imprimerFormulaire.naissance.checked == true ){
compteur = compteur + 27;
}
if(document.imprimerFormulaire.age.checked == true ){
compteur = compteur + 11;
}
if(document.imprimerFormulaire.adresse.checked == true ){
compteur = compteur + 101;
}
if(document.imprimerFormulaire.tel1.checked == true ){
compteur = compteur + 21;
}
if(document.imprimerFormulaire.tel2.checked == true ){
compteur = compteur + 21;
}
if(document.imprimerFormulaire.email.checked == true ){
compteur = compteur + 41;
}
var mode = document.imprimerFormulaire.mode.value;
if (mode == "portrait" && compteur > 171) {
alert ("Vous avez sélectionné trop d'information. Passez en paysage" );
return false;
}
if (mode == "paysage" && compteur > 258) {
alert ("Vous avez sélectionné trop d'information." );
return false;
}

Il s'agit de checkbox qui si elles sont cochées permettront l'impression de la colonne correspondante. Le compteur permet de limiter le nombre en fonction de la largeur de la page.
Le mode est un bouton radio qui permet de choisir entre portrait et paysage

Pour info, la page PHP qui gère tout ça fonctionne très bien : c'est donc juste le Javascript qui foire

Merci

Message édité par bepimaco le 24-04-2005 à 09:28:37

---------------
Bepimaco

Publicité

bepimaco

Festina lente

La totalité de la page :

Code :

<!-- Insertion du fichier contenant l'entête de la page
Ne pas oublier de remplir les champs titre et description-->
<?php
$titre="Procédure d'impression - [étape 1/2]";
$base = "liste";
include '../inclus-entetebdd.php';
?>
<script language="JavaScript">
<!--
function valider(){
if(document.imprimerFormulaire.nom.checked == false){
alert("Vous ne pouvez supprimer le nom" );
return false;
}
if(document.imprimerFormulaire.prenom.checked == false){
alert("Vous ne pouvez supprimer le prénom" );
return false;
}
var compteur = 54;
if(document.imprimerFormulaire.unite.checked == true){
compteur = compteur + 35;
}
if(document.imprimerFormulaire.numad.checked == true){
compteur = compteur + 13;
}
if(document.imprimerFormulaire.naissance.checked == true){
compteur = compteur + 27;
}
if(document.imprimerFormulaire.age.checked == true){
compteur = compteur + 11;
}
if(document.imprimerFormulaire.adresse.checked == true){
compteur = compteur + 101;
}
if(document.imprimerFormulaire.tel1.checked == true){
compteur = compteur + 21;
}
if(document.imprimerFormulaire.tel2.checked == true){
compteur = compteur + 21;
}
if(document.imprimerFormulaire.email.checked == true){
compteur = compteur + 41;
}
var mode = document.imprimerFormulaire.mode.value;
if (mode=="portrait" && compteur>171) {
alert ("Vous avez sélectionné trop d'information. Passez en paysage" );
return false;
}
if (mode=="paysage" && compteur>258) {
alert ("Vous avez sélectionné trop d'information." );
return false;
}
if(document.imprimerFormulaire.mode.value == "paysage" ){
alert("Lors de l'impression, sélectionnez paysage" );
return false;
}
}
// -->
</script>
<style type="text/css">
h1 {
text-align: center;
color: black;
z-index: 3;
font-size: 2em;
margin: 0;
padding: 0;
}
table {
text-align: left;
font-family: arial, helvetica, verdana, sans-serif;
background-color : transparent;
margin-top: 10px;
margin-left: 10px;
}
#ajout1{
position: absolute;
margin-top: 150px;
width: 554px;
left: 50%;
margin-left: -277px;
}
#ajout2{
position: absolute;
text-align: center;
background-image: url('../../styles/bdd/images/ecran-chapeau.gif');
width: 554px;
height: 43px;
}
#ajout3{
position: absolute;
margin-top: 43px;
background-image: url('../../styles/bdd/images/ecran-titre.gif');
width: 554px;
height: 43px;
color: black;
z-index: 2;
}
.ajout4{
position: absolute;
margin-top: 86px;
background-image: url('../../styles/bdd/images/ecran-corps.gif');
width: 554px;
height: 339px;
}
#ajout5{
position: absolute;
background-image: url('../../styles/bdd/images/ecran-action.gif');
width: 554px;
height: 43px;
margin: 0;
padding: 0;
z-index: 2;
}
#ajout6{
position: absolute;
margin-top: 431px;
background-image: url('../../styles/bdd/images/ecran-base.gif');
width: 554px;
height: 48px;
z-index: 1;
}
select{
background: #cccccc;
border: solid black 1px;
}
textarea{
background: transparent;
border: solid black 1px;
}
</style>
</head>
<?php
include '../inclus-ambiancebdd.php';
?>
<div id="ajout1">
<div id="ajout2"></div>
<div id="ajout3">
<h1>Impression sélective</h1>
</div>
<div class="ajout4">
<form action="adherents-imprimer2.php" method="post" name="imprimerFormulaire" onSubmit="return valider()">
<table>
<tr>
<td><input name="nom" type="checkbox" value="nom" checked="checked"></td>
<td>Nom</td>
</tr>
<tr>
<td><input name="prenom" type="checkbox" value="prenom" checked="checked"></td>
<td>Prénom</td>
</tr>
<tr>
<td><input name="unite" type="checkbox" value="unite"></td>
<td>Unité</td>
</tr>
<tr>
<td><input name="numad" type="checkbox" value="numad"></td>
<td>Numéro d'adhérent</td>
</tr>
<tr>
<td><input name="naissance" type="checkbox" value="naissance"></td>
<td>Date de naissance</td>
</tr>
<tr>
<td><input name="age" type="checkbox" value="age"></td>
<td>Age</td>
</tr>
<tr>
<td><input name="adresse" type="checkbox" value="adresse"></td>
<td>Adresse</td>
</tr>
<tr>
<td><input name="tel1" type="checkbox" value="tel1"></td>
<td>Numéro de téléphone 1</td>
</tr>
<tr>
<td><input name="tel2" type="checkbox" value="tel2"></td>
<td>Numéro de téléphone 2</td>
</tr>
<tr>
<td><input name="email" type="checkbox" value="email"></td>
<td>Adresse e-mail</td>
</tr>
<tr>
<td><input name="mode" type="radio" value="portrait" checked="checked"></td>
<td>Impression en portrait</td>
</tr>
<tr>
<td><input name="mode" type="radio" value="paysage"></td>
<td>Impression en paysage</td>
</tr>
</table>
<div id="ajout5">
<div id="annuler"><a href="../zone-a/adherents-afficher.php"></a></div>
<div id="valider"><input name="envoyer" type="image" src="../../styles/bdd/images/vide.gif"></div>
</div>
</form>
</div>
<div id="ajout6"></div>
<?php
//récupération des valeurs des champs:
$requete = $_GET["query"] ;
echo $requete ;
?>
</div>
</body>
</html>

Message édité par bepimaco le 22-04-2005 à 18:59:09

---------------
Bepimaco

FlorentG

Déjà les trucs du style :

document.imprimerFormulaire.nom...

Sont à bannir totalement, car ce sont des méthodes d'il y a 20 ans qui ne sont plus valides.
Donc, faisons la mise à jour.

1) Le formulaire. On va dire qu'on garde le tableau, vu que t'as l'air d'avoir trié ça sur deux colonne : champs & libellés de part et d'autres. Déjà t'as pas associé les libellé et les contrôle. Exemple :

<tr>
<td><input id="naissance" name="naissance" type="checkbox" value="naissance"></td>
<td><label for="naissance">Date de naissance</label></td>
</tr>

Donc note l'ajout de l'attribut id dans le input, et l'élément <label>, qui a un attribut for qui contient l'id du contrôle associé.

2) Le JS. Maintenant que tous tes contrôles ont un id, tu peux faire :

if(document.getElementById('naissance').checked == false){
alert("Vous ne pouvez supprimer le nom" );
return false;
}

Et là ça commence à p00trer.

Aussi, on ne met jamais de :

Mais :

Mais le mieux pour ce genre d'erreurs et de balancer ta page au validateur du w3

Sinon, pour ton erreur, y'aurait pas un problème avec les deux premier test qui renvoient [i]false[i] en cas d'erreur, et qui feraient sortir de la fonction, sans passer aux autres tests ?

bepimaco

Festina lente

Merci de ton aide. J'ai troqué les name contre des id, j'ai ajouté les labels. Et pour ce qui est du javascript, je sais pas pourquoi j'ai mis celui-là : une bête erreur de copié collé.

Encore merci, tout fonctionne à merveille

---------------
Bepimaco

cerel

Faut pas enlever les "name". Les name doivent etre uniquement utilise a l'interieur d'un formulaire, et pas ailleurs.

Donc tu remets les name, mais tu laisse les id.

bepimaco

Festina lente

En fait, j'ai mis name + id merci de l'info

---------------
Bepimaco

FORUM HardWare.fr

Programmation

HTML/CSS

[Résolu] Javascript qui plante

Sujets relatifs
[RESOLU] compiler sources c et c++ avec autoconf/automake	[Résolu] Insérer dans un champ texte en fonction de checkbox
[résolu] Signature forum hardware.fr -> cURL	[javascript] pb expression régulière
[C] Valeur reçue différente de la valeur retournée [Résolu]	qui peut dire ce qui cloche dans mon mini prog ?? [ RESOLU ]
le javascript classic	Arrondir un filesize [RESOLU]
[Java] [Resolu] Erreur ODBC État de curseur non valide	Problème Javascript InnerHTML avec firefox
Plus de sujets relatifs à : [Résolu] Javascript qui plante

Page générée en 0.096 secondes