Serohin serhii hw 03 - #40
Conversation
|
👉 View analysis in DeepCode’s Dashboard | Configure the bot |
| @@ -0,0 +1,41 @@ | |||
| #include "cstring" | |||
| #include <bits/stdint-uintn.h> | |||
There was a problem hiding this comment.
а зачем тут этот заголовочный файл? есля я правильно понимаю то он не переносимый
| #include <cstring> | ||
| #include <iostream> | ||
|
|
||
| static bool string_reverse(char *string, size_t len); |
There was a problem hiding this comment.
как по мне и частый на практике это пишут сразу тут определение, так как потом по коду проще понимать как юзается ниже эта функция
| std::cin >> temp; | ||
| std::cout << std::endl; | ||
|
|
||
| if (!array_init_sorted(temp, array, NEW_SIZE)) { |
There was a problem hiding this comment.
а зачем тут делать NEW_SIZE как i + 1, чтобы потом в array_init_sorted сделать size - 1?
| #include <cstdlib> | ||
| #include <iostream> | ||
|
|
||
| bool array_init_sorted(uint32_t new_member, uint32_t *array, size_t size); |
| size_t selected_size{}; | ||
| std::cout << "Input an array size in range [1 ... " << MAX_SIZE << " ]" | ||
| << std::endl; | ||
| param_init_safety("Array size -->", &selected_size); |
There was a problem hiding this comment.
а почему это не сделать как возращаемое значение, зачем его передавать как ссылку
| std::cout << "IN RANGE! "; | ||
| std::cout << MIN << " <= " << array[i] << " && " << MAX | ||
| << " >= " << array[i] << std::endl; | ||
| array_remove_cell(array, size, i); |
There was a problem hiding this comment.
а зачем ее делать как bool и ее не проверять?
| static inline char to_upper(char letter) { | ||
|
|
||
| constexpr char LOW_CASE_FLAG{0x60}; | ||
| constexpr char UP_CASE_FLAG{0x40}; |
There was a problem hiding this comment.
а что такое 0x60 и 0x40? может лучше явно писать символьные литералы?
| return letter & LOW_CASE_FLAG ? (letter & ~LOW_CASE_FLAG) | UP_CASE_FLAG | ||
| : letter; |
There was a problem hiding this comment.
чем этот код лучше если вы напишите чем простой if?
зачем писать код в который нужно вникать чтобы понять что вы просто преобразовываете в нижний регистр
| static bool is_digit(char digit) { | ||
|
|
||
| constexpr char DIGIT_FLAG{0x30}; | ||
| return (digit & DIGIT_FLAG) == DIGIT_FLAG; |
There was a problem hiding this comment.
return digit >= '0' && digit <= '9';
просто и понятно
2 participants
No description provided.