Калькулятор

[ezhidudek]

SOS SOS SOS

Ответил препод.

вот,что он написалперевожу с другого языка,поэтому изложу только суть)
"Уже почти все в порядке.ПРограмма некорректно работает при задании типа "12/a" .Еще использована неудобная функция gets(),вместо нее лучше использовать fgets(),также лучше использовать getchar()
вместо getch(),после этого не нужно INCLUDOVAT(ИНКЛУДИРОВАТЬ,ВТФ ЧТО ЗА СЛОВО Я НЕ ЗНАЮ)нестандартный файл conio.h.
И последнее - обратите внимание на переменную в скобках цикла for - в СИ это не работает"
Вот версия,которую ты кинул в тему..Скажи пожалуйста,это сильно сложно исправить так,как он написал?

#include <math.h>
 #include <stdio.h>
 #include <conio.h>
 #include <string.h>

 int main()
 {
 float a,b;
 char ch;
 char str[50], buf[50];
 int k = 0;

 printf ("Zadejte vyraz:\n");

 gets(str);

 for(int i=0; i<strlen(str); i++)
 if(str[i] != ' ')
 buf[k++] = str[i];

 buf[k] = '\0';

 sscanf (buf, "%f%c%f",&a,&ch,&b);
 switch (ch)
 {
 case '+': printf("=%g",a+b); break;
 case '-': printf("=%g",a-b); break;
 case '*': printf("=%g",a*b); break;
case '/': if(b == 0) printf("Chyba: deleni nulou"); else printf("=%g",a/b);
 break;
 case '%': printf("=%g",fmod(a,b)); break;
 default: printf("Chyba: neplatne zadani");
 }
 getch();

 return 0;
 }

[BDA]

