Opened 3 years ago

Closed 3 years ago

#19 closed ожидаются исправления (задача сдана)

Лаба 2

Reported by: antonenko.mixail Owned by: Святослав Власов
Component: WW_strings Version: 1.0
Keywords: Cc:

Description


Change History (5)

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

Component: HW #2 (X0)WW_strings
Type: ожидается проверкаожидаются исправления

Корректность -- нет замечаний, 7 баллов.
Стиль -- 1.8

Замечания по стилю:

  1. Makefile написан неправильно. Проект не должен пересобираться целиком при изменении одного файла. Неправильно указаны зависимости. После сборки автоматически ничего запускаться не должно.
  1. Имена аргументов нужно сделать более осмысленными. s1 и s2 в сигнатуре функции my_strcpy совершенно неинформативны, непонятно что откуда и куда копируется. В идеале при взгляде на сигнатуру my_strcpy или my_strcat должно быть понятно как её вызывать -- какой аргумент строка-источник, а какой буфер-приемник. Хорошие кандидаты на такие имена -- это src и dst (source и destination).
  1. Твой кодстайл записи аргументов функций выглядит откровенно странно. Он выглядел бы нормально, будь у тебя 10 длинных аргументов, но когда длина аргументов меньше 80 символов, их обычно не пишут на отдельных строках.

Я за это баллы не снижаю, но рекомендую исправить.

  1. Когда тело оператора if/for/while содержит всего 1 строчку её можно как обрамлять в фигурные скобки, так и не делать этого. Но нельзя перемешивать эти варианты. Выберите один из них и придерживайтесь его в своем коде.
  1. Инклюдник <stdio.h> избыточен. Для включения size_t достаточно <stddef.h>

comment:2 Changed 3 years ago by antonenko.mixail

  1. Подписи взяты из стандарта по формулировке задачи.

В итоге баллы снижены только за пункты 4 и 5?

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

Баллы снижены за 1 и 4

За 2 не снижаю, но рекомендую исправить.

comment:4 Changed 3 years ago by antonenko.mixail

Спасибо, Makefile обновлён.

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

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

Когда ты исправил замечания, нужно переводить тикет в состояние "Ожидается проверка" и обновлять версию, иначе я могу не проверить.

Makefile исправлен не до конца -- заголовочных файлов нет в зависимостях.
Остальное тоже не исправлено. Оценка за стиль остается прежней.

Note: See TracTickets for help on using tickets.