Opened 3 years ago

Closed 3 years ago

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

WW_intusive_list lab_04 rachkin.vladimir

Reported by: Владимир Рачкин (rachkin.vladimir) Owned by: Владимир Рачкин (rachkin.vladimir)
Component: WW_intrusive_list Version:
Keywords: Cc:

Description

Пока не реализованны тесты

Change History (4)

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

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

Какие тесты?

Makefile явно неправильный. Предупреждения компилятора плохие!

Ну собсна и падает. Неинициализированные значения, ещё много чего, Valgrind с отладчиком нам в помощь.

Что у тебя за чтение, которое наполовину это самостоятельно написанные стандартные функции, когда есть scanf/snscanf? Первого бы хватило за уши (" %s" для команды, остальное по необходимости).

Тем более ты делаешь fgets больше чем есть в буфере аааа.

Стиль всё ещё страдает, хочу пробелы приличные:

if (same_prefix(input, "add") == 1) {
    add_point(&l, get_nth_int(input, 2), get_nth_int(input, 3));
} else if (same_prefix(input, "rm") == 1) {

Если уж завёл списке фиктивный элемент (это нормально), зачем его на кучу класть?

3/12.

comment:2 Changed 3 years ago by Владимир Рачкин (rachkin.vladimir)

Owner: changed from Владимир Рачкин (rachkin.vladimir) to Дмитрий Лапшин (lapshin)
Type: ожидаются исправленияожидается проверка
Version: 2.0

Я не додумался, как реализовать автоматические тесты(

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

Owner: changed from Дмитрий Лапшин (lapshin) to Владимир Рачкин (rachkin.vladimir)
Type: ожидается проверкаожидаются исправления
Version: 2.0

Ещё раз, какие тесты? Я не просил никаких тестов. Ну и нет версий у лабок, нет.

Makefile всё ещё неправильный, он каждый раз линкует программу, а на изменения исходников-незаголовков не реагирует.

Сильно нарушен формат вывода. Читаешь числа зачем-то руками, и вообще у тебя опасные размеры строк.

И потом что-то всё ещё не работает: списки у тебя почему-то задом наперёд.

У тебя сложные циклы бегания по списку двумя указателями, одним было бы проще.

А зачем в структуре списка тебе указатель на элемент, который точно лежит рядом и никогда не меняется?

Кажется, тут ещё что-то чинить.

7/12.

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

Resolution: задача сдана
Status: assignedclosed
Note: See TracTickets for help on using tickets.