Opened 5 years ago

Closed 5 years ago

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

WW #4 fomenko.ivan

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

Description

Я запутался в самой сортировке, как нам правильно "переходить на следующий шаг", там где иф на i++ и j++, и при этом иметь только одну функцию мерджсорт, поэтому вернулся к собираемому этапу

Change History (5)

comment:1 Changed 5 years ago by fomenko.ivan

Owner: changed from < ldvsoft> to ldvsoft

Я запутался в самой сортировке, как нам правильно "переходить на следующий шаг", там где иф на i++ и j++, и при этом иметь только одну функцию мерджсорт, поэтому вернулся к собираемому этапу.
А еще прошу прощения, потому что в 20 30, я создал билетик с вашим логином в скобках, надеюсь, еще возможно отослать

comment:2 Changed 5 years ago by fomenko.ivan

Owner: changed from ldvsoft to Дмитрий Лапшин (lapshin)

Кажется, я снова указал не тот ваш ник, но меня вновь поправили

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

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

Да, чот оно лажааааает. Но память аккуратно.

Стиль:

  1. atoi/sscanf ради всех святых.
  2. strcmp ради всех проклятых.
  3. Имена переменных можно делать понятнее, тебе же поможет!

Ну так да, писать надо дальше.

comment:4 Changed 5 years ago by fomenko.ivan

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

Я только сейчас вспомнил про стиль, который ради всех святых и проклятых, боюсь его править и что-то сломать

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

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

Стиль:

  1. atoi/sscanf ради всех святых. Выкидываешь свой get, пишешь atoi.
  2. strcmp ради всех проклятых.
  3. Зачем тебе next_step в main? array[i] = argv[i + 2][0]?
  4. Пробелы местами теряешь.
  5. Дублирующиеся выражения просятся в переменные.

Работает, но писать можно аккуратнее. 7/10.

Note: See TracTickets for help on using tickets.