|
<System.Runtime.Serialization.DataMember>
Private mLastUpdated As DateTime, _
mLastUpdatedBy As String, _
mClearanceRequired As Int64, _
mClearanceIsRequired As Boolean
Visual basic code; class members for serialization - declaring a group of variables as data members.
|
|
|
|
|
I would say it depends. But sure I would not go with this practice as it leads little confusion.
Thanks,
|
|
|
|
|
How is it confusing?
I would rather not write the datamember attribute for each member
|
|
|
|
|
I'm not commenting on whether the code is acceptable, but I am suggesting that posting this question in the Lounge might not be.
Cheers,
Peter
Software rusts. Simon Stephenson, ca 1994. So does this signature. me, 2012
|
|
|
|
|
For lazing about and discussing anything in a software developer's life that takes your fancy.
The Lounge is rated PG. If you're about to post something you wouldn't want your kid sister to read then don't post it. No flame wars, no abusive conduct and please don't post ads.
What do you think is a valid thing to post about in the code project?
|
|
|
|
|
As a general rule, explicit intent is better coding practice than implied intent.
|
|
|
|
|
I rather have white space, but agreed.
|
|
|
|
|
Please ask in the proper forum.
The report of my death was an exaggeration - Mark Twain
Simply Elegant Designs JimmyRopes Designs
I'm on-line therefore I am.
JimmyRopes
|
|
|
|
|
For lazing about and discussing anything in a software developer's life that takes your fancy.
The Lounge is rated PG. If you're about to post something you wouldn't want your kid sister to read then don't post it. No flame wars, no abusive conduct and please don't post ads.
This tickles my fancy.
|
|
|
|
|
Colborne_Greg wrote: This tickles my fancy.
You conveniently left out:
Quote: Technical discussions are welcome, but if you need specific help please use the programming forums.
The difficult we do right away...
...the impossible takes slightly longer.
|
|
|
|
|
I DO NOT NEED HELP
Quit behaving like you know what the fook is going on.
THE QUESTION IS - is this an acceptable practice, please learn to read English before getting on my case.
|
|
|
|
|
Your question will only get opinions for an answer. The answer all depends on who you ask.
I hate it because it isn't explicit. It requires you to know how the compiler is going to behave.
|
|
|
|
|
It is only one line of code, broken into multiple lines via the _ character.
<System.Runtime.Serialization.DataMember>
Private mLastUpdated As DateTime, mLastUpdatedBy As String, mClearanceRequired As Int64, mClearanceIsRequired As Boolean
|
|
|
|
|
So what?
It seems like you're doing this and your looking for absolution from the community. Again, it's a matter of opinion and in a real environment with coding standards, what you've done may be outlawed.
|
|
|
|
|
White space.
It's more important to people.
Also as a programmer if you do not understand how a compiler is going to handle your code, you might consider another profession.
|
|
|
|
|
Yes, it's a matter of knowing how the compiler is going to handle the code, but it's more about maintainability. The format of the code is not to please the compiler. It's there to convey, without a doubt, what you intended. It's better to be explicitly expressive than it is to assume functionality with a lack of expression. Think about what the next guy who has to maintain your code is going to think when he sees it.
Again, you're not looking for opinions. You're shooting down everything everyone says about what you wrote in an attempt to justify it.
|
|
|
|
|
You are not reading every comment.
Maintainability is a good reply.
|
|
|
|
|
Message Closed
modified 18-Aug-15 18:12pm.
|
|
|
|
|
|
More satisfying to post in an "improper" forum. Stuffed Shirt proper? 
The fredster
|
|
|
|
|
System.Runtime.Serialization sucks.
|
|
|
|
|
|
Whehe; the compiler accepts it - what trouble are you having with it, and what makes it unacceptable to you?
Bastard Programmer from Hell 
If you can't read my code, try converting it here[^]
|
|
|
|
|
I have yet to find a compiler that has to support and maintain the code it compiles.
|
|
|
|
|
Visual Studio 2013 for windows 8.1 apps
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
