Ciclo for e getelementbyid

[pippuccio76]

Salve ho creato questa funzione :

function mostracopertina(){
    
    for(var i=1 , i<=12 ,i++){
        var foto ='foto'+i;
        var cop ='cop'+i;
        
        if(document.scelta_immagini.foto.checked){
            
            document.getElementById(cop).style.display = 'inline';
            
            
        }
        else{
            document.getElementById(cop).style.display = 'none';
        }
    }
    
    
}

Vorrei che nell' if else dentro al for venga sostituito al nome della variabile il contenuto ad esempio nell 'if al posto di foto ci sia foto1 foto2 ecc così come al posto di cop .

Perchè invece viene riscritto solamente ?

[KillerWorm]

Ciao, stai sbagliando approccio. Per accedere dinamicamente alle proprietà di un oggetto dovresti utilizzare la notazione a parentesi quadre:

if(document.scelta_immagini[foto].checked){

Deduco che ti mancano delle nozioni base sulla programmazione ad oggetti in JavaScript.
Può essere utile dare uno sguardo a qualche guida di base, ad esempio questa.

[pippuccio76]

Ciao, stai sbagliando approccio. Per accedere dinamicamente alle proprietà di un oggetto dovresti utilizzare la notazione a parentesi quadre:

if(document.scelta_immagini[foto].checked){

Deduco che ti mancano delle nozioni base sulla programmazione ad oggetti in JavaScript.
Può essere utile dare uno sguardo a qualche guida di base, ad esempio questa.

La console di chrome mi dice : Unexpected token [

[Vindav]

posta l'html della pagina

[pippuccio76]

posta l'html della pagina

Eccolo è codice creato con php:

<form name="scelta_immagini" method="POST" action="immagine_scelta.php">
<input type="hidden" name="imgselect" value="X">




							<div class="col-md-3">
	




				 		<input type="checkbox" name="foto1" value="1" id="fotoimm1" onClick="numimmagini(this);check_allok();mostracopertina();" />
						<br/>
						<div id="cop1">
						SELEZIONA QUESTA IMMAGINE COME IMMAGINE DI COPERTINA<input type="radio" name="copertina" value="1"/>
						</div>
														</div>


									</div>


							<div class="col-md-3">




				 		<input type="checkbox" name="foto2" value="1" id="fotoimm2" onClick="numimmagini(this);check_allok();mostracopertina();" />
						<br/>
						<div id="cop2">
						SELEZIONA QUESTA IMMAGINE COME IMMAGINE DI COPERTINA<input type="radio" name="copertina" value="2"/>
						</div>
													</div>


						<div class="row">
						<div class="col-md-3">


				 		<input type="checkbox" name="foto3" value="1" id="fotoimm3" onClick="numimmagini(this);check_allok();mostracopertina();" />
						<br/>
						<div id="cop3">
						SELEZIONA QUESTA IMMAGINE COME IMMAGINE DI COPERTINA<input type="radio" name="copertina" value="3"/>
						</div>
													</div>


							<div class="col-md-3">






				 		<input type="checkbox" name="foto4" value="1" id="fotoimm4" onClick="numimmagini(this);check_allok();mostracopertina();" />
						<br/>
						<div id="cop4">
						SELEZIONA QUESTA IMMAGINE COME IMMAGINE DI COPERTINA<input type="radio" name="copertina" value="4"/>
						</div>
													</div>




					</div>


			<hr/>




							<div class="col-md-3">




				 		<input type="checkbox" name="foto5" value="1" id="fotoimm5" onClick="numimmagini(this);check_allok();mostracopertina();" />
						<br/>
						<div id="cop5">
						SELEZIONA QUESTA IMMAGINE COME IMMAGINE DI COPERTINA<input type="radio" name="copertina" value="5"/>
						</div>
													</div>


							<div class="col-md-3">






																	</div>




							<div class="col-md-3">
	
																	</div>


							<div class="col-md-3">


																	</div>




						<div class="row">
						<div class="col-md-3">


																	</div>




							<div class="col-md-3">




				 		<input type="checkbox" name="foto12" value="1" id="fotoimm12" onClick="numimmagini(this);check_allok();mostracopertina();" />
						<br/>
						<div id="cop12">
						SELEZIONA QUESTA IMMAGINE COME IMMAGINE DI COPERTINA<input type="radio" name="copertina" value="12"/>
						</div>
													</div>




					</div>


			<hr/>


							<div class="col-md-3">
																	</div>


							<div class="col-md-3">
																	</div>




	




							<div class="col-md-3">
		




																	</div>


							<div class="col-md-3">




																	</div>












<div class="row">
	<div class="col-md-12 text-center">


			<button type="button" id="invio_offerta" onclick = "document.scelta_immagini.submit()"  >
			CARICA L'IMMAGINE
			</button>
	</div>
</div>




</form>

[Vindav]

i div dove si trovano le copertine sono tutti nascosti in partenza? se si penso che il ciclo sia inutile puoi fare qualcosa del genere:

function mostracopertina(foto_sel){
    index = foto_sel.name.replace("foto", "");
        if(foto_sel.checked){
            
            document.getElementById("cop"+index).style.display = 'inline';
            
            
        }
        else{
            document.getElementById("cop"+index).style.display = 'none';
        }
    }

la funziona va richiamata cosi: mostracopertina(this);

[pippuccio76]

i div dove si trovano le copertine sono tutti nascosti in partenza? se si penso che il ciclo sia inutile puoi fare qualcosa del genere:

function mostracopertina(foto_sel){
    index = foto_sel.name.replace("foto", "");
        if(foto_sel.checked){
            
            document.getElementById("cop"+index).style.display = 'inline';
            
            
        }
        else{
            document.getElementById("cop"+index).style.display = 'none';
        }
    }

la funziona va richiamata cosi: mostracopertina(this);

Funziona grazie ....