Comment puis-je retourner un Pointeur Null dans une fonction C++

Je suis actuellement en train de travailler sur un morceau de code qui va chercher dans un vecteur de type Personne (dont j'ai défini dans le code et de s'afficher si nécessaire). Si elle trouve la personne, il retourne leur nom. C'est actuellement en train de travailler, mais si il ne trouve pas la personne, il est censé retourner un pointeur Null. Le problème est, je ne peux pas comprendre comment le faire retourner un pointeur Null! On ne conserve que soit le plantage du programme à chaque fois.

Code:

Person* lookForName(vector<Person*> names, string input)
{
    string searchName = input;
    string foundName;
    for (int i = 0; i < names.size(); i++) {
        Person* p = names[i];
        if (p->getName() == input) {
            p->getName();
            return p; //This works fine. No problems here
            break; 
        } else {
            //Not working Person* p = NULL; <---Here is where the error is happening
            return p;
        }
    }
}
  • Juste retour " 0 " (zéro). C'est la représentation d'un pointeur null en C++.
  • nullptr serait mieux, mais sérieusement, utilisez std::find_if et, de préférence, un std::vector<Person> sans le pointeur.
  • Oh, et aussi que vous n'étiez pas à l'initialisation de P dans presque tous les cas, il aurait une valeur != 0. Rappelez-vous que C++ n'est pas l'initialiser des variables automatiquement.
  • Je le vois en cours d'initialisation.
  • Vous avez une grave erreur de logique ici - cette boucle ne sera jamais exécuter plus d'une fois, chaque branche de la if retourne.
  • Merci pour toutes ces idées! Je suis d'essayer d'utiliser std::find_if mais vais avoir quelques erreurs de compilation. Je vais continuer à bien!

InformationsquelleAutor Silmarilos | 2013-08-24
  1. 3

    Vous pouvez utiliser std::find_if algorithme:

    Person * lookForName(vector<Person*> &names, const std::string& input)
{
    auto it = std::find_if(names.begin(), names.end(),
              [&input](Person* p){ return p->getName() == input; });


    return it != names.end() ? *it : nullptr; //if iterator reaches names.end(), it's not found
}

    Pour C++03 version:

    struct isSameName
{
    explicit isSameName(const std::string& name)
    : name_(name)
    {
    }

    bool operator()(Person* p)
    {
       return p->getName() == name_;
    }
    std::string name_;
};

Person * lookForName(vector<Person*> &names, const std::string& input)
{
    vector<Person*>::iterator it = std::find_if(names.begin(), names.end(),
                           isSameName(input));


    return it != names.end() ? *it : NULL;
}
    • Merci pour l'idée! Un rapide Q si, j'ai collé ce code à la place de mon code, et il est venu avec une "erreur:" il n'est pas le nom d'un type." Dois-je supprimer une partie du code? ou dois-je besoin de le déclarer quelque part d'autre?
    • Il me semble que vous êtes de la compilation avec un C++03 compilateur, ou autrement, en C++03 mode. La solution ci-dessus dépend en C++11 caractéristiques, à savoir auto l'inférence de type, les lambdas et nullptr. Vous pouvez dérouler la boucle manuellement ou définir un foncteur hors-ligne (en dehors de la fonction) pour utiliser std::find_if
    • Ai eu de travail! Un million de mercis pour votre aide!
    • content que ça a aidé!
    InformationsquelleAutor billz
  2. 1

    Semble que vous avez juste à retourner la valeur Null, nullptr, ou 0.

    codeproject

    InformationsquelleAutor James Robert Albert
  3. 1

    Si le nom que vous recherchez n'est pas sur le premier élément, puis vous n'êtes pas à la recherche dans le reste des éléments.

    Vous avez besoin de faire quelque chose comme -

    for (int i = 0; i<names.size(); i++){
    Person* p = names[i];
    if (p->getName() == input) {
        return p;

        //Placing break statement here has no meaning as it won't be executed.
    } 
}

//Flow reaches here if the name is not found in the vector. So, just return NULL
return NULL;

    Comme Chris l'a suggéré, essayez d'utiliser std::find_if algorithme.

    InformationsquelleAutor Mahesh
  4. 0

    Juste utilisez le code suivant:

    return NULL;
    InformationsquelleAutor olegarch