請問這段程式碼所出現的問題原因為何？

低調的宅男

1. #include <iostream.h>
   
2. #include <vector>
   
3. using namespace std;
   
4. int main()
   
5. {   
   
6.     int x;
   
7.     const int j = x;
   
8.     int iarr[j];
   
9.     cout<<"輸入一個數：";
   
10.     cin>>x;
    
11.     for (int j=1;j<=x;j++)
    
12.     {
    
13.      iarr[j]=j;
    
14.      cout<<iarr[j]<<endl;
    
15.     }
    
16.     system ("pause");
    
17.     return 0;   
    
18. }

複製代碼

小弟用Dev C++寫得
在跑得時候出現一個問題，當我輸入一個數為12以上時會出現錯誤
而且視窗出現的只出現到12

EX：我輸入10
1
2
3
4
5
6
7
8
9
10

EX：我輸入15
1
2
3
4
5
6
7
8
9
10
11
12

出現錯誤
請各位大大解答...

瀏覽完整內容，請先 註冊 或 登入會員

kaworucloud

先看看你的 array 多大吧

1. cout << "Array size: " << ( sizeof( iarr ) / sizeof( int ) ) << endl;

複製代碼

因為未給x值

1.     int x;         //未給初始值的x
   
2.     const int j = x;     //j 的值未知
   
3.     int iarr[j];            //宣告一個不明大小的陣列

複製代碼

大概是x殘值是12吧…

ponchi96

1. #include <iostream.h>
   
2. #include <vector>
   
3. using namespace std;
   
4. int main()
   
5. {   
   
6.     int x;//顯示前一定會輸入x值 未設就算了
   
7.     int iarr[100];//直接宣告陣列大小
   
8.     //const int j = x;  int iarr[j]; 這兩行我不太懂意義是什麼
   
9.     cout<<"輸入一個數：";
   
10.     cin>>x;
    
11.     for (int j=1;j<=x;j++)
    
12.     {
    
13.      iarr[j]=j;
    
14.      cout<<iarr[j]<<endl;
    
15.     }
    
16.     system ("pause");
    
17.     return 0;   
    
18. }

複製代碼

算了,當作要省空間好了,輸入X後,宣告一個X大的陣列

1. #include <iostream.h>
   
2. #include <vector>
   
3. using namespace std;
   
4. int main()
   
5. {   
   
6.     int x;
   
7.     cout<<"輸入一個數：";
   
8.     cin>>x;
   
9.     int iarr[x];
   
10.     for (int j=1;j<=x;j++)
    
11.     {
    
12.      iarr[j]=j;
    
13.      cout<<iarr[j]<<endl;
    
14.     }
    
15.     system ("pause");
    
16.     return 0;   
    
17. }

複製代碼

...

瀏覽完整內容，請先 註冊 或 登入會員

goodbye_mylove

回復

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

低調的宅男

0. 你的書太老了

1. #include <iostream.h>
   
2. using namespace;

複製代碼

你的書可以拿去丟了

1. #include <iostream>
   
2. using namesapce std;

複製代碼

1. const int j 未給予明確數值

1.     int x;
   
2.     const int j = x;

複製代碼

你的 j 從頭到尾都會是綁定的 x，不過在 assing to j 時， x 卻是未知的，run time error

2. 變數建議、index error

1.     for (int j=1;j<=x;j++)
   
2.     {
   
3.          iarr[j]=j;
   
4.          cout<<iarr[j]<<endl;
   
5.     }

複製代碼

我覺得你一開始就已經有 const int j 了，
這裡面的 loop index 就不要用 j，
另外 index 應是從 0 到 x-1 ，而不是從 1 到 x

3. 你沒用到 vector

你有 include <vector> ，不過一直沒用到它，我假設你是要用 vector 方式去解這題，
簡單的方式大概會長這樣

1. #include <iostream>
   
2. #include <vector>
   
3. using namespace std;
   
