ошибка C2064: термин не оценивает функцию, принимающую 1 аргумент - PullRequest
3 голосов
/ 20 мая 2011
class Student {
    // ...
    bool Graduate() { return m_bGraduate; }
    // ...
};

class School {
    vector<Student*> m_vecStudents;

    void DelAndNullify(Student* &pStd);
    void Fun1();
};

void School::DelAndNullify(Student* &pStd)
{
    if ( (pStd != NULL) && (pStd->Graduate()) )
    {
        delete pStd;
        pStd = NULL;
    }
}

void School::Fun1()
{
    for_each(m_vecStudents.begin(), m_vecStudents.end(), mem_fun(&School::DelAndNullify));
}

Ошибка 1 ошибка C2064: термин не оценивается для функции, принимающей 1 аргумент C: \ Program Files \ Microsoft Visual Studio 10.0 \ VC \ include \ алгоритма 22 1 Моделирование

Почему я получаю эту ошибку?


обновлено

изменить Student на pStd


обновлено // файл алгоритма

template<class _InIt, class _Fn1> inline
_Fn1 _For_each(_InIt _First, _InIt _Last, _Fn1 _Func)
{
    // perform function for each element
    for (; _First != _Last; ++_First)
        _Func(*_First); // <<<<<<<< this line!
    return (_Func);
}

Кстати, если я определю DelAndNullify как static, то следующая строка пройдет компилятор

for_each(m_vecStudents.begin(), m_vecStudents.end(), ptr_fun(&School::DelAndNullify));

Обновлено 05.09.2012

#include <string>
#include <fstream>
#include <vector>
#include <algorithm>
#include <iterator>
#include <string>
#include <iostream>
#include <iomanip>
#include <functional>
#include <boost/bind.hpp>

class Student {
public:
    Student(int id, bool bGraduate) : m_iID(id), m_bGraduate(bGraduate) {}
    bool Graduate() const { return m_bGraduate; }
private:
    int  m_iID;
    bool m_bGraduate;
};

class School {
public:
    School(int numStudent)
    {
        for (int i=0; i<numStudent; ++i)
        {
            m_vecStudents.push_back(new Student(i+1, false));
        }
    }

    ~School() 
    {   
        // deallocate the allocated student resource to prevent memory leak!
    }

    void DelAndNullify(Student* &pStd);
    void Fun1();

private:
    std::vector<Student*> m_vecStudents;

};

void School::DelAndNullify(Student* &pStd)
{
    if ( (pStd != NULL) && (!pStd->Graduate()) )
    {
        delete pStd;
        pStd = NULL;
    }
}

void School::Fun1()
{   // http://stackoverflow.com/questions/6065041/error-c2064-term-does-not-evaluate-to-a-function-taking-1-arguments
    std::for_each(m_vecStudents.begin(), m_vecStudents.end(), std::bind1st(std::mem_fun(&School::DelAndNullify), this));
    //boost::bind(&School::DelAndNullify, this, _1);
}

int main(int /*argc*/, char* /*argv*/[])
{
    School school(10);
    school.Fun1();
    return 0;
}

Ошибка 1 ошибка C2535: «void std :: binder1st <_Fn2> :: operator () (Student * &) const»: функция-член уже определена или объявлена ​​c: \ Program Files \ Microsoft Visual Studio 10.0 \VC \ include \ xfunctional 299

Ответы [ 3 ]

6 голосов
/ 20 мая 2011

std::mem_fun(&School::DelAndNullify) возвращает двоичный функтор, который принимает School* и Student*, но std::for_each ожидает, что унарный функтор принимает только Student*. Используйте Boost . Bind вместо:

std::for_each(
    m_vecStudents.begin(),
    m_vecStudents.end(),
    boost::bind(&School::DelAndNullify, this, _1)
);

Если у вас достаточно свежий компилятор, вы можете использовать std::bind или std::tr1::bind вместо библиотеки Boost; или, если вы используете компилятор с поддержкой лямбда-выражений в C ++ 11, вы можете сделать следующее вместо использования bind:

std::for_each(
    m_vecStudents.begin(),
    m_vecStudents.end(),
    [this](Student*& s){ DelAndNullify(s); }
);
1 голос
/ 20 мая 2011

это

mem_fun(&School::DelAndNullify)

возвращает двоичную функцию, ожидая School* и Student*.

Используйте

bind1st(mem_fun(&School::DelAndNullify), this)

вместо.

1 голос
/ 20 мая 2011

Похоже, mem_fun превращает вашу функцию-член в "статическую" функцию, которая принимает объект в качестве первого параметра, например:

static void DelAndNullfify(Student *pStudent);

Но у вас уже есть параметр в функции pre-mem_fun'd, поэтому вы получите:

static void DelAndNullfify(School *pSchool, Student* &prStudent);

Это один параметр слишком много.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...