0

カスタム クラスである Elevator の std::vector を並べ替えた後、問題が発生しています。

クラス定義:

class Elevator
{
private:

        uint16_t m_id;
        char *m_channel;
        ElevatorType m_type;

public:

        Elevator(uint16_t id, const char *channel, ElevatorType type);
        Elevator(const Elevator &other);
        ~Elevator();

        char *ToCString();

        bool operator<(const Elevator& rhs) const;

        static vector<Elevator> *ElevatorsFromWorkbook(const char *path);

};

less 演算子は次のように実装されます。

bool Elevator::operator<(const Elevator& rhs) const
{
    return (this->m_id < rhs.m_id);
}

ToCStringメソッドは次のように実装されます。

char *Elevator::ToCString()
{
    char *output = (char*)malloc(sizeof(char) * (8+strlen(m_channel)));
    char type = 'U';
    if (m_type == ELEVATOR_TYPE_A)
    {
        type = 'A';
    }
    else if (m_type == ELEVATOR_TYPE_G)
    {
        type = 'G';
    } 
    else if (m_type == ELEVATOR_TYPE_M)
    {
        type = 'M';
    }
    sprintf(output,"%d %s %c",m_id,m_channel,type);
    return output;
}

コピー コンストラクター:

Elevator::Elevator(const Elevator &other)
{
    m_id = other.m_id;
    m_channel = (char*)malloc(sizeof(char)*(strlen(other.m_channel)+1));
    strcpy(m_channel,other.m_channel);
    m_type = other.m_type;

}

問題の例:

std::vector<Elevator> elevators;
elevators.push_back(Elevator(4569,"CHANNEL3",ELEVATOR_TYPE_G));
elevators.push_back(Elevator(4567,"CHANNEL3",ELEVATOR_TYPE_G));

printf("%s\n",elevators.at(0).ToCString()); //Prints "4567 CHANNEL1 G"
std::sort(elevators.begin(),elevators.end());
printf("%s\n",elevators.at(0).ToCString()); //ISSUE: Prints "4567 4567 ▒#a G"

私は何を間違っていますか?御時間ありがとうございます!

コンパイルに関する注意: Cygwin を次のフラグで使用しています: -Wall -Wextra -fno-exceptions -fno-rtti -march=pentium4 -O2 -fomit-frame-pointer -pipe

4

1 に答える 1

1

標準によるとstd::sort、オブジェクトはスワップ可能である必要があります。つまり、オブジェクトを呼び出すことができますswapswapオーバーロードがないので、それはstd::swap. そしてstd::swap言う

必須: タイプ T は MoveConstructible (表 20) および MoveAssignable (表 22) でなければなりません。

(これは C++11 です。C++03 では、「移動」を「コピー」に置き換えます。)

オーバーロードがないため、クラスは CopyAssignable ではないためoperator=、それを追加する必要があります。( What is The Rule of Three?を参照してください。 ) コピーの構築は、デフォルトの構築と から合成できることに注意してくださいoperator=

于 2013-01-31T02:27:47.947 に答える