Vector sort не сортирует

@Eumi · Регистрация: 12.06.2012

Студворк — интернет-сервис помощи студентам

Добрый день, подскажите, пожалуйста. Мне необходимо отсортировать вектор в котором хранится мой класс.

Вроде все сделала, переопределеила <, но сортировка не работает. Подскажите, чего я еще не замечаю.

Player.cpp в нем переопределила operator< (73 строка)

Кликните здесь для просмотра всего текста

C++ (Qt)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
#include "player.h"
 
Player::Player(QTextEdit *debugEdit)
{
    debugEdit->append("Работает конструктор сущности Player");
}
 
Player::~Player()
{
 
}
 
int Player::getNumberPlayedGames() const
{
    return numberPlayedGames;
}
 
void Player::setNumberPlayedGames(int value)
{
    numberPlayedGames = value;
}
 
int Player::getNumberWinGames() const
{
    return numberWinGames;
}
 
void Player::setNumberWinGames(int value)
{
    numberWinGames = value;
}
 
int Player::getTeamNumber() const
{
    return teamNumber;
}
 
void Player::setTeamNumber(int value)
{
    teamNumber = value;
}
 
string Player::getTeamRole() const
{
    return teamRole;
}
 
void Player::setTeamRole(const string &value)
{
    teamRole = value;
}
 
string Player::getBirthDay() const
{
    return birthDay;
}
 
void Player::setBirthDay(const string &value)
{
    birthDay = value;
}
 
string Player::getFio() const
{
    return fio;
}
 
void Player::setFio(const string &value)
{
    fio = value;
}
 
bool Player::operator<(const Player &rhs) const { return teamNumber < rhs.teamNumber; }
 
 
string Player::playerToString()
{
    return to_string(teamNumber);
}

Team.cpp - здесь вызываю сортировку (32 строка)

Кликните здесь для просмотра всего текста

C++ (Qt)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
#include "team.h"
#include <player.h>
#include <QDebug>
#include <algorithm>
 
Team::Team()
{
    teamList.reserve(10);
    sizeTeam = 0;
}
 
Team::~Team()
{
 
}
 
vector<Player *> Team::getTeamList() const
{
    return teamList;
}
 
Player* Team::getPlayer(int playerNumber)
{
    return teamList[playerNumber];
}
 
int Team::getSizeTeam() const
{
    return sizeTeam;
}
 
void Team::sortTeam()
{
    qDebug()<<"Sort!";
    sort(teamList.begin(), teamList.end());
 
}
 
void Team::addPlayerToTeam(Player *aPlayer){
    teamList[sizeTeam] = aPlayer;
    sizeTeam++;
}

@dawn artist · 20.06.2017, 23:18

Сообщение от Eumi

вектор в котором хранится мой класс

Сообщение от Eumi

чего я еще не замечаю

Что в векторе хранится указатель, а не класс.

@Eumi · 20.06.2017, 23:38 **[ТС]**

Сообщение от daun-autist

Что в векторе хранится указатель, а не класс.

Понятно, спасибо, буду править =(

@Eumi · 21.06.2017, 00:41 **[ТС]**

Не хочу темы плодить, поэтому задам вопрос здесь.
Если я передлываю вот так вот, чтобы избавиться от указателя, то приложение крашиться. Если встать в отладке повялется вот такое сообщение

Кликните здесь для просмотра всего текста

C++ (Qt)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
#include "team.h"
#include <player.h>
#include <QDebug>
#include <algorithm>
 
Team::Team()
{
    teamList.reserve(10);
    sizeTeam = 0;
}
 
Team::~Team()
{
 
}
 
vector<Player> Team::getTeamList() const
{
    return teamList;
}
 
Player* Team::getPlayer(int playerNumber)
{
    return &teamList[playerNumber];
}
 
int Team::getSizeTeam() const
{
    return sizeTeam;
}
 
void Team::sortTeam()
{
    qDebug()<<"Sort!";
    sort(teamList.begin(), teamList.end());
 
}
 
void Team::addPlayerToTeam(Player *aPlayer){
    teamList[sizeTeam] = *aPlayer;
    sizeTeam++;
}

Кликните здесь для просмотра всего текста

C++ (Qt)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
#ifndef TEAM_H
#define TEAM_H
#include <player.h>
#include <vector>
 
using namespace std;
 
class Team
{
public:
    Team();
    ~Team();
    void addPlayerToTeam(Player *);
    vector<Player> getTeamList() const;
    Player *getPlayer(int playerNumber);
 
    int getSizeTeam() const;
    void sortTeam();
    void teamListString();
 
private:
    vector<Player> teamList;
    int sizeTeam;
};
 
#endif // TEAM_H

@dawn artist · 21.06.2017, 00:58

Не по теме:

А ведь можно было просто лямбду в std::sort передавать...

Добавлено через 1 минуту

Сообщение от Eumi

C++
1
2
3
4
Player* Team::getPlayer(int playerNumber)
{
    return &teamList[playerNumber];
}

После вставки более 10 значений никто не гарантирует валидность этого указателя.
Вообще, зачем он нужен, этот указатель, в свете того, что вектор могут отсортировать.

Добавлено через 5 минут

Сообщение от Eumi

C++
1
int sizeTeam;

Вектор не знает своего размера, давайте ему поможем!11

Добавлено через 5 минут

Сообщение от Eumi

C++
1
2
3
4
void Team::addPlayerToTeam(Player *aPlayer){
    teamList[sizeTeam] = *aPlayer;
    sizeTeam++;
}

Ну и конечно пропустил самое главное. Это просто пушка!

@Eumi · 21.06.2017, 01:05 **[ТС]**

Сообщение от daun-autist

Не по теме:
А ведь можно было просто написать оператор< для указателей или лямбду в std::sort передавать...

что-то вроде такого?

C++ (Qt)
1
bool Player::operator<(const Player *rhs) const { return this->getTeamNumber() < rhs->getTeamNumber(); }

ЗЫ так не работает

@dawn artist · 21.06.2017, 01:29

C++
1
std::sort(teamList.begin(), teamList.end(), [](auto p1, auto p2) { return *p1 < *p2; });

Добавлено через 20 минут
Впрочем, ничего всё равно сортировать не будет, т.к. сортировать нечего.

@Eumi · 21.06.2017, 01:29 **[ТС]**

Сообщение от daun-autist

C++
1
std::sort(teamList.begin(), teamList.end(), [](auto p1, auto p2) { return *p1 < *p2; });

auto не работает
use of 'auto' in lambda parameter declaration only available with -std=c++14 or -std=gnu++14

попробовала так, не канает:

C++
1
sort(teamList.begin(), teamList.end(), [](const Player* p1, const Player* p2) -> bool { return p1->getTeamNumber() < p2->getTeamNumber() ; });

Въехала=))) спасибо большое =)

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

Vector sort не сортирует

Решение