Reklamer

Java Validation

Forum: Andet

#1 - Jon Kanon - Thursday 17-09-2009 kl. 11:42

Hey.. Er der nogen af jer der kan hjælpe med noget java validation. Arbejder ikke med java normalt..

Først og fremmest kan jeg ikke få isNaN til at virke som jeg vil, se her:
function validate_form()
{
// validates that the entry is formatted as an e-mail address





var reg = /^([A-Za-z0-9_\-\.])+\@([A-Za-z0-9_\-\.])+\.([A-Za-z]{2,4})$/;
var address = document.buoy.emailAddr.value;


errmsg="";

if (document.buoy.emailAddr.value=="")
{alert("Email Field is Blank \n\nPlease insert your email");
return false;}
if(reg.test(address) == false) {
alert("Please enter a valid email");
return false;}

for(var i=1;i<=number;i++)
{
var lat="document.buoy.latd"+i+".value";
var lon="document.buoy.lond"+i+".value";

//Check the value numbers
if (eval(lat)lat2||eval(lat)=="")
{errmsg +="LATITUDE FOR BUOY "+i+": Choose a value between "+lat1+"N and "+lat2+"N\n";}
if (isNaN(lat))
{errmsg +="LATITUDE FOR BUOY "+i+": Have to be a number\n" ;}

if (eval(lon)lon2||eval(lon)=="")
{errmsg +="LONGITUDE FOR BUOY "+i+": Choose a value between "+-1*lon1+"W and "+lon2+"E\n";}
//if (isNaN("buoy.lond"+i+".value"))
// {errmsg +="LONGITUDE FOR BUOY "+i+": Have to be a number\n" ;}
}



if (errmsg=="")
{return true;}
else{
alert(" Following Error Have Occured:\n\n"+errmsg);
return false;}
}


Hvis der er en der er super god(Allan??) til java, og det kun tager 5 min, bliver jeg bare så glad, hvis i/du kunne hjælpe:)

#2 - Jon Kanon - Thursday 17-09-2009 kl. 11:47

.. edit .. edit ..
ændret: Thursday, September 17, 2009 kl. 15:48

#3 - Riley - Thursday 17-09-2009 kl. 12:17

http://www.w3schools.com/jsref/tryit.asp?filename=tryjsref_isnan

kig der, har lidt svært ved at afgøre om din java kode gør det rigtige, men det ser pænt korrekt ud!

#4 - Jon Kanon - Thursday 17-09-2009 kl. 12:43

Well den siger at det ikke er et nummer lige meget hvad. Så jeg tænkte at problemmet muligvis lå i variablen jeg henter fra min form, altså:
var lat="document.buoy.latd"+i+".value";

er der en måde at trimme denne på og hvorfor mon den første eval virker nå den anden ikke gør :o Det er jo også et nummer den bruger..

Vil prøve at smide min kode ind i linket senere i aften, er på vej ud af døren:) Men den virker jo ikke på hjemmesiden, så ved ikke hvorfor det skulle virke der??

#5 - Jon Kanon - Friday 18-09-2009 kl. 04:51

løste problemmet ved at definere en array først således:
var lat_ar=new Array("",buoy.latd1.value,buoy.latd2.value,buoy.latd3.value,buoy.latd4.value,buoy.latd5.value,buoy.latd6.value,buoy.latd7.value,buoy.latd8.value,buoy.latd9.value,buoy.latd10.value);
og derefter loope over
if (isNaN(lat_ar[i]))
{errmsg +="LATITUDE FOR BUOY "+i+": Have to be a number\n" ;}

Det vil være et større problem hvis man f.eks. har 1000 indicer man skal igennem, men her funger det fint, og har ikke tænkt mig at bruge mere tid på det:)

edit: det sidste jeg skrev med 1000 indicer er selvfølgelig noget vrøvl, da man ligesom i alle andre sprog, helt sikkert kan definere en array inde i en løkke. Min pointe var bare at jeg ikke vidste how:)
ændret: Sunday, September 20, 2009 kl. 07:44

#6 - Kare1234 - Friday 18-09-2009 kl. 05:46

Nice. Jeg fatter absolut minus af Java, men måske skulle du overveje at skralle ting af funktionen indtil du finder ud af, hvad der er galt. Btw: hvad med syntax highlight på spo?

#7 - Jon Kanon - Friday 18-09-2009 kl. 07:03

jep, jeg udkommenteret tingene, så jeg fandt ud af at det var "" der var problemmet. En lille rettelse er at man bliver nød til at flytte .value i array'en ind under lykken, ellers leder den efter en værdi, som muligvis ikke eksistere (for i<10)..

