2016-10-07 11 views
-2

私はこのコードを関数に入れる方法を理解しようとしていますので、私のメインで呼び出すことができますが、何も動作していません。私はプログラミングにかなり新しいです、事前に感謝! (目標はユーザーに数字を入力させ、最高のものを選ぶことです)。Function C++のユーザー入力配列

ループ "のために" これは二つの異なるべきであるように

for (z = 0; z < 10; z++) { 
    cin >> array[z]; 


    for (int i = 0; i < 10; i++) 
    { 
     if (array[i] > temp) 
      temp = array[i]; 
    } 

} 
+3

便利な問題の説明ではありません「何も作業していません」。また、この質問のコードは[mcve]の要件を満たしていません。そのような答えは不可能である。 –

+0

あなたはほとんど問題を説明していません。あなたはあなたの問題が何であるかを明示する必要があります。 – Carcigenicate

+0

親愛なるサム、私は関数に入れようとしましたが、関数の表記法でエラーが出るか、最も高い数値が計算されません。あなたの最後の基準では、変数を宣言してコードを拡張する必要があるとは思っていません。問題は、このコードを関数に入れると、このコードがうまく実行されないことです。悪い問題の説明を申し訳ありません。私の英語はうんざりです。速い返答をありがとう。 – Jeroen

答えて

1

は、それはあなたが、Z、TEMPおよび配列を宣言するのを忘れて見えます[]

EDIT:あなたは、配列を初期化した場合に忘れないでください[]

int z, i; 
int temp = 0; 
int array[10] = {0,0,0,0,0,0,0,0,0,0}; 

for (z = 0; z < 10; z++) { 
    cin >> array[z]; 
} 


for (i = 0; i < 10; i++) 
{ 
    if (array[i] > temp) 
     temp = array[i]; 
} 

番目:関数内であなたはこれを試すの集約

を含める必要がこの関数は引数を取りませんでしょうが、あなたはそれを宣言して、それを呼ぶだろう「機能()」

SECOND編集:私は実際に動作するように、このような関数を得ることができた、それは次のようになります。

をそこ
#include <iostream> 

using namespace std; 

void function(); 

int main() { 
    cout << "enter 10 numbers: " << endl; 

    function(); 

    return 0; 
} 

void function() { 
    int z, i; 
    int temp = 0; 
    int array[10] = {0,0,0,0,0,0,0,0,0,0}; 

    for (z = 0; z < 10; z++) { 
    cin >> array[z]; 
    } 


    for (i = 0; i < 10; i++) { 
     if (array[i] > temp) 
      temp = array[i]; 
    } 

    cout << "your largest number is: " << temp; 
} 

幸運男

+0

ああ私の神!これをありがとう!私の間違いは、配列を関数のパラメータに入れようとしたことでした。もう一度ありがとう。 – Jeroen

関連する問題