Opened 3 years ago

Closed 3 years ago

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

WW_intrusive_list xargeliya.sergej

Reported by: Сергей Харгелия Owned by: Дмитрий Лапшин (lapshin)
Component: WW_intrusive_list Version:
Keywords: Cc:

Description


Change History (5)

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

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

Если уж хранишь в списке фиктивный элемент (это нормально), зачем его на кучу класть? Тем более каким-то хаком удаляешь в main.

if {
}
else

Мне его трудно принимать)

9/12.

comment:2 Changed 3 years ago by Сергей Харгелия

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

Честно говоря, я совсем не понял, куда относится часть комментария про
if {
}
else
Можете, пожалуйста, сказать про это чуть подробнее, если ошибка осталась?
Всё остальное, вроде бы, исправил

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

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

Нуууублин, init_list мне совсем не нравится. То есть теперь ты просишь фиктивный элемент передать снаружи. Так можно было бы попросить всю лишнюю работу за нас и сделать. И он всё ещё лежит как бы прыжком по указателю.

Вопрос про переносы, популярен стиль

if () {
} else if () {
} else {
}

10/12.

Last edited 3 years ago by Дмитрий Лапшин (lapshin) (previous) (diff)

comment:4 Changed 3 years ago by Сергей Харгелия

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

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

Resolution: → задача сдана
Status: assigned → closed

Ок. Я проглядел что ты malloc не проверил, ай-яй. 12.

Note: See TracTickets for help on using tickets.