2017-09-22 63 views
-1

我的代碼返回一個分段錯誤。我剛開始編寫功能initdrawC中的十五個遊戲。分割錯誤

它是用C編寫的,來自cs50的東西。

這是我的代碼:

/** 
* fifteen.c 
* 
* Implements Game of Fifteen (generalized to d x d). 
* 
* Usage: fifteen d 
* 
* whereby the board's dimensions are to be d x d, 
* where d must be in [DIM_MIN,DIM_MAX] 
* 
* Note that usleep is obsolete, but it offers more granularity than 
* sleep and is simpler to use than nanosleep; `man usleep` for more. 
*/ 

#define _XOPEN_SOURCE 500 

#include <cs50.h> 
#include <stdio.h> 
#include <stdlib.h> 
#include <unistd.h> 

// constants 
#define DIM_MIN 3 
#define DIM_MAX 9 

// board 
int board[DIM_MAX][DIM_MAX]; 

// dimensions 
int d; 

// prototypes 
void clear(void); 
void greet(void); 
void init(void); 
void draw(void); 
bool move(int tile); 
bool won(void); 

int main(int argc, string argv[]) { 
    // ensure proper usage 
    if (argc != 2) 
    { 
     printf("Usage: fifteen d\n"); 
     return 1; 
    } 

    // ensure valid dimensions 
    d = atoi(argv[1]); 
    if (d < DIM_MIN || d > DIM_MAX) 
    { 
     printf("Board must be between %i x %i and %i x %i, inclusive.\n", 
      DIM_MIN, DIM_MIN, DIM_MAX, DIM_MAX); 
     return 2; 
    } 

    // open log 
    FILE *file = fopen("log.txt", "w"); 
    if (file == NULL) 
    { 
     return 3; 
    } 

    // greet user with instructions 
    greet(); 

    // initialize the board 
    init(); 


    // accept moves until game is won 
    while (true) 
    { 
     // clear the screen 
     clear(); 

     // draw the current state of the board 
     draw(); 

     // log the current state of the board (for testing) 
     for (int i = 0; i < d; i++) 
     { 
      for (int j = 0; j < d; j++) 
      { 
       fprintf(file, "%i", board[i][j]); 
       if (j < d - 1) 
       { 
        fprintf(file, "|"); 
       } 
      } 
      fprintf(file, "\n"); 
     } 
     fflush(file); 

     // check for win 
     if (won()) 
     { 
      printf("ftw!\n"); 
      break; 
     } 

     // prompt for move 
     printf("Tile to move: "); 
     int tile = get_int(); 

     // quit if user inputs 0 (for testing) 
     if (tile == 0) 
     { 
      break; 
     } 

     // log move (for testing) 
     fprintf(file, "%i\n", tile); 
     fflush(file); 

     // move if possible, else report illegality 
     if (!move(tile)) 
     { 
      printf("\nIllegal move.\n"); 
      usleep(500000); 
     } 

     // sleep thread for animation's sake 
     usleep(500000); 
    } 

    // close log 
    fclose(file); 

    // success 
    return 0; 
} 

/** 
* Clears screen using ANSI escape sequences. 
*/ 
void clear(void) { 
    printf("\033[2J"); 
    printf("\033[%d;%dH", 0, 0); 
} 

/** 
* Greets player. 
*/ 
void greet(void) { 
    clear(); 
    printf("WELCOME TO GAME OF FIFTEEN\n"); 
    usleep(2000000); 
} 

/** 
* Initializes the game's board with tiles numbered 1 through d*d - 1 
* (i.e., fills 2D array with values but does not actually print them). 
*/ 
void init(void) 
{ 
    int x = 0; 
    int z = 0; 
    int y = d * d; 
    int w = 1; 
    for (x = 0; x < d; x++) 
    { 
     for (z = 0; z < d;x++) 
     { 
      board[x][z] = y - w; 
      w++; 
     } 
    } 
    board[d-1][d-1] = y; 

    if (y % 2 == 0) 
    { 
     board[d - 1][d - 2] = 2; 
     board[d - 1][d - 3] = 1; 
    } 
} 

/** 
* Prints the board in its current state. 
*/ 
void draw(void) 
{ 
    int q = 0; 
    int r = 0; 

    for (q = 0; q < d; q++) 
    { 
     for (r = 0; r < d; r++) 
     { 
      printf("|%2i|", board[q][r]); 
     } 
    } 

    if (board[q][r] == 0) 
    { 
     printf(" |__|"); 
    } 
} 

/** 
* If tile borders empty space, moves tile and returns true, else 
* returns false. 
*/ 
bool move(int tile) { 
    // TODO 
    return false; 
} 

/** 
* Returns true if game is won (i.e., board is in winning configuration), 
* else false. 
*/ 
bool won(void) { 
    // TODO 
    return false; 
} 

正如你可以看到它仍然沒有完成。 我有點卡在這裏。也許有人知道我怎麼能解決這個問題。 在此先感謝您的幫助。

+2

SO正在提示您輸入更多信息,並欺騙它並不會產生任何好處。您尚未提供任何問題描述,只是您希望我們調試的一些不完整的長代碼。 –

+0

'draw'中的if(board [q] [r] == 0)超出範圍:'q == d'和'r == d'(對於'd == DIM_MAX')。 –

+0

恩,我很抱歉。我提到在代碼中一切都很好,直到init函數出現,所以錯誤應該在那裏。這是我第一次提交。我會更好下一次 – zorange

回答

3

更改for (z = 0; z < d; x++)for (z = 0; z < d; z++)而且您將不會再收到init(...)的段錯誤。你只需要使用調試器來找出這些類型的東西。如果您使用的是gcc,請嘗試gcc -g file.c,然後運行gdb a.out並鍵入運行

+0

好趕上!愚蠢的錯誤... –

+0

這是一個偉大而銳利的眼睛,你已經到了那裏。謝謝 ! 你們在這麼短的時間內幫助我很棒! – zorange