[OstryKepucz]

Ktoś mi wytłumaczy dlaczego jak wcześniej przeciążałem operator konwersji z mojej klasy na stringa, wszystko działało xD i kompilator nie miał nic przeciwko, a teraz po kilku godzinach nagle wywala mi komunikat :
_**"ambigous conversion for C-style cast from 'String' to 'std::__cxxx..."**

String to moja własna klasa będąca mniej czy więcej odwzorowaniem klasy string z biblioteki.
W klasie mam napisaną metodę przeciążającą operator konwersji ( jawny operator konwersji do std::string):

explicit operator std::string() const {

string tmp;

for(int i=0; i<= this->size_of_text; i++){

tmp+=this->text[i];

}

return tmp;

}

Iiiii wcześniej przy np:

String s2="abcd";

string a=(string) s2;
wszystko śmigało, zakomentowałem sobie ten kod, po chwili przypadkiem od komentowałem iiii się okazuje, że nagle dla kompilatora jest to już be (╥﹏╥)

#programowanie #cpp #cplusplus

[MamCieNaHita]

@OstryKepucz: obawiam się, że ten wycinek to za mało, żeby zdiagnozować problem.
komunikat mówi, że ma przynajmniej 2 sposoby na zrobienie konwersji, może je wskazuje?

[OstryKepucz]

@MamCieNaHita:tutaj masz całość-> https://pastebin.com/6wY5GAAA
wg mnie jest tylko jedna opcja... Po za tym wcześniej bez przeciążenia operatora nie dało się wykonać takiej konwersji, natomiast teraz bez tej metody jakoś cudownie działa xD

[SirKir]

@OstryKepucz: Zgaduje że dodałeś #include i przestało działać ;)

[OstryKepucz]

@SirKir: string był dodany jeszcze przed tym :/ No naprawdę nie rozumiem... Wszystko działało nic nie zmieniałem, nawet programu nie resetowałem... i nagle boom kompilator mówi "you shall not pass!" XD
edit. próbowałem nawet wszystkie biblioteki wywalić i zostawić tylko iostream i dalej to samo :(

[tell_me_more]

@OstryKepucz: https://en.cppreference.com/w/cpp/string/basic_string/operator%3D

Jak dla mnie chodzi o to, że std::string ma kilka operatorów= zdefiniowanych, m.in. z charów i std::stringa. Twój typ ma konwersję do charów i do std::stringa. Kompilator nie wie, którą ścieżkę wybrać.

A "wcześniej działało", bo pewnie nie przekompilowałeś całego, albo nie zapisałeś zmian przed przekompilowaniem, albo zapomniałeś, że nie działało :)

PS ogólnie to rozumiem, że ten kod jest do nauki operatorów. Bo w prawdziwym życiu kombinowanie z

[OstryKepucz]

@tell_me_more: o kurcze rzeczywiście operator const char* był wcześniej z błędem i naprawiłem go później i to on miesza (ง ͠° ͟ل͜ ͡°)ง. To teraz kolejne pytanie jak to ominąć? :/
Edit. wystarczyło "wyłączyć" niejawną konwersję przez explicit dla konwersji do const chara ( ͡° ͜ʖ ͡°)

[tell_me_more]

@OstryKepucz: ogólnie IMHO przedefiniowanie operatorów to code smell, powinno się to robić w ostateczności. A już szczególnie nie należy ruszać operatorów rzutowania. Przedefiniowanie operatorów rzutowania to świetny sposób na to, żeby Twój kod robił 5 różnych rzeczy w tle, których się nie spodziewasz.

[lionbest]

@OstryKepucz: @tell_me_more: Tak, bo to operator const char*() powinien być przede wszystkim explicit.
Z reguły jest masz operator bool() to wszystko, co się implicit do niego konwertuje powinno być explicit.

Dziwne trochę, że u mnie na clangu to działa bez problemu, a co ciekawe przy GCC tylko od C++17.

[tell_me_more]

Z reguły jest masz operator bool()

@lionbest: to już jest #!$%@? :)

[lionbest]

@tell_me_more: Dlaczego, to bardzo przydatny operator.

[tell_me_more]

@lionbest: masz na swojej klasie operator rzutowania do boola. Porównujesz 2 elementy tej klasy. Nie masz operatora==. I kod typu:

MojString haslo("TAJNE_HASLO");
MojString wartoscZBazy = pobierzZBazy();
...
if (haslo == wartoscZBazy) { wystrzelRakiety(); }

wchodzi do ifa niezaleznie od hasla, bo skonwertował sobie oba na boola (a skoro oba są niepuste to oba są true bo tak "dla wygody" zdefiniowałeś operator na boola).

Nie, dzięki. Explicit is better than implicit jak

[OstryKepucz]

@tell_me_more: dziękuję w sumie za info, sam jakbym nie musiał to bym za wszelką cenę nie chciał tykać operatorów ale po to stworzyłem ta klasę żeby przećwiczyć na niej operatory (｡◕‿‿◕｡)

[OstryKepucz]

@tell_me_more: a z tymi rakietami to zaskoczę pana prowadzącego ( ͡° ͜ʖ ͡°) ( ͡° ͜ʖ ͡°)ﾉ⌐■-■

[lionbest]

@tell_me_more: @OstryKepucz: Ale pierwszy będzie użyty operator == (MojString, MojString) jeżeli taki istnieje. Jeżeli nie to cóż, rakiety polecą.