Pk ça boucle pas :'(

Pk ça boucle pas :'( - HTML/CSS - Programmation

Marsh Posté le 12-02-2006 à 13:08:54    

voila le script,
 
et le résultat : "2"
 
oui mais moi c 1 puis 2 puis 3 que je voudrais voir affiché en boucle
 
 

Code :
  1. <SCRIPT type="text/javascript">
  2. function montre(object)
  3.   {
  4.   if (document.layers && document.layers[object])
  5.     {
  6.     document.layers[object].visibility = 'visible';
  7.     }
  8.   else if (document.all)
  9.     {
  10.     document.all[object].style.visibility = 'visible';
  11.     }
  12.   else if (document.getElementById)
  13.     {
  14.     document.getElementById(object).style.visibility = 'visible';
  15.     }
  16.   }
  18. function cache(object)
  19.   {
  20.   if (document.layers && document.layers[object])
  21.     {
  22.     document.layers[object].visibility = 'hidden';
  23.     }
  24.   else if (document.all)
  25.     {
  26.     document.all[object].style.visibility = 'hidden';
  27.     }
  28.   else if (document.getElementById)
  29.     {
  30.     document.getElementById(object).style.visibility = 'hidden';
  31.     }
  32.   }
  33. i=1;
  34. function changeMessage()
  35. {
  36. if (i==1)
  37. {
  38.   montre('txtNom');
  39.   cache('txtPrenom');
  40.   cache('txtAge');
  41. }
  42. else
  43. {
  44.   if (i==2)
  45.   {
  46.    cache('txtNom');
  47.    montre('txtPrenom');
  48.    cache('txtAge');
  49.   }
  50.   else
  51.   {
  52.    if (i==3)
  53.    {
  54.     cache('txtNom');
  55.     cache('txtPrenom');
  56.     montre('txtAge');
  57.    }
  58.   }
  59. }
  60. }
  61. i=i+1;
  62. if (i==4)
  63. {
  64. i=1;
  65. }
  67. setInterval("changeMessage()",1000);
  69. </SCRIPT>
  70. <div id=VIP>
  71.   <table id="txtNom" style="visibility:hidden;">
  72.   <tr>
  73.   <td>1</td>
  74.   </tr>
  75.   </table>
  76.   <table id="txtPrenom" style="visibility:hidden;">
  77.   <tr>
  78.   <td>2</td>
  79.   </tr>
  80.   </table>
  81.   <table id="txtAge" style="visibility:hidden;">
  82.   <tr>
  83.   <td>3</td>
  84.   </tr>
  85.   </table>
  86. </div>


Message édité par Dorian BAC+4 le 12-02-2006 à 15:39:26
Reply

Marsh Posté le 12-02-2006 à 13:08:54   

Reply

Marsh Posté le 12-02-2006 à 13:38:15    

La flemme de chercher, mais qu'est-ce que le javascript est mal codé.
 
tes if enchainés sont particulièrement horribles.

Reply

Marsh Posté le 12-02-2006 à 14:21:27    

et la balise [code] ??

Reply

Marsh Posté le 12-02-2006 à 18:24:20    

up non?

Reply

Marsh Posté le 12-02-2006 à 18:25:46    

Ton truc est khakha
 
 
ca c'est pas caca

Code :
  1. <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
  3. <html>
  4. <head>
  5. <title></title>
  6. <script type="text/javascript">
  7. var txtBlock = new Array("txtPrenom","txtNom","txtAge" );
  8. var txtIndex=0;
  10. function changeMessage(){
  11.  for(j=0; j<=txtBlock.length-1; j++){
  12.   if (document.getElementById(txtBlock[j])) {
  13.    document.getElementById(txtBlock[j]).style.display = (j==txtIndex) ?  "block" : "none";
  14.   }
  15.  }
  16.  (txtIndex<txtBlock.length-1) ? txtIndex++ : txtIndex=0;
  17. }
  18. window.onload = function() {
  19.  var timer = setInterval("changeMessage()",500);
  20. }
  21. </script>
  22. <style>
  23. #txtPrenom, #txtNom, #txtAge {
  24.  display:none;
  25. }
  26. </style>
  27. </head>
  29. <body>
  30. <div id="txtPrenom">1</div>
  31. <div id="txtNom">2</div>
  32. <div id="txtAge">3</div>
  33. </body>
  34. </html>


 
Il y a trop d'erreurs dans ton code, ton problème à toi est simple et il se situe au fait que ta condition
if(i==4)....
n'est pas incluse dans la fonction changemessage, du coup c'est normal que ce ne marche pas et que ca t'affiche 2 ;)
 
et puis tu as utilisé plein de vieux code dépassé,
tes fonctions cache et montre devaient se simplifier à : document.getElementById("iddelobjet" ).style.display.
 
ensuite il y a 2 choses à savoir :  
la propriété visibility:hidden cache l'objet mais laisse l'espace qu'il prend dans la page
la propriété display:none cache l'objet et l'enlève entièrement de la page
 
Et ton vieux paté de If imbriqués tu aurais largement pu l'éviter quand même.


Message édité par gatsu35 le 12-02-2006 à 18:55:24
Reply

Marsh Posté le 12-02-2006 à 20:48:48    

lol ok merci pour ton edit :) c cool de m'avoir expliké pour le if i ==4
j'ai testé ton script ça marche nikel
 
merci bcq gars

Reply

Marsh Posté le 12-02-2006 à 21:28:04    

Je tiens juste à te mettre en garde et ça je le vois souvent :  
Tu fais des études pour être ingénieur (cf poste cat emploi), actuellement tu es BAC+4 (enfin niveau BAC+4) (cf pseudo)
 
quand je vois un code comme tu viens de me pondre, je me dis que tu ne sais pas programmer, et ça tu n'es pas le seul, dans mon ancienne formation sur 28 personnes, il y en avait 14 qui codaient comme les pieds et qui auraient pondu un code aussi crade que le tiens pour faire le peu que je viens de faire....
 
PS: Moi je viens de finir mes études et j'ai un BAC+4


Message édité par gatsu35 le 12-02-2006 à 21:28:21
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed