Moi!
Tässä koodissa on if-lauseke jossa on currentLevel[y][x] == "S" mutta se ei toimi vaikka sen pitäisi. Testasin myös currentLevel[1][1] jossa "S" sijaitsee mutta ei toimi silti. Eli "if" ei toimi vaikka pitäisi toimia. Koodi:
for (int y = 0; y < 15; y ) {
for (int x = 0; x < 20; x ) {
if(inFile.hasNext()) {
line = inFile.next();
currentLevel[y][x] = line;
//Nämä if eivät toimi vaikka tuolla arrayssa on S.. Testasin myös laittaa tohon suoraan currentLevel[1][1] jossa S on...
if(currentLevel[y][x] == "S") {
System.out.println("Tää toimii!");
PlayerX = x * 35;
PlayerY = y * 35;
PlayerMoveY = y;
PlayerMoveX = x;
} else if(currentLevel[y][x] == "F") {
MoneyInLevel ;
} else if(currentLevel[y][x] == "E") {
LevelFinishX = x * 35;
LevelFinishY = y * 35;
}
}
}
}
Ei toimi vaikka pitäisi
7
244
Vastaukset
- ööp
Kokeile .equals()
- pcmuseo
Onko S merkkijono vai merkki?
0x53,0x00 != 0x53
Eli pitäisikö verrata 'S' ?
http://docs.oracle.com/javase/tutorial/java/data/characters.html - dadaak
Vosiko olla Joku ylimääräinen merkki esim '\n' perässä. trim ja sitten vertailu
- Tutki-sitä
Katso linen sisältö inFile.next():n jälkeen.
- nuubi
Merkkijonoa verrataan .equals() tai .matches()
String-luokassa on myös equalsIgnoreCase, joka jättää isot ja pienet kirjaimet huomiotta, sama kumpi on. Lisäksi esimerkissäsi et kasvata for-silmukoissa x:n ja y:n arvoja jostain syystä. Käy koko matriisi läpi. Ja tarkista, että currentLevel on String-taulukko. Alustuksen pitäisi olla tämä: String[][] currentLevel = new String[15][20];
Lisäksi oletko sijoittanut hakemaasi matriisin alkioon jonkin merkkijonon, vai tuleeko NullPointerException. Debuggaa koodi, ja tarkista arvot joka kierroksella. Kyllä se ratkeaa siitä.- Vika_for_loopissa
Pitää suurentaa muuttujia joka kierroksella x ja y .
Ketjusta on poistettu 0 sääntöjenvastaista viestiä.
Luetuimmat keskustelut
Taitaa jäädä kotimaiset mansikat ostamatta
Kotimainen mansikka on niin kallista, että en ole vielä ainuttakana maistanut. Jos hinta pysyy näin korkealla niin tästä792322Sinkkumiehet hukkaavat tärkeän ässän hihastaan kun
...eivät suostu kavereiksi naisten kanssa. Mikä voi olla heillä syynä? Hyväksyvät vain naisen, joka suorastaan anelee sa1371481- 1331452
- 1541193
- 1081013
"Kaikkien miesten asia" - kampanja on alkanut
Miehillä on naisiin kohdistuvan väkivallan lopettamisessa merkittävä rooli. Ei riitä, ettei itse tee väkivaltaa. Miesten373927Tiedät, että en voi enää laittaa viestiä
Aikaa kulunut. Eikä se näyttäisi enää luontevalta vastata näin pitkän ajan jälkeen. Tiedän myös, että sinä et enää lait82774Lautakunta käsittelee Iisalmen kulttuuri- ja vapaa-aikajohtajan virkasuhteen purkua koeajalla:
Lautakunta käsittelee Iisalmen kulttuuri- ja vapaa-aikajohtajan virkasuhteen purkua koeajalla: "Aina valinta ei mene nap55724Lienee aika luopua siitä kaikesta
mitä meillä ikinä olikaan. Hassua, koska juuri mitään ei ole edes ollutkaan. En vaan jaksa tätä mahdotonta juttua enää j67684Kun kohtaatte rakkauden, tarttukaa siihen
Toimisinko jälkiviisaana toisin? Varmasti. Vaikka silloin kuvittelin tekeväni, niin kuin on oikein. Mahdollisimman siist50683