Opened 5 years ago

Closed 5 years ago

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

WW #4

Reported by: luchinin.aleksey Owned by: Дмитрий Свиридкин
Component: WW_mergesort Version: 3.0
Keywords: Cc:

Description


Change History (5)

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

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

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

  1. Зачем в mergesort.h подключен stdio? size_t объявлен в stddef.h
  1. Цикла с memcpy можно заменить на один вызов memcpy
  1. Выделять память на каждом уровне рекурсии -- очень дорогое удовольствие. Попробуйте обойтись одним выделением.
  1. Внутри одного блока кода удалять память стоит через то же имя указателя, для которого она и выделялась.

5.* Не используйте variable-length arrays. От них больше вреда, чем пользы. Выделяйте память честно.
src/main.c:30:8: runtime error: variable length array bound evaluates to non-positive value 0
01-empty-char test failed (output).

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

7/10

comment:2 Changed 5 years ago by luchinin.aleksey

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

Попытался исправить недочёты

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

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

Процедуру слияния надо бы оформить отдельной функцией

9 + 1

comment:4 Changed 5 years ago by luchinin.aleksey

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

Попытался исправить недочёт

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

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

Зачем вы указатели туда-сюда переименовываете? Они передаются в функции по значению и никак не меняются
buf -> start -> buf ?

9.5 + 1

Note: See TracTickets for help on using tickets.