0

Я перегружаю оператор :

void operator +=(pair<string, string> eng_rus) {
        root = insert(eng_rus.first, eng_rus.second, root);
    }

Но при этом мне так же надо перегрузить оператор ещё таким образом :

void operator +=(pair<char, string> eng_rus) {
    string s;
    s += eng_rus.first;
    root = insert(s, eng_rus.second, root);
}

И тд. Каким образом я могу избежать дублирования кода, возможно ли это в данной ситуации?

8
  • странный код честно говоря, зачем второй оператор? он же делает тоже самое, только с символом (что можно вполне интерпретировать как строку). Но даже если и есть на это какая-то гипотетическая причина, то можно в одну отдельную функцию просто перенести.
    – ampawd
    Commented 5 сент. 2021 в 20:33
  • += на неинициализированной переменной мне кажется проблемой;)
    – eri
    Commented 6 сент. 2021 в 0:12
  • это об s+=... ?
    – KoVadim
    Commented 6 сент. 2021 в 6:14
  • Если конструктор вызван, значит инициализирована.
    – maestro
    Commented 6 сент. 2021 в 7:16
  • переиспользовать оператор для pair<string, string>? this += { string { eng_rus.first }, eng_rus.second };
    – IR42
    Commented 6 сент. 2021 в 14:00

2 ответа 2

1

Так как в комментариях вы согласились с моим предложением, отвечу, что достаточно одного оператора:

void operator +=(pair<string, string> eng_rus) {
    root = insert(eng_rus.first, eng_rus.second, root);
}

Который может корректно принимать все указанные вами типы входных данных: <char,string>,<string,char>,<char,char>

Тоесть символьный тип char абсолютно спокойно можно интерпретировать как строку std::string, который может инициализироваться одним символом.

Таким образом убирается дублирование операторов принимающих все эти типы.

edit пример вызова

struct S
{
    void operator += (const std::pair<std::string, std::string>& eng_rus) {
        std::string s;
        s += eng_rus.first;
        //root = insert(s, eng_rus.second, root);
    }
};

int main()
{
    S s;
    s += std::pair<std::string, std::string>(std::string(1, '1'), std::string(1, '2'));    
    return 0;
}
2
  • Можно пример вызова этого оператора с char-ом? Commented 7 сент. 2021 в 18:30
  • @HolyBlackCat добавил пример
    – ampawd
    Commented 7 сент. 2021 в 18:42
0

Я не вижу здесь дублирования кода. Если вся начинка вынесена в функцию insert, вы уже избавились от дублирования.


Вторую функцию можно немного сократить:

root = insert(std::string(1, eng_rus.first), eng_rus.second, root);
1
  • Мне необходимо чтобы оператор += работал как с парами <string,string>, так и с парами <char,string>,<string,char>,<char,char>. А в аргументы ф-ии у меня сейчас может поступать только пара str,str ,а для добавления условий с char получается такой же оператор за исключением пары моментов => не прям полное, но всё же существенное дублирование кода.
    – s1rGAY
    Commented 6 сент. 2021 в 14:43

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.