jQuery nie działa pokazywanie spana o określonym ID

pytanie zadane 9 marca 2017 w JavaScript przez Miles Nowicjusz (170 p.)

napisałem kod który ma pokazywać i chować spana po kliknięciu na guzik ale kod nie działa, dlaczego???

$(document).ready(function(){
	$('#okno')
		.css({background:'red'})
		;//.hide(0);
		
})

{
	var okno = document.getElementById('okno');
	var guzik = document.getElementById('guzik');
if(guzik.innerHTML=="Pokaż")
{
	guzik.innerHTML = "Ukryj";
	
	}else{
	
	guzik.innerHTML = "Pokaż";
	}	}

$("#guzik").click(zmien());
</script>
<title>Strona Główna</title>
</head>
<body>
<div>
<span id="okno">okno</span>
<button id="guzik">Pokaż</button><br />

//jQuery podpięty

2 odpowiedzi

odpowiedź 9 marca 2017 przez obl Maniak (51,280 p.)
wybrane 9 marca 2017 przez Miles

Najlepsza

To:

$("#guzik").click(zmien());

nie zadziała bo nie masz funkcji zmień zadeklarowanej w powyższym kodzie a tak w ogóle to jeżeli już byś ją miał to powinno być:

$("#guzik").click(zmien);

bo masz tu przekazać wskaźnik na funkcję, która zostanie wywołana gdy przycisk zostanie wciśnięty. Być może chciałeś zrobić coś takiego:

zmien{
    var okno = document.getElementById('okno');
    var guzik = document.getElementById('guzik');
if(guzik.innerHTML=="Pokaż")
{
    guzik.innerHTML = "Ukryj";
     
    }else{
     
    guzik.innerHTML = "Pokaż";
    }   }
 
$("#guzik").click(zmien);

Tylko, że w twoim kodzie wciąż brakuje śladów chęci zaimplementowania tego co chcesz uzyskać. Jedyne co tam jest robione to podmieniany jest tytuł przycisku na "Ukryj" lub "Pokaż" co też jest dość słabe.

odpowiedź 9 marca 2017 przez hoktaur Pasjonat (22,250 p.)

Nie wiem czy to źle okroiłeś ale ja bym się doczepił w tym miejscy (brak; na końcu i brak przed klamrą zmiennej/funkcji.

       ;//.hide(0);
         
})
 
{
    var okno = document.getElementById('okno');

komentarz 9 marca 2017 przez Patryk Rafał Bywalec (2,700 p.)
edycja 9 marca 2017 przez Patryk Rafał

widzę że korzystasz z jquery to nie musisz szukać elementu document..... tylko var okno = $("#okno"); nie prawda że krócej, a co do twojego problemu to ja bym to zrobił tak

napisałem ci pomocny kod , myślę że juz sobie poradzisz

<!DOCTYPE html>
<html>
<head>
<script src="https://ajax.googleapis.com/ajax/libs/jquery/3.1.1/jquery.min.js"></script>
<script>
$(document).ready(function(){
var licznik = 1;
	$("#hide").click(function(){
    	licznik++;
        if(licznik % 2 == 0){
        	$("p").hide();
            $("button").text("Pokaz");
        }else if(licznik % 2 == 1){
        	$("p").show();
            $("button").text("Ukryj");
        }
    });
  
});
</script>
</head>
<body>

<p>If you click on the "Hide" button, I will disappear.</p>

<button id="hide">Hide</button>


</body>
</html>