If-setning som går meg på nervene

28. februar 2012

Har følgende kode:

Kode

if(!(tekstfelt1.length() > 0) || !(tekstfelt2.length() > 0)) 
					Toast.makeText(this, "Feil input", Toast.LENGTH_SHORT).show();

Men programmet bare kræsjer når tekstfelt2 er tom. Hvor surrer jeg? Altfor mye koding på en dag, skjønner jeg..

Butterfly

Evil Panda

551 218

28. februar 2012

Nå har jeg ikke kodet i php før, men lurer på hvorfor du negerer istedenfor å endre > til <= ?

Opous

Trådstarter

185 15

28. februar 2012

Glemte å nevne at det er java, bruker bare PHP taggene for å marker kode.

Stringen tekstfelt1 inneholder dataene fra et tekstfelt, om strengen ikke er større en 0, så skal en Toast raises.

s1gh

1.785 384

28. februar 2012

Hva med å vise litt mer av koden? Og hvilken feilmelding får du?

Opous

Trådstarter

185 15

28. februar 2012

Det er i grunn bare denne biten som er aktuell. Er jo i realiteten en enkel sjekk på om to tekstfelter ikke er fyllt inn. Men alikevell:

Kode

String tekstfelt1 = tekst1.getText().toString();
String tekstfelt2 = tekst2.getText().toString(); 

//MÅ FIKSE DENNE SJEKKEN
if(!(tekstfelt1.length() > 0) || !(tekstfelt2.length() > 0)) 
	Toast.makeText(this, "Feil input", Toast.LENGTH_SHORT).show();
else {         	
	//to funksjonskall som jeg vet fungerer og får riktig input
}

I følge debuggern får jeg at tekstfelt2 inneholder " ".

Herregud. Problemet lå rett og slett i en annen funksjon som setter tekstfelt2 = " " hvis en checkbox er unchecked..Etter 12 timer med koding går det lett i surr....

Sist endret av Opous; 28. februar 2012 kl. 21:26.

Butterfly

Evil Panda

551 218

28. februar 2012

Nå vil vel aldri Stringene her være null, men just in case kan du sjekke det før du sjekker lengden, så unngår du iallefall en NullPointerException:

Kode

String tekstfelt1 = tekst1.getText().toString();
String tekstfelt2 = tekst2.getText().toString(); 

if((tekstfelt1 == null || tekstfelt1.length() < 1 ) || (tekstfelt2 == null || tekstfelt2.length() < 1)) 
    Toast.makeText(this, "Feil input", Toast.LENGTH_SHORT).show();
else {             
    //to funksjonskall som jeg vet fungerer og får riktig input
}

Edit: ahh du fant ut av det.

Sist endret av Butterfly; 28. februar 2012 kl. 21:34.