Opened 5 years ago

Closed 5 years ago

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

WW #2 krasov.vladislav

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

Description


Change History (5)

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

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

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

  1. Неверное имя исполяемого файла программы.
  2. str.h: сигнатуры функций не соответствуют стандарту Си. Подсказка: в str.c вроде получше. А почему нет ошибки компиляции?
  3. А так работает же.
  4. Нарушение интерфейса теста: main должен вызывать 4 функции, каждая нормально тестирует соответствующую одну функцию.

Стиль:

  1. Полезно в str.c подключать str.h. Сразу бы ошибку нашёл.
  2. Тем более stdio.h не нужен там.
  3. Пробелы в заголовке функции (void foo(int a) {).
  4. Пробелы в управляющих конструкциях (while (2 < 3) {).
  5. strcat: дублирование кода.
  6. strcmp очень сложный. Можно упрощать.
  7. Неконсистентные пробелы в #include.

5/10.

comment:2 Changed 5 years ago by krasov.vladislav

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

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

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

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

  1. Неверное имя исполяемого файла программы.
  2. Заголовок strlen не соответствует стандарту Си.

Стиль:

  1. Мёртвые комментарии в коде ни к чему.

7/10.

comment:4 Changed 5 years ago by krasov.vladislav

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

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

Resolution: задача сдана
Status: assignedclosed

Кстати пробел между именем функции и аргументами.

10.

Note: See TracTickets for help on using tickets.