如何在班级中定义和访问列表-如何在纸牌游戏中访问玩家的手

Question

我意识到以前可能已经问过这个问题，但是我一直在寻找，虽然我在理论上理解我需要做的事情，但实际上使我真正起作用的方法却在暗示我。

我正在尝试通过一个项目来学习C＃和面向对象的编程概念。 这个特别的项目是复制顶级特朗普纸牌游戏。 我的主题是《权力的游戏》。

我的问题是，在初始化播放器1和计算机指针时，我很困惑。 我可以看到卡片从卡组中消失了，但是在玩家的手中却看不到它们（尽管当我执行断点/固定到源时。我可以看到cardinHand正在填充。

问题在于，游戏最终只拥有60张牌，而不是15张（创建Deck时加载的XML中的30行）。 您能检查一下我的代码，提供指向我哪里出错的指针。 我知道我需要传递卡片对象，我担心我只是缺少一些简单的东西。 我觉得这是很重要的部分，因为我想根据玩家的手来显示卡片形式。

这是GameStart方法

public static void startGame()
    {
    Player player1 = new Player();
    Player computer = new Player();

    var newdeck = new Deck();
    newdeck.Shuffle();

    while (newdeck.CountCards() != 0)
    {
        newdeck.dealCards();
    }

    MessageBox.Show("New Game Started");

这是我的甲板课

public class Deck
{

private List<Card> deckofCards = new List<Card>();


//indexer on the deck
// Define the indexer, which will allow client code 
// to use [] notation on the class instance itself. 

public Card this[int i]
{
    get
    {
        // This indexer is very simple, and just returns or sets 
        // the corresponding element from the internal array. 
        return deckofCards[i];
    }
}


public Deck()
{
    resetDeck();
}

public void resetDeck()
{

    XmlDocument xmldata = new XmlDocument();
    xmldata.Load(@"C:\GameofThronesXml.xml");
    XmlNamespaceManager nsmgr = new XmlNamespaceManager(xmldata.NameTable);

    XmlNodeList nodeList;
    XmlElement root = xmldata.DocumentElement;
    nodeList = xmldata.SelectNodes("//CharacterData/record", nsmgr);

    foreach (XmlElement data in nodeList)
    {
        Card singlecard = new Card();
        singlecard.CardID = int.Parse(data.SelectSingleNode("./CardID").InnerText);
        singlecard.Name = data.SelectSingleNode("./Name").InnerText;
        singlecard.Intelligence = int.Parse(data.SelectSingleNode("./Intelligence").InnerText);
        singlecard.Ruthlessness = int.Parse(data.SelectSingleNode("./Ruthlessness").InnerText);
        singlecard.Status = data.SelectSingleNode("./Status").InnerText;
        singlecard.Prestige = int.Parse(data.SelectSingleNode("./Prestige").InnerText);
        singlecard.FightingSkill = int.Parse(data.SelectSingleNode("./FightingSkill").InnerText);
        singlecard.AppearedIn = int.Parse(data.SelectSingleNode("./AppearedIn").InnerText);
        singlecard.Description = data.SelectSingleNode("./Description").InnerText;
        deckofCards.Add(singlecard);
    }

    //string path = @"C:\GameofThronesXml.xml";
    //XElement doc = XElement.Load(path);
    //deckofCards = (from items in doc.Descendants("CharacterData")
    //                select new Card(
    //                        int.Parse(items.Element("CardID").Value),
    //                        items.Element("Picture").Value,
    //                        items.Element("Name").Value,
    //                        int.Parse(items.Element("Intelligence").Value),
    //                        int.Parse(items.Element("Ruthlessness").Value),
    //                        items.Element("Status").Value,
    //                        int.Parse(items.Element("Prestige").Value),
    //                        int.Parse(items.Element("FightingSkill").Value),
    //                        int.Parse(items.Element("AppearedIn").Value),
    //                        items.Element("Description").Value)).ToList();
}

public void Shuffle()
{

    deckofCards = deckofCards.OrderBy(c => Guid.NewGuid())
                 .ToList();

}

public int CountCards()
{

    int number = deckofCards.Count();
    return number;

}

public Card dealCards()
{

    Hand hand = Player.hand;
    //Hand computerHand = Player.hand;

    if (this.deckofCards.Count == 0)
        throw new InvalidOperationException("There are no cards to deal.");

    Card card = deckofCards[0];

    hand.cardsinHand.Add(card);
    deckofCards.RemoveAt(0);
    return card;

 }
}

手班

public class Hand
  {

    /// <summary>
    /// The deck in the hand
    /// </summary>
    public List<Card> cardsinHand = new List<Card>();

    public int GetNumberofCards()
    {
        int count = cardsinHand.Count;
        return count;
    }

    public void AddCard(Card card)
    {
        cardsinHand.Add(card);
    }



}

玩家等级

public class Player
{

    public static Hand hand = new Hand();


}

tl; dr-想要有两名玩家级别的玩家， 在这种情况下，我可以访问发牌的数量， 每张15张 -我知道这与hand.cardsinhand.Count有关-计数-我只是无法访问它与定义的玩家类别。

解决方案 -

        Player Player1 = new Player();
        Player Computer = new Player();

        Deck newdeck = new Deck();
        Hand CompHand = new Hand();
        Hand PlyrHand = new Hand();

        newdeck.Shuffle();

        while (newdeck.CountCards() != 0)
        {
            //The Hand adds the return dealt cards to the respective lists.
            CompHand.AddCard(newdeck.dealCards());
            PlyrHand.AddCard(newdeck.dealCards());
        }

        MessageBox.Show("New Game Started");

        //Snoopy Dance - Each card has 15 cards in each hand.
        MessageBox.Show(CompHand.cardsinHand.Count().ToString());
        MessageBox.Show(PlyrHand.cardsinHand.Count().ToString());

Answer 1

startGame代码startGame继续处理，直到没有剩余牌为止：

while (newdeck.CountCards() != 0)
{
    newdeck.dealCards();
}

但是dealCards正在访问静态（即，由该类的所有实例共享） Player.hand ，因此所有卡都被添加到了其中。 （我想当您为两个玩家添加每张卡时您将获得60，而当其中一位玩家被注释掉时，您只有30。）

您不希望所有玩家共享同一手牌，因此Player.Hand更改Player.Hand静态（即删除static关键字）。 然后将您的两个初始化的玩家的手（ player1.hand和computer.hand ）作为参数dealCards到dealCards ：

public Card dealCards(Hand hand, Hand computerHand)
...

我认为关于哪些类负责做某事有一些整理工作（可能在游戏运行后发布到codereview.stackexchange.com），但是此更改应使您开始工作。