Petite fonction toute simple, mais

**SpaceFrog** · 20/07/2006, 16h06

arf ...

bon tests faits, le break fait sortir du switch à la première occurence du break d'un case true...

parcontre contrairement à ce que je pensais si l'on ne met pas de break il ne passe pas par les cases suivant mais effectue quand mêem toutes les instructions des case ...

curieux ...

**FremyCompany** · 20/07/2006, 16h09

curieux ? lol, mois je dirais logique

Et si on en revenait au if ? mdrr

**SpaceFrog** · 20/07/2006, 16h11

lol oui à regrets ...

**linar009** · 20/07/2006, 16h13

Rien de tel que les if lol

**renaud26** · 20/07/2006, 16h14

euh...je suis un peu largué, là...

Mais bon, je vais donc revenir aux if...si ce sont les maîtres qui le disent...

**FremyCompany** · 20/07/2006, 16h14

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
39
function getHTML(txt) {
  var att=new Array() // Ensemble des balises à utiliser
  // Ajoute une balise et ses attributs au tableau
  if (document.getElementById("bold").checked) {
    att[att.lenght]="b"
  }
  if (document.getElementById("italic").checked) {
    att[att.lenght]="i"
  }
  if (document.getElementById("underline").checked) {
    att[att.lenght]="u"
  }
  if (document.getElementById("greater").checked) {
    att[att.lenght]="big"
  }
  if (document.getElementById("smaller").checked) {
    att[att.lenght]="small"
  }
  if (document.getElementById("fixefont").checked) {
    att.push("font face='Courier New'");
  }
  // Construit le HTML à utiliser
  var str = ""
  // Balises ouvrantes
  for (var i=0; i<att.length; i++) {
     str += "<" + att[i] + ">"
  }
  // Texte central
     // Conversion du texte au format HTML
     var Div = document.createElement("DIV");
     Div.appendChild(document.createTextNode(txt));
  str += Div.innerHTML;
  // Balises fermantes
  for (var i=att.length; i>-1; i--) {
     str += "</" + att[i].split(" ")[0] + ">"
  }
  // str vaut maintenant le HTML final
  return str
}

Utilisation :

Code :

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

document.getElementById("theSpan").innerHTML=getHTML(document.getElementById("theInput").value)

**linar009** · 20/07/2006, 16h18

Ne sois pas méprisant! lol
Y'a toujours de la place à gagner ...

Code :

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

1
2
3
4
5
6
7
if (document.getElementById("bold").checked) att[att.lenght]="b";
if (document.getElementById("italic").checked) att[att.lenght]="i";
if (document.getElementById("underline").checked) att[att.lenght]="u";
if (document.getElementById("greater").checked) att[att.lenght]="big";
if (document.getElementById("smaller").checked) att[att.lenght]="small";
if (document.getElementById("fixefont").checked) att.push("font face='Courier New'");
}

**FremyCompany** · 20/07/2006, 16h29

Envoyé par linar009

Ne sois pas méprisant! lol
Y'a toujours de la place à gagner ...

Code :

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

1
2
3
4
5
6
7
if (document.getElementById("bold").checked) att[att.lenght]="b";
if (document.getElementById("italic").checked) att[att.lenght]="i";
if (document.getElementById("underline").checked) att[att.lenght]="u";
if (document.getElementById("greater").checked) att[att.lenght]="big";
if (document.getElementById("smaller").checked) att[att.lenght]="small";
if (document.getElementById("fixefont").checked) att.push("font face='Courier New'");
}

Je sais, mais je hais cette synthaxe car j'estime qu'elle n'est pas claire !
Regarde et tu comprendras :

Code :

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

1
2
if (true) alert("x")
alert("y")

Code :

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

1
2
3
if (true)
   alert("x")
alert("y")

Code :

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

1
2
3
if (true)
alert("x")
alert("y")

Code :

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

1
2
3
if (true) alert("x")
   alert("x")
alert("y")

Ce n'est pas clair... Qu'est-ce qui dépend du if ? Ca ne se voit pas au premier regard... avec les {}, oui !

**linar009** · 20/07/2006, 16h30

Tout à fait d'accord, après il faut s'adapter au code

Code :

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

1
2
3
if (true) alert("x")
   alert("x")
alert("y")

n'est pas clair

Code :

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

1
2
3
4
5
6
if (document.getElementById("bold").checked) att[att.lenght]="b";
if (document.getElementById("italic").checked) att[att.lenght]="i";
if (document.getElementById("underline").checked) att[att.lenght]="u";
if (document.getElementById("greater").checked) att[att.lenght]="big";
if (document.getElementById("smaller").checked) att[att.lenght]="small";
if (document.getElementById("fixefont").checked) att.push("font face='Courier New'");

