Opened 5 years ago

Closed 5 years ago

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

WW #2 bubnov.danil

Reported by: Бубнов Данил Константинович Owned by: Дмитрий Лапшин (lapshin)
Component: WW_strings Version:
Keywords: Cc:

Description


Change History (6)

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

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

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

  1. strcpy: возвращаемое значение.
  2. strcat: туда же.
  3. strlen: сигнатура не соотвествует стандарту.

Стиль:

  1. Пробелы вокруг условий управляющих конструкций (if (a < b) {).
  2. strcat — дублирование кода.
  3. strcmp можно и попроще.

5/10.

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

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

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

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

Стиль:

  1. В файл реализации полезно тянуть свой заголовок.
  2. Кстати, так бы нашёл ошибку: заголовок strlen неправильный. #include <stddef.h> хватит, чтобы пришёл size_t.

8/10.

comment:4 Changed 5 years ago by Бубнов Данил Константинович

Добавил заголовок и увидел ошибку: в заголовочном файле неправильно был указан тип strlen. Заменил заголовочный файл для size_t

comment:5 Changed 5 years ago by Бубнов Данил Константинович

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

Добавил заголовок и увидел ошибку: в заголовочном файле неправильно был указан тип strlen. Заменил заголовочный файл для size_t

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

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

Ага, ты не назначит вовремя. В этот раз прощается.

Классный strcmp!

Стиль:
Зачем в str.c ещё раз stddef.h?

Но ладно, хорошо сделано, 10.

Note: See TracTickets for help on using tickets.