[c] char zadeva

Pozdravljeni!

Sem še bolj med začetnimi programerji in me tukaj nekaj muči pri char tipu. Vem da je majcena zankica, vendar je ne znam rešiti, in sicer takole. Koda zgleda takole:


int main()
{
char a,b,vsota,razlika,kvocient,produkt,zelja,s;


while(zelja!=5)
{


printf("Vnesi prvo stevilo a:");
scanf("%c%c",&a);
printf("Vnesi drugo stevilo b:");
scanf("%c%c",&b);
system("cls");

printf("Pritisnite 1 za izracun vsote\nPritisnite 2 za izracun produkta\nPritisnite 3 za izracun kvocienta\nPritisnite 4 za izracun razlike\nPritisnite 5 za izhod iz programa\nPritisnite S za izris smeskota\n");
scanf("%c%c",&zelja);
if(zelja=='1')
{
vsota=a+b;
printf("Vsota teh dveh stevil je = %c\n",vsota);
}
else if(zelja=='2')
{
produkt=a*b;
printf("Produkt teh dveh stevil je = %c\n",produkt);
}
else if(zelja=='3')
{
kvocient=a/b;
printf("Kolicnik teh dveh stevil je = %c\n",kvocient);
}
else if(zelja=='4')
{
razlika=a-b;
printf("Razlika teh dveh stevil je = %c\n",razlika);
}

else if(zelja=='5')
{
system("pause");
return 0;
}
else if(zelja=='s' || zelja=='S')
{
printf("\n\n * *\n");
printf(" - \n");
printf(" |________|\n");
}

}

}

Mene muči sedaj, ko naprimer pritisnem "1" da sešteje 2 števili, mi namesto številke vsote, izpiše isto znak. In ko želim sešteti 5 + 5 je rezultat "J". Zelim da pa bi bil stevilka 10.
Char sem uporabil samo za to, da ko pritisnes S da ti izrise smeskota. (malce je zamaknjen v kodi, čeprav izpis je zelo drugačen)

Hvala za vse.

Lp, Cyborg

char a;
printf("Vnesi prvo stevilo a:");
scanf("%c%c",&a);

Prvič, ne vem zakaj preberes dva znaka.
Drugič, namesto znaka raje preberi int (ali pa double)

int a;
printf("Vnesi prvo stevilo a:");
scanf("%d",&a);

Za željo lahko še vedno uporabljaš char.

Kot prvo in prvo. Nimaš kontrole na deljenje z 0 (če je b=0 se bo program neslavno sesul).

Hm. Zakaj char ?

Kot drugo in drugo. Nimaš postavljenih začetnh vrednosti za a in b

Programira se nikoli kar iz glave.

Vedno se najprej naredi načrt programa in na njem preveri delovanje. Kot lahko vidiš lahko že v par vrsticah zakriviš "division with zero".

Ko se uvede novo spremenljivko se vedno psotavi tudi začetne vrednosti. Neupoštevanje tega pravila vodi k veliki zgubi časa pri odkrivanju vzroka "kraških pojavov".

Uporabi možgane, zato jih pa imaš. Če imaš a in b tipa int ali float, potem morajo biti istega tipa tudi vsi možni rezultati (produkt, vsota, ... - sploh ne vem zakaj imaš različna imena). Drugače pa kakor si že opazil ti "%c" v printf formau pove da izpisuješ char, se pravi znak in ne številke, kar moraš povedati z "%d"za inte oziroma "%f" za floate.

Zakaj je scanf("%d%c", a) narobe? Resda ti prebere še enter takoj za znakom a tega nikamor ne pospraviš. Toliko kolikor spremenljivk prebereš, toliko jih moraš shraniti. Bolje bi bilo scanf("%d%d", a, dummy), pri čemer spremenljivke dummy pač nikjer ne uporabiš. Mislim pa da deluje tudi naslednji način scanf("%d\n", a). Drugače si pa poglej specifikacijo scanf-a za njegovo pravilno uporabo.

Daj double. Kje pa sploh uporabljaš spremenljivko s?

Če že praviš, da delaš v C++ si olajšaj življenje in uporabi std::string ter tokove.

Je pa programiranje, ali ne? Mas več jezikov, c, c#,c++,visual basic c... (sorry če se motim)