ça ça l'est ..

non?

**FremyCompany** · 20/07/2006, 16h35

Envoyé par linar009

Tout à fait d'accord, après il faut s'adapter au code

Code :

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

1
2
3
if (true) alert("x")
   alert("x")
alert("y")

n'est pas clair

Code :

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

1
2
3
4
5
6
if (document.getElementById("bold").checked) att[att.lenght]="b";
if (document.getElementById("italic").checked) att[att.lenght]="i";
if (document.getElementById("underline").checked) att[att.lenght]="u";
if (document.getElementById("greater").checked) att[att.lenght]="big";
if (document.getElementById("smaller").checked) att[att.lenght]="small";
if (document.getElementById("fixefont").checked) att.push("font face='Courier New'");

ça ça l'est ..

non?

Quitte à choisir entre une synthaxe parfois claire et parfois pas, et une autre qui l'est toujours, autant choisir celle qui l'est toujours ! Même si parfois elle est plus longue... Mais c'est vrai que quand je fais du code "abrégé", j'écris

Code :

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

if (true) {alert("x")}

... Au final, ca ne coute que deux pauvres caractères en plus, et si je dois rajouter du code dans mon if, je n'ai pas grand chose à faire !
ex :

Code :

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

if (true) {alert("x"); alert("y")}

**renaud26** · 20/07/2006, 17h24

J'ai finalement réussi à faire ma fonction avec un mix de ce qui a été dit, tout en essayant de comprendre pour me coucher moins naze en JS...
J'ai un petit problème, encore: je voudrais ajouter un attribut de couleur.
Mon form comporte une fonction JS qui affiche une palette de couleurs, et lorsque le user select sa couleur, le champ text couleur se remplit tout seul.
Mais par contre, la couleur ne s'applique pas. J'ai mis un innerText pour vérif le code HTML généré, il est ok. Vous voyez pourquoi ?

la ligne de fonction concernée

Code :

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

1
2
3
 
 
str = "<font color=" + document.form1.coul_texte.value + ">" + str + "</font>";

...et dans le form:

Code :

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

1
2
3
4
5
6
 
 
<img src="../images/color.gif" onClick="Couleurs('coul_texte', 'right') ;">
 
<input name='coul_texte' type='text' value="#000000" 
onChange="affiche_mot();"></td>

Vous voyez le loup ?

**FremyCompany** · 20/07/2006, 17h26

Pour comprendre, il nous faudrait le code exact de tes fonctions

**renaud26** · 20/07/2006, 17h37

ben c'est une fonction color_picker qui ajoute le code hexadécimal dans le champ texte dès que le user a cliqué sur la couleur de son choix. Ce qui ne doit pas aller c'est peut être le onChange...

**FremyCompany** · 20/07/2006, 17h41

Je parlais du code de "affiche_mot();"
C'est la que le faute est...

**renaud26** · 20/07/2006, 17h51

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
 
 
function affiche_mot(){
var str = document.form1.aff_mot_cle.value ;
 
if (document.form1.gras.checked==true){
str = "<b>" + str + "</b>"; 
}
if (document.form1.ital.checked==true){
str = "<i>" + str + "</i> ";
}
if (document.form1.underline.checked==true){
str = "<u>" + str + "</u> ";
}
if (document.form1.overline.checked==true){
str = "<div style='text-decoration:overline'>" + str + "</div>";
}
if (document.form1.coulText.checked==true){
str = "<font color=" + document.form1.coul_texte.value + ">" + str + "</font>";
}
}

Mais en fait, je crois avoir capté pourquoi. En fait, une autre fonction cache ce champ de choix couleurs. Il ne s'affiche que si le user coche la checkbox "texte_coul". et moi, dans ma fonction, j'ai mis

Code :

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

1
2
3
4
5
 
 
if (document.form1.coulText.checked==true){
str = "<font color=" + document.form1.coul_texte.value + ">" + str + "</font>";
}

quelle est l'expression qui remplace document.form1.coulText.checked==true
pour un champ texte ?

**FremyCompany** · 20/07/2006, 18h15

J'ai pas assez d'information mais essaie déjà ces pistes la :
disabled==false ?
style.display=="none" ?
style.visibility=="hidden" ?

**renaud26** · 20/07/2006, 18h32

Excuse moi, je ne vois pas comment tester ces pistes...
Mais si on part du principe que la fonction s'exécute si checkbox.checked==true comment je peux formuler if(champText.coul_texte != vide) ?
C'est à dire executer la fonction dès que le color_picker remplit le champ coul_texte...

