• Toll-free  888-665-8637
  • International  +1 717-220-0012
Welcome Guest! To enable all features please Login or Register.

Notification

Icon
Error

CorneliuTusnea
#1 Posted : Wednesday, November 8, 2006 7:57:35 AM(UTC)
CorneliuTusnea

Rank: Member

Joined: 8/17/2006(UTC)
Posts: 681

Hi,


Could you guys please remove (or at least not add new) duplicated code?



Move it to utility classes as shared functions but don't Copy&Pasted it around as it makes it very hard to customize the code.

Few examples:

- Force Image Size code (both old & new code)

- AddressIsValid, RegionIsRequired & the rest of ***IsRquired functions.

(by new code I mean code that was added in SP1)



I have my own Force Image Size code and it's quite painfull to replace it every time :)



Thanks,

Corneliu.
http://www.bestgames.com.au
http://www.bestchess.com.au



BV Product Links, Details and Signatures: Improve your customer experience:

http://www.acorns.com.au/projects/bv/quicklink/

jetheredge
#2 Posted : Wednesday, November 8, 2006 10:28:20 AM(UTC)
jetheredge

Rank: Member

Joined: 3/1/2006(UTC)
Posts: 1,142

Sorry, I meant to go back through and clean that up before release. In fact, I'm going to add that to my list to clean up before SP1 final. So much code, so little time.
Justin Etheredge
Senior Software Engineer
BVSoftware
Forum Jump  
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.

©2025 Develisys. All rights reserved.
  • Toll-free  888-665-8637
  • International  +1 717-220-0012