Level Extreme platform
Subscription
Corporate profile
Products & Services
Support
Legal
Français
Best way to re-factor
Message
From
14/02/2013 13:19:08
 
General information
Forum:
ASP.NET
Category:
Coding, syntax and commands
Environment versions
Environment:
C# 4.0
OS:
Windows 7
Network:
Windows 2003 Server
Database:
MS SQL Server
Miscellaneous
Thread ID:
01565967
Message ID:
01566041
Views:
56
>>Convention is properties start with UpperCaseLikeThis.
>>
>>What are the datatypes of the properties?
>>
>>>One more question - what is the common convention - have properties to start with upper case letter or lower case? E.g. should I use it exactly as
>
>Here is what I have now:
>
>
>/// <summary>
>        /// Class used to parse tcDetails into common properties (StartTime, EndTime, closeOperator, closeSalesPoint)
>        /// </summary>
>        private class DrawerDetails
>        {
>            public DateTime StartTime { get; private set; }
>            public DateTime EndTime { get; private set; }
>            public String CloseOperator { get; private set; }
>            public String CloseSalespoint { get; private set; }
>
>            public void ParseDetails(String cDetails, String currentOperator, String currentSalespoint)
>            {
>                CloseOperator = cDetails.ParseStringFromSqml("CLOSE_OP");
>                if (String.IsNullOrWhiteSpace(CloseOperator))
>                    CloseOperator = currentOperator;
>                CloseSalespoint = cDetails.ParseStringFromSqml("CLOSE_SP");
>                if (String.IsNullOrWhiteSpace(CloseSalespoint))
>                    CloseSalespoint = currentSalespoint;
>
>                StartTime = cDetails.ParseDateTimeFromSqml("START_TIME");
>                EndTime = cDetails.ParseDateTimeFromSqml("END_TIME");
>            }
>
>        }
>
>and one more question - when I pass this class as a parameter among other parameters - should this class parameter be first? Just want to follow the convention.


Conventionwise - Even Craig won't be able to answer this
Gregory
Previous
Next
Reply
Map
View

Click here to load this message in the networking platform