C＃：比較List中的字符串 <string> 在單獨的List中的對象中使用字符串屬性 <Object> 使用Obj.getName（）

Question

我正在寫一個程序，我有一個卡片組（MyDeck對象），其中包含一個列表（卡類型列表），其中是我的卡（卡對象）。 還有一個雜志（雜志對象），其中包含待售卡片列表（類型字符串列表）。 我想把雜志中的卡片添加到我的套牌中，只要我還沒有卡片。 困難的部分是我的卡片是對象，而雜志中的那些是字符串，所以我必須調用mycard.getCardName（）來獲取我的卡片的名稱和雜志中的卡片我可以簡單地在列表中運行因為它們只是一個字符串列表。

這是我的代碼，問題是只有雜志中的前兩張卡被添加到我的套牌中。

也許我正在接近這個問題。

public static class MyDeck
{
    public static List<Card> Cards = new List<Card>();
    //Other Members
}

public class Card
{
    //associated memebers   
}

public class Magazine
{
    public List<string> MagCards = new List<string>();
    //Other Members
}


private void AddCardsToMyDeck(List<string> MagCards)
{
    //If I have no cards in my deck then add the first one on the magazine to my deck
    if(MyDeck.MyCards.Count == 0)
    {
       Card c = new Card(MagCards[0]);
       MyDeck.MyCards.Add(c);
    }

    //for each name of each card in the magazine       
    foreach(string MagCard in MagCards)
    {
      //for each card in my deck
      foreach (Card crd in MyDeck.MyCards)
      {
         //if the my card is in the magazine then don't add it, else add it
         if (MagCards.Contains(crd.getCardName()))
         {
            break;
         }
         else
         {
            Card cd = new Card(MagCard);
            MyDeck.MyCards.Add(cd);
         }
      }
    }
}


// NOTE: MyDeck.Cards is empty
Magazine.MagCards.Add("B. Ruth");
Magazine.MagCards.Add("M. Piazza");
Magazine.MagCards.Add("J. Robinson");
AddCardsToMyDeck(Magazine.MagCards);

//Only Ruth and Piazza are added

Answer 1

我認為你正在倒退一部分。 你的問題的文字說你想要將卡片從雜志中添加到你的套牌中，如果卡片還沒有放在你的套牌中，但是代碼會查看你牌組中的卡牌，如果他們不在雜志中，他們會嘗試做一些事情。 無論哪種方式，你都是通過編寫for循環來做到這一點：

private void AddCardsToMyDeck(List<string> MagCards)
{
    var newCards = MagCards.Except(MyDeck.MyCards.Select(c => c.getCardName()));
    MyDeck.MyCards.AddRange(newCards.Select(c => new Card(c)).ToArray());
}

Answer 2

我不確定我理解你的問題是什么，但是為了解決你提出的問題，你可以使用linq將列表首先轉換為名稱列表。

  foreach (string crd in from c in MyDeck.MyCards select c.getCardName() )
  {
     if ( MagCards.Contains( crd ) )
     {
        break;
     }
     else
     {
        Card cd = new Card(MagCard);
        MyDeck.MyCards.Add(cd);
     }
  }

作為一個設計問題，Joel Coehoorn為此提出了一個很好的答案。

Answer 3

// get card names in your deck
var deckCardNames = MyDeck.Cards.Select( c => c.getCardName() );
// get card names in magazine cards that are not in your deck
var missingMagCardNames = MagCards.Except( deckCardNames );
// instantiate card objects for each missing card name
var missingCards = missingMagCardNames.Select(mc => new Card(mc));
// add missing cards to your deck
MyDeck.Cards.AddRange( missingCards );

// whole thing inline
MyDeck.Cards.AddRange( MagCards.Except( MyDeck.Cards.Select( c => c.getCardName() ) ).Select (mc => new Card(mc) ) );