我如何在这里获得超出范围的例外?

时间:2016-08-02 17:55:07

标签: c# .net algorithm data-structures

一直在看这个并且无法弄明白。正如下面的评论所指出的那样,有关指数列在清单中。

using System;
using System.IO;
using System.Linq;
using System.Collections.Generic;

public class Mine
{
    public int Distance { get; set; }
    public int Gold { get; set; }
}

public class Move
{
    public int SourceIndex { get; set; }
    public int DestinationIndex { get; set; }
    public int Cost { get; set; }   
}

public class Program
{
    public static void Main()
    {

        var mines = new List<Mine> () {
            new Mine() { Distance = 10, Gold = 1 },
            new Mine() { Distance = 20, Gold = 2 },
            new Mine() { Distance = 25, Gold = 1 }
        };

        // Cost of consolidating the gold from mines[i1] to mines[i2]
        Func<int,int,int> Cost = (i1, i2) => Math.Abs(mines[i1].Distance - mines[i2].Distance) * mines[i1].Gold;

        // Number of mines to consolidate the gold into 
        int k = 1;


        var bestMove = new Move() { SourceIndex = -1, DestinationIndex = -1, Cost = Int32.MaxValue }; 

        // total cost
        int sum = 0;

        while(mines.Count != k)
        {
            var indices = Enumerable.Range(0, mines.Count).ToArray();
            for(int i = 0, j = 1; j < indices.Length; ++i, ++j)
            {
                int cost_ij = Cost(i,j);
                if(cost_ij < bestMove.Cost)
                {
                    bestMove.SourceIndex = i; 
                    bestMove.DestinationIndex = j; 
                    bestMove.Cost = cost_ij;
                }

                int cost_ji = Cost(j,i);
                if(cost_ji < bestMove.Cost)
                {
                    bestMove.SourceIndex = j; 
                    bestMove.DestinationIndex = i; 
                    bestMove.Cost = cost_ji;
                }
            }
            Console.WriteLine("bestMove.SourceIndex = {0}, bestMove.DestinationIndex = {1}", bestMove.SourceIndex, bestMove.DestinationIndex); // prints "bestMove.SourceIndex = 2, bestMove.DestinationIndex = 1"
            sum += bestMove.Cost;
            mines[bestMove.DestinationIndex].Gold += mines[bestMove.SourceIndex].Gold; // this is throwing an exception "Index was out of range. Must be non-negative and less than the size of the collection."
            mines.RemoveAt(bestMove.SourceIndex); 

        }
        Console.WriteLine(sum);
    }
}

小提琴:https://dotnetfiddle.net/hYa3A0

这是没有道理的,因为当时

            Console.WriteLine("bestMove.SourceIndex = {0}, bestMove.DestinationIndex = {1}", bestMove.SourceIndex, bestMove.DestinationIndex); // prints "bestMove.SourceIndex = 2, bestMove.DestinationIndex = 1"
            sum += bestMove.Cost;
            mines[bestMove.DestinationIndex].Gold += mines[bestMove.SourceIndex].Gold; // this is throwing an exception "Index was out of range. Must be non-negative and less than the size of the collection."
            mines.RemoveAt(bestMove.SourceIndex); 

第一次运行

bestMove.DestinationIndex = 2
bestMove.DestinationIndex = 1

mines.Count = 3

也许我只是疯了。

2 个答案:

答案 0 :(得分:2)

指数是基于零的指数。

将for循环更改为:

for(int i = 0, j = 1; j < indices.Length-1; ++i, ++j)

答案 1 :(得分:0)

问题在于您的计数器j。 j在一个点被分配4,超过你的地雷集合的大小(3)。

将while循环中的第一行更改为:

var indices = Enumerable.Range(0, mines.Count - 1).ToArray();