C++コンストラクタから関数への配列の受け渡し

Question

配列を含むコンストラクタを持つクラスがあり、この配列の各メンバをこのクラスの関数に渡して追加するにはどうすればよいですか？

コンストラクタの実装

person::person 
{ 
    double NoPerson[150]; //no of people 
    for(int j=0;j<=150;j++) 
    { 
    NoPerson[j]=1.00;//has to set to 1 pound 
    } 

    double person::aveWage(double NoPerson[]) 
    { 
    double total; 
    double NoPerson[150]; 

    using namespace std; 

    for(int i=0;i<=150 ;i++) 
    { 
     total=total+cashCus[i]; 
    } 
    cout<<"total cash for all customers "<<total <<endl; 
} 

は、誰もがこれを達成することができる方法を知っていますか？

Answer 1

私はあなたが正しいかどうか分かりませんが、コンストラクタで初期化したいクラス（NoPerson）の配列があり、そのクラスの他の関数から使用したいと思うようです。

その場合クラスの他の関数がそれにアクセスできないので、ローカル変数にしないでください。あなたはそれを関数のようにクラスのメンバー変数にします。

このような何か：

class Person 
{ 
public: 
    Person() 
     { 
      for (int i = 0; i < 150; i++) 
       NoPerson[i] = 1; 
     } 

    double total() 
     { 
      double total; 

      for (int i = 0; i < 150; i++) 
       total += NoPerson[i]; 

      return total; 
     } 

private: 
    // Variables and functions in the 'private' section can only be 
    // referenced by the function in this class. 

    int NoPerson[150]; // This is a member variable 
} 

あなたは上記の例でわかるように、変数NoPersonが機能でクラスずに定義されています。これにより、クラス内のすべての関数にアクセスできるようになります。

Answer 2

コードが壊れています。実際には、コード内に 'NoPerson'という3つの異なる配列があります。一つはコンストラクタのローカル、aveWageの引数はaveWageの中にローカルで宣言されたNoPerson配列です。さらに、何も配列を使用していないため、コンストラクタを終了するときにコンストラクタの配列が削除されます。 aveWageの引数としてNoPerson配列を使ってコンストラクタからaveWageを呼び出す場合は、これを行うことができますが、aveWageのローカル宣言はそれをマスクします。

Answer 3

本当に配列でなければなりませんか？そうであれば、それに沿ってサイズを渡す必要があります。 int add(int[] items, int size)。このチュートリアルでは、配列の基本について説明します。http://www.cplusplus.com/doc/tutorial/arrays/

しかし、配列は「プレーンなC」のものです。 C++を使用している場合、ほとんどの場合、vectorを使用する方がよいでしょう。

Answer 4

1）配列の境界が間違っています（< 150）。

2）C++ 98/03では、メンバ配列を初期化できません。 C++ 11では、ゼロ以外の150個の値を初期化するのは面倒ですが、person::person() : NoPerson{1., 1., 1.} { }（実際には150個）です。 （それはしかし、この問題は、適切な設計のリビジョンを生き残るとは考えにくいです。）

3）aveWageループはstd::accumulate（例えばstd::cout << "total = " << std::accumulate(cashCus, cashCus + 150, 0) << std::endl;で書くことができます。あなたはtotalを初期化することはありませんので、現在お使いのコードが壊れている。