Do you want CardPrinted renamed into PaperCard?
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins
6 posts
• Page 1 of 1
Do you want CardPrinted renamed into PaperCard?
by Max mtg » 25 Feb 2013, 07:33
Recently I made IPaperCard interface, and like the idea so much that would love to rename all out-of-game cards and tokens to Paper*.
Would you call this a good idea?
Would you call this a good idea?
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: Do you want CardPrinted renamed into PaperCard?
by moomarc » 25 Feb 2013, 17:46
What is the purpose of renaming them all? Is it just because you find it would be easier to interperet the usage in the code, or is there another reason? If it's for easier code reading, I don't mind either way.Max mtg wrote:Recently I made IPaperCard interface, and like the idea so much that would love to rename all out-of-game cards and tokens to Paper*.
Would you call this a good idea?
-Marc
-
moomarc - Pixel Commander
- Posts: 2091
- Joined: 04 Jun 2010, 15:22
- Location: Johannesburg, South Africa
- Has thanked: 371 times
- Been thanked: 372 times
Re: Do you want CardPrinted renamed into PaperCard?
by Max mtg » 25 Feb 2013, 19:19
I think new names reflect closer the essence of the class.
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: Do you want CardPrinted renamed into PaperCard?
by friarsol » 25 Feb 2013, 19:21
I'd say OracleCard and OracleToken would reflect it even better, since the card on paper for older cards isn't necessarily the state of the Card officially.Max mtg wrote:I think new names reflect closer the essence of the class.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Do you want CardPrinted renamed into PaperCard?
by Max mtg » 25 Feb 2013, 19:31
No, they are not Oracle cards. ArtIndex, foil - aren't properties of oracle card. It is CardRules that might be named OracleCard, though it's current name seems good enough for now.friarsol wrote:I'd say OracleCard and OracleToken would reflect it even better, since the card on paper for older cards isn't necessarily the state of the Card officially.Max mtg wrote:I think new names reflect closer the essence of the class.
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: Do you want CardPrinted renamed into PaperCard?
by friarsol » 25 Feb 2013, 20:48
Ahh right, I always confuse those in my head for some reason. I'm pretty ambivalent here.Max mtg wrote:No, they are not Oracle cards. ArtIndex, foil - aren't properties of oracle card. It is CardRules that might be named OracleCard, though it's current name seems good enough for now.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
6 posts
• Page 1 of 1
Who is online
Users browsing this forum: No registered users and 50 guests