Opened 3 years ago
Closed 3 years ago
#103 closed ожидаются исправления (задача сдана)
WW_mergesort yusufov.arsenij
| Reported by: | Arseny Yusufov | Owned by: | Дмитрий Лапшин (lapshin) |
|---|---|---|---|
| Component: | WW_mergesort | Version: | 2.0 |
| Keywords: | Cc: |
Description
Change History (8)
comment:1 Changed 3 years ago by
| Summary: | lab_03 yusufov.arsenij → WW_mergesort yusufov.arsenij |
|---|
comment:2 Changed 3 years ago by
- Стиль, скорее всего, соблюдается не везде. Это я знаю.
- Интерфейс работы "как консольная утилита" еще не реализован. Можно просто ./lab_03, продемонстрирует работу реализации, но ввода пока нет.
- Документации и разумных комментариев пока нигде нет.
comment:3 Changed 3 years ago by
| Component: | HW #1 (BMP) → WW_mergesort |
|---|
comment:5 Changed 3 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to Arseny Yusufov |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
| Version: | 1.0 |
Слегка нарушаешь формат вывода. Valgrind пищит везде.
Если что требование про my_ было только в прошлой лабе, но ладно.
char memory_error[] = "";
const char memory_error[] тогда уж, это важно.
Не очень понимаю, почему компаратор у тебя по смыслу противоположен strcmp.
Если память выделить не получилось, ты пишешь сообщение о ошибке, но исполнение не прерываешь, интересный план.
Объявление вложенной функции вижу я. И не вижу определения. Как оно работает плохо представляю, если надо выноси наружу.
Вижу цикл копирования памяти по элементу. И кажется цикл объединения буферов можно как-то меньше дублировать.
5/10.
comment:6 Changed 3 years ago by
- Вы не могли бы пояснить, в чем нарушение формата вывода?
- my_* – не из-за требований, а из-за конфликта с stdlib, в слаке спраишвал, разрешили так.
- Кажется, нам не рассказывали про завершение работы, я учту это. Как я понял, нужно просто exit.
comment:7 Changed 3 years ago by
| Owner: | changed from Arseny Yusufov to Дмитрий Лапшин (lapshin) |
|---|---|
| Version: | → 2.0 |
comment:8 Changed 3 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
Формат всё ещё нарушен на пустом выводе.
А, это у тебя была проблема с my_, ок.
Нет, exit всегда можно вызвать, но в main вообще-то есть return.
В main формально есть утечка памяти, если my_mergesort вернёт ошибку.
В цикле объединения двух кусков дублируется код, кажется просто правильное условие решило бы проблему (есть есть левая доля и (нет правой или она хуже), то вставляем левую).
Но так ок! Бонус в зачёт, получаем 10.