4. int main()
   
5. {   
   
6.     int x;
   
7.     vector<int> iarr;
   
8.     cout<<"輸入一個數：";
   
9.     cin>>x;
   
10.     iarr.resize(x); // 重設 iarr 大小   
    
11.    
    
12.     for (int j=0;j<x;j++)
    
13.     {
    
14.         iarr[j]=j;
    
15.         cout<<iarr[j]<<endl;
    
16.     }
    
17.     system ("pause");
    
18.     return 0;   
    
19. }

複製代碼

...

瀏覽完整內容，請先 註冊 或 登入會員

goodbye_mylove

回復

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

ponchi96

1.     cin>>x;
   
2.     int iarr[x];

複製代碼

我很好奇你和原 po 的 compiler 是哪一套？
這段碼可以過讓我有點意外。
standard C/C++ 應不支援 VAL 吧？...

瀏覽完整內容，請先 註冊 或 登入會員

低調的宅男

回復

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

ponchi96

感謝~
您第二個程式碼正式我想要得那種

我找到問題了

1. #include <iostream>
   
2. using namespace std;
   
3. int main()
   
4. {   
   
5.     int x;
   
6.     cout<<"輸入一個數：";
   
7.     cin>>x;
   
8.     int iarr[x];   //我這行程式碼放在cout<<"輸入一個數：";前面，導致它無法出現全部的數字
   
9.     for (int j=1;j<=x;j++)
   
10.     {
    
11.      iarr[j]=j;
    
12.      cout<<iarr[j]<<endl;
    
13.     }
    
14.     /*vector<int> ivector (iarr, iarr+5);
    
15.     for (int i=0;i<ivector.size();i++)
    
16.     {
    
17.      cout<<ivector[i]<<" ";
    
18.     }
    
19.     cout<<endl;
    
20.     */
    
21.     system ("pause");
    
22.     return 0;   
    
23. }

複製代碼

vevtor那段我後來發現還真沒用到......

瀏覽完整內容，請先 註冊 或 登入會員

ponchi96

回復

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

goodbye_mylove


我是用devc++ compile
原諒我見識短淺= =
standard C/C++ 應不支援 VAL
我不太懂意思
是邏輯問題還是寫法有問題
他寫的應該是c++沒錯吧

goodbye_mylove

回復

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

ponchi96

VAL : Variable Array Length, 可變陣列長度。

C/C++ 在「執行期前」就必須確定陣列的大小，但剛的例子是

1. int x;
   
2. cin >> x;
   
3. int arr[x];

複製代碼

很明顯是在 「執行期當時」才可以確定陣列的大小，這似乎是 Compiler 實做出來的。具小弟不才所知，目前 VC, BCB 並不支援 VAL (GCC 不知道有沒有支援)

於是靜態陣列比較常用下列做法

1. #define N 20
   
2. const int x=10
   
3. 
   
4. int arr[N]; // 可過，define 是於「編譯期前」就代入的東西
   
5. int arr2[x]; // 可過， const 是於「編譯期」時的東西，「編譯期」的確是於「執行期」前

複製代碼

至於要執行期才決定陣列大小的話，就用 new/malloc 或是用 vector，
所以小弟才納悶您是用哪套 compiler...

瀏覽完整內容，請先 註冊 或 登入會員

ponchi96

回復

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

goodbye_mylove

正如你所說
我通常陣列也是先宣告大小

1. #     int x;
   
2. #     const int j = x;
   
3. #     int iarr[j];

複製代碼

只是我看沒宣告x值都可以過了
所以也就照著打了
想說他要省空間
結果竟然compile可以過XD...

瀏覽完整內容，請先 註冊 或 登入會員

kaworucloud

variable-length arrays 是 C99 的特性
Borland 跟 MS 主力都放在 C++，不支援 C99
這裡有列出支援 C99 特性的編釋器

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

goodbye_mylove

回復

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

kaworucloud


謝謝指正。