Discussion :

[Gunner4902]

Bonjour,

J'ai actuellement la commande :

Code :

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

<span onClick="hide('singlepart_cover'), hide('singlepart_link'), hide('multipart_link');" >

est-il possible de simplifier cette ligne en évitant de répéter "hide" trois fois ?

Merci

[thibaut]

Salut,

Tu definis une fonction:

Code :

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

1
2
3
4
5
6
7
8
9
 function hideAll(_tab) {
    if (_tab) {
        for (var i = 0, length = _tab.length; i < length; i++) {
            hide(_tab[i]);
        }
    }
 
    delete _tab;
}

Code :

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

<span onclick="hideAll(['singlepart_cover', 'singlepart_link', 'multipart_link']);" >

A+

[Gunner4902]

J'ai oublié de dire que j'avais bien fait une fonction

Code :

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

function hide(nom_div) {document.getElementById(nom_div).style.display="none";}

par contre si je fais

Code :

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

<span onclick="hideAll(['singlepart_cover', 'singlepart_link', 'multipart_link']);" >

la console d'erreur d'Opera me dit :

Event thread: click
Error:
name: TypeError
message: Statement on line 3: Could not convert undefined or null to object
Backtrace:
(document.getElementById(nom_div)).style.display = "none";
Line 1 of script
affiche("JWZ159MP.php"), hide(["singlepart_cover", "singlepart_link", "multipart_link"]), show("multipart_name");
At unknown location
[statement source code not available]

[thibaut]

tu as bien écrit la fonction hideAll() ???

[Sheriff]

salut !
remplace les virgules par les point-virgules.

Code :

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

<span onClick="hide('singlepart_cover'); hide('singlepart_link'); hide('multipart_link');" >

++

[thibaut]

Essaie ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
 function hide() {

    for (var i = 0, length = parameters.length; i < length; i++) {
        if (document.getElementById(arguments[i])
            && document.getElementById(arguments[i]).style.display) {
            
            document.getElementById(arguments[i]).style.display = 'none';
        }
    }
}

Code :

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

<span onclick="hide('singlepart_cover', 'singlepart_link', 'multipart_link');" >

A+

[Gunner4902]

Le dernier code ne fonctionne pas, j'ai une console d'erreur qui s'affiche…

Ce n'est pas grave, car finallement la fonction à déclarer est beaucoup plus longue que celle déjà en place, et celà alourdit la page web au lieu de l'alléger…

Merci quand même