Conversation
| #include <cmath> | ||
| #include <cstring> | ||
| #include <iostream> | ||
| using namespace std; |
There was a problem hiding this comment.
это лишнее, понимаю что в качестве примеров для дз это ок, но для будущего старайтесь такие вещи просто избегать. конструкция типа using namespace std; может привести к неприятным сюрпризам
| double a; | ||
| double b; | ||
| double c; | ||
| double x; |
There was a problem hiding this comment.
старайтесь обьявлять переменные по мере их использования. Т.е. не обьязательно все сразу обьявлять вначале функции.
Например
double a;
std::cout << "Enter a: ";
std::cin >> a;
double b;
std::cout << "Enter b: ";
std::cin >> b;
| double b; | ||
| double c; | ||
| double x; | ||
| std::cout << "Enter a: "; |
There was a problem hiding this comment.
зачем вы вначале тогда подключали using namespace std;
если вы пишите std::cout
выражение using namespace std; пишут для того чтобы лишний раз не писать std при использования функционала со стандартной библиотеки
| std::cin >> b; | ||
| std::cout << "Enter c: "; | ||
| std::cin >> c; | ||
| if ((b * b - 4 * a * c) >= 0) // If >=0 |
There was a problem hiding this comment.
у вас выражение (b * b - 4 * a * c) повторяеться в трех местах
сделайте константу и используйте ее в коде
auto kDesc = (b * b - 4 * a * c);
| } | ||
|
|
||
| int value; | ||
| cin >> value; |
There was a problem hiding this comment.
можно еще попробовать std::cin.get()
должно работать
2 participants
No description provided.