Opened 5 years ago
Closed 5 years ago
#270 closed ожидаются исправления (задача сдана)
WW #4
| Reported by: | zhemchuzhina.elizaveta | Owned by: | zhemchuzhina.elizaveta |
|---|---|---|---|
| Component: | WW_mergesort | Version: | 2.0 |
| Keywords: | Cc: |
Description
Change History (4)
comment:1 Changed 5 years ago by
| Owner: | changed from Дмитрий Свиридкин to zhemchuzhina.elizaveta |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 5 years ago by
| Owner: | changed from zhemchuzhina.elizaveta to Дмитрий Свиридкин |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
| Version: | 1.0 → 2.0 |
понадобилось серьезное усилие воли, но я приняла решение переписать мэйн: замалочила все и сделала не режущие глаз вызовы функций мержа
comment:3 Changed 5 years ago by
| Owner: | changed from Дмитрий Свиридкин to zhemchuzhina.elizaveta |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
Тесты пройдены
В заголовок надо подключить что-то, объявляющее size_t. У вас сейчас компилируется только потому, что в каждом .c файле также подключается нечто, прямо или косвенно его затягивающее.
И все-таки зачем вам копировать строки?
Вынесите процедуру слияния в отдельную функцию, так жить станет проще.
7.9
comment:4 Changed 5 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Тесты не пройдены: Существуют строки, длиннее 254 символов :) Зачем вы их копируете?
Не используйте variable-length arrays. От них больше проблем чем пользы. Честно выделяйте память в куче.
Лучше разбить main на три функции.
Используйте С11 и не мучайтесь: объявляйте переменную-счетчик прямо в заголовке цикла for.
Не копируйте байтики вручную, есть же memcpy!
6/10