std :: list.sort (Predicate) Ошибка компилятора

Question

Теперь я следую вашим советам. У меня есть еще одна проблема.

ifndef MARKERCONTAINE_H
define MARKERCONTAINE_H
include <list>
include <string>
include "Marker.h"
namespace Ableton {

    struct DeferencedEqual{

        DeferencedEqual(const Marker* compare_to) : compare_to_(compare_to) { }

        bool operator()(Marker* c1) const {return  *c1 == *compare_to_;}  

        private:
            const Marker* compare_to_;
    };

    bool compare(Marker* const p1, Marker* const p2){return p1<p2;}

class MarkerContainer {

private:
    std::list<Marker*> list;
    double end_t;
    inline bool exists(std::list<Marker*> *input_list, Marker* check);
   // inline int computeInterval(double to_convert, bool beat_t=true);

public:
    MarkerContainer();
    MarkerContainer(double end_time);
    bool addMarker(Marker* input);
    double computeBeatTime(double sample_t);
    double computeSampleTime(double beat_t);    
    void printAll();
    void sort();

};



}

Функция сортировки теперь:

 namespace Ableton {

    void MarkerContainer::sort(){

       list.sort(compare);
    }

    }

И мой main.cpp:

include "Marker.h"
include "MarkerContainer.h"
include <iostream>
include <list>
include <string>
include <cctype>

using namespace std;

int main (int argc, const char * argv[])
{

    Ableton::MarkerContainer::MarkerContainer container =  Ableton::MarkerContainer::MarkerContainer(10.0);
    Ableton::Marker::Marker* one = new Ableton::Marker::Marker(1.0, 1.0);
    Ableton::Marker::Marker* two = new Ableton::Marker::Marker(2.0, 1.0);
    Ableton::Marker::Marker* three = new Ableton::Marker::Marker(3.0, 1.0);

    if(!container.addMarker(one))
        cout << *one << "NOT Added" << endl;
    if(!container.addMarker(two))
        cout << *two << "NOT Added" << endl;
    if(!container.addMarker(three))
        cout << *three << "NOT Added" << endl;
    cout << "-- LIST " << endl;

    container.printAll();

    cout << "-- LIST after sort" << endl;
    //container.printAll();

}

Теперь я улавливаю эту ошибку ld, она звучит странно для меня ... нет пути для меня в C ++?: -)

ld: дубликат символа Ableton :: Сравнить (Ableton :: Marker *, Ableton :: Marker *) в /..../MarkerContainer.o и /..../main.o для архитектуры x86_64 clang: ошибка: сбой команды компоновщика с кодом выхода 1 (используйте -v для просмотра вызова) Команда / Разработчик / usr / bin / clang ++ завершилась с кодом выхода 1

Еще раз спасибозаранее, можете ли вы предложить мне какую-нибудь книгу о STL и о том, как она работает «под землей», чтобы лучше понять, что произошло в этих случаях с технической точки зрения

pedr0

Answer 1

Проблема в том, что вы включили определение для:

Ableton::compare(Ableton::Marker*, Ableton::Marker*)    

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

Решение:
Поместите определение в файл cpp или пометьте его как inline, если хотите, чтобы оно было в файле заголовка.

Answer 2

Я думаю, вы должны использовать ссылки в компараторе. Попробуйте этот вариант

bool compare(const Marker*&  p1, const Marker*&  p2){return p1<p2;}