Kalkulator w C

Bobas · **Dołączył(a):** 24 lis 2007, o 16:03 **Posty:** 89

Witam, napisałem właśnie mój pierszy kalkulator

Podspodem źródło. Chętnie wysłucham komentarzy

Kod:

unsigned char liczba1, liczba2, dzialanie, wynik;
                                                 
printf("podaj liczbe 1: ");                      
scanf("%d", &liczba1);                           
printf("podaj liczbe 2: ");                      
scanf("%d", &liczba2);                           
printf("podaj dzialanie (* / + -) :");           
scanf("%c", &dzialanie);                         
                                                 
switch (dzialanie)                               
   {                                            
   case '*':                                    
      wynik = liczba1 * liczba2;               
      break;                                   
   case '/':                                    
      wynik = liczba1 / liczba2;               
      break;                                   
   case '+':                                    
      wynik = liczba1 + liczba2;               
      break;                                   
   case '-':                                    
      wynik = liczba1 - liczba2;               
      break;                                   
   }                                            
printf("wynik %d", wynik);

hak3r · **Dołączył(a):** 2 sty 2009, o 14:38 **Posty:** 153

Troche to dziurawe, nie widzę default w switch co znaczy że jak ktoś poda inne działanie niż oczekiwane to wynik będzie jakąś przypadkową wartością

nenpa8lo

Typ wyniku powinien być na tyle duży by pomieścić wynik mnożenia obu liczb, a zarazem musi być to typ całkowity aby wynik dzielenia był podazany z dokładnością po przecinku np. float.

linuxnewbie · **Dołączył(a):** 4 lut 2009, o 18:56 **Posty:** 64

takze unsigned char to troche malo na liczby

Zrob wszystkie wspomniane poprawki i pokaz kalkulator

Bobas · **Dołączył(a):** 24 lis 2007, o 16:03 **Posty:** 89

Uwzględniłem wasze poprawki i kod teraz wygląda tak:

Kod:

float liczba1, liczba2, wynik;
char dzialanie, blad = 0;
                                                  
printf("podaj liczbe 1: ");
scanf("%f", &liczba1);                            
printf("podaj liczbe 2: ");                       
scanf("%f", &liczba2);                            
printf("podaj dzialanie (* / + -) :");            
scanf("%c", &dzialanie);                          

switch (dzialanie)                                
    {                                             
    case '*':                                     
        wynik = liczba1 * liczba2;                
        break;                                    
    case '/':                                     
        wynik = liczba1 / liczba2;                
        break;                                    
    case '+':                                     
        wynik = liczba1 + liczba2;                
        break;                                    
    case '-':                                     
        wynik = liczba1 - liczba2;                
        break;                                    
    default:
       blad = 1
    }
if (blad == 1)
   printf("Bledne dzialanie!");                        
else
   printf("Wynik: %f ", wynik);

Jeszcze coś źle?

hak3r · **Dołączył(a):** 2 sty 2009, o 14:38 **Posty:** 153

Brak ochrony przed dzieleniem przez zero

nenpa8lo

Duże to jest. Taki program to powinno być pare linii.

prog · **Dołączył(a):** 19 lut 2009, o 02:11 **Posty:** 13

Powinno być w pętli tak żeby program po wykonaniu dalej się wykonywał. Chyba, że użytkownik zadecyduje inaczej (jakiś specjalny przycisk do zamknięcia).

linuxnewbie · **Dołączył(a):** 4 lut 2009, o 18:56 **Posty:** 64

Jak zrobisz to umieść źródło i exe czy też out

zobaczymy jak to śmiga na różnych OSach

Bobas · **Dołączył(a):** 24 lis 2007, o 16:03 **Posty:** 89

Więc w chwili obecnej kod wygląda tak:

Kod:

void powitaj(char *nazwa)
{
   printf("Kalkulator by %s", nazwa);
}

void zegnaj(char *nazwa)
{
   printf("Do zobaczenia %s", nazwa);
}


main()
{
   char exit = 0;dzialanie, nazwa[]="Twoj nick";
   float wynik, liczba1, liczba2;


   while (1)
   {
      powitaj(nazwa);
      printf("Podaj liczbe: ");
      scanf("%f", &liczba1);
      printf("Podaj dzialanie ( * / - + ) :);
      scanf("%c", &dzialanie);
      printf("Podaj liczbe: ");
      scanf("%f", &liczba2);
       
      switch(dzialanie)
      {
         case '-':
            wynik = liczba1 - liczba2;
            break;
            
         case '+':
            wynik = liczba1 + liczba2;
            break;
            
         case '*':
            wynik = liczba1 * liczba2;
            break;
            
         case '/':
            if (liczba2 == 0)
               {
               exit = 1;
               } 
            else
               {
               wynik = liczba1 / liczba2;
               }
            break;
            
         default:
            exit = 1;
            break;
      }
      if (exit == 1)
         {
         printf("Niepoprawne dzialanie");
         }
      else
         {
         printf("Wynki %f", wynik);
         }
      printf("Nacisnij: 1-licz ponownie,  2-zakoncz");
      scanf("%c", &dzialanie);
      if (dzialanie == '2')
         {
         zegnaj(nazwa);
         break;
         }
   }
}

Chyba już wystarczy tych szlifów? Jaki można napisać teraz program, żęby się bardziej wkręcić w tematykę?

Kalkulator w C

Kto przegląda forum