Getter 和分段错误

Question

我的程序需要一些帮助：

#include <iostream>
#include<conio.h>

using namespace std;


class Position {
public :
    int             line;
    int             column;

                    Position(int,int);
                    Position();
};
Position::Position(int n, int m) : line{n},column{m}{}

class Board {
private :
    int**           tab;
    int             nbline;
    int             nbcolumn;

public  :
                    Board(int, int);
    void            setValue(Position&, int);
    int             getValue(Position&);
    int             getNbline();
    int             getNbcolumn();
};


class Play {
private :
 // Play         m_instance;
    void            moves(Board, Position&); // quand le joueur joue
 // void            moves(Board, Position); // quand l'IA joue. Mettre une énum pour direction,
    bool            wincondition(Board);
    Play&           operator=(const Play&);

public :
// static Play&    Instance();

};

Board::Board(int m, int n) : tab{new int*[m]}, nbline{m}, nbcolumn{n}{

   int  x(0);
   for (int i = 0; i < m; ++i){
        tab[i] = new int[n];

        for(int j = 0; j < n; ++j) {
            tab[i][j] = x; x++;}}
}


void     Board::setValue(Position& p, int value)         { tab[p.line][p.column] = value; }
int      Board::getValue(Position& p)                    { return tab[p.line][p.column];  }
int      Board::getNbline()                              { return nbline;                 }
int      Board::getNbcolumn()                            { return nbcolumn;               }



void Play::moves(Board tab, Position& blank) {
   /* int             c = getch() ;
    Position        tmp;

    if(c==0 || c==224) {c = getch();


    switch(c){
    case 75 : //left
        if(blank.column-1>=0) {
                tmp.column = blank.column;
                tmp.line   = blank.line;

                tab.setValue(blank,tab.getValue(tmp));
                blank.column++;
                tab.setValue(blank, 0);
                }
        break;

    case 72 : // haut
        if(blank.line+1<=0) {
                tmp.column = blank.column+1;
                tmp.line   = blank.line;

                tab.setValue(blank,tab.getValue(tmp));
                blank.column++;
                tab.setValue(blank, 0);
            }
        break;


    case 77 ://droit
        if(blank.column+1<=tab.getNbcolumn()) {
                tmp.column = blank.column;
                tmp.line   = blank.line;

                tab.setValue(blank,tab.getValue(tmp));
                blank.column--;
                tab.setValue(blank, 0);
            }
        break;

    case 80 : //bas
        if(blank.line+1<=tab.getNbline()) {
                tmp.column = blank.column+1;
                tmp.line   = blank.line;

                tab.setValue(blank,tab.getValue(tmp));
                blank.column++;
                tab.setValue(blank, 0);
            }
        break;
    default : cout << "\n ERROR " << endl; break; // erreur
    }
}*/
}

int main()
{
    int             lines, columns;


    cout << "Enter number of lines" << endl;
    cin >> lines;
    cout << "Enter number of columns" << endl;
    cin >> columns;

    Board           tab(lines,columns);
    Position        pos(lines,columns);
    Position&       p = pos;


    for (int i = 0; i<lines;i++)
    {
        for(int j = 0; j<columns;j++)
        {
            cout << tab.getValue(p) << " ";
            if (i == lines) { cout << endl;}
        }
    }
    return 0;
}

当我在第 139 行调用 getValue 时，出现分段错误。获取值在第 57 行定义。执行 getValue 时，p.line 和 p.column 都在 main 函数的开头获取了正确的值。

程序没有错误，只有 2 个警告，因为我没有使用 Play::moves 个参数（因为目前在 /* */ 之间，等待测试）。我使用 Code::Blocks 和 -Wall -Wextra -pedantic -std=c++11.

我真的看不出出现分段错误的原因。我错过了什么吗？

Answer 1

您调用 get 的位置设置为您的棋盘大小。由于数组是基于 0 索引的，因此数组的大小实际上是数组末尾的一个。

const int size = 100
int arr[size];  //0, 1, 2, ... 98, 99
arr[size];  // fail arr is 0-99 and size is 100

Getter 和分段错误

Getter and Segmentation Fault

c++

getter

tabs

reference

segmentation-fault