Opened 5 years ago

Closed 5 years ago

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

WW #2 sunko.elena

Reported by: sunko.elena Owned by: Дмитрий Лапшин (lapshin)
Component: WW_strings Version:
Keywords: Cc:

Description


Change History (6)

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

Component: WW_make → WW_strings

Это вторая лаба, как я понимаю?

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

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

Корректность: твои тесты падают.

comment:3 Changed 5 years ago by sunko.elena

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

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

Owner: changed from Дмитрий Лапшин (lapshin) to sunko.elena
Summary: WW #2 → WW #2 sunko.elena
Type: ожидается проверка → ожидаются исправления

Корректность:

  1. strcpy возвращает неверное значение.
  2. strcat дублирует код.
  3. strcpy очень страшно некорректен, пихая всю программу в неопределённое поведение.
  4. Заголовок strlen не соответствует стандарту.

Стиль:

  1. Зачем ты один аргумент, который в оригинальном значении не сдался, копируешь в переменную?
  2. Пустые строчки между функциями.
  3. Напрашивается упрощение в strcmp.
  4. test.c: отступы!
  5. В файлах реализации крайне полезно подключать свои заголовки.

4/10.

comment:5 Changed 5 years ago by sunko.elena

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

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

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

Стиль:

  1. strcmp: воспользоваться тем фактом, что '\0' == 0 меньше, чем всё остальное.

А так норм. 9/10.

Note: See TracTickets for help on using tickets.