WW #4

[Дмитрий Свиридкин]

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

1. В заголовок mergesort.h надо подключить stddef.h для использования size_t.

2. Разбейте main на три функции

3. Не используйте variable-length arrays. Честно выделяйте память.

4. mergesort буз рекурсии это хорошо. Но зачем вы тогда память в цикле выделяете?

5. В функции сортировки следует вынести отдельно процедуру слияния.

6. Используйте C11, чтобы объявлять переменные там, где они нужны, а не заранее сверху.

7. переменные-счетчики в циклах имеют более сложное назначение, чем просто индексация в массиве. Это повод задуматься об их переименовании.

8. Можно использовать memcpy

7.8/10

[Дмитрий Свиридкин]

1.
char **ar_str = malloc(elements * 100 * sizeof(char));

А откуда эта магическая константа?

argv[i + 2] что ж массив не сдвинули, а длину сдвинули?

3.

 size_t it1, it2;
 it1 = 0;
 it2 = 0;

Не разрывайте объявление и инициализацию, если не хотите иметь дело с UB.

4.
Ваш вариант реализации процедуры слияния пока лидирует по неочивидности, покрепленной однобуквенными параметрами.

Может все-таки merge(dst, src1, src1size, src2, src2size, elementsize, comparator) ?

5.
Счетчики так и остаются загадкой

6. Выделение памяти может зафелиться не только в сортировке.

---

8.2 + 0.8