Ошибка Segmentation fault

@Anton-KRaV · Регистрация: 28.07.2013

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

Всем доброго дня.

Люди добрые, помогите, кто чем может. При вызове метода hand.dealToPlayers(0) или hand.dealToPlayers(1) ошибок нет, но при вызове hand.dealToPlayers(2) при исполнении программы появляется ошибка "Segmentation fault". При закомментировании строки 37 исполнение происходит без ошибок.

Голову сломал, не могу понять что не так. Заранее благодарю, и, если что, простите новичка =)

P.S. Код сокращен до предела.

C++
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
class Player
{
private:
    vector<Card> hand;
public:
    void setCardToPlayersHand(Card card)
        { hand.push_back(card); }
};
 
/*----------------------------------------------------------------------------*/
 
class Poker
{
private:
    vector<Player> players;
public: 
    vector<Player> getPlayers() const
    {
        return players;
    }
};
 
/*----------------------------------------------------------------------------*/
 
class Hand: public Poker
{
private:
    vector<Card> cardDeck;
public:
    void dealToPlayers(short count)
    {
        short i = 0;
        while(i < count)
        {
            for(vector<Player> :: iterator it = getPlayers().begin(); it < getPlayers().end(); it++)
            {
                (*it).setCardToPlayersHand(cardDeck.front());
                cardDeck.erase(cardDeck.begin());
            }
        i++;
        }
    }
};
 
/*----------------------------------------------------------------------------*/
 
