Evitare codice ridondante

[cecco]

Ciao a tutti,
vorrei sottoporvi questo problema che nella mia immaginazione potrebbe essere risolto, spero anche nella realtà :master:

Ho un filmato con 4 movieclips, che si chiamano container001_mc, container002_mc, container003_mc, container004_mc.

Ciascuno di questi contiene altri movieclips e tutti fanno esattamente le stesse cose, cioè caricare un'immagine, aprire una popup con un ingrandimento, aprire un link ad un pdf.

Il problema è che ogni volta mi sembra di scrivere codice ripetitivo, deve esserci un'altro metodo!
Mi aiutate?

Il codice che ho utilizzato io (funzionante) è questo:

//CONTAINER001
this.container001_mc.loadmovie("001-p.jpg");
this.container001_mc.light_btn.onRelease = function() {
      getURL("001.jpg", "_blank");
}
this.container001_mc.download_btn.onRelease = function() {
      getURL("001.pdf", "_blank");
}
//CONTAINER002
this.container002_mc.loadmovie("002-p.jpg");
this.container002_mc.light_btn.onRelease = function() {
      getURL("002.jpg", "_blank");
}
this.container002_mc.download_btn.onRelease = function() {
      getURL("002.pdf", "_blank");
}
//CONTAINER003
this.container003_mc.loadmovie("003-p.jpg");
this.container003_mc.light_btn.onRelease = function() {
      getURL("003.jpg", "_blank");
}
this.container003_mc.download_btn.onRelease = function() {
      getURL("003.pdf", "_blank");
}
//CONTAINER004
this.container004_mc.loadmovie("004-p.jpg");
this.container004_mc.light_btn.onRelease = function() {
      getURL("004.jpg", "_blank");
}
this.container004_mc.download_btn.onRelease = function() {
      getURL("004.pdf", "_blank");
}

Grazie,
Cecco

[telegio]

se ne hai meno di 10 pulsanti puoi fare così:

for (i=1; i<5; i++){
this["container00"+i+"_mc"].loadmovie("00"+i+"-p.jpg");
this["container00"+i+"_mc"].light_btn.onRelease = function() {
      getURL("00"+i+".jpg", "_blank");
}
this["container00"+i+"_mc"].download_btn.onRelease = function() {
      getURL("00"+i+".pdf", "_blank");
}
} 

se ne hai di più è meglio che togli dai nomi 001, 002 e lasci 1, 2 etc..
provala perchè non ho potuto testarla in quanto dove sono NON ho flash..

[cecco]

Ne ho di più infatti.. Finora sono 67, ma il numero è in crescita. Il fatto è che i file si chiamano così, anche per questione di ordine.. Ora cmq provo.

Grazie

[telegio]

allora aspetta perchè ti conviene chiamare i pulsanti 1, 2 eccetera e anche le immagini, sennò tocca fare degli if che se i < 10 aggiunge 2 zeri, se i < 100 ne aggiunge 1 solo..
intanto ti conviene provare questo..

for (i=1; i<150; i++){
this["container"+i+"_mc"].loadmovie(i+"-p.jpg");
this["container"+i+"_mc"].light_btn.onRelease = function() {
      getURL(i+".jpg", "_blank");
}
this["container"+i+"_mc"].download_btn.onRelease = function() {
      getURL(i+".pdf", "_blank");
}
} 

ma il mio script ti funziona?

[silver55]