#8 - Kare1234 - Friday 18-09-2009 kl. 08:22

Ha! Jeg plejer bare at bruge try-except. SÃ¥ crasher koden aldrig!

#9 - Jon Kanon - Friday 18-09-2009 kl. 09:39

Jep, men er det ikke lidt overkill, i forhold til et lille program:)

#10 - Kare1234 - Friday 18-09-2009 kl. 10:00

Det er udemærket til at teste, fordi man kan få fejlene smidt i hovedet uden at programmet stopper fordi man (f.eks.) har en syntaxfejl. Jeg ved godt, at Allan mener at det er lidt til "the dark side" men wtf.

#11 - Riley - Friday 18-09-2009 kl. 11:23

nono try catch er fint hvis du så bare håndterer fejlene... bruger for det meste selv alerts til at tracke hvornår fejlen sker, men det er mere fordi js er det argeste lort at debugge

#12 - Jon Kanon - Friday 18-09-2009 kl. 14:24

#11 Jaah det har jeg fundet ud af:)

#13 - brian - Sunday 20-09-2009 kl. 05:45

@6: stensikkert syntaxhighlighting på 5p0. Men måske vigtigere; code tag så begyndende whitespace ikke går tabt..

Ellers ret irriterende kode at læse på spo. Jeg ved det er en indivudiel ting, men må jeg evt foreslå at du sætter dine afsluttende '}' på en efterfølgende linie, i samme indrykningsniveau som blokken du afslutter, medmindre det er en blok på en linie?

Din isNaN returnerer sandt fordi du spørger den om en streng er et tal. Jeg vil skyde på at /if (isNaN(lat))/if (isNaN(eval(lat)))/ vil kunne løse dit problem?
Jeg er nysgerrig; hvad gør fremhævede?
if (eval(lat)lat2||eval(lat)=="")

#14 - Jon Kanon - Sunday 20-09-2009 kl. 07:30

Forstår din frustration mht {} og jeg plejer at følge nogle regler for mig selv der. hvis det er en if eller for funktion, plejer jeg normalt at holde dem på en linie, med mindre det kan stå på en linie, with wrap.

Well koden kom til at se således ud og det virker:):

var lat_ar=new Array("",buoy.latd1, buoy.latd2 ,buoy.latd3, buoy.latd4 ,buoy.latd5, buoy.latd6,buoy.latd7, buoy.latd8, buoy.latd9, buoy.latd10);
for(var i=1;i<=number;i++)
{
if (isNaN(lat_ar[i].value)){}
}

Forstår din forvirring og ved ikke hvorfor det er skrevet sådan i spo, men den rigtige syntax er:
if (eval(lat)>lat1||eval(lat)<lat2||eval(lat)=="")
ændret: Sunday, September 20, 2009 kl. 11:31
ændret: Sunday, September 20, 2009 kl. 11:37
ændret: Sunday, September 20, 2009 kl. 11:37

#15 - brian - Sunday 20-09-2009 kl. 07:48

Nu ved jeg ikke hvor meget du har ændret i dit oprindelige kode, men bare lige for at klarificere hvad jeg mente med /noget/noget andet/;

Når du spørger:
if (isNaN(lat))

er 'lat' en tekstreng og svaret derfor altid sandt - "bla bla 43" er ikke et tal. Hvis du evaluerede den ville du (måske) få et tal, fordi lat er en kommando du har opbygget:
if (isNaN(eval(lat)))

... eller sagt på en anden måde /noget/noget andet/ betyder udskift noget med noget andet..

#16 - Jon Kanon - Monday 21-09-2009 kl. 04:07

Aha Brian.. Nu fik jeg rent faktisk tjekket det igen og
if(isNaN(eval(lat))) virkede rent faktisk..

Jeg havde vist prøvet at skrive følgende

var lon=eval("document.buoy.lond"+i+".value");

og det virkede ikke.. Men jeg forstår det hele lidt bedre nu:)

#17 - Jon Kanon - Monday 21-09-2009 kl. 05:27

Brian (eller en anden), kan du så gennemskue hvorfor dette ikke virker:
var db = "buoy.depth"+i+;

if (!eval(db)[0].checked && !eval(db)[1].checked)
{
errmsg +="error1\n";
}

som bruges til at checke om der er klikket på en radiobutton med 2 felter og hvor jeg igen har en i-løkke rundt om. Har løst det på en anden måde, men kunne da være interessant at finde ud af:)

#18 - Jon Kanon - Monday 21-09-2009 kl. 06:11

Ups: F5;)
ændret: Monday, September 21, 2009 kl. 10:11

Du skal vaere logget ind og godkendt for at kunne skrive en kommentar

Reklamer