Validere button med en prompt

else
  return

Det hjælper ikke rigtig, den sender det afsted alligevel..

Hej nannami,


I din else returnerer du false, altså:

function question()
{
  answer = "2007";
  question = prompt("Hvilket årstal har vi?",'');

  if (question==answer)
    alert('Svar korrekt! Klik OK for at udføre!');
  else 
    return false;
} 


Og når du tilføjer en metode til din onclick event tilføjer du et return, altså:


BtnSend.Attributes.Add("onclick", "return question();");


Det burde gøre tricket! :)


Mvh.

- Snap

Hejsa crazysnap.

Det ser næsten ud til at virke. Man trykker på knappen, svarer forkert og der sker ikke nogget. Men trykker man på knappen igen derefter, udføres eventet uden spørgsmål..

Hej nannami,


Ja, det har du jo fuldstændig ret i og grunden er sådan set meget simpel. Din variabel 'qeustion' er ikke erklæret, dvs du mangler 'var' foran. Så sammenlagt og for at få det til at virke skal du ændre:


question = prompt("Hvilket årstal har vi?",'');

til:

var question = prompt("Hvilket årstal har vi?",'');




For nemhedsskyld har jeg lige pastet den revideret javascript metode ind: :)


function question()
    {
        answer = "2007";
        var question = prompt("Hvilket årstal har vi?",'');

        if (question==answer)
        {
            alert('Svar korrekt! Klik OK for at udføre!');
            return true;
        }
       
        alert('Svaret er forkert!');
        return false;
  } 


Håber det virker nu! :)


Mvh.

- Snap

Ja nu virker det! :-) Er det noget med at variablen bliver "re-erklæret" hver gang der bliver kørt eller hvordan? Jeg kan også se at du i det reviderede har lavet lidt om på konstruktionen og fjernet else?

Hej nannami,


Grunden til det ikke virkede før var fordi det faktisk ikke var valid javascript da man assigner til en variabel der ikke er blevet erklæret (gøres med 'var'). Og helt galt var det i dette tilfælde da 'question' også har samme navn som selv metoden, det betyder at den faktisk prøvede at assigne prompt-svaret til metoden, hvilket jo overhovedet ikke giver mening. :)

Ja jeg flyttede lige lidt rundt på koden, ikke noget du behøver at addaptere overhovedet (du kan bare tilføje 'var' før question variablen), men synes bare det gjorde scriptet lidt mere overskueligt samt tydeliggjorde at der skal returneres 'true' når svaret er korrekt! :)



Mvh.

- Snap

Okay - fin forklaring! Så er jeg med. :-) Jeg vil gerne smide en .toLowercase() på et sted, så der ikke tages højde for hvordan et svar indtastes (hvis vi siger svaret er et ord frem for 2007)

Hvor kan jeg sætte det på`?

Jeg får det ikke til at virke med:

var question = prompt("Hvilket årstal har vi?",'');
question = question.toLowercase();

Det er måske igen fordi question har samme navn som selve metoden?

Hej igen,


1) det er toLowerCase()    (med stort c)  :)

2) Du skal også gøre det på 'answer' variablen, plus at der lige skal en 'var' foran 'answer' variablen (havde jeg overset).


Så samlet kan det se sådan ud:


function question()
    {
        var answer = "Hej Med Dig";
        var question = prompt("Skriv hej med dig?",'');

        if (question.toLowerCase() == answer.toLowerCase())
        {
            alert('Svar korrekt! Klik OK for at udføre!');
            return true;
        }
       
        alert('Svaret er forkert!');
        return false;
  } 


Mvh.

- Snap :)

Ah.. det c skulle være med stort, det er ikke altid til med det case sensivity :-)

Nu virker det jo fint som det skal! Tak for hjælpen!

Det var så lidt! :)


Tak for pointene og held og lykke med det!


Mvh.

- Snap :)