Evo koda, sesuje se, ko zelim sesteti, se pravi ko pritisnem stevilko 1.

#include <stdio.h>
#include <stdlib.h>

int main()
{
char zelja,s;
float a,b,vsota,razlika,kvocient,produkt;



while(zelja!=5)
{


printf("Vnesi prvo stevilo a:");
scanf("%f",&a);
printf("Vnesi drugo stevilo b:");
scanf("%f",&b);
system("cls");

printf("Pritisnite 1 za izracun vsote\n
Pritisnite 2 za izracun produkta\n
Pritisnite 3 za izracun kvocienta\n
Pritisnite 4 za izracun razlike\n
Pritisnite 5 za izhod iz programa\n
Pritisnite S za izris smeskota\n");

scanf("%c%c",&zelja);
if(zelja=='1')
  {
     vsota=a+b;
     printf("Vsota teh dveh stevil je = %f\n",vsota);
  }
else if(zelja=='2')
     {
       produkt=a*b;
       printf("Produkt teh dveh stevil je = %f\n",produkt);
     }
else if(zelja=='3')
       {
           kvocient=a/b;
           printf("Kolicnik teh dveh stevil je = %f\n",kvocient);
       }
else if(zelja=='4')
         {
            razlika=a-b;
            printf("Razlika teh dveh stevil je = %f\n",razlika);
         }

else if(zelja=='5')
            {
               system("pause");
               return 0;
            }
else if(zelja=='s' || zelja=='S')
  {
    printf("            * *\n");
    printf("             - \n");
    printf("         |________|\n");
  }

}

}

scanf("%c%c",&zelja);

bereš 2 characterja, shraniš pa samo enega. ni čudno da se sesuje.
daj za scanf še en getch() če se hočeš znebit entra.

Poskusil si:

scanf("%c", &zelja);
getch();

?

Lepo bi bilo tudi zelja inicializirati, while se v prvi ponovitvi obnaša nedefinirano.

Owca, ne z getch(); ne deluje, sm probal pred scanf in za scanf. Ter oboje in ni delovalo. Sem pa probal od Gundolf-a moznost, ter sm uvedel se eno spremenljivko, dummy, ki prebere enter, druga pa potem mojo zeljo prebere. Tako da program sedaj deluje. Amapk kako bi lahko naredil brez dummy spremenljivke. Slisal sem da %*c ne prebere entra iz prejsne vrstice. Ta "*".

Koda je pa sedaj takšna.

#include <stdio.h>
#include <stdlib.h>

int main(void){
char zelja,s,dummy;
float a,b,vsota,razlika,kvocient,produkt;

do
{


printf("Vnesi prvo stevilo a:");
scanf("%f",&a);
printf("Vnesi drugo stevilo b:");
scanf("%f",&b);
system("cls");

printf("Pritisnite 1 za izracun vsote\n
Pritisnite 2 za izracun produkta\n
Pritisnite 3 za izracun kvocienta\n
Pritisnite 4 za izracun razlike\n
Pritisnite 5 za izhod iz programa\n
Pritisnite S za izris smeskota\n");
scanf("%c%c",&dummy,&zelja);
if(zelja=='1')
{
vsota=a+b;
printf("Vsota teh dveh stevil je = %f\n",vsota);
}
else if(zelja=='2')
{
produkt=a*b;
printf("Produkt teh dveh stevil je = %f\n",produkt);
}
else if(zelja=='3')
{
kvocient=a/b;
printf("Kolicnik teh dveh stevil je = %f\n",kvocient);
}
else if(zelja=='4')
{
razlika=a-b;
printf("Razlika teh dveh stevil je = %f\n",razlika);
}

else if(zelja=='5')
{
system("pause");
return 0;
}
else if(zelja=='s' || zelja=='S')
{
printf("            * *\n");
printf("             - \n");
printf("         |________|\n");
}
}
while(zelja!=5);

}

Hvala za pomoč !

Ja, ravnokar sm delal na programu da bi se tega znebil in dela tudi takole: scanf("%*c%c",&zelja);

Lp, Cyborg

Sj ql, program lahko potem poenostavljaš! Vseeno hvala za vaš trud!