Ошибка в объявлении функции пропуск основного текста функции

Ее не необходимо объявлять дружественной. Ее необходимо сделать отдельной внешней функцией, а не методом класса.

А уж дружественной ее придется сделать в тех случяах, когда/если вам надо лезть во внутренности класса, которые обычным внешним функциям просто так не доступны. Если вам понадлобится лезть в такие внутренности, то придется объявить ее дружественной. А не понадобится — можно не объявлять. Если вы будуте реализовать вашу функцию, как просто return a*p;, то дружественность вам не нужна.

Также частным побочным эффектом дружественности является возможность запихать определение внешней функции внутрь определения класса. Часто дружественность применяют именно ради этого. Без дружественности это определение функции следует делать снаружи определения класса.

Другими словами, в вашем случае никакой необходимости в дружественности нет. Уберите дружественность, и объявите ваш оператор снуражи класса как inline функцию — все будет работать и так.

Есть вектор с указателями на книги. Надо его отсортировать по дате(по году, потом по месяцу и дню). Еще кстати не знаю, если в векторе указатели, то и в comparator надо ставить * или &?
Питаюсь сделать так

  bool comparator( Book& a, Book& b) {
    if (a.GetYear() < b.GetYear()) return true;
    if (a.GetYear() > b.GetYear()) return false;
    if (a.GetMonth() < b.GetMonth()) return true;
    if (a.GetMonth() > b.GetMonth()) return false;
    return a.GetDay() < b.GetDay();
}
sort(books.begin(), books.end(), comparator)

Выдает такие ошибки

error C3867: "HeroFunctions::comp": нестандартный синтаксис; используйте "&", чтобы создать указатель на член
error C2672: "sort": не найдена соответствующая перегруженная функция
error C2780: void std::sort(const _RanIt,const _RanIt): требует аргументов: 2, имеется: 3

Пытался сделать с перегрузкой

bool operator<(Book* book1, Book* book2)
    {
        if (book1->GetYear() == book2->GetYear())
        {
            if (book1->GetMonth() == book2->GetMonth())
                return book1->GetDay() < book2->GetDay();
 
            return book1->GetMonth() < book2->GetMonth();
        }
        return book1->GetYear() < book2->GetYear();
    }
    sort(books.begin(), books.end())

Выдает это

error C2804: бинарный "operator <" имеет слишком много параметров
error C2333: HeroFunctions<T,U>::operator <: ошибка в объявлении функции; пропуск основного текста 
  функции
error C2333: HeroFunctions<std::string,Hero>::operator <: ошибка в объявлении функции; пропуск 
 основного текста функции

Вот тут проект. Думаю будет проще, если запустите и там увидете ошибку и сможете исправить

There’s a vector with book signs. We’ll have to sort it out by day. Also, I don’t know if the vector’s indicator, you’re gonna have to put it in the comparator or something.
I’m trying to do that.

  bool comparator( Book& a, Book& b) {
    if (a.GetYear() < b.GetYear()) return true;
    if (a.GetYear() > b.GetYear()) return false;
    if (a.GetMonth() < b.GetMonth()) return true;
    if (a.GetMonth() > b.GetMonth()) return false;
    return a.GetDay() < b.GetDay();
}
sort(books.begin(), books.end(), comparator)

Makes such mistakes.

error C3867: "HeroFunctions::comp": нестандартный синтаксис; используйте "&", чтобы создать указатель на член
error C2672: "sort": не найдена соответствующая перегруженная функция
error C2780: void std::sort(const _RanIt,const _RanIt): требует аргументов: 2, имеется: 3

Trying to overload.

bool operator<(Book* book1, Book* book2)
    {
        if (book1->GetYear() == book2->GetYear())
        {
            if (book1->GetMonth() == book2->GetMonth())
                return book1->GetDay() < book2->GetDay();
        return book1-&gt;GetMonth() &lt; book2-&gt;GetMonth();
    }
    return book1-&gt;GetYear() &lt; book2-&gt;GetYear();
}
sort(books.begin(), books.end())


Gives it up.
error C2804: бинарный "operator <" имеет слишком много параметров

error C2333: HeroFunctions<T,U>::operator <: ошибка в объявлении функции; пропуск основного текста

функции

error C2333: HeroFunctions<std::string,Hero>::operator <: ошибка в объявлении функции; пропуск

основного текста функции


 https://github.com/LavrovSergey/test 

#ifndef moveHare
#define moveHare

//Create new class for Hare movement roll
class moveHare
{
//Initialize variables to be used
    int hareMovement = 0;
    int hareRoll = rand() % 100 + 1;

private:
//if statements to compare the random int to the movement int 
//Error happens at first if statement.`enter code here`
    if (hareRoll > 50) {
        hareMovement = hareMovement + 3;
    }

    else if (hareRoll > 50 && hareRoll < 71) {
        hareMovement = hareMovement + 6;
    }
};

#endif

2 ответа

Лучший ответ

#pragma once

#ifndef movement
#define movement





int moveHare() {

        int hareMovement = 0;
        int hareRoll = rand() % 100 + 1;

        if (hareRoll > 50) {
            hareMovement = hareMovement + 3;
        }

        else if (hareRoll > 50 && hareRoll < 71) {
            hareMovement = hareMovement + 6;
    }
        return hareMovement;
}


#endif

 #ifndef moveHare
 #define moveHare

 //Create new class for Hare movement roll
 class MoveHare
 {
     private:
         int hareMovement = 0;
         int hareRoll = rand() % 100 + 1;

     public:
         int MoveTheHare(void);
 };

 #endif

int MoveHare::MoveTheHare(void)
{
    if (hareRoll > 50)
    {
        hareMovement = hareMovement + 3;
    }

    else if (hareRoll > 50 && hareRoll < 71)
    {
        hareMovement = hareMovement + 6;
    }

    return hareMovement;
}