Conversation
| #include <cstdio> | ||
| #include <iostream> | ||
| int main(int argc, char const *argv[]) { | ||
| std::cout << "Вас приветствует програма Quadratic_equation." << std::endl; |
There was a problem hiding this comment.
Я не очень сильно рекомендую использовать руский текст в консольных приложениях. Ну это на ваше усмотрение)
| // Если А=0, то у нас нет квадратного уравнения | ||
| while ((A < kShortMin) && (A > kShortMax) && (A == 0)) { | ||
| std::cout << "Параметр А задан не верно\n"; | ||
| return 1; |
There was a problem hiding this comment.
ну тут немного не верно, если вы делаете while это значит что вы даете пользователю возможность повторно ввести данные, а вы делаете return 1, т.е. в таком случае лучше написать if а не цикл. Или используйте цикл как у вас это есть, но тогда уберите return а добавте cin
| << kShortMin << " до " << kShortMax << std::endl; | ||
| std::cin >> A; | ||
| // Если А=0, то у нас нет квадратного уравнения | ||
| while ((A < kShortMin) && (A > kShortMax) && (A == 0)) { |
There was a problem hiding this comment.
у вас эта проверка никогда не выполниться, т.к. тип переменной A и kShortMin одного типа.
Вам нужно сделать тип переменной A больше чем тип переменной kShortMin. Например int A
| std::cout << "Параметр С задан не верно\n"; | ||
| return 1; | ||
| } | ||
| auto D = B ^ 2 - 4 * A * C; |
There was a problem hiding this comment.
оператор ^ в языке С++ это не оператор возведение в степень, это побитовый xor. Вам нужно просто написать B * B
| auto D = B ^ 2 - 4 * A * C; | ||
| if (D < 0) { | ||
| std::cout | ||
| << "Заданое квадратное уравнение не имеет действительных решеий\n"; |
There was a problem hiding this comment.
и что дальше пошли выполнять код? в этом случае вы получили решение и должны сделать return 0 дальше выполнять код не имеет смысла
2 participants
No description provided.