Self Teaching Help!

-=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- (c) WidthPadding Industries 1987 0\|742\|0 -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=-
Socoder -> C/C++/C#/Other -> Self Teaching Help!
Sun, 17 Apr 2011, 21:21
killa57	Hi im in need of some help i have a programming project from a book and cant figure it out? it asks to make a class named Card that represents like a standard playing card. And every card have a suit and a card number. and the program deals five random cards...... So im new to this how would i go about starting this ? I've created a class called Card i feel like im not even going about this right it has me confused even when i read the chapter more im like umm what? :S So does it look right so far? how would i define the card name and suits? in the program or the Class? This is the Class that i have so far using System; using System.Collections.Generic; using System.Linq; using System.Text; namespace standard_playing_card { public class Card { private int facevalue; const int Max1 = 13; const int Max2 = 4; Random generator; private String name; public Card(String suits) { name = suits; generator = new Random(DateTime.Now.Millisecond); facevalue = 1; } } } --v This is the program i have so far namespace standard_playing_card { class Program { static void Main(string[] args) { Card Random1 = new Card("Hearts"); Card Random2 = new Card("Dimaonds"); Card Random3 = new Card("Spades"); Card Random4 = new Card("Clubs"); } } } --v
Mon, 18 Apr 2011, 03:19
Jayenkai	When I used to deal with card things, the whole Suit/Value thing used to annoy me like MAD. I'm sure there's an easy way to deal with Suit/Value, but at some point you just have to give up! Instead, don't think of 4 small decks of cards. When you're playing with cards, you instead clump them all together, to make one big deck. Try it like that.. Instead of each card having 2 values (Suit, Value), give each card only one (card number). Stick with this single number, right up until the point that you need to know the Suit/Value, at which point it's just a case of division to work them out. Assuming Card = 0->51 Suit = floor(Card/13); // Will return a value 0->3 Value = (Card % 13); // Will return a value 0->13 Then simply draw/write whichever is associated at the time. By sticking with a full deck, as opposed to having 4 groups, you'll find it much easier to do card-like things with them. Shuffling is WAY easier like this! As for whether it's Class or Program, I'm not the world's greatest OOP user, but if Class is the theme of this lesson, then try to do as much as you possibly can within the class. The more that happens in the class, the better you'll learn to do things that way. -=-=- ''Load, Next List!''
Mon, 18 Apr 2011, 04:00
Afr0	There's nothing wrong with your program so far, at least. Except you should be consistent when naming variables, and there's no need to be using 'System.Linq'. Also don't call your argument 'suits'. Call it 'suit'. You're passing a string, not an array of strings! Be consistent, don't confuse yourself and anyone else reading your code! Edit: I also noticed you haven't defined an access level for your 'generator' variable. C# defaults to 'private' when you have not set an access level, but you should get used to setting access levels. It makes your code easier to understand and read, and will help you in the future (IIRC C++ (and other OOP languages) requires you to explicity set your access levels). -=-=- Afr0 Games Project Dollhouse on Github - Please fork!
Mon, 18 Apr 2011, 06:09
Evil Roy Ferguso	The lovely thing about coding is that you'll ask 10 people a question and get 23 different answers. For what it's worth: - Cards shouldn't randomly set their suit / value in the constructor -- this is kind of like if at a playing card factory, each card off the printing press had a random suit and value. Instead, you'll want to tell each card what suit / value it has when you create it. - After you've created all the cards, shuffle the list. A Knuth-Fischer-Yates shuffle is both effective and easy to implement. Creating the whole deck and shuffling is necessary because otherwise you could easily end up with more than one of the same card. - For the moment, I'd just go with an array lookup for the number ( static readonly string[] _valueNames = new[] { "Ace", "Deuce", ... "Queen", "King" } -- might get fancy later on. If your Suit is an Enum, just a ToString() will work for now. This is about what I arrived at: using System; using System.Collections.Generic; using System.Linq; namespace ConsoleApplication4 { public enum Suit { Hearts, Diamonds, Clubs, Spades } public class Card { private static readonly string[] ValueNames = new[] {"Ace", "Deuce", "Three", "Four", "Five", "Six", "Seven", "Eight", "Nine", "Ten", "Jack", "Queen", "King"}; public Suit Suit { get; private set; } public int Value { get; private set; } public Card(Suit suit, int value) { Suit = suit; Value = value; } public override string ToString() { return string.Format("{0} of {1}", ValueNames[Value], Suit); } } public class ListShuffler { private readonly Random _random; public ListShuffler() { _random = new Random(); } // This is a generic method -- if you don't want to deal with this yet, // you can replace the next line with: //public void ShuffleList(IList<Card> listToShuffle) public void ShuffleList<T>(IList<T> listToShuffle) { // Knuth-Fischer-Yates shuffle - loops from the back of the array to the front. // Items after the "i" have been shuffled, before the i are still unshuffled. // The value of list[i] is swapped with the value of one of the "unshuffled" // cards before i is decremented. When i = 0, all of the cards have been shuffled. for (int i = listToShuffle.Count - 1; i > 0; i--) { int swapWithIndex = _random.Next(i + 1); T temp = listToShuffle[i]; listToShuffle[i] = listToShuffle[swapWithIndex]; listToShuffle[swapWithIndex] = temp; } } } class Program { static void Main(string[] args) { // Create the deck using a LINQ "select many" operation. // This could alternately be written as nested for loops over the suits and values, // inside of which cards were appended to a list. IList<Card> deck = (from suit in new[] {Suit.Hearts, Suit.Diamonds, Suit.Clubs, Suit.Spades} from value in Enumerable.Range(0, 13) select new Card(suit, value)).ToList(); var shuffler = new ListShuffler(); shuffler.ShuffleList(deck); // Take the first five cards from the shuffled list with LINQ's "Take". foreach (Card card in deck.Take(5)) { Console.WriteLine(card.ToString()); } Console.ReadLine(); } } } --v