Viopen Java-kurssilla, vertailuoperaattoritehtävä

Anonyymi

Voisko joku fiksumpi kertoa mitä koodissani on väärin? Olen kokeillut ihan kaikkea ja googlaillut monta tuntia ja useampana päivänä, mutta en vaan saa tätä oikein!!
Kiitos tuhannesti jo etukäteen jos joku osaa auttaa :)

Tehtävänanto:
Tee ohjelma, joka kysyy ensin käyttäjän sukupuolen ja tämän jälkeen iän. Lopuksi ohjelma tulostaa jonkin lausahduksen seuraavan ohjeen mukaisesti:
- Mies 0-55 vuotta: "Olet mies parhaassa iässä!"
- Mies 56-100 vuotta: "Olet viisas mies!"
- Nainen 0-55 vuotta: "Olet neito kauneimmillaan!"
- Nainen 56-100 vuotta: "Olet nuori ikäiseksesi!"
- muuten "Ohjelmassa tapahtunut virhe!"

Ikä käsitellään kokonaislukumuuttujana ja sukupuoli merkkimuuttujana (m/n). Isoja kirjaimia ei tarvitse huomioida.


Yritykseni ratkaista:

import java.io.*;

public class Tulostus {
public static void main(String[] args) {
int ika;

BufferedReader in = new BufferedReader(new InputStreamReader(System.in));

try {
System.out.print("Oletko mies vai nainen (m/n)?");
String sukupuoli = in.readLine();
char jonon_eka_merkki = sukupuoli.charAt(0);

System.out.println("Anna ikäsi:");
ika = Integer.parseInt(in.readLine());

if (ika >= 0 && ika <= 55 && jonon_eka_merkki.equals('m'))
System.out.println("Olet mies parhaassa iässä!");
else if (ika >= 56 && ika <= 100 && jonon_eka_merkki.equals('m'))
System.out.println("Olet viisas mies!");
else if (ika >= 0 && ika <= 55 && jonon_eka_merkki.equals('n'))
System.out.println("Olet neito kauneimmillaan!");
else if (ika >= 56 && ika <= 100 && jonon_eka_merkki.equals('n'))
System.out.println("Olet nuori ikäiseksesi!");
else System.out.println("Ohjelmassa tapahtunut virhe!");

} catch(Exception e) {
System.out.print("Antamasi syöte oli virheellinen...");
}
}
}

11

