Reprise du message précédent :

Ils sont morts depuis longtemps les traumatisés du goto, on peut p-e prendre du recul maintenant ?

Cherchez pas, c'est comme ça episétou    
Ça fait des années que j'évite de le faire et que ça m'hérisse le poil de voir ça, mais je ne me rappelle plus de la raison invoquée  

A part que t'es un gros mauvais?

 
T'es du genre à faire
 

if(!stop)
{
    // 50.000 lignes de code
}
else
{
    return false;
}

 
Plutôt que :
 

if(stop)
    return false;
 
// 50.000 lignes de code

 
?

Nul.
Question réparti, c'est pas encore ça.
 
Sinon, y a bien des conventions dans lequel tu te contraints depuis quelques années, et que tu n'as pas remis en question avec ton recul actuel, non ?

Qu'il y a plusieurs portes des sortie probablement.
Perso ça ne m'a jamais gêné une seule fois, par contre c'est super pratique (ça évite les blocs conditionnels dans tous les sens etc, et avec une coloration basique on voit direct les différentes "portes de sortie" ).

 
oui. Par exemple, insulter les gens qui racontent des conneries sur les forums.

Plus stupide tu meurs
 
GTFO

 
CCP

if(!stop)
{
    // 50.000 lignes de code
   result = true;
}
else
{
    if(!stip)
    {
    //20000 de plus
    result=true;
    }
result=false
}
return result;

Ca marche pas ton truc il faut une else {} de plus

ah oui.

Attendez que je retrouve ma raison ultime, et vous vous prosternerez à genou en vos flagellant avec vos cables d'écouteurs, bande d'hérétiques

 
Ca sert, mais je ne sais pas comment
 
Bon, maintenant que j'ai mis tout le code sur le haut, mes listeners ne s'appliquent plus, voilà pourquoi j'avais du séparer le code en deux

 
la raison donnée par mon prof d'algo cette année etait : "plus simple a debogguer"
 
Mouais

Et allah inventa onDomReady

huhu.

En même temps, cette affirmation est indiscutable dans le cadre de l'utilisation d'un débugger. T'as un breakpoint unique, et t'es obligé de passer par là

 
Ben ptin, faudra que ton prof m'explique un peu en quoi c'est plus simple à debugger parce que là je vois pas C'est même plutôt le contraire

 
Oui, c'est le "dom:loaded" dans prototype, mais pour le coup ça ne vas pas.

ouais, je l'ai jamais écouté
 
je prenais même plaisir a lui sortir des algos de tri écris d'une manière tordue, avec des noms de variables a la con =) et des multi returns !

 
Ca a été discuté plus haut :
document.observe ("dom:loaded", function () { /* ... */ })
 

et pouquoi ? ca ne s'éxecute pas ? j'ai loupé un épisode ?

Donc, pour résumer le bouzin :

Le html de ma page :

<form>
  <input type="hidden" value="2" id="nbrLines">
  
  <div id="tree"></div>
</form>

(oui, le form est vide )

Tout le reste du html étant ensuite généré via JS.
Donc, à l'initialisation de l'arbre, il y a une première branche de créée, avec à l'intérieur trois liens ayant pour id parentLine2, addValue2 et addAttribute1Tag2 (toujours généré via JS).

Je veux appliquer des écouteurs à ces liens, je fais donc cela :

document.observe("dom:loaded", function() {
      $('parentLine2').observe('click', addLine);
      $('addValue2').observe('click', addInputClick);
      $('addAttribute1Tag2').observe('click', addInputClick);
});

Les écouteurs ne sont pas appliqués et voilà ce que me retourne la console d'erreurs :

La ligne 199 correspond à celle-ci : $('parentLine2').observe('click', addLine);.

Bien sûr, il ne trouve pas parentLine2 puisqu'il est généré via JS.

@koskoz
oui à quel moement tu généres l'element parentLine2 ? et pourquoi ne pas lui assigner l'evenement lors de sa creation ?
 
Aah mais à tout les coups il s'agit d'un element d'un arbre ?

<form>
  <input type="hidden" value="2" id="nbrLines">
  
  <div id="tree"></div>
</form>

document.observe("dom:loaded", function() {
      $('parentLine2').observe('click', addLine);
      $('addValue2').observe('click', addInputClick);
      $('addAttribute1Tag2').observe('click', addInputClick);
});

Ben pourquoi tu n'ajoute pas l'écouteur après la création en JS de l'objet ?

Ils se génèrent à l'initialisation de l'arbre :

