search

maxkholin / s21_codereview

0 stars 0 forks source link

Task 3 #2

Open bgenia opened 3 months ago

bgenia commented 3 months ago

Хорошая декомпозиция, код понятный и простой. Вот некоторые замечания:

https://github.com/maxkholin/s21_codereview/blob/789aadb51e5467ead164a342de5c79525eefbd71/task_3/main.c#L11

Опечатка в слове Height.

https://github.com/maxkholin/s21_codereview/blob/789aadb51e5467ead164a342de5c79525eefbd71/task_3/main.c#L48-L49

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

https://github.com/maxkholin/s21_codereview/blob/789aadb51e5467ead164a342de5c79525eefbd71/task_3/main.c#L52-L54

Вместо 1 и 0 лучше использовать true и false из stdbool.h.

https://github.com/maxkholin/s21_codereview/blob/789aadb51e5467ead164a342de5c79525eefbd71/task_3/main.c#L57

По названию функции не понятно что она проверят и зачем ей для этого чей-то рост. Лучше дать более говорящее название вроде isHumanHigherThan.

https://github.com/maxkholin/s21_codereview/blob/789aadb51e5467ead164a342de5c79525eefbd71/task_3/main.c#L35-L40

Две одинаковых проверки роста подряд и два вывода, можно делать по 1 разу:

if (!checkHumanHeight(...)) continue;

if (count > 0) { print(", "); } else { count++; }

print(human);

https://github.com/maxkholin/s21_codereview/blob/789aadb51e5467ead164a342de5c79525eefbd71/task_3/main.c#L14 https://github.com/maxkholin/s21_codereview/blob/789aadb51e5467ead164a342de5c79525eefbd71/task_3/main.c#L27

Оба этих числа должны быть неотрицательными, можно было задать как unsigned int или проверять.

maxkholin commented 3 months ago

Большое спасибо за столь развернутое ревью, о многих вещах я даже не догадался