Скажу по поводу 2 последних замечаний:
1) Не нужно писать #include <conio.h>
2) for(int i=0;... - так можно писать только начиная со спецификации C99 (не уверен в терминологии, но если простым языком, то в "обычном" си так нельзя).
Нужно написать int i; for (i=0;... и т.д.

[ezhidudek]

<BDA>,понял,а как быть с остальными замечаниями?
спасибо.

[BDA]

Вместо gets(str); использовать fgets(str, 50, stdin);
Вместо getch(); использовать getchar();
Насчет неправильности 12/a.
Я бы сделал, как советовал <- DelpHIFanat ->, т.е. посимвольный разбор.
Буквально пару недель назад писал "красивый" разбор (с использованием scanf) строки на параметры для учебной задачи - 6 строчек кода потребовали около 2 часов времени (в этот день сделал только эти 6 строчек).

Вы должны обрабатывать количество распознанных параметров (число, которое возвращается sscanf).

[ezhidudek]

BDA,
вот что получилось,но теперь калькулятор не считает..где я ошибся?

#include <math.h>
 #include <stdio.h>
 #include <conio.h>
 #include <string.h>

 int main()
 {
 float a,b;
 char ch;
 char str[50], buf[50];
 int k = 0;

 printf ("Zadejte vyraz:\n");



   
   while(1)
   {
    
     if( scanf("%d %c %d ", &a, &ch, &b) == 3 ) break;
     printf("\"%c\" NE CISLO !!!\n", getchar());
     fflush(stdin);
   }
   getchar(); 
  
 

 fgets(str, 50, stdin);

 for(int i=0; i<strlen(str); i++)
 if(str[i] != ' ')
 buf[k++] = str[i];

 buf[k] = '\0';

 sscanf (buf, "%f%c%f",&a,&ch,&b);
 switch (ch)
 {
 case '+': printf("=%g",a+b); break;
 case '-': printf("=%g",a-b); break;
 case '*': printf("=%g",a*b); break;
case '/': if(b == 0) printf("Chyba: deleni nulou"); else printf("=%g",a/b);
 break;
 case '%': printf("=%g",fmod(a,b)); break;
 default: printf("Chyba: neplatne zadani");
 }
getchar();;

 return 0;
 }

[BDA]

#include <math.h>
#include <stdio.h>
#include <string.h>
#include <locale.h>
#include <ctype.h>

int
main() {
    struct lconv l_struct;
    struct lconv *l_ptr = &l_struct;
    l_ptr = localeconv();
    float a, b;
    char ch;
    char str[50], buf[50];
    int k = 0, i;
    printf ("Zadejte vyraz:\n");
    fgets(str, 50, stdin);
    for(i = 0; i < strlen(str); ++i) {
        if(isdigit(str[i]) ||
           str[i] == l_ptr->decimal_point ||
           str[i] == '+' ||
           str[i] == '-' ||
           str[i] == '*' ||
           str[i] == '/' ||
           str[i] == '%') {
            buf[k++] = str[i];
        } else if (!isspace(str[i])) {
            printf("ERROR\n");
            getchar();
            return 0;
        }
    }
    buf[k] = '\0';
    sscanf (buf, "%f%c%f", &a, &ch, &b);
    switch (ch) {
    case '+':
        printf("=%g",a+b);
        break;
    case '-':
        printf("=%g",a-b);
        break;
    case '*':
        printf("=%g",a*b);
        break;
    case '/':
        if(b == 0) printf("Chyba: deleni nulou");
        else printf("=%g",a/b);
        break;
    case '%':
        printf("=%g",fmod(a,b));
        break;
    default:
        printf("Chyba: neplatne zadani");
    }
    getchar();
    return 0;
}

[ezhidudek]

ошибка..
ISO C++ forbids comparison between pointer and integer

и это же С++..а нельзя как-нибудь исправить то,что я прислал?.

[BDA]

Во-первых, на какой строке выдается предупреждение?
Во-вторых, это Вы используете (имхо) с++ компилятор вместо с компилятора - в самом коде, имхо, нет никаких "фишек" с++.
В-третьих, это и есть Ваш исправленный код, просто во это:

   while(1)
   {
     if( scanf("%d %c %d ", &a, &ch, &b) == 3 ) break;
     printf("\"%c\" NE CISLO !!!\n", getchar());
     fflush(stdin);
   }
   getchar(); 
 fgets(str, 50, stdin);

- бред.

[ezhidudek]

BDA,

ошибка выдается на 20 строчке.
str[i] == l_ptr->decimal_point ||

вот здесь



не буду с вами спорить,просто выглядит замудренно,не поверит,что я такое написал сам.

[BDA]

Я добавил l_ptr->decimal_point, чтобы получать десятичный разделитель от системы, но Вы можете это выбросить:

#include <math.h>
#include <stdio.h>
#include <string.h>
#include <ctype.h>

int
main() {
    float a, b;
    char ch;
    char str[50], buf[50];
    int k = 0, i;
    printf ("Zadejte vyraz:\n");
    fgets(str, 50, stdin);
    for(i = 0; i < strlen(str); ++i) {
        if(isdigit(str[i]) ||
           str[i] == '.' ||
           str[i] == '+' ||
           str[i] == '-' ||
           str[i] == '*' ||
           str[i] == '/' ||
           str[i] == '%') {
            buf[k++] = str[i];
        } else if (!isspace(str[i])) {
            printf("ERROR\n");
            getchar();
            return 0;
        }
    }
    buf[k] = '\0';
    sscanf (buf, "%f%c%f", &a, &ch, &b);
    switch (ch) {
    case '+':
        printf("=%g",a+b);
        break;
    case '-':
        printf("=%g",a-b);
        break;
    case '*':
        printf("=%g",a*b);
        break;
    case '/':
        if(b == 0) printf("Chyba: deleni nulou");
        else printf("=%g",a/b);
        break;
    case '%':
        printf("=%g",fmod(a,b));
        break;
    default:
        printf("Chyba: neplatne zadani");
    }
    getchar();
    return 0;
}

А код проверки, имхо, простой:
Считываем строку и переносим ее в буфер по правилам - если цифра или разрешенный символ, то переносим, иначе, если не пробельный символ, то ошибка.
На самом деле, можно было бы читать посимвольно в буфер.