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

572

    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. Mistä puhuitte viimeksi kun näitte

      Kerro yksi aiheista
      Ikävä
      107
      7793
    2. 113
      6064
    3. Se on hyvästi

      Toivottavasti ei tavata.
      Ikävä
      83
      5197
    4. Olenko saanut sinut koukkuun?

      Hyvä. Rakastan sua.
      Ikävä
      139
      4556
    5. Alavuden sairaala

      Säästääkö Alavuden sairaala sähkössä. Kävin Sunnuntaina vast. otolla. Odotushuone ja käytävä jolla lääkäri otti vastaan
      Ähtäri
      11
      3230
    6. Sisäsiittosuus

      Tämän kevään ylioppilaista 90% oli sama sukunimi?
      Suomussalmi
      62
      3024
    7. Miksi sä valitsit

      Juuri minut sieltä?
      Ikävä
      58
      2948
    8. Törkeää toimintaa

      Todella törkeitä kaheleita niitä on Ylivieskassakin. https://www.ess.fi/uutissuomalainen/8570818
      Ylivieska
      10
      2454
    9. Kerro nyt rehellisesti fiilikset?

      Rehellinem fiilis
      Suhteet
      61
      2437
    10. Hei........

      Pelkkä sun näkeminen saa mut hymyilemään pitkin iltaa. Oot niin 🤩😘 Edellinen poistettiin.
      Ikävä
      56
      2066
    Aihe