お土産を分配するためのクラス

Question

お土産を人に分配するプログラム内で、お土産の数を計算するクラスを作りました。お土産を人数で割ったときの余剰分を、リストの上にいる人から一つずつわけていくことを意図しています。（あるいは、トランプを一枚ずつ配るときのように、全員にお土産を１つずつ順番に配っていき、お土産がなくなるまで続ける方法も可です。）

しかし、せっかく分配のためのコードを書いたものの、return値があるため、最後のdivision + 1が適応されて、すべての人に同じ数でtotalSouvenirsの数を超えたかたちになってしまいます。なにかよい方法はないものでしょうか。もしくは、intでなくて、voidにしたらうまく行くでしょうか。解決法を探しています。

class SouvenirDistribution
    {
        
        public int Distribute(int totalSouvenirs, List<Person> p)
        {

            int numberOfPpl = p.Count;
            int division = totalSouvenirs / numberOfPpl;
            int remainder = totalSouvenirs % numberOfPpl;
            int result = 0;
            int i = 0;


            if (remainder == 0)
            {
                for (i = 0; i < p.Count; i++)
                {
                    result = division;

                }

            }


            else if(i<remainder)
            {
                for (i = remainder; i < p.Count; i++)
                {
                    result = division;

                }
                for (i = 0; i < remainder; i++)
                {
                    result = division + 1;
                }

            }
           
            return result;
           
        }
    }
}

メインウィンドウ内のDistributeメソッド適用箇所です↓BtnDivide_Clickは、ボタンにより分配が実行されます。

//nameとageプロパティが入っています↓
List<Person> names = new List<Person>();

    private void BtnDivide_Click(object sender, RoutedEventArgs e)
        {
            SouvenirDistribution sd = new SouvenirDistribution();
            int totalSouvenirs = int.Parse(TxtTotalSouvenirs.Text);
          
            int i = 0;
            int numberOfSouvenirs = sd.Distribute(totalSouvenirs,names);
            for(i = 0; i < names.Count; i++)
            {  
              　//Personクラス内のToString()メソッド内の内容で構成された人名リストに、お土産の分配数が追加される形です。
                NameList.Items[i] += $"  {numberOfSouvenirs} 個";
               
            }

こちらが、Personクラスです。

 public class Person
     {
        
        //プロパティ
        public int Age { get; set; }
        public string FirstName { get; set; }
        public string LastName { get; set; }
        public string Name { get; set; }
               
        //コンストラクタ
        public Person(string name, int age) 
        {
            Age = age;            
            Name = name;
        }

        // 名前リストに表示される内容　「名前,年齢才」
        public override string ToString()
        {
            return  Name + ", " + Age + "才";

        }

     }

Comments

[takabosoft]

Distribute関数は何を引数に取って、どんな処理をして、何を返す関数なのでしょうか？

[papinianus]

仕様をより具体的に言語化するか、これを利用しているコードとPersonの定義の開示をするか、どちらかをご対応ください。

[YAmaGNZ]

タイトルを意味の分かるものにしてください

[irm]

ありがとうございます。追記対応しました。

Answer 1

こんなの如何でしょう？
肝はPersonを拡張してプロパティを増やすところかな？

public class Person
{
    public string Name { get; set; }
}

public class PersonSharing : Person
{
    public int Share { get; set; }
}

public void Distribute(int totalSouvenirs, List<PersonSharing> persons)
{
    var remaining = totalSouvenirs;

    persons.ForEach(p =>
    {
        p.Share = totalSouvenirs / persons.Count;
        remaining -= p.Share;
    });

    foreach(var person in persons)
    {
        if (remaining < 1) break;

        person.Share ++;
        remaining--;
    }
}

Comments

[irm]

ありがとうございます。なるほど、こういう書き方があるのですね。Shareプロパティの作成もそうですが、数的な処理の仕方など、全体的な考え方がとても参考になりました。自分のコードに当てはめようとしていますが…。

Answer 2

ロジックを考えるヒントだけです。

すでに多くのコメントがされていますが、作成されたDistribute関数は何をしたいのでしょうか。一人当たりの分配数を返すものを想定しているのだと思いますが、端数がある場合、多い人と少ない人ができるわけですが、その情報はどうするのでしょう。

呼び出し側ではその情報を持ち合わせず知りえないわけですから、関数の考え方の方向を変える必要があります。

• あくまで(誰か一人の)分配数を返したい

引数でメンバーの誰かを指定できるようにする。

• 戻り値の仕様を変更してもいいなら

分配数のリストを返すようにする(papinianusさんの回答)

• 戻り値が無くてもいいなら

関数内で各メンバーの分配数を設定する(hihijijiさんの回答)

Comments

[irm]

ありがとうございます。そうですね、お土産を人数で割った商のいわゆる余りの扱いに困っていました。ゆっくり読ませていただきます。

Answer 3

そういう使い方がしたいなら、こうかな(関数の実装はともかく、int[]を返す)

public int[] Distribute(int totalSouvenirs, List<Person> p)
{
    int division = totalSouvenirs / p.Count;
    int remainder = totalSouvenirs % p.Count;
    return Enumerable.Repeat(division + 1, remainder).Concat(Enumerable.Repeat(division, p.Count - remainder)).ToArray();
}

で

int[] numberOfSouvenirs = sd.Distribute(totalSouvenirs,names);
for(i = 0; i < names.Count; i++)
{  
    //Personクラス内のToString()メソッド内の内容で構成された人名リストに、お土産の分配数が追加される形です。
    NameList.Items[i] += $"  {numberOfSouvenirs[i]} 個";
}

Comments

[irm]

こちらでできました^^ありがとうございます！intの配列的なものも思いつき質問前に試そうとしたのですが、いまいちうまくいかずにいました。return部分が参考になります。