Opened 3 years ago

Closed 3 years ago

#43 closed ожидается проверка (задача сдана)

Prudnikov WW_Strings lab_02

Reported by: Марк Прудников Owned by: Дмитрий Лапшин (lapshin)
Component: WW_strings Version:
Keywords: Cc:

Description

Добрый вечер. Тестов нет, знаю, поздно опомнился, сдаю так, чтобы хоть какой-то фидбек получить.

Change History (5)

comment:1 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Марк Прудников
Type: ожидается проверка → ожидаются исправления
Version: 1.0

Makefile неправильный. Он ещё и программу с тестами запускает по умолчанию, нинада пжлст. И не прячь вывод команд без просьбы, неприятно.

my_strcat дублирует код, в my_strcmp слишком сложный финал.

Зачем в str.c подключается stdio.h и test_str.h?

Include guards?

6/10.

comment:2 Changed 3 years ago by Марк Прудников

Owner: changed from Марк Прудников to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка
Version: → 2.0

Исправления после вашего ревью.

comment:3 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Марк Прудников
Type: ожидается проверка → ожидаются исправления
Version: 2.0

Пробельчики в while (/if ( и в ) {. Отступы в тестах очень плохие (табы с пробелами, бббр) и между файлами разные, как так?

Зачем str.h подключает stdio.h?

8/10.

comment:4 Changed 3 years ago by Марк Прудников

Owner: changed from Марк Прудников to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка

Поменял подключаемый заголовок после вашего ответа на почту и исправил отступы.

comment:5 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Resolution: → задача сдана
Status: assigned → closed

Пробелы всё ещё в разнобой, не вижу единого стиля. 8.

Note: See TracTickets for help on using tickets.