Информация об изменениях

Сообщение Re[4]: Тестовое задание Senior .Net - критикуйте от 17.08.2022 14:12

Изменено 17.08.2022 14:29 Shmj

Re[4]: Тестовое задание Senior .Net - критикуйте
Здравствуйте, vaa, Вы писали:

vaa>ОК! Понеслась!


Во-первых, сразу ошибка —

var fileName = new Generator().Generate(2_000);
await new Sorter().Sort(fileName, 500);


Должно получаться 4 файла а получается 3. Какой тут сеньор, скажите? Причина — неумение писать понятный код.

Вот это:



— содержит баг. Сначала читается строка и потом проверяется конец файла — по этому эта последняя строка не будет учтена. Исправить очень просто и наглядно вот так, пишу как для дебилов — а то вы все умные, пока практики не коснется:

  Скрытый текст


Ниже еще один баг:



— это нужно для случаев, если i != 0 как раз.

Сама сортировка вроде тоже с ошибкой — еще не проверял. За это уже ставлю неуд.
Re[4]: Тестовое задание Senior .Net - критикуйте
Здравствуйте, vaa, Вы писали:

vaa>ОК! Понеслась!


Во-первых, сразу ошибка —

var fileName = new Generator().Generate(2_000);
await new Sorter().Sort(fileName, 500);


Должно получаться 4 файла а получается 3. Какой тут сеньор, скажите? Причина — неумение писать понятный код.

Вот это:



— содержит баг. Сначала читается строка и потом проверяется конец файла — по этому эта последняя строка не будет учтена. Исправить очень просто и наглядно вот так, пишу как для дебилов — а то вы все умные, пока практики не коснется:

  Скрытый текст


— хотя в моей записи тоже не совсем точно — предусмотреть случай, когда файл пустой и первой строки нет — тоже желательно. Но это мелочи.

Ниже еще один баг:



— это нужно для случаев, если i != 0 как раз.

Сама сортировка вроде тоже с ошибкой — еще не проверял. За это уже ставлю неуд.