Opened 3 years ago

Closed 3 years ago

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

ww_strings

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

Description


Change History (7)

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

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

За корректность -- 0/7.

  1. Программа не собирается:
$ make
mkdir -p obj
gcc -c -o obj/test.o -I INCLUDE src/test.c
src/test.c:1:10: fatal error: str.h: No such file or directory
    1 | #include "str.h"
      |          ^~~~~~~
compilation terminated.
make: *** [Makefile:9: obj/test.o] Error 1

В линуксе путь регистрозависим.

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

За стиль -- 1.7/3
Замечания:

  1. Нет include guards
  2. include <stdlib.h> в test.c избыточен
  3. Память после тестов не освобождается
  4. Не обязательно юзать malloc, можно использовать память на стеке, например:
char dst[20] = "Hello";
my_strcat(dst, " world!");

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

Version: 1.02.0

Корректность 4/7.
Твой же собственный тест не проходит:

lab_02: src/test_str.c:42: test_strcat: Assertion `my_strcmp(my_strcat(dst2, "lol"), "lol") == 0' failed.

Стиль - 2.8/3

  1. Инклюды от остального кода нужно отделять двумя пустыми строками
  2. Конструкции вида
if (...) {
  return ...
} else {
  return ...
}

лучше заменять на

if (...) {
  return ...
}
return ...

Они эквивалентны, но вторая выглядит лучше

comment:3 Changed 3 years ago by petrov.vladimir

Version: 2.03.0

comment:4 Changed 3 years ago by petrov.vladimir

Постарался поправить старые баги на падающем тесте

comment:5 Changed 3 years ago by petrov.vladimir

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

comment:6 Changed 3 years ago by petrov.vladimir

Загрузил изменения в 20-00, до дедлайна, но забыл изменить статус "ожидается проверка". Извините, постараюсь больше не упускать этот момент

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

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

10/10

Note: See TracTickets for help on using tickets.