int main()
{
    hand.dealToPlayers(2);

Tulosba · 13.05.2015, 14:15

Anton-KRaV, судя по всему cardDeck у тебя не содержит достаточного кол-ва элементов, а то и вовсе пустой. Заполнение cardDeck в коде не увидел.

@Anton-KRaV · 13.05.2015, 16:05 **[ТС]**

Спасибо за ответ, но к сожаление вроде нет, вот алгоритм заполнения вектора:

C++
1
2
3
4
5
6
7
for(short i = 1; i<= 4; i++)
{
    for(short j = 2; j <= 13 + 1; j++)
    {
        cardDeck.insert(cardDeck.end(), Card(j, i));
    }
}

Tulosba · 13.05.2015, 16:21

Сообщение от Anton-KRaV

к сожаление вроде нет

Лучше бы увидеть минимально работающий (с ошибкой) пример, а не

Сообщение от Anton-KRaV

Код сокращен до предела.

Тогда можно будет говорить о чем-то конкретном, а не о догадках.

@Anton-KRaV · 13.05.2015, 16:27 **[ТС]**

Конечно, благодарю за проявленное внимание. Код под спойлером, строка, выдающая ошибку закоменченна:

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

C++
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
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
#include <iostream>
#include <string>
#include <vector>
#include <iterator>
#include <algorithm>
#include <random>
 
using namespace std;
/*----------------------------------------------------------------------------*/
class Card
{
private:
    short numb;
    short suit;
public:
    Card(short Numb, short Suit)
    {
        numb = Numb;
        suit = Suit;
    }
    short getNumb() const
        { return numb; }
    short getSuit() const
        { return suit; }
    friend ostream& operator << (ostream&, Card&);
};
 
/*----------------------------------------------------------------------------*/
 
ostream& operator << (ostream& stream, Card& out)
{
    stream << out.getNumb() << out.getSuit();
    return stream;
}
 
/*----------------------------------------------------------------------------*/
 
class Player
{
private:
    string username;
    short numb;
    vector<Card> hand;
    double bankroll;
public:
    Player(short Numb)
    {
        numb = Numb;
    }
    Player(short Numb, string Username)
    {
        numb = Numb;
        username = Username;
    }
    void setUsername(string Username)
        { username = Username; }
    void setNumb(short Numb)
        { numb = Numb; }
    void setHand(vector<Card> Hand)
        { hand = Hand; }
    void setBankroll(double Bankroll)
        { bankroll = Bankroll; }
    void setCardToPlayersHand(Card card)
        { hand.push_back(card); }
    string getUsername() const
        { return username; }
    short getNumb() const
        { return numb; }
    vector<Card> getHand() const
        { return hand; }
    double getBankroll() const
        { return bankroll; }
};
 
/*----------------------------------------------------------------------------*/
 
class Poker
{
private:
    vector<Player> players;
public: 
    Poker()
    {
        for(short i = 1; i <= 9; i++)
        {
            players.push_back(Player(i));
        }
    }
    vector<Player> getPlayers() const
    {
        return players;
    }
};
 
/*----------------------------------------------------------------------------*/
 
class Hand: public Poker
{
private:
    vector<Card> cardDeck;
    vector<Card> table;
public:
    Hand()
    {
        for(short i = 1; i<= 4; i++)
        {
            for(short j = 2; j <= 13 + 1; j++)
            {
                cardDeck.insert(cardDeck.end(), Card(j, i));
            }
        }
    }
    void dealToPlayers(short count)
    {
        short i = 0;
        while(i < count)
        {
            for(vector<Player> :: iterator it = getPlayers().begin(); it < getPlayers().end(); it++)
            {
                /*(*it).setCardToPlayersHand(cardDeck.front());*/
                cardDeck.erase(cardDeck.begin());
            }
        i++;
        }
    }
    void dealToTable()
    {
        table.push_back(cardDeck.front());
        cardDeck.erase(cardDeck.begin());
    }
};
 
int main()
{
    Hand hand;
    hand.dealToPlayers(2);
    cout << "OK";
    
    return 0;
}

Tulosba · 13.05.2015, 16:45

Anton-KRaV, у тебя getPlayers() возвращает копию, а должен ссылку. Замени сигнатуру на такую:

C++
1
vector<Player>& getPlayers()

В первом приближении работает.

@Anton-KRaV · 13.05.2015, 17:16 **[ТС]**

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

у тебя getPlayers() возвращает копию, а должен ссылку.

Прошу прощения, ни с указателями, ни со ссылками подружиться не смог, поэтому будут два вопроса:
1. Почему возврат копии не подходит в данном случае?
2. Заменять в описании метода?

Tulosba · 13.05.2015, 17:28

Сообщение от Anton-KRaV

Почему возврат копии не подходит в данном случае?

Во-первых, потому что в цикле ты хочешь модифицировать состояние игроков (setCardToPlayersHand), а не их копий.
Во-вторых, на каждой цикла:

C++
1
2
for(vector<Player> :: iterator it = getPlayers().begin();  it < getPlayers().end(); it++ 
)

получаются разные копии vector<Player>. По шагам:

C++
1
2
vector<Player> :: iterator it = getPlayers().begin(); // получили копию-1, указываем на начало копии-1
it < getPlayers().end() // получили копию-2 и сравниваем итератор копии-1 с конечным итератором копии-2!!!

дальше уже не важно, т.к. сравнение итераторов, которые не относятся к одному и тому же контейнеру приводит к неопределенному поведению (UB).

Сообщение от Anton-KRaV

Заменять в описании метода?

Строка 89 твоего кода.

Не по теме:

Сообщение от Anton-KRaV

ни с указателями, ни со ссылками подружиться не смог

пока начинать дружбу :)

@Anton-KRaV · 13.05.2015, 18:45 **[ТС]**

Немного понял, благодарю!

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .
Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL3_image 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	Влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	13.05.2015, 14:15
	Anton-KRaV, судя по всему cardDeck у тебя не содержит достаточного кол-ва элементов, а то и вовсе пустой. Заполнение cardDeck в коде не увидел. 0

@Anton-KRaV 1 / 1 / 1 Регистрация: 28.07.2013 Сообщений: 41
	13.05.2015, 18:45 [ТС]
	Немного понял, благодарю! 0