onclick w funkcji Opcje

[imosek8]

Witam! Mam mały problem, otóż, próbuję zrobić coś takiego...

[HTML] pobierz, plaintext 
<script language="JavaScript">
function showelement(id)
{
 alert("get.ElementById('id').style.display = this.checked ? 'block' : 'none'");
}
</script>
[HTML] pobierz, plaintext 

i potem w kodzie daje:

[HTML] pobierz, plaintext 
<input type="radio" name="xxx" onclick="java script:showelement(coach);" />
[HTML] pobierz, plaintext 

coach to id div'a, który ma zostać wyświetlony po naciśnięciu. Mam parę takich div'ów, dlatego takie ujęcie w funkcji byłoby dla mnie wygodne. Jestem początkujący, więc nie wiem co tu robię źle, że to nie działa..;/ bo jak dam kod w onclick z alertu to działa....

[WiruSSS]

Jezeli już musisz to robić w ten sposób to wystarczy:

[HTML] pobierz, plaintext 
<input type="radio" name="xxx" onclick="showelement('coach');" />
[HTML] pobierz, plaintext 

ale sugerowałbym przenieść cały JS do osobnego pliku. Można by nadać name inputów odpowiednie do id checkboxów i przeskanować wszystkie inputy (lub jeśli chcesz ograniczyć to nadać jakąś klasę tym co mają być przeszukwiane) i tam podpiąć zdarzenia.

Ten post edytował WiruSSS 27.08.2013, 00:26:01

[imosek8]

No i u mnie u groy wyskakuje komunikat z zawartoscia alert...a powinien pokazywac sie div...

Nie wiem wlasnie jak u mnie to zrobic, bo ogolnie mam 3 div'y. i teraz w jednym inpucie daje przy onclick:

[HTML] pobierz, plaintext 
onclick="document.getElementById('player').style.display = this.checked ? 'block' : 'none'; document.getElementById('coach').style.display = this.checked ? 'none' : 'block'; document.getElementById('team').style.display = this.checked ? 'none' : 'block'; " "
[HTML] pobierz, plaintext 

w drugim

[HTML] pobierz, plaintext 
onclick="document.getElementById('coach').style.display = this.checked ? 'block' : 'none'; document.getElementById('player').style.display = this.checked ? 'none' : 'block'; document.getElementById('team').style.display = this.checked ? 'none' : 'block'; " "
[HTML] pobierz, plaintext 

w trzecim

[HTML] pobierz, plaintext 
onclick="document.getElementById('team').style.display = this.checked ? 'block' : 'none'; document.getElementById('coach').style.display = this.checked ? 'none' : 'block'; document.getElementById('player').style.display = this.checked ? 'none' : 'block'; " "
[HTML] pobierz, plaintext 

i wtedy dziala dobrze...ale pomyslalem ze przez funckje bedzie lepiej bo mniej kodu....jak to najlepiej zrobic?

[nospor]

No ale po grzyba wkladasz to w alert? Alert sluzy do wyswietlania tekstu a nie wykonywania kodu. Wywal ten alert a bedzie dzialac

[imosek8]

ja juz nic nie rozumiem mam dokladnie teraz tak:

w head

[HTML] pobierz, plaintext 
<script language="JavaScript" type="text/javascript">
	function showelement(id)
	{
	 document.getElementById('id').style.display = this.checked ? 'block' : 'none'";
	}
</script>
[HTML] pobierz, plaintext 

w body

[HTML] pobierz, plaintext 
<input type="radio" name="BOX" onclick="showelement('coach');">
<div id="coach" style="display: none;">
<input type="text" name="login" placeholder="Login" class="forlog2">
<input type="text" name="name" placeholder="Imię" class="forlog">
<input type="text" name="surname" placeholder="Nazwisko" class="forlog">
<input type="text" name="license" placeholder="Licencja" class="forlog2"><input type="text" name="email" placeholder="Podaj swój e-mail" class="forlog2">
<input type="password" name="pass" placeholder="Hasło" class="forlog2">
<input type="password" name="pass2" placeholder="Potwierdź hasło" class="forlog2">
<input type="checkbox" name="reg"><span> Akceptuj</span> <a href="/regulamin.php" class="odnosnik">regulamin</a>
</div>
[HTML] pobierz, plaintext 

Nie wiem, może źle coś przekazuje do funkcji albo coś?

[nospor]

No ale przekaz jeszcze do funkcji sswoj radio, bo w funkcji this. nie odnosi sie do niego

[_Borys_]

Skoro zaznaczasz radio to już nie możesz odznaczyć i wystarczy to:

[JAVASCRIPT] pobierz, plaintext 
function showelement(id)
 {
 document.getElementById(id).style.display = 'block';
 }
[JAVASCRIPT] pobierz, plaintext 

a gdybyś chciał this.checked to musisz przekazywać kliknięty obiekt do funkcji czyli
onclick="showelement('coach',this);"
a odbierać

[JAVASCRIPT] pobierz, plaintext 
<script type="text/javascript">
function showelement(id,input)
	{
	 document.getElementById('id').style.display = input.checked ? 'block' : 'none'";
	}
 
</script>
[JAVASCRIPT] pobierz, plaintext 

ale to raczej dla checkbox-a