1407

    Vastaukset

    Anonyymi (Kirjaudu / Rekisteröidy)
    5000
    • Anonyymi
      • Anonyymi

        Kiitos paljon vastauksesta! Kokeilin myös noita aaltosulkeita, mutta se ei tuntunut auttavan. Myöskään kurssin esimerkissä ei käytetty aaltosulkeita if ja if else -lauseissa, ilmeisesti siksi että ovat try - catch -lauseen sisällä. Sain koodin onnistumaan pelkkää ikää kysyttäessä, joten uskoisin että vika on tuossa sukupuolen kysymisessä.

        Tämä tehtävä on kyllä tosi hankala!


      • Anonyymi kirjoitti:

        Kiitos paljon vastauksesta! Kokeilin myös noita aaltosulkeita, mutta se ei tuntunut auttavan. Myöskään kurssin esimerkissä ei käytetty aaltosulkeita if ja if else -lauseissa, ilmeisesti siksi että ovat try - catch -lauseen sisällä. Sain koodin onnistumaan pelkkää ikää kysyttäessä, joten uskoisin että vika on tuossa sukupuolen kysymisessä.

        Tämä tehtävä on kyllä tosi hankala!

        Aaltosulkeita ei tarvitse, jos suoritettavassa lohkossa on vain yksi lause (eli jos aaltosulkeita ei laita, niin if ottaa vain seuraavan yhden lauseen "sisälleen"). Suositeltavaa niitä on kyllä käyttää aina, niin ei tule epäselvyyksiä.


      • minkkilaukku kirjoitti:

        Aaltosulkeita ei tarvitse, jos suoritettavassa lohkossa on vain yksi lause (eli jos aaltosulkeita ei laita, niin if ottaa vain seuraavan yhden lauseen "sisälleen"). Suositeltavaa niitä on kyllä käyttää aina, niin ei tule epäselvyyksiä.

        Tietenkin, jos se siinä on vain joku hyvin lyhyt yksinkertainen, joka mahtuu samalle riville, niin sitten ehkä aaltosulkeet on sallittua jättää pois.


    • Anonyymi

      Tsekkaa ensin että onko merkkijono m TAI n JA (ikä >= 0 JA ikä <= 100). Jos ei ole niin tulosta "virhe". Sen jälkeen tsekkaa Ifissä oliko merkkijono m, jonka jälkeen tsekkaat iän ja tulostat jomman kumman mies-kommentin. Else-lohkossa tsekkaat iän ja tulostat sopivan nais-kommentin.

    • Anonyymi
    • Korvaa tuo equals vertailu ==:lla.
      Eli esim
      jonon_eka_merkki == 'm'

      • Anonyymi

        Kiitos!! Tuo equalsin korvaaminen == -merkillä auttoi! Jee! Tehtävä meni läpi. Itse olin jostain syystä uskossa että == -merkkiä saa käyttää vain jos on numeroista kyse ja muissa tapauksissa tulisi käyttää equalsia.


      • Anonyymi
        Anonyymi kirjoitti:

        Kiitos!! Tuo equalsin korvaaminen == -merkillä auttoi! Jee! Tehtävä meni läpi. Itse olin jostain syystä uskossa että == -merkkiä saa käyttää vain jos on numeroista kyse ja muissa tapauksissa tulisi käyttää equalsia.

        Jaavasta en tiedä, mutta ainakin ehdassa C-kielessä char on vastaavanlainen numeerinen arvo kuin vaikkapa int. Merkkijono on sitten tyyppiä char *, eli arvo on muistiosoitin.


      • Anonyymi kirjoitti:

        Kiitos!! Tuo equalsin korvaaminen == -merkillä auttoi! Jee! Tehtävä meni läpi. Itse olin jostain syystä uskossa että == -merkkiä saa käyttää vain jos on numeroista kyse ja muissa tapauksissa tulisi käyttää equalsia.

        Joo char on primitiivi, kuten int, double yms. Tuo equals-metodi on objekteille.

        Primitiiveillekin on sitten olemassa wrapper-luokat (eli objekti, joka vain pitää sisällään kyseistä primitiivin arvoa) esim Character.

        Eikös sulla muuten kääntäjä kertonut sitä että virhe on noissa neljässä kohdassa?


      • Anonyymi
        minkkilaukku kirjoitti:

        Joo char on primitiivi, kuten int, double yms. Tuo equals-metodi on objekteille.

        Primitiiveillekin on sitten olemassa wrapper-luokat (eli objekti, joka vain pitää sisällään kyseistä primitiivin arvoa) esim Character.

        Eikös sulla muuten kääntäjä kertonut sitä että virhe on noissa neljässä kohdassa?

        Linuxin javac herjaa ainakin "error: char cannot be dereferenced".


    Ketjusta on poistettu 0 sääntöjenvastaista viestiä.

    Luetuimmat keskustelut

    1. Kiva kun SDP alkaa hallitsemaan Suomea

      Vanhat hyvät ajat taas palaavat ja kansa vaurastuu. Muistatteko vielä Sorsan aikakauden? Silloin Suomessa tehtiin jopa
      Maailman menoa
      260
      6470
    2. SDP:n lyhyt selviytymisopas

      1. Komitea on vastaus, oli kysymys mikä tahansa Jos maailma on muuttumassa tai jossain palaa, demari ei hätiköi. Ensin p
      Maailman menoa
      79
      5128
    3. Keskisen kyläkaupassa Temun vaatteita myynnissä?

      Siis mitä? Miksi siis ei itse tilaisi Temusta samoja.
      Maailman menoa
      96
      4037
    4. SÄHKÖLASKU 1600 EUROA Ei tässä ole mitään järkeä

      https://www.is.fi/taloussanomat/art-2000011804443.html Kyllä täysin huijausta nämä sähkömarkkinat. Varsinkin pörssisähk
      Maailman menoa
      268
      3512
    5. Säästäminen on typerää, muistakaa äänestää demareita

      Säästäminen on typerää, koska aiheuttaa vain talouden taantumista ja lopulta tappaa potilaan. Demareiden tapa on satsat
      Maailman menoa
      63
      3284
    6. Toivoisitko

      Toivoisitko, että kaivattusi olisi introvertimpi tai extrovertimpi? Itsenäinen tai tarvitsisi enemmän apua/sinua? Osoit
      Ikävä
      146
      3089
    7. Olli Rehn: Eläkkeistä pitää leikata. Nyt tuli Lindtmanille kauhun paikka

      jos johtaa seuraavaa hallitusta. Purra: eläkkeisiin ei kosketa. Eikä tällä hallituskaudella varmasti kosketa, mutta seur
      Maailman menoa
      230
      2657
    8. Venäläiset varkaudessa

      Ja venäjä mieliset. Teitä kiinnostaa toisinajattelijat ja trafikista päätellen aika paljonkin. Kunhan aika koitaa tulee
      Varkaus
      270
      2000
    9. SDP:n budjetin peruskivi: "Rahaa nimittäin on!"

      Demarien talouspolitiikan ydin on usein tiivistetty klassiseen meemiin: rahaa on, kunhan se on jonkun muun rahaa. Vuoden
      Maailman menoa
      39
      1538
    10. Lupajaoston esityslista

      Ilmoitus rakennusvalvonntaan luvattomasta rakentamisesta. esityslistan 6pykälä Kyse on kulttuuritalon rakentamisesta jo
      Pyhäjärvi
      49
      1435
    Aihe