功能调用重复没有明显的原因...战舰AI故障

时间:2013-10-02 09:33:06

标签: c#

我在这里有点斗智斗勇...任何帮助都将非常感激:此代码适用于在此论坛上托管的战舰AI竞赛,然后被指定为A.I.的项目。班级

我的问题(除了在C#上相当无能)是在Get Shot功能中,它似乎反复调用了镜头策略功能,从而增加了stratlist的大小。 我不想只是解决我的问题,而是我想从概念上理解为什么会发生这种情况,以及如何避免它。任何其他风格的建议或提示也将非常受欢迎。对任何违反礼仪的行为提前道歉......这是我发布的第一个问题。

在第二次阅读时,这有点模糊,所以我会尝试清除它:

get shot函数声明一个名为shot的Point;然后,它通过从保留列表中获取随机值(50个值中)来为拍摄值分配值。它继续以几种方式评估这个镜头。下次我们来到获取拍摄功能并进入第二行时,我的列表已经增长了两倍,无论我怎样或在哪里,我都无法弄清楚原因。

以下是代码:

namespace Battleship
{
    using System;
    using System.Collections.ObjectModel;
    using System.Drawing;
    using System.Collections.Generic;
    using System.Linq;

    public class Potemkin : IBattleshipOpponent
    {
        public string Name { get { return "Potemkin"; } }
        public Version Version { get { return this.version; } }

        Random rand = new Random();
        Version version = new Version(1, 1);
        Size gameSize;
        bool shotstat = false;
        List<Point> stratlist = new List<Point>();
        List<Point> aimlist = new List<Point>();

        public void NewGame(Size size, TimeSpan timeSpan)
        {
            this.gameSize = size;
            shotstrategy();            
        }

        public void PlaceShips(ReadOnlyCollection<Ship> ships)
        {
            foreach (Ship s in ships)
            {
                s.Place(
                    new Point(
                        rand.Next(this.gameSize.Width),
                        rand.Next(this.gameSize.Height)),
                    (ShipOrientation)rand.Next(2));
            }
        }

        private void shotstrategy()
        { 
            for (int x = 0; x < gameSize.Width; x++)
                for(int y = 0; y < gameSize.Height; y++)
                    if ((x + y) % 2 == 0)
                    {
                        stratlist.Add(new Point(x, y));
                    }
        }

        public Point GetShot()
        {
            Point shot;
            shot = this.stratlist[rand.Next(stratlist.Count())];
            if (shotstat == true)
            {
                if (aimlist.Count == 0)
                {
                    fillaimlist(shot);
                }
                while (aimlist.Count > 0)
                {
                    shot = aimlist[0];
                    aimlist.RemoveAt(0);
                    return shot;
                }
            }

            return shot;

        }

        public void NewMatch(string opponent) { }
        public void OpponentShot(Point shot) { }
        public void fillaimlist(Point shot)
        {
            aimlist.Add(new Point(shot.X, shot.Y + 1));
            aimlist.Add(new Point(shot.X, shot.Y - 1));
            aimlist.Add(new Point(shot.X + 1, shot.Y));
            aimlist.Add(new Point(shot.X - 1, shot.Y));        
        }
        public void ShotHit(Point shot, bool sunk)
        {   
            if (!sunk)
            {
                shotstat = true;
            }
            else
            {
                shotstat = false;
            }  
        }
        public void ShotMiss(Point shot) { }
        public void GameWon() { }
        public void GameLost() { }
        public void MatchOver() { }
    }
}


}

2 个答案:

答案 0 :(得分:0)

您确定shotstrategy未被其他地方调用吗?因为GetShot似乎没有问题。代码中唯一可以增加列表大小的部分是shotstrategy函数。你需要仔细看看它的电话。

答案 1 :(得分:0)

GetShot()功能似乎没有增加stratlist。只有shotstrategy()函数会增加值。

你的问题可能是这一行

stratlist.Add(new Point(x, y));

每次新游戏开始时,您都会添加更多积分。例如,在8乘8网格中,您将有32个方格作为目标。在下一场比赛中;这将添加另外32个等等。

您可以通过重置stratlist功能中的NewGame值来解决此问题。

public void NewGame(Size size, TimeSpan timeSpan)
{
    stratlist = new List<Point>
    this.gameSize = size;
    shotstrategy();
}