Opened 3 years ago

Closed 3 years ago

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

WW #2

Reported by: kutyavin.denis Owned by: Святослав Власов
Component: WW_strings Version: 3.0
Keywords: Cc:

Description


Change History (6)

comment:1 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления

Корректность -- 4/7.
Ошибки:

  1. Порядок параметров и сигнатуры функций не соответствуют стандарту.
  2. Твои же тесты падают, потому что выходят за границы памяти. Подсказка: после вызова malloc выделенная память не обнуляется.
  3. Память после тестов не освобождается

Стиль -- 1.5/3
Замечания:

  1. В str.c не включен заголовочный файл str.h, то же касается test_str.c
  2. Кривые отступы в test_str.c

comment:2 Changed 3 years ago by kutyavin.denis

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 3 years ago by Святослав Власов

Корректность -- 7/7

Стиль -- 2.5/3

  1. Включение malloc.h в str.[h,c] избыточно. Для включения size_t достаточно подключить файл <stddef.h>
  2. В test_str.c весь код внутри функций слеплен в один большой блок, это тяжко читать. Код намного лучше читается, когда код внутри функций разделен на некоторые блоки по смыслу и блоки отделены друг от друга пустыми строками.

comment:4 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления

comment:5 Changed 3 years ago by kutyavin.denis

Type: ожидаются исправленияожидается проверка
Version: 2.03.0

comment:6 Changed 3 years ago by Святослав Власов

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

10/10

Хотя функцию test_strcat я бы еще поделил на блоки, она по прежнему смотрится слишком громоздко

Note: See TracTickets for help on using tickets.