Conversation
| int main() { | ||
| long a; | ||
| long b; | ||
| long c; |
There was a problem hiding this comment.
зачем все переменные обьявлять в одном месте? моя рекомендация обьявлять переменные по мере их использования. Это намного удобней с точки зрения читабельности текста
| long c; | ||
| const short four = 4; | ||
| const short two = 2; | ||
| const short one = 1; |
There was a problem hiding this comment.
рекомендую для констант придумать свой код стайл чтобы их отличать от других переменных. Т.е. чтобы было видно по коду что вы юзаете константу а не просто изменяемую переменную
| << " You entered an incorrect number, try entering the number again"; | ||
| return 0; | ||
| } | ||
| auto D = (std::pow(b, two) - (four * a * c)); |
There was a problem hiding this comment.
зачем для этого использовать отдельную функцию? можно просто b * b
| std::cout << " Discriminant = " << D << std::endl; | ||
| if (a != 0) { | ||
| if (D >= 0) { | ||
| auto X1 = ((-b) + (std::sqrt(D))) / (two * a); |
There was a problem hiding this comment.
в редких случаях но предлагаю вам использовать такие вещи как std::sqrt(D) как отдельная константа. Т.к. функция вычесления корня не такая уже и простая функция, и вличет за собой накладные расходы, небольшие, но есть)
поэтому рекоменую такие вещи выносить в отдельные переменные
2 participants
No description provided.