Opened 5 years ago

Closed 5 years ago

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

WW #4

Reported by: smirnov.ilya Owned by: Дмитрий Свиридкин
Component: WW_mergesort Version: 2.0
Keywords: Cc:

Description


Change History (8)

comment:1 Changed 5 years ago by smirnov.ilya

не могу понять,как сделать свою функцию перекопирования для строк(не понимаю,как контролировать размер строк),поэтому memcpy

Last edited 5 years ago by smirnov.ilya (previous) (diff)

comment:2 Changed 5 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to smirnov.ilya
Type: ожидается проверка → ожидаются исправления

Все просто: не копируйте строки :)

И не использйте variable-length arrays

И еще кое-что:

lab_04/bin/lab_04 int 
src/main.c:35:13: runtime error: variable length array bound evaluates to non-positive value 0
AddressSanitizer:DEADLYSIGNAL
=================================================================
==14056==ERROR: AddressSanitizer: stack-overflow on address 0x7ffcaea64ff8 (pc 0x5631806ffd56 bp 0x7ffcaea65020 sp 0x7ffcaea64ff0 T0)
    #0 0x5631806ffd55 in mergesort (lab_04/bin/lab_04+0x5d55)
    #1 0x5631806ffd86 in mergesort (lab_04/bin/lab_04+0x5d86)
    #2 0x5631806ffd86 in mergesort (lab_04/bin/lab_04+0x5d86)
    #3 0x5631806ffd86 in mergesort (lab_04/bin/lab_04+0x5d86)
    ......

comment:3 Changed 5 years ago by smirnov.ilya

Owner: changed from smirnov.ilya to Дмитрий Свиридкин
Version: 1.0 → 2.0

comment:4 Changed 5 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to smirnov.ilya

Тесты пройдены

Подключение заголовков в заголовочных файлах тоже стоит помещать под include_guard

Лучше разбить main на три функции

функция mergesort должна сама корректно работать с пустым массивом, без необходимости внешней проверки.

7.5

comment:5 Changed 5 years ago by smirnov.ilya

Owner: changed from smirnov.ilya to Дмитрий Свиридкин
Type: ожидаются исправления → ожидается проверка

comment:6 Changed 5 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to smirnov.ilya
Type: ожидается проверка → ожидаются исправления

Форматирование поехало

Почему функции называются merge_TYPENAME, когда они не мерджат, а сортируют?

Подключать в заголовке mergesort string.h -- это как-то круто для всего лишь size_t. Поищите минимально достаточный заголовок, содержащий size_t.

8.8

comment:7 Changed 5 years ago by smirnov.ilya

Owner: changed from smirnov.ilya to Дмитрий Свиридкин
Type: ожидаются исправления → ожидается проверка

comment:8 Changed 5 years ago by Дмитрий Свиридкин

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

argc - 2 От вот этой штуки в 12 мастах можно избавиться

9

Note: See TracTickets for help on using tickets.