Здравей,

Прегледах ти кода, но дори да работи за някои примери, това все още не е правилното решение. Дефинираш много излишни неща, които изобщо не използваш.

Компилационната грешка идва, защото вътре в един клас, който има template T, дефинираш друг template T и го "скрива" (отвори и компилирай .cpp файла с Code::Blocks и ще видиш грешката, не знам защо Visual Studio я игнорира - като цяло ако не ти се компилира нещо в judge системата, пробвай да компилираш с Code::Blocks и най-вероятно ще видиш грешката). Изобщо този Comparator клас който дефинираш е излишен. Най-вероятно си го направил, защото ти е дало грешка за Comparator<T> less, който дефинираш по-долу (но не ползваш никъде). Това е на верен път - този less трябва да го ползваш в operator() за да вземеш резултата му за двата параметъра и да го обърнеш наобратно. Но го декларираш грешно. Comparator е template, което означава, че буквално се заменя с кода, който е подаден от отвън. От отвън се подава LessThan<Song> примерно, което означава, че текста Comparator се заменя с текста LessThan<Song>. Ти като напишеш Comparator<T> less; се получава LessThan<Song><Song> less; тоест повтаряш template-а.

Просто го остави Comparator less; - това ще се замени с LessThan<Song> less; ако бъде повикано за Song и LessThan, както в третия случай в main файла.

И не забравяй на този less да му викаш operator() в overload-а за operator() на Reverse - НЕ трябва да правиш обратното на знак < (това правиш в момента в Reverse) - това е напълно грешно, защото предполагаш, че двата обекта се сравняват с оператор <, което не е вярно - сравняват се с Comparator, което може да бъде LessThan, което ползва operator<, ама може да е и какъвто и да е друг клас който има operator(), който ти не знаеш какво точно прави. Не трябва да предполагаш какво прави Comparator, трябва просто да му извикаш operator() и той ще ти каже каква е подредбата на двата параметъра.

И също така в LessThan ползваш operator>, вместо да ползваш operator<, което означава, че редът ще ти излезе наобратно. Сигурно пак си се опитвал да компенсираш за нещо друго, но правилното поведение на LessThan е да ползва operator<.

Направи тези корекции, махни всичките излишни декларации и typedef-ове (не ги ползваш за нищо) и ще ти се компилира и работи правилно.

На упражненията днес ще решаваме подобни задачи, така че ще имаш време да си прегледаш нещата до утре вечерта ако този отговор не те ориентира напълно.

Поздрави,

Жоро