Funktion nimmt nicht 0 Argumente

Question

Die Groß() Funktion ein Teil meiner Klasse ist:

class MyString { 
public: 
    MyString(); 
    MyString(char cstring[]); 
    void uppercase(); 

private 

}; 

Und die Umsetzung für Groß nicht beendet ist, sondern sieht wie folgt aus:

void MyString::uppercase() 
{ 
    cout << "need to implement"; 
} 

Als ich rufen sie die Funktion, sieht es wie folgt aus:

//Output streaming Operator Overload 
ostream& operator<<(ostream& os, const MyString& string) 
{ 
    if (MyString::printAsUppercase == true) 
     uppercase(); 
    else 
     os << string.data; 

    cout << "(" << string.length << ")"; 
    return os; 

} 

Wenn ich versuche, den Code zu kompilieren, erhalte ich folgende Fehlermeldung:

Ich verstehe das wirklich nicht, wie ich den Prototyp erklärte, um keine Argumente zu nehmen, und folgte damit in der Implementierung. Die Funktion sollte keine Argumente annehmen müssen. Warum passiert das?

Answer 1

Sie müssen angeben, dass Sie Ihre Großbuchstabenmethode aufrufen möchten. Der Fehler besagt, dass Sie "Std :: Großbuchstaben" < < Std aufrufen :: Sie müssen Ihre Methode auf Ihre Klasse aufrufen.

Kommentar bearbeiten: François Andrieux erwähnt, dass dies wahrscheinlich ein Problem ist, weil Sie "using namespace std;" Daher werden alle Funktionsaufrufe in der Std gesucht. (Und hes Recht damit :))

Answer 2

std::uppercase ist ein legitimes E/A-Stream-Objekt in der Standardbibliothek. Da Sie im std-Namespace codieren, kann der Compiler nicht feststellen, ob Sie Ihre Funktion oder das Bibliotheksobjekt verwenden. Eine einfache Lösung ist, ändern Sie den Namen Ihrer Funktion oder geben Sie Ihre Klasse MyClass :: Großbuchstaben (...). Eine bessere Lösung wäre es, den Code in Ihren eigenen Namespace einzubinden oder die Kodierung im std-Namespace mit der Deklaration "using namespace std" zu vermeiden. Meiner Meinung nach ist es klarer, den std-Namespace bei jeder Verwendung anzugeben. Zum Beispiel, anstatt "using namespace std;" „Cout < < ...“ schreiben Sie einfach „std :: cout < < ...“ dann

Answer 3

ich den folgenden Code hoffen helfen könnte, die Punkte zu verstehen, die von anderen Benutzern angezeigt. Es kompiliert, aber ... es auf jeden Fall bei weitem nicht optimal ist:

main.cpp 

#include <iostream> 
#include "MyString.h" 

int main(int argc, char** argv) 
{ 
    // Needs arguments from command line!!! 
    MyString teststring(argv[1]); 
    std::cout << teststring << std::endl; 
    return 0; 
} 

MyString.h 

class MyString { 
public: 
    MyString(); 
    MyString(char *cText); 
    void myUpperCase(); 
    int myStringLenght(); 

protected: 
    bool printAsUppercase; 
    char *data; 
    friend std::ostream& operator<<(std::ostream& os, 
            MyString mystring); 
}; 

MyString.cpp 

#include <cstdlib> 
#include <cstring> 
#include <iostream> 
#include "MyString.h" 

MyString::MyString(char *cText) 
{ 
    data = (char *)std::malloc(strlen(cText)); 
    std::strcpy(data, cText); 
} 

void MyString::myUpperCase() 
{ 
    std::cout << "not yet implemented"; 
} 

int MyString::myStringLenght() 
{ 
    return int(strlen(data)); 
} 

std::ostream& operator<<(std::ostream& os, 
          MyString mystring) 
{ 
    // to be deleted - only for testing 
    mystring.printAsUppercase = true; 

    if (mystring.printAsUppercase == true) 
     mystring.myUpperCase(); 
    else 
     os << mystring.data; 

    std::cout << "(" << mystring.myStringLenght() << ")"; 
    return os; 
}