Wataru

Есть. Складывайте элементы array в HashMap (храните там количетсво каждого элемента).

Потом пройдитесь по list и, если в HashMap текущий элемент есть с ненулевым счетчиком, выводите другим цветом и уменьшайте счетчик.

Как выводить в цвете - зависит от языка и платформы. Можно хоть символами "*" ***выделять*** при выводе.

Подсчитайте, сколько элементов в каждом множестве. Вспомните определение "инъективного отображения". Посчитайте, сколько возможных значений отображаения для первого элемента, для второго и т.д. Перемножте.

Зачем buffer[i] *= 1;?

Есть ошибка с тем, что у вас код работает только с длиной равной степеням двойки, но нигде этого не проверяется. Обычно, чтобы не возиться с частыми случаями, расширяют буфер до степени двойки.

Если же делать с нечетными n, то эти стандартные формулы не работают.

Еще возможно по коду разбросаны всякие off by one error и перепутанные знаки (почему в w() аргумент у синуса/косинуса берется со знаком минус?)

Еще недочет - вы 2 раза вызываете w(), когда как можно было бы и запомнить результат первого вызова.

И еще, обычно рекурсию разворачивают в циклы. В английской вики приведен псевдокод.

Можно просто удалять через delete после вызова, но это ужасный подход.

На самом деле, вам тут не нужна динамическая память. Сам класс Vector не жрет много памяти. Поэтому mulVec может возвращать просто Vector, а не указатель на него. Cам Matrix4x4 тоже не жрет память, поэтому не надо заводить и на него указатели. Вся память у него внутри в указателе data (кстати, вы же в деструкторе ~Matrix4x4 эту data удаляете же?).

Если же там еще что-то и Vector большой, то возвращайте unique_ptr или shared_ptr на него. Тогда все само удалится, когда надо.

И еще, в вашем текущем коде вы в main выделяете test, а потом перезаписываете. Тут у вас происходит утечка памяти. Его надо было delete-нуть перед перезаписью, это же указатель. Это хороший пример, почему вот так использовать сырые указатели где не надо - это плохо. Легко ошибиться: надо всегда помнить, кто ответственен за удалиение указателя.

Если это, таки, С++, то используйте std::vector. Одна функция его создает и возвращает, другая печатает.
Чтобы не было лишнего копирования передавайте как константную ссылку:

std::vector<int> MakeArray();
void Print(const std::vector<int> array);

Если надо работать с сишными массивами, то тогда придется длину возвращать отдельно через выходной параметр. Передавайте как указатель + длина.

int* MakeArray(int *len);
void Print(int *array, int len);

Это какой-то карго-культ получается. Корткие фнукции понятны - поэтому все функции должны быть короткими. Нет, они, а вернее, весь код должны быть понятными.

Если несколько тривиальных строчек кода понятнее трех однострочных функций, то не надо их переписывать, только потому что введен этот принцип.

Вам нужна тупо функция, которая читает К строк и которая пропускает K строк.

Тогда ваш код становится в Skip(N-1), Read(M-N+1), Read(K-M+1) и читает файл ровно один раз.