// Structure de départ de l'arbre
var struct1 = [
{
    'id' : 'item1',
    'txt' : 'Nom du fichier de règles',
    'open' : true,
    'items' : [
    {
        'id' : 'item'+nbrLines,
        'txt' : line()
    }
    ]
}
];
 
function line() {
  var line = '<div id="l'+ nbrLines +'" class="line">';
  
  line += '  <a href="#" id="parentLine'+ nbrLines +'">+</a>'              
  line += '  <select name="tag'+ nbrLines +'">'
  line += '    <option value=""></option>'
  line += '    <option value="8">azerty</option>'
  line += '    <option value="6">blabla</option>'      
  line += '    <option value="4">fgh</option>'
  line += '    <option value="3">test</option>'
  line += '    <option value="1">Test tag</option>'
  line += '  </select>'
  line += '  <a href="#" id="addValue'+ nbrLines +'">Value</a>'
  line += '  <div id="value'+ nbrLines +'" style="display: inline;">'
  line += '  </div>'
  line += '  <input type="hidden" value="1" id="attributeNbrLine'+ nbrLines +'">'
  line += '  <a href="#" id="addAttribute1Tag'+ nbrLines +'">Attribute</a>'
  
  line += '</div>'
  
  return line;
}
 
function TafelTreeInit () {  
  tree1 = new TafelTree ('tree', struct1, {  
  'generate' : true,  
  'imgBase' : 'img/',  
  'defaultImg' : 'page.gif',  
  'defaultImgOpen' : 'folderopen.gif',  
  'defaultImgClose' : 'folder.gif',  
  'onDrop' : dropConfirm,  
  'behaviourDrop' : 'child', // child ou sibling  
  // ou childcopy ou siblingcopy  
  'dropALT' : true, // defaut : true  
  'dropCTRL' : true // defaut : true  
  });
}

Edit ratibus : je ne sais pas où ajouter l'écouteur après la création en JS de l'objet.

Tu m'expliques comment la lib pourrait observer un élément qui n'existe pas

 
Bah oui, je sais c'est bien ce que je dis
Mais je ne vois pas comment observe mon élément créé à l'initialisation (cf. post du dessus).

utilise la version de jquery hébergée par google
http://encosia.com/2008/12/10/3-re [...] y-for-you/

Toi t'as la gueule d'un programmeur cobol ou AS400

Tu setup l'observer après que l'élément ait été créé

Nan mais si tu réfléchis au problème et essaye de trouver une solution adaptée, s'pas du jeu aussi
 
 
 

if(!stop)
{
    // 50.000 lignes de code
}
else
{
    return false;
}

if(stop)
    return false;
 
// 50.000 lignes de code

T'es du genre à suivre des guidelines où il tolèrent une fonction avec 50.000 lignes de code ?  

 
Genre c'était pas une façon de parler

FlorentP

 
J'ai bien compris le principe, mais sérieusement je sais pas où les foutre (dtc proof).
 
Les éléments sont créés dans ma fonction "line" qui est appelée dans la structure de l'arbre qui elle même est appelé dans la fonction d'initialisation de l'arbre.
 
J'ai essayé de mettre les observers dans la fonction d'initialisation de l'arbre, tout à la fin, mais ça revient au même.

Ya pas vraiment d'intéret à minifier si tu utilises gzip (c'est les devs de prototype qui le disent ).

Personne lui a encore gueulé dessus pour ca ?

Je fais du Fortran 2003, ça compte ?

if(!stop)
{
    // 50.000 lignes de code
}
else
{
    return false;
}

if(stop)
    return false;
 
// 50.000 lignes de code

 
Perso je préfère ça:

if(stop)
    return false;
else
{
    // 50.000 lignes de code
    return true;
}

 
Primo parce que j'utilise régulièrement des langages qui n'ont pas de return (VB6/Script et Pascal/delphi) et secundo parce qu'on peut avoir une idée de la structure du code rien quen regardant l'indentation et sans avoir à lire toutes les instructions.

 
theredled c'est pas la même unité monétaire
 
TafelTreeInit est appelé où ?

Outre le fait que tu pourrais assigner l'event juste après la creation du truc, tu pourrais aussi assigner l'event à une classe CSS plutot qu'à un ID, car ce dernier si j'ai bien compris s'incrémente tt le temps. Dans le code de la fonction assignée il te suffirait de reperer l'origine de l'event.
 
A noter que sous jQuery (1.3+) il existe une fonction Live (http://docs.jquery.com/Events/live) qui permet d'assigner un evenement à une classe donnée existante OU qui va exister.