Opened 5 years ago

Closed 4 years ago

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

WW #4 sunko.elena

Reported by: sunko.elena Owned by: sunko.elena
Component: WW_mergesort Version:
Keywords: Cc:

Description


Change History (2)

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

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

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

  1. В main, когда сортируются строчки, у тебя заводится буфер на стеке для указателей. Мне кажется, можно и без него.
  2. Не проверяется malloc.

Стиль:

  1. Пустые строчки в конце зачем-то.
  2. У тебя ж есть strcmp.
  3. *argv[i] есть более читаемые варианты.
  4. Имена переменных не оч.
  5. Пробелы вокруг инфиксных операторов.
  6. Пробелы в управляющих конструкциях (for (int i = 0; i < n; ++i) {).
  7. unsigned int vs size_t.
  8. Отступы.
  9. copy_bytes в цикле... А может без цикла.

Но в целом ок. 7/12.

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

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