私の配列のコーディングに何が問題なのですか？

Question

これは、各しかし、それがコンパイルされないバック0に戻される第6及び第13のスロットを除いて、その中に4を有していなければならない、私は14個のスロットを有するアレイのために書かれたコードです。私がここで間違ったことを誰かが知っていますか？

using namespace std; 
#include <iostream> 

const int MAX = 14; 

int main() 
{ 
    void printArray(); 

    system ("pause"); 
    return 0; 
} 

void startArray (int beadArray[MAX]) 
{ 
    for(int i=0; i<MAX; i++) 
    { 
     beadArray[i]=4; 
    } 
    beadArray[6]=0; 
    beadArray[13]=0; 
} 

//**********************************************// 

void printArray() 
{ 
    startArray (int beadArray[MAX]); 
    for(int i=0; i<MAX; i++) 
    { 
     cout<<i; 
    } 
} 

Answer 1

いくつかの修正ミス：あなたはoutsiteそれを使用したい場合は

• はstartArray（）関数呼び出しの外で、あなたの配列を宣言します。
• あなたは
• coutの< < beadArray [i]の代わりに、coutの< <のI

それを変更したい場合は、参照として配列を渡します。

using namespace std;  
#include <iostream> 

const int MAX = 14; 

int main()  
{ 
    void printArray(); 
    system ("pause"); 
    return 0; 
} 

void startArray (int &beadArray[MAX]) 
{ 
    for(int i=0; i<MAX; i++) 
     beadArray[i]=4; 
    beadArray[6]=0; 
    beadArray[13]=0; 
} 

//**********************************************// 

void printArray() 
{ 
    int beadArray[MAX]; 
    startArray (beadArray); 

    for(int i=0; i<MAX; i++) 
     cout<<beadArray[i]; 
} 

Answer 2

startArray (int beadArray[MAX]); 

あなたはbeadArrayを宣言し、ワンステップでそれを使用しようとしています。あなたはそれを使用する前にそれを宣言する必要があります

int beadArray[MAX]; 
startArray (beadArray); 

をまた他の問題の多くを持っている：<iostream>は#includeがまだdをされていないため

• using namespace std;は効果がありません。グローバルusing namespace std;も使用しないでください。

• system ("PAUSE");を置き換える必要があります。私は個人的に使用します。

  cin.syncを（）; cin.get（）;

• main()のコンパイラは、関数についてはわかりません。 main()の前に、プロトタイプを配置する必要があります。

  void printArray（）; void startArray（int []）;

• main()void printArray();と言っています。関数を呼び出すときは、関数名と引数を使用します。

  printArray（）;

• printArray()の場合、beadArray [i]の代わりにiを出力しています。スペーシングもありません。

• グローバル定数は、使用する悪いことです。

私はこのようなルックスを持っていた固定コード：

#include <iostream> 

const int MAX = 14; 

void startArray (int (&beadArray)[MAX]); 
void printArray(); 

int main() 
{ 
    printArray(); 

    std::cout << "\n\nPress enter to continue..."; 
    std::cin.sync(); 
    std::cin.get(); 
    return 0; 
} 


void startArray (int (&beadArray)[MAX]) 
{ 
    for(int i=0; i<MAX; ++i) 
     beadArray[i]=4; 

    beadArray[6]=0; 
    beadArray[13]=0; 
} 

void printArray() 
{ 
    int beadArray[MAX]; 
    startArray (beadArray); 

    for(int i=0; i<MAX; i++) 
     std::cout << beadArray[i] << ' '; 
} 

私は一定のままでしたが、あなたはそれを置き換えるために行うことができますがたくさんあります。