Opened 5 years ago

Closed 4 years ago

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

WW #4 shein.andrey

Reported by: Андрей Шеин Owned by: Андрей Шеин
Component: WW_mergesort Version:
Keywords: Cc:

Description


Change History (5)

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

Owner: changed from Дмитрий Лапшин (lapshin) to Андрей Шеин
Summary: WW #4 → WW #4 shein.andrey
Type: ожидается проверка → ожидаются исправления

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

  1. Сыыыыпется. Читаем условие, ищем крайние случаи.
  2. Твои буферы на стеке самонадеяны это мягко сказано. Сортируй строки, чтобы не ограничивать их длину.

Стиль:

  1. Что за coping? Почему он в заголовке?
  2. В конце merge а нельзя ли попроще? Без цикла?
  3. Как насчёт промежуточных значений в переменных? А то читать сортировку сложно.

5/10.

comment:2 Changed 5 years ago by Андрей Шеин

Owner: changed from Андрей Шеин to Дмитрий Лапшин (lapshin)

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

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

Корректность: ему хорошо.

Стиль:

  1. Есть длинные выражения с дублирующимися кусками. Переменные могут помочь.

9/10.

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

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

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

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