Opened 5 years ago

Closed 5 years ago

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

WW #4 obryadina.aleksandra

Reported by: Obryadina Alexandra Owned by: Дмитрий Лапшин (lapshin)
Component: WW_mergesort Version:
Keywords: Cc:

Description


Change History (6)

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

Owner: changed from Дмитрий Лапшин (lapshin) to Obryadina Alexandra
Summary: WW #4 → WW #4 obryadina.aleksandra

Корректность:

  1. Числа и строчки тебе не дались. Оч смешная бага почему.
  2. Надеяться что 200 байт на строчку хватит довольно невинно...

Стиль:

  1. А зачем их вообще копировать?
  2. Переменные непонятны.
  3. А может промежуточные результаты куда-то складывать, чтобы не копировать этих монстров сложения?
  4. Цикл копирования просится в функцию.
  5. Переменную можно объявлять ближе к использованию.

5/10.

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

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

comment:3 Changed 5 years ago by Obryadina Alexandra

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

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

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

Корректность:

  1. Работа со строчками жжжжжжесть. Тебе точно нужны отдельные указатели на куче? В которые ты копируешь... Куда? Сколько? А сколько там места?! Ааааа undefined behaviour повреждение кучи даже valgrind сдался segmentation fault падение приложения.

Стиль:

  1. Цикл, делающий copy, кажется может быть заменён на один copy.
  2. Всё ещё копируешь пытаешься копировать строчки.
  3. Имена переменным можно лучше, чем array3.
  4. И может конструкцию вида src + index_XX * element_size можно как-то пореже повторять?

7/10.

comment:5 Changed 5 years ago by Obryadina Alexandra

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

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

Resolution: → задача сдана
Status: assigned → closed
  1. Пробельчик один потерялся.
  2. Выражения можно было таки и вынести...

9/10.

Note: See TracTickets for help on using tickets.