Es geht um diese Funktion:

unsigned SYSInfo::getName(){ 
  if (dev[0].name == 0) return{};
  return dev[0].name;   
}

Der Fehler ist:

keine Übereinstimmung für »operator==« in »((exsystem::SYSInfo*)this)->exsystem::SYSInfo::dev.std::vector<_Tp, _Alloc>::operator[] [mit _Tp = exsystem::face, _Alloc = std::allocator<exsystem::face>, std::vector<_Tp, _Alloc>::reference = exsystem::face&, std::vector<_Tp, _Alloc>::size_type = long unsigned int](0ul).exsystem::face::name == 0«

Zur Funktion, in diesem Fall steht "lo" gespeichert in name und soll geholt werden.

Bei meiner Funktion:

unsigned SYSInfo::getBytesr(){ 
  if (dev[0].bytes_received == 0) return{};
  return dev[0].bytes_received;   
}

funktioniert alles da steht allerdings eine Zahl drin.

LG Enno

Hey ich mal wieder.

Es geht um diese Funktion:

unsigned SYSInfo::getName(){ 
  if (dev[0].name == 0) return{};
  return dev[0].name;   
}

Das geht doch nicht. Eine Zeichenkette kann doch nicht Null sein. Oder hast du z.B. schon mal eine Person kennengelernt die Null heißt? Manche Personen sind vielleicht eine Null, aber das ist doch kein Name. Ich denke mal, du willst testen, ob die Zeichenkette leer ist. Dann mach if (dev[0].empty()) return{};

Was ist "dev[0].name" für ein Typ?

Und was willst du mit "return{}" bewirken?

Oh, ja die Infos fehlen glaube ich noch:

std::vector<face> dev

struct face{ 
    std::string name;

std::string SYSInfo::getName(){ 
  if (dev[0].name.empty()) return{};
  return dev[0].name;   
}

Weiß jemand Rat zu diesem Fehler? Kommt Bei beiden Funktionen von oben!

LG Enno

Was ist der Unterschied zu?:

std::string SYSInfo::getName(){ return dev[0].name; }

Weiß jemand Rat zu diesem Fehler? Kommt Bei beiden Funktionen von oben!

Ich sehe keinen Fehler, nur eine WARNUNG. Und ich finde die Warnung ziemlich eindeutig.

std::string SYSInfo::getName(){ 
  if (dev[0].name.empty()) return{};
  return dev[0].name;   
}

Was ist der Unterschied zu?:

std::string SYSInfo::getName(){ return dev[0].name; }

Ich überprüfe vorher ob name leer ist?

Weiß jemand Rat zu diesem Fehler? Kommt Bei beiden Funktionen von oben!

Ich sehe keinen Fehler, nur eine WARNUNG. Und ich finde die Warnung ziemlich eindeutig.

Kann man das ignorieren? oder bekommt man das irgendwie weg?

Btw. das ist kein Chat ... Bitte lesen, denken, ausprobieren, schreiben, ... in genau dieser Reihenfolge!

Und gibst dann was zurueck?

0 oder nicht?

das return{} gibt eine Inizializer-List mit 0 Elementen zurück. Was du meinst, ist return "";

bzw. besser wäre es sogar so:

const std::string& SYSInfo::getName(){ 
    if (dev[0].name.empty()) 
      return "";
    return dev[0].name;   
}

da hier der String nicht kopiert werden muss.

Aber mal etwas nachgedacht, wie sieht ein leerer String aus? genau: "". Du überprüfst also, ob der String leer ist, wenn ja, gibst du "" zurück. sonst den String -> Überprüfung ist redundant. Ergo:

const std::string& SYSInfo::getName(){ 
    return dev[0].name;   
}

knivil schrieb:

Und gibst dann was zurueck?

0 oder nicht?

Nein, auf raten wuerde ich verzichten! Schon mal deinen Rueckgabewert angesehen? 0 ist eine Zahl und kein std::string.

bzw. besser wäre es sogar so:

const std::string& SYSInfo::getName(){ 
    if (dev[0].name.empty()) 
      return "";
    return dev[0].name;   
}

da hier der String nicht kopiert werden muss.

Nicht gut, da eine Referenz auf ein lokales temporäres Objekt zurückgegeben wird.

const std::string& SYSInfo::getName() const
{ 
    return dev[0].name;   
}

Spielt keine Rolle, da const-Referenzen auf temporäre Objekte erlaubt sind.

Na dann ist es ja gut, dass ich das angesprochen habe. Wie lange lebt denn das temporäre Objekt?

das return{} gibt eine Inizializer-List mit 0 Elementen zurück. Was du meinst, ist return "";

Nein, das gibt einen default-konstruierten String zurück. Ist also nicht falsch (vermutlich allerding eher durch Zufall als durch planmäßiges Vorgehen), aber natürlich trotzdem überflüssig.

Spielt keine Rolle, da const-Referenzen auf temporäre Objekte erlaubt sind.

Die Lebenszeit des referenzierten Objekts wird aber nur für Variablen im lokalen Scope verlängert.

const int& i = 3; // ok

const std::string& SYSInfo::getName() const
{ 
    return dev[0].name;   
}

Kann das bitte noch einmal genau erklärt werden warum die zwei const da stehen?

LG Enno

Kannst du dir vielleicht mal endlich ein Buch kaufen?

Hab den Primer.
Meinste ich kann da einfach unter

const std::sting beispiel() const{
.
.
.
}

gucken?

Du weisst aber hoffentlich schon, dass du durch den blossen Besitz eines Buches nicht schlauer wirst, oder?

Ich hab das ganze Buch, da ich es als PDF aufm Rechner hab, per such Funktion nach const String durchsucht ... Hab nichts gefunden was mir helfen könnte.
Google bringt auch nichts vor. Da mir das in diesem Thread vorgeschlagen wurde das es die beste Möglichkeit ist dachte ich irgendwer ist so nett das dann auch zu erklären oder mir ein link zu geben. Da ich das gerne verstehen und lernen möchte. Solche Kommentare helfen mir gar nicht und sind nur nervig.

Beides kommt laut Inhaltsverzeichnis im ersten Teil ("Basics") dran (2.4 2.5 (References) und 7.7 (Class Member Functions)).

Setz dich mal einen Abend hin und lies das Ding von vorne bis hinten durch. Es ist dabei beim ersten Mal überhaupt nicht notwendig, dass du alles verstehst, Hauptsache die Grundkonzepte bleiben hängen.

Beides kommt laut Inhaltsverzeichnis im ersten Teil ("Basics") dran (2.4 und 7.7).

Bei welcher Version? 5? Ich hab 4.