Opened 5 years ago
Closed 4 years ago
#289 closed ожидаются исправления (задача сдана)
WW #4 anisimova.karina
Reported by: | anisimova.karina | Owned by: | anisimova.karina |
---|---|---|---|
Component: | WW_mergesort | Version: | |
Keywords: | Cc: |
Description
Пока что работает только на char, но я исправлю.
Change History (7)
comment:1 Changed 5 years ago by
comment:2 Changed 5 years ago by
Owner: | changed from Дмитрий Лапшин (lapshin) to anisimova.karina |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Корректность:
- Valgrind тебя ругает.
- Иногда оч плохо ему, подумай о пустых.
Стиль:
- Компаратор строчек
const
теряет. - Ой лишний раз строчки копируешь)
- Пробееелы. Длинные строооки.
- Наверняка можно что-то в переменную сложить, чтобы было удобнее.
- Зачем два malloc я не понял.
- И цикл копирования памяти можно в функцию...
Наводи порядок. 5/10.
comment:3 Changed 5 years ago by
Owner: | changed from anisimova.karina to Дмитрий Лапшин (lapshin) |
---|---|
Type: | ожидаются исправления → ожидается проверка |
comment:4 Changed 5 years ago by
Owner: | changed from Дмитрий Лапшин (lapshin) to anisimova.karina |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Корректность:
- Что-то ему не сказать, чтобы лучше.
valgrind
в помощь. - Арифметика указателей с
void *
запрещена.
Стиль:
mergesort
явно просит пустых строк для выделения кусков.- Где-то пробелов не хватает.
Но это уже лучше. 7/10.
comment:5 Changed 5 years ago by
Owner: | changed from anisimova.karina to Дмитрий Лапшин (lapshin) |
---|---|
Type: | ожидаются исправления → ожидается проверка |
comment:6 Changed 5 years ago by
Owner: | changed from Дмитрий Лапшин (lapshin) to anisimova.karina |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Корректность ок.
Пробелам лучше не стало. 9/10.
comment:7 Changed 4 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Теперь должен работать не только при char