C++中的矩陣輸出不正確

Question

我正在編寫一個代碼來創建一個值，並創建一個大小爲1s的金字塔的矩陣，因此頂部是x-1元素。例如，如果輸入是五，則輸出將是：

0 0 0 0 1 0 0 0 0 
0 0 0 1 0 1 0 0 0 
0 0 1 0 0 0 1 0 0 
0 1 0 0 0 0 0 1 0 
1 0 0 0 0 0 0 0 1 

問題是代碼生成的值，不應該是一個，或它是不輸出矩陣的正確方法。我試圖在代碼中添加一個cout，它將一些值定義爲1，這些值都是正確的，所以我相信問題出在輸出中。例如，x的輸入會給

0 0 0 0 1 0 0 0 1 
0 0 0 1 0 1 0 1 0 
1 0 1 0 0 0 1 0 0 
0 1 0 0 0 1 0 1 0 
1 0 1 0 0 0 0 0 1 

的代碼如下：

#include <iostream> 
using namespace std; 

int main() { 
     cout << "Matrix size:"; 
     int x; 
     cin >> x; 
     int arr1[2*x][x]; 
     for (int initialX=0; initialX<x; initialX++){ 
       for (int initialY=0; initialY<=((2*x)-2); initialY++){ 
         arr1[initialX][initialY]=0; 
       } 
     } 
     for (int set=0; set<x; set++){ 
       arr1[set][x-1-set]=1; 
       arr1[set][x-1+set]=1; 
     } 
     for (int outX=0; outX<x; outX++){ 
       for (int outY=0; outY<(2*x-1); outY++){ 
         cout << arr1[outX][outY] << " "; 
       } 
       cout << endl; 
     } 
     return 0; 
} 

這與矩陣工作在C我第一次++，因此關於矩陣任何幫助表示讚賞。另外，如果它非常重要，我正在使用C++ 11。先謝謝你。

Answer 1

需要切換陣列尺寸。取而代之的

int arr1[2*x][x]; 

使用

int arr1[x][2*x]; 

因爲這個錯誤，你正在訪問數組越界，這會導致不確定的行爲。

由於VLA不是標準C++，我強烈推薦使用std::vector。

std::vector<std::vector<int>> arr1(x, std::vector<int>(2*x)); 

這也將所有元素初始化爲0，並避免了需要編寫顯式代碼數組元素初始化爲0。

Answer 2

我想你可能已經交換了你的設置的索引值。另外，我不建議使用set作爲變量名稱。試試這個：

for (int i=0; i<x; i++){ 
    arr1[x-1-i][x]=1; 
    arr1[x-1+i][x]=1; 
} 

如果有疑問，試着用小的值來代碼，比如x = 2。

Answer 3

#include <iostream> 
using namespace std; 

int** CreateMatrix(int size); 

int main() 
{ 
    cout << "Matrix size:"; 
    int x; 
    cin >> x; 

    int** intArray = CreateMatrix(x); 

    int max = (x * 2) - 1; 

    for (int row = 0; row < x; row++) 
    { 
     for (int col = 0; col < max; col++) 
     { 
      cout << intArray[row][col] << " "; 
     } 
     cout << endl; 
    } 
    return 0; 
} 

int** CreateMatrix(int size) 
{ 
    int max = (size * 2) -1; 
    int** intArray = new int*[size]; 
    for (int index = 0; index < size; index++) 
     intArray[index] = new int[max]; 

    int middle = size - 1; 
    int low = middle; 
    int high = middle; 

    for (int row = 0; row < size; row++) 
    { 
     for (int col = 0; col < max; col++) 
     { 
      intArray[row][col] = (col == low || col == high) ? 1 : 0; 
     } 
     low--; 
     high++; 
    } 
    return intArray; 
}