7 replies to this topic

[thatsme]

here is the method of class TrainingProgram. Program_list is the list of strings. This method checks if the given string is already in program_list or not. I get such an error: . C:\Users\admin\Desktop\Documents\C\training_functions.cpp|559|error: 'struct std::_List_iterator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > >' has no member named 'compare'|
Any help? Thanks in advence


bool TrainingProgram::trainingProgram_isExercise(string exercise)
{

for (it3 = program_list.begin(); it3 != program_list.end(); ++it3)
{

if (it3.compare(exercise) == 0) //error in that line
return true;

}
return false;

}

[abzero]

the compare method is part of std::string? it3 is the iterator, so you want to use *it3 to get to what the iterator is actually pointing to.

[thatsme]

Yes, the compare is method of std::string. *it3 didn't help. The same error occurs

[abzero]

try (*it3).compare, or it3->compare()

[Flying Dutchman]

Post code for it3 - what kind of iterator is it? Also, is program_list a std::list?

[julmuri]

I would just use std::find from algorithm.

bool TrainingProgram::trainingProgram_isExercise( const std::string& exercise )
{
    return std::find( program_list.begin(), program_list.end(), exercise )
        != program_list.end();
}

Does the same thing with less hassle.

[thatsme]

I just did that: string str = *it3 and then str.compare(exercise) worked well. thanks for help

[abzero]

although that's probably ok for a string. For other data types you might want to use a reference or pointer rather then the normal object.