0

Techies-- このコードは、チャンネルが 1 つしかない場合、または均等に分割されている場合に機能します。残りがある場合はほとんど機能します...次のチャネルを作成し、アイテムを引き込みますが、範囲外になります。これらの行が問題の核心です。

    items_per_batch = batchcount / (int)channels; //
    subsets = batch.Split(items_per_batch);

items_per_batch は実際には、Split 拡張機能に、分割するアイテムの数に関する一般的な数のアイデアを与えるために使用されます。残りがある場合は、別のサブセット配列を作成するだけです。私が本当にする必要があるのは、アイテムの長さを追跡することです。私は試した：

    int idx2 = subset.GetLength(1)

ある時点で、その値を使用する forloop も範囲外になりました。誰にも提案はありますか？

    static void channelassign()
    {
        int THRESHOLD = 2;
        string[] batch = new string[]
        { "item1", "item2", "item3", "item4","item5","item6","item7" };
        int batchcount = batch.Count();
        int items_per_batch;
        string[][] subsets;
        int idx1;
        int idx2;


        if (THRESHOLD != 0) //avoid accidental division by 0.
        {

            float channels = batchcount / THRESHOLD;
            if (channels < 1)
            {
                channels = 1; // only 1 channel is needed
                items_per_batch = batchcount; // process all items
                idx1 = 1; // fix value to a single channel
                idx2 = (batchcount - 1); // true start of array is 0
                subsets = batch.Split(batchcount); //splits correctly
            }
            else
            {
              // decide how many items will be included per batch
              channels =  (int)Math.Round(channels, 
                  MidpointRounding.ToEven); //determines channel number
              items_per_batch = batchcount / (int)channels; //
              subsets = batch.Split(items_per_batch); 
              idx1 = subsets.GetLength(0); // gets channel# assigned by split
              // idx2 = subsets.GetLength(1); // gets items back from splits

            }

            //distribute contents of batch amongst channels


            for (int channel = 0; channel < idx1; channel++)
            {
                for (int i = 0; i < items_per_batch; i++)
                {
                    Console.WriteLine(" Channel:" + channel.ToString() + " 
                       ItemName: {0} ", subsets[channel][i]);
                }
            }


        }
        else
        {
            Console.WriteLine("Threshold value set to zero. This 
               is an invalid value. Please set THRESHOLD.");
        }
4

1 に答える 1

2

分裂

float channels = batchcount / THRESHOLD;

はints に対して実行されるため、float channels常に次の整数値になります。

floor(batchcount / THRESHOLD)

しかし、それはあなたの問題の原因ではありません。

for (int channel = 0; channel < idx1; channel++)
{
    for (int i = 0; i < items_per_batch; i++)

batchcountがの倍数でない場合channels、一部のチャンネルのアイテム数は より少なくなりitems_per_batchます。したがって、内側のループは、subsets[channel][i]存在しない a にアクセスしようとします。

于 2012-07-31T18:09:40.673 に答える