**SpaceFrog** · 21/07/2006, 08h21

Je reviens juste sur le switch ...

J'aime pas que ça me résiste

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
<script type='text.javascript'>
 
var att=new Array()
for (i=0; i<4;i++){
switch (true){
case (document.getElementById("bold").checked && (" "+att.join(" ")+" ").indexOf(' b ')==-1) :  att.push("b");
																													break;
case document.getElementById("italic").checked && (" "+att.join(" ")+" ").indexOf(' i ')==-1):   att.push("i");
																													break;	
case document.getElementById("underline").checked && (" "+att.join(" ")+" ").indexOf(' u ')==-1): att.push("u");
																													 break;	
case document.getElementById("greater").checked && (" "+att.join(" ")+" ").indexOf(' big ')==-1):  att.push("big");
																														break;
case document.getElementById("smaller").checked && (" "+att.join(" ")+" ").indexOf(' small ')==-1): att.push("small")
}
}
 
var str = ""
for (var i=0; i<att.length; i++) {
   str += "<" + att[i] + ">"
}
str += letexte
for (var i=att.length; i>-1; i--) {
   str += "</" + att[i] + ">"
}
 
</script>

je n'ai plus de regrets ...

**FremyCompany** · 21/07/2006, 10h48

Désolé de te contredire mais je suis persuadé que ca ne marche pas mieux qu'avant

Envoyé par SpaceFrog

Je reviens juste sur le switch ...

J'aime pas que ça me résiste

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
<script type='text.javascript'>
 
var att=new Array()
for (i=0; i<4;i++){
switch (true){
case (document.getElementById("bold").checked && (" "+att.join(" ")+" ").indexOf(' b ')==-1) :  att.push("b");
																													break;
case document.getElementById("italic").checked && (" "+att.join(" ")+" ").indexOf(' i ')==-1):   att.push("i");
																													break;	
case document.getElementById("underline").checked && (" "+att.join(" ")+" ").indexOf(' u ')==-1): att.push("u");
																													 break;	
case document.getElementById("greater").checked && (" "+att.join(" ")+" ").indexOf(' big ')==-1):  att.push("big");
																														break;
case document.getElementById("smaller").checked && (" "+att.join(" ")+" ").indexOf(' small ')==-1): att.push("small")
}
}
 
var str = ""
for (var i=0; i<att.length; i++) {
   str += "<" + att[i] + ">"
}
str += letexte
for (var i=att.length; i>-1; i--) {
   str += "</" + att[i] + ">"
}
 
</script>

je n'ai plus de regrets ...

**SpaceFrog** · 21/07/2006, 11h35

Hormis ton erreur précedente dasn la boucle décroissante que j'ai corrigée, essaye ça et tu me diras si c'est toujours aussi drôle ...

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=windows-1252">
 
<title>Test</title>
<script type='text.javascript'>
function enrichit(){
var att=new Array()
for (i=0; i<4;i++){
switch (true){
case (document.getElementById("bold").checked && (" "+att.join(" ")+" ").indexOf(' b ')==-1) :  att.push("b");
																													break;
case (document.getElementById("italic").checked && (" "+att.join(" ")+" ").indexOf(' i ')==-1) :   att.push("i");
																													 break;	
case (document.getElementById("underline").checked && (" "+att.join(" ")+" ").indexOf(' u ')==-1): att.push("u");
																													 break;	
case (document.getElementById("greater").checked && (" "+att.join(" ")+" ").indexOf(' big ')==-1):  att.push("big");
																														break;
case (document.getElementById("smaller").checked && (" "+att.join(" ")+" ").indexOf(' small ')==-1): att.push("small")
}
}
 
var str = ""
for (var i=0; i<att.length; i++) {
   str += "<" + att[i] + ">"
}
str += document.getElementById('mazone').value
for (var i=att.length-1; i>-1; i--) {
   str += "</" + att[i] + ">"
}
document.getElementById('mazone').value=str
document.getElementById('resultat').innerHTML=str
}
</script>
 
 
 
 
</head>
 
<body>
gras <input type='checkbox' id="bold" /><br/>
italique <input type='checkbox' id="italic"/><br/>
souligné <input type='checkbox' id="underline" /><br/>
grand <input type='checkbox' id="greater"><br/>
petit <input type='checkbox' id="smaller"><br/>
 
<br/>
<textarea id="mazone"></textarea>
<input type="button" onclick="enrichit()" value="go" />
<div id='resultat'></div>
</body>
</html>

Petite fonction toute simple, mais

JavaScript

Discussions similaires

Partager

Partager