"CODE EN DOUBLON qui ne sert à rien selon la communauté"

+1 !

"je garderai dorénavant mes scripts...trop écoeuré de la mentalité sur ce site"

Non mais lol, tu tend le bâton pour te faire battre et viens te plaindre ensuite !!

"je n'avais pas demandé de passer devant un jury, ce script devait juste servir aux newbies"

Je répète ce que j'ai dit, le newbie à besoin de base solid si ils veulent évoluer dans le language, si tu respectes pas la mise en forme avec les css, et leur fournit une fonction seulement pour des div, tu leurs rends pas service.

"POUR INFO, j'utilise JQuery et Prototype...donc pas besoin de me faire un cours dont je connais déjà le contenu"

Lol ba regarde les fonctions du même type dans prototype et jquery, ont en reparlera après !

"Ramener sa science auprès des autres n'a jamais rien amené de bon.."

Voilà ce que ca donne quand on est tros fière pour accepter la critique. On s'en prend aux autres, alors que ceux-ci veulent seulement t'aider.

de rien ;)

normalement ca fait rien jdmcreator.
Sinon pour ta source masternico, tu dois certainement pas connaitre la proriété ccText de l'objet style :

document.getElementById('test').style.cssText += 'border : 1px solid black; background : red';

marche sur ie6.

a+

daccord avec kazma, c'est vue et revue. Moi j'aurais dit Mootools pawa :).

Sinon quand on programme en js et html , on essaye de le faire bien.
On évite d'écrire le css dans une balise. Si un débutant essaye ton script , et si lui à respecter cette règle il fait quoi? Il s'énerve devant son pc , comme toi tu viens de le faire.

De plus, si ce même débutant utilise un display 'inline' pour un span , ou 'inline-block', il fait quoi?

Bref on a bien compris que tu cherchais a faire simple, mais faut aussi chercher a faire fonctionnel non?

bon je viens d eme reveiller :)

var toogleDisplay = function(id, display){
   var el = document.getElementById(id),
   currentDisplay = getStyle(el, 'display');
  
   if(currentDisplay == 'none'){
       el.style.display = display || el.style.__originalDisplay ||  'block';
   }else{
       el.style.__originalDisplay = currentStyle;
       el.style.display = 'none';
   }
  
}

toujours pas tester, mais ca devrait le faire !

c'est plus un snippet.

Comme tu le dit display n'a pas nécessairement comme valeur block;
De plus, si le style de ton élément est dans une css, pour récupérer le display , il faut passé par une méthode spéciale et non via la propriété style.

Donc il te faut une méthode cross-browser pour récupérer le style, plus ajouter un argument à ta fonction pour spécifier le display ou sauvegarder l'état initiale du display sauf si sa valeur est à none;

concernant le style en volà une toute prête.

var getStyle = function(/*DOMObject*/element, /*string*/rule){//peux etre definit en globale car très utilise, aucune dépendance...
    var camelRule=rule.replace(/\-(\w)/g, function (strMatch, p1){return p1.toUpperCase();}),//supprime les tiré et met en majuscule la lettre suivante
        value = element.style[camelRule];
    if (!value){
        if(document.defaultView && document.defaultView.getComputedStyle){
            value = document.defaultView.getComputedStyle(element, "").getPropertyValue(/*fait l inverse de camelRule*/rule.replace(/[A-Z]/g, function(match){return '-'+match.charAt(0).toLowerCase();}));
        }else{//specifique ie
            value = element.currentStyle[camelRule] ;
        }
    }
    return value == 'auto' ? undefined : value;
};

pour ta fonction voir ca pourrait donner ca :

var toogleDisplay = function(id, display){
   var el = document.getElementById(id),
   currentDisplay = getStyle(el, 'display');
  
   if(currentStyle == 'none'){
       el.style = display || el.style.__originalDisplay ||  'block';
   }else{
       el.style.__originalDisplay = currentStyle;
       el.style = 'none';
   }
  
}

brute de fonderie à tester. Mais comme quoi c'est pas si simple !

a++

ca sert à quoi? je connais un peu les iterateurs en php, ils ont été créer pour boucler sur des objets, mais en js, je vois pas l'intérêt vu qu'il y'a la boucle for in. De plus dans ton exemple tu utilises un tableau, ce qui est inutile, et consomme des ressources pour rien.

Il y'a qu'un intérêt, celui de créer des listes spécifiques , comme les ensembles , les pile, listes chaînées ect et leurs fournir des méthodes communes, mais bon faut pas oublié que le js est interprète, de telles procédures sont très couteuse en temps d'exécution et sont vraiment limité a des cas très particulier. Bref autant utiliser les procédures native et les adapté si le besoin sans fait sentir, surtout que dans tout les cas la quantité de code à rajouté est très limité.

Il a des librairies comme dojo ou extjs qui ont implémenté ce genre de classe, mais j'ai jamais vue un bout de code les utiliser.

Pour finir tu postes en expert, alors que pour moi c'est tout juste un niveau débutant.

justement, grâce à la méthode native hasOwnProperty qui permet de savoir si une propriété est définit directement dans l'instance et non dans le prototype , je crois pas qu'il y ait des bug sur cette fonction, mais à tester quand même sous ie6.

par exemple une base pourrait ressembler à ca :

        function Hash(obj){
    for(var i in obj)
    this[i] = obj[i];
}

Hash.prototype.each = function(fn, context){
    context = context || this;
    for(var i in this){
    if(this.hasOwnProperty(i))
    fn.call(context, this[i], i);
}
}
    

var jo = new Hash({
    test1 : 'ok1',
    test2 : 'ok2'
});

jo.each(function(value, index){
    alert(index +' à comme valeur ' + value);
});

salut, je me serais pris autrement. Au lieu de cloner l'objet et de le référencer par this.object, j'aurais étendue à l'instance toute les variables de l'objet. Ensuite la fonction hasOwnProperty permet de savoir si une propriété est définit dans le prototype ou dans l'insatnce.

Par exemple pour la fonction keys sa donnerait

keys:function(){
    var a=[];
    for(var b in this){
        if(this.hasOwnProperty(b))
            a.push(b);
    }
    return a;
}

salut!

Je sais pas tros à quoi sert ton cript, n'étant pas un pro de la finance.

Est-ce que ton script pourrais s'adapter à d'autre données, enfin pour faire de simple graph?
Là y'a pas de légende, description c'est normale?

Ta classe jscandles n'exploite pas le prototype, qui est le modèle de classe en javascript, c'est là ou l'on définit les membres de la classe, sans, pas d'héritage, de plus ca ralentie l'instanciation (du moin si tu dois créer plusieurs instances).

Ta méthodes setData  est inutile, pas d'encapsulation en js, pas besoin d'un setters juste pour initialiser une valeur.

Quand tu boucles sur un tableaux, n'utilises pas dans ta zone conditionnelle une comparaison  sur la variable length du tableau , mais enregistre celle ci avant dans une variable, sans quoi a chaque itérations l'interpréteur recalculera la taille du tableau.

La fonction inverseArray  est inutile, il existe la méthode Array, reverse pour ça.

Perso j'aurais utilisé une librairie comme excanvas pour générer le canvas (12 ko compilé), car ie il connait pas :(

Il existe une super librairie de graph http://highcharts.com/ jette un coup d'½il je suis sur que ca t'intéressera.

a++