[krillov]

Hi,


I´d like to use sort algorithm in my program, but somehow it doesn´t wanna work. I get this error message in Visual Studio 2010:

...\projekt\jukebox.cpp(260): error C3867: 'Jukebox::sortAlbum': function call missing argument list; use '&Jukebox::sortAlbum' to create a pointer to member
...\projekt\jukebox.cpp(260): error C2780: 'void std::sort(_RanIt,_RanIt)' : expects 2 arguments - 3 provided
e:\instprograms\msvisualstudio\vc\include\algorithm(3639) : see declaration of 'std::sort'

void Jukebox::sortAlbumByName()
{
	std::sort(albums.begin(), albums.end(), sortAlbum);
}

bool Jukebox::sortAlbum(const Album &pAlbum1, const Album &pAlbum2)
{
	std::string album1 = pAlbum1.getAlbumTitle();							
	std::string album2 = pAlbum2.getAlbumTitle();

	std::transform(album1.begin(), album1.end(), album1.begin(), (int (*)(int))tolower);
	std::transform(album2.begin(), album2.end(), album2.begin(), (int (*)(int))tolower);

	return ((album1.compare(album2)) == -1);	
}

[Skydiver]

This error:

krillov, on 17 March 2013 - 09:12 AM, said:

[color="#FF0000"]...\projekt\jukebox.cpp(260): error C3867: 'Jukebox::sortAlbum': function call missing argument list; use '&Jukebox::sortAlbum' to create a pointer to member

is referring to

03    std::sort(albums.begin(), albums.end(), sortAlbum);

You declared your function sortAlbum() as an instance method. Change it to a static method and things should work better.

[krillov]

I tried to change to a static method but it didn´t work or maybe I did it wrong. I moved the function outside of the class declaration and it´s working now but I don´t know if it´s an acceptable solution. Any idea? I got the same problems when I try to use for_each algorithm.

[IngeniousHax]

std::sort takes two arguments in general, you're passing it 3, but I believe you're passing the third argument incorrectly. Have a look at this reference for std::sort

[sepp2k]

IngeniousHax, on 17 March 2013 - 09:23 PM, said:

std::sort takes two arguments in general

It has one overload that takes two arguments and one that takes three.

Quote

but I believe you're passing the third argument incorrectly.

Other than the fact that, as Skydiver pointed out, the function needs to be static (or free) in order to be referenced in that way, there's nothing wrong with how the OP passes the third argument.

krillov, on 17 March 2013 - 06:32 PM, said:

I tried to change to a static method but it didn´t work or maybe I did it wrong.

Well, how did you do it and what was the error that you got?

This post has been edited by sepp2k: 17 March 2013 - 01:37 PM

[krillov]

I fixed. I made a dumb mistake and wrote static in the cpp file and not in the header file only. It works correctly now with three arguments. Thanks everyone. :-)

This post has been edited by krillov: 17 March 2013 - 02:53 PM