...puoi anche generalizzare formattando con zeri in base alla lunghezza della var i ( evitando cosi' gli if e mantenendo i nomi esistenti )

// ... conversione di un contatore in string  
//     a lunghezza fissa con riempimento di zeri 

// ... es. conv. in String a 4 cifre con ZERI  
var zeri :String = "0000";       

var ctr  :String; 

for (var i:uint=1; i < 15; i++)
{
    ctr = zeri + String(i);
    ctr = ctr.substr((String(i).length), zeri.length);

    // ... test 
    trace("container" + ctr + "_mc");
    trace( ctr + "-p.jpg");
    trace( ctr + ".pdf");

    // ...     
    this["container" + ctr + "_mc"].loadmovie( ctr + "-p.jpg"); 

    this["container" + ctr + "_mc"].light_btn.onRelease = function()
    { 
        getURL(ctr + ".jpg", "_blank"); 
    }

    this["container" + ctr + "_mc"].download_btn.onRelease = function()
    { 
        getURL(ctr + ".pdf", "_blank"); 
    } 
} 

[cecco]

@telegio: i tuoi script li ho provati.. sia uno che l'altro, ma non vanno. Mi ero dimenticato nel postare il codice iniziale che dentro container00x c'è loadervuoto_mc, cmq anche facendo la modifica non cambia nulla.. Non carica le anteprime, non vanno i collegamenti.

@silver55: il tuo script non l'ho potuto provare nel sito, perchè questo è composto da 650 pagine in Actionscript 2.0, mentre il tuo codice è in AS 3.0.. Il problema è che il 2 non ha la funzione uint, non so con cosa sostituirla..

------

Io ho cmq provato a scrivere un codice ex novo, con un array, avevo già pensato alla cosa del'if che citava telegio, e ho scritto così:

var btnList:Array = [btn0, btn1, btn2, btn3, btn4, btn5, btn6, btn7, btn8, btn9, btn10, btn11, btn12, btn13, btn14, btn15, btn16, btn17, btn18, btn19, btn20, btn21, btn22, btn23, btn24, btn25, btn26, btn27, btn28, btn29];

for (var b:Number = 0; b<btnList.length; b++) {
    if (this.id<=9) {
        eval("container00"+b+"_mc").loadervuoto_mc.loadMovie("rassegna_st/00"+b+"-p.jpg");
        eval("container00"+b+"_mc").download_btn.id = b;
        eval("container00"+b+"_mc").light_btn.id = b;
        eval("container00"+b+"_mc").download_btn.onPress = function() {
            getURL("rassegna_st/00"+b+".pdf", "_blank");
        };
        eval("container00"+b+"_mc").light_btn.onPress = function() {
            getURL("rassegna_st/00"+b+".jpg", "_blank");
        };
    } else if (this.id>9) {
        eval("container0"+b+"_mc").loadervuoto_mc.loadMovie("rassegna_st/0"+b+"-p.jpg");
        eval("container0"+b+"_mc").download_btn.id = b;
        eval("container0"+b+"_mc").light_btn.id = b;
        eval("container0"+b+"_mc").download_btn.onPress = function() {
            getURL("rassegna_st/0"+b+".pdf", "_blank");
        };
        eval("container0"+b+"_mc").light_btn.onPress = function() {
            getURL("rassegna_st/0"+b+".jpg", "_blank");
        };
    }
} 

Anche se mi sembra una cavolata farlo così, però funziona per le prime 9, cioè se i<=9. L'else if non va per niente.

[silver55]

... si scusa, puoi eliminare la dichiarazione
questo e' in AS2

var zeri :String = "0000";      
var ctr  :String; 

for (i=1; i < 15; i++)
{
    ctr = zeri + String(i);
    ctr = ctr.substr((String(i).length), zeri.length);

    trace("container" + ctr + "_mc");
    trace( ctr + "-p.jpg");
    trace( ctr + ".pdf");

    // ... etc ...     
} 

[cecco]

Infatti avevo tentato proprio così e.... FUNZIONA!!!

Posto il codice corretto:

// ... conversione di un contatore in string   
//     a lunghezza fissa con riempimento di zeri 

// ... es. conv. in String a 3 cifre con ZERI   
var zeri:String = "000";
var ctr:String;
for (var i = 1; i<68; i++) {
    ctr = zeri+String(i);
    ctr = ctr.substr((String(i).length), zeri.length);
    // ... test 
    trace("container"+ctr+"_mc");
    trace(ctr+"-p.jpg");
    trace(ctr+".pdf");
    // ...      
    this["container"+ctr+"_mc"].loadervuoto_mc.loadMovie("rassegna_st/"+ctr+"-p.jpg");
    this["container"+ctr+"_mc"].light_btn.onRelease = function() {
        getURL("rassegna_st/"+ctr+".jpg", "_blank");
    };
    this["container"+ctr+"_mc"].downloadpdf_btn.onRelease = function() {
        getURL("rassegna_st/"+ctr+".pdf", "_blank");
    };
} 

Grazie a entrambi per la collaborazione!
Cecco

[cecco]

Aspè.. vedi cosa succede a parlare troppo presto?

Succede una cosa stranissima.
Le anteprime vengono caricate tutte correttamente tramite loadMovie.
Però i link funzionano male, non so perchè. Su qualunque anteprima io faccia click, si apre sempre o 067.jpg, o 067.pdf. Se cambio la dicitura

for (var i = 1; i<68; i++) 

e scrivo qualunque altro numero, tipo:

for (var i = 1; i<15; i++) 

Si apre sempre la 14.. Ovvero sempre un numero inferiore a quello che c'è scritto nella dichiarazione in mezzo..

Che faccio?

[cecco]

Sto provando in mille modi, ma non riesco assolutamente.. Qualche idea?
Devo postare il link al filmato, così lo vedete?

Cecco