Tarkistussilmukka pielessä?

Hamppuravatti

Olen kirjoittamassa ohjelmaa, jossa syötetään arvosanoja. Arvosanoista lasketaan keskiarvo.

Tämä alla oleva tarkistussilmukka ilmeisesti ei jostain syystä kelpaa. Osaisko joku sanoa mitä vikaa siinä on. Ideana siis on, että vain luvut 1-5 kelpaavat arvosanoiksi.


while(arvosana != VARTIJA && arvosana < 1 || arvosana > 5)
      {
         printf("\nVain luvut 1-5 kelpaavat (99 lopetukseen)");
         printf("\nAnna ensimmäinen arvosana > ");
         scanf("%d", & arvosana);
      
         summa = arvosana;
      }

10

585

    Vastaukset

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

      (arvosana < 1 || arvosana > 5) ?

    • korjauksia

      Ylempänä tuli jo yksi korjaus. Lisäksi virheelliset sekä lopetusarvot lisätään tuossa versiossa summaan.

      • CÄijö

        Minummielestä VARTIJA pitää kirjottaa pienellä näin: vARTIJA


      • Hamppuravatti

        Kirjoitin tuon silmukkarakenteen hieman uusiksi. Ohjelma toimii nyt kuten pitääkin sisempää silmukkaa (lukuvälitarkistus) lukuunottamatta. Sanoisitko miten tuon sisemmän silmukan ehto pitäisi kirjoittaa, jotta se lopettaa VARTIJAN arvolla?

        while(luku != VARTIJA)
           {
              while(luku < 1 || luku > 5)
              {
                 printf("\nVain luvut 1-5 kelpaavat (%d lopetukseen)", VARTIJA);
                 printf("\nAnna sopiva luku > ");
                 scanf("%d", & luku);
                 
              }


      • snöööög
        Hamppuravatti kirjoitti:

        Kirjoitin tuon silmukkarakenteen hieman uusiksi. Ohjelma toimii nyt kuten pitääkin sisempää silmukkaa (lukuvälitarkistus) lukuunottamatta. Sanoisitko miten tuon sisemmän silmukan ehto pitäisi kirjoittaa, jotta se lopettaa VARTIJAN arvolla?

        while(luku != VARTIJA)
           {
              while(luku < 1 || luku > 5)
              {
                 printf("\nVain luvut 1-5 kelpaavat (%d lopetukseen)", VARTIJA);
                 printf("\nAnna sopiva luku > ");
                 scanf("%d", & luku);
                 
              }

        while (1


      • snöööög
        Hamppuravatti kirjoitti:

        Kirjoitin tuon silmukkarakenteen hieman uusiksi. Ohjelma toimii nyt kuten pitääkin sisempää silmukkaa (lukuvälitarkistus) lukuunottamatta. Sanoisitko miten tuon sisemmän silmukan ehto pitäisi kirjoittaa, jotta se lopettaa VARTIJAN arvolla?

        while(luku != VARTIJA)
           {
              while(luku < 1 || luku > 5)
              {
                 printf("\nVain luvut 1-5 kelpaavat (%d lopetukseen)", VARTIJA);
                 printf("\nAnna sopiva luku > ");
                 scanf("%d", & luku);
                 
              }

        while (!(luku < 1 || luku > 5))

        Jota ehkä yritit.


      • Hamppuravatti
        snöööög kirjoitti:

        while (1

        Tuo ei ole vastaus esittämääni kysymykseen, koska tarkistussilmukka toimii esittämänäni versiona kun syötetään arvoja, jotka ovat pienempiä kuin 1 tai suurempia kuin 5.

        Pulmanani on saada tuo sisempi tarkistussilmukka toimimaan siten, että jos sen kuluessa syötetään VARTIJA:ssa määritelty arvo (99), niin koko ohjelman tulisi loppua.


    • c-burnout

      Jos sisäkkäiset while silmukat menevät solmuun niin on joskus parempi tehdä se toisin. Ja sen sijaan että tarkistaa kaikki ehdot while silmukoissa, ne voi yhtä hyvin tarkistaa erillisillä if-lauseilla jolloin voi paremmin kontrolloida mitä ohjelma tekee missäkin vaiheessa. Itse toteuttaisin ohjelman esimerkiksi tällä tavalla (vaihtoehtoisia tapoja on toki monia monia muitakin):

      #include

      #define VARTIJA -1

      int main(void)
      {
      int luku = 0;
      int summa = 0;
      int maara = 0;

      printf("\nAnna lukuja 1-5 joista lasketaan summa ja keskiarvo. (%d lopetukseen)\n", VARTIJA);

      do {

      printf("Anna sopiva luku > ");
      scanf("%d", & luku);

      if (luku >= 1 && luku

    • c-burner

      Foorumi toimii taas päin helvettia, joten moneskohan kerta on kun yritän saada tän viestin lähetettyä. No tässä mun tapa ratkaista ongelma (ellei ole pakko käyttää kahta while -silmukkaa):

      #include

      #define VARTIJA -1

      int main(void)
      {
      int luku = 0;
      int summa = 0;
      int maara = 0;

      printf("\nAnna lukuja 1-5 joista lasketaan summa ja keskiarvo. (%d lopetukseen)\n", VARTIJA);

      do {

      printf("Anna sopiva luku > ");
      scanf("%d", & luku);

      if (luku >= 1 && luku

    • c-burner

      Foorumi toimii taas päin helvettia, joten moneskohan kerta on kun yritän saada tän viestin lähetettyä. No tässä mun tapa ratkaista ongelma (ellei ole pakko käyttää kahta while -silmukkaa):

      #include

      #define VARTIJA -1

      int main(void)
      {
      int luku = 0;
      int summa = 0;
      int maara = 0;

      printf("\nAnna lukuja 1-5 joista lasketaan summa ja keskiarvo. (%d lopetukseen)\n", VARTIJA);

      do {

      printf("Anna sopiva luku > ");
      scanf("%d", & luku);

      if (luku >= 1 && luku

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

    Luetuimmat keskustelut

    1. Voiko normaali ihminen ryhtyä vasemmistolaiseksi?

      Tätä jäin pohdiskelemaan.
      Maailman menoa
      202
      4208
    2. SDP haluaa 40 000 nettomaahanmuuttajaa

      SDP:n Suunnanmuutos-vaihtoehtobudjetissa, käy ilmi, että demarit itse asiassa vaativat räjähdysmäistä ”työperäisen” maah
      Maailman menoa
      146
      3778
    3. Orpo: Velkajarrua vastustavaa puoluetta vaikea ajatella hallitukseen

      No Minja Koskelan kommunistipuolue jäi ulos tuosta. Kaikki eduskuntapuolueet vasemmistoliittoa lukuun ottamatta sopivat
      Maailman menoa
      84
      3132
    4. Hienoa! Eduskunta luopui käteisen käytöstä

      Nyt tuo sama muutos pitää saada myös muuhun yhteiskuntaan. Käteistähän ei tarvitse tänä päivänä enää kuin rikolliset.
      Maailman menoa
      43
      1579
    5. Ikävä sinua mies

      Vuosia kuluu, mutta tunteet ei ole hävinnyt. Tasoittuneet toki, kun ei olla nähty. Järki palannut päähän kuitenkin. Se i
      Ikävä
      19
      1418
    6. Sulla on avaimet ja keinot

      Jos haluat jatkaa tutustumista. Itse olen niin jäässä etten pysty tekemään enää mitään. Pidempi keppi johon on helpompi
      Ikävä
      25
      1365
    7. Mikä tämä henkilö mahtaa touhuta Parkanossa

      Kamalaa https://www.ylasatakunta.fi/teksti/pirkanmaan-karajaoikeus-vangitsi-koiran-tappamisesta-epaillyn-6.68.127794.b58
      Parkano
      32
      1357
    8. Orpo loukkaantui fasismiin viittaavasta sanavalinnasta

      Mutta miksi loukkaantui? Orpohan on tehnyt yhteistyötä fasistien kanssa jo vuonna 2019, siis jo neljä vuotta ennen loukk
      Maailman menoa
      27
      1341
    9. Kiinnostaa - ei kiinnosta - kiinnostaapas

      Selittäkää hämmentyneelle miksi miehiä ei ikinä kiinnosta silloin, kun sitä olisi itsekin kiinnostunut? Sitten kun siirt
      Sinkut
      116
      1130
    10. Martina haluaa Marbellaan

      Martinan tekisi mieli ottaa lennot Marbellaan, jossa näkisisi kauniita ja hyväntuulisia ihmisiä. No sitten pitää matkust
      Kotimaiset julkkisjuorut
      215
      1053
    Aihe