Correctement à l'aide de LINQ to shuffle un pont

J'essaie d'écrire un simple jeu de cartes. Dans un effort pour arriver à un bon brassage de l'algorithme, je suis tombé sur Jeff Atwood est post sur le code de l'Horreur.

Cependant Quand j'ai vue le contenu de l'objet après l'appel du Constructeur, ils ne sont pas mélangées.

Voici ma tentative d'utilisation de Jeff Solution:

class MainDeck : List<Card>
{
   public MainDeck()
    {
        this.Add(new Card(1, "Hearts"));
        this.Add(new Card(2, "Hearts"));
        this.Add(new Card(3, "Hearts"));
        ...

        this.OrderBy(a => Guid.NewGuid());
    }
}

voici le code de la Carte:

class Card
    {
        string suit;
        int value;

        public Card(int value, string suit)
        {
            this.value = value;
            this.suit = suit;
        }

        bool isFaceCard()
        {
            if (value >= 11 || value == 1)
                return true;
            else
                return false;
        }

        public override string ToString()
        {
            return (value +", " + suit);
        }
    }

Que dois-je modifier pour que le brassage de travail?

S'il vous plaît montrer le code de la Carte. Peut-être que c'est juste moi, mais sans que je ne vois pas comment nous pouvons vous aider.
Vous devez comprendre que OrderBy retourne une collection ordonnée - il n'a pas de tri en place. Je voudrais aussi valoir que Jeff est uniquement de transférer à partir d'une source de la pseudo-aléatoire à l'autre en utilisant une version modifiée de Fisher-Yates shuffle avec un décent source de l'aléatoire est plus propre qu'à l'aide de OrderBy, de l'OMI.
Code Ajouté. Toutefois, la question n'a rien à voir avec la Carte de la Classe, il a à faire avec n'importe quel type d'objet énumérable
Comment pourrais-je commande le contenu de mon objet par le Guid alors?
En utilisant le Guid pour le tri peut être une mauvaise idée. Voir Eric Lippert commentaire de une réponse différente que les états "Utilisation guid pour générer de l'unicité, de ne jamais l'aléatoire".

InformationsquelleAutor jth41 | 2013-10-05

c#linq list playing-cards

14

LINQ méthodes ne sont pas la mutation des collections existantes. Si cette déclaration ne fait rien du tout: this.OrderBy(a => Guid.NewGuid());
Aussi, je suis sûr que vous ne pouvez pas attribuer à this, donc, il faut soit ne pas hériter de List<T> (qui est bonne), ou de faire quelque chose comme ceci:
```
var sorted = this.OrderBy(a => Guid.NewGuid()).ToList();
this.Clear();
this.AddRange(sorted);
```
Aussi regarder ce DONC réponse, il est plus correct de brassage de l'algorithme.
- Où avez-vous la AddAll fonction?
- c'était une erreur (ne me souviens pas le nom de la méthode). Édité.
- à l'aide de votre solution, this se termine avec 0 éléments, après avoir tenté this.AddRange(sorted);
- Mis à jour.
- votre requête ne fonctionne pas sans ToList(), car il n'est pas exécutée jusqu'à ce que AddRange est appelé, de sorte que vous êtes l'exécution de la requête sur la liste vide (comme Clear a déjà été appelée). Vous devez toujours vous souvenir que les requêtes LINQ sont paresseux et différé.
- oups, vous attrapé moi! (Je ne sais ce) 🙂
- Trouvé un intéressant commentaire par Eric Lippert sur une réponse différente expliquant pourquoi vous ne devriez pas utiliser le Guid à trier: "il est légal pour les guid être générés de façon séquentielle à partir d'un premier élément aléatoire.... Utilisation guid pour générer de l'unicité, de ne jamais l'aléatoire" (édité par souci de concision, l'accent mis par moi).
- Je ne "utilisé" pour garder le code évidemment similaire pour le code en question. bien sûr, c'est la solution insuffisante.
InformationsquelleAutor Sarge Borsch
6

Essayer cette
```
 public void Shuffle()
 {
     Random r = new Random();
     this.Sort((x, y) => r.Next(-1, 1));
 }
```
En raison de Linq différés exécution de la ligne suivante n'est pas exécutée.
```
this.OrderBy(a => Guid.NewGuid());
```
Cela crée juste de la requête, mais jamais exécuté. Même s'il est exécuté, il ne changera pas votre collection.

N'oubliez pas de Linq est une manière d'interroger les données, pas les transformer.
- c'est pale
- Vous ne devriez pas utiliser des fonctions non-pures dans le Tri, parce que (par contrat) .Sort() est libre d'évaluer votre lambda plusieurs fois. Au lieu de cela, vous devez utiliser .Select() pour ajouter un nombre aléatoire dans un anonyme struct, et puis trier par qui.
InformationsquelleAutor Sriram Sakthivel

Utiliser cette méthode d'extension

public static IEnumerable<T> Randomize<T>(this IEnumerable<T> source)
{
    Random rnd = new Random();
    return source.OrderBy<T, int>((item) => rnd.Next());
}

InformationsquelleAutor Masoud Darvishian

1

Linq ne fonctionne pas par référence.

vous devriez obtenir le résultat dans une variable.
Créer la liste d'objets à l'intérieur de la classe au lieu d'hériter de la liste comme suit:
```
class MainDeck
{
   public List<Card> lstCards;
   public MainDeck()
    {
        this.lstCards.Add(new Card(1, "Hearts"));
        this.lstCards.Add(new Card(2, "Hearts"));
        this.lstCards.Add(new Card(3, "Hearts"));
        ...

        this.lstCards = this.lstCards.OrderBy(a => Guid.NewGuid()).ToList();
    }
}
```
Veuillez marquer comme réponse ou utile si cette réponse aide.
- this = this.OrderBy(a => Guid.NewGuid()); Sera cette compilation?
- J'ai écrit juste en dessous de ce que je n'ai pas la prétention d'être correct. MAIS c'réponses à la question pourquoi son code ne fonctionne pas. Je vais mettre à jour ma réponse. Merci de remarquer.
- Je sais que c'est pas intentionnellement, fait ou de la faute de frappe ou juste une idée, mais des débutants vais essayer et cela ne fonctionne pas. pour eux, c'est juste un contenu fallacieux. N'est-ce pas? et vous êtes absent .ToList() trop 🙂
InformationsquelleAutor Tauseef

Vous devez vous connecter pour publier un commentaire.