Programming standards...no flaming please!
-
leckey wrote:
Unfortunately, it's my BOSS.
Then tread with care, but still indicate that a lot of refactoring is required in the interest of maintainability.
leckey wrote:
I am hoping to persuade said boss to let me rewrite this evil thing in C#.
I've done conversions of large projects to C# using automated tools, and it went pretty well. I wouldn't advocate a move of language on the basis that this would solve any problems, but if you're more comfortable with C# ( as I am ), then it's a good move to start with. It also has the benefit that perhaps the boss doesn't know C#, so he won't be able to tinker with it again.... Christian Graus - Microsoft MVP - C++ Metal Musings - Rex and my new metal blog
10K lines in one function? Heck, *I'll* come smack your boss. And if he gives you the "what is wrong with the code" look/comment, it is a *BAD* sign. :) Btw, I'll second the recommendation on Code Complete. I do not agree with everything he says, but I would categorize the book as an excellent source of how to code. It should be required reading for every programmer. What I cannot agree with is the relatively arbitrary 30 lines/function rule (give or take). Some concepts that code implements don't make sense broken up - the multiple functions don't help the clarity. And clarity is the goal. Charlie Gilley Will program for food... Whoever said children were cheaper by the dozen... lied. My son's PDA is an M249 SAW.
-
led mike wrote:
hehehehe if I don't stop laughing soon I will need to got to hospital
I am glad I've had a few :beer: so far :)
-
10K lines in one function? Heck, *I'll* come smack your boss. And if he gives you the "what is wrong with the code" look/comment, it is a *BAD* sign. :) Btw, I'll second the recommendation on Code Complete. I do not agree with everything he says, but I would categorize the book as an excellent source of how to code. It should be required reading for every programmer. What I cannot agree with is the relatively arbitrary 30 lines/function rule (give or take). Some concepts that code implements don't make sense broken up - the multiple functions don't help the clarity. And clarity is the goal. Charlie Gilley Will program for food... Whoever said children were cheaper by the dozen... lied. My son's PDA is an M249 SAW.
charlieg wrote:
relatively arbitrary 30 lines/function rule
I tend to regard that as a good guide, but I would say people who convolute their code to reach that aim are doing as much harm as people who don't pay attention at all. I find it's a good guide, but I won't even make my code more complex in order to reach it. It's more a good starting point than an immutable law, IMO. Christian Graus - Microsoft MVP - C++ Metal Musings - Rex and my new metal blog
-
charlieg wrote:
relatively arbitrary 30 lines/function rule
I tend to regard that as a good guide, but I would say people who convolute their code to reach that aim are doing as much harm as people who don't pay attention at all. I find it's a good guide, but I won't even make my code more complex in order to reach it. It's more a good starting point than an immutable law, IMO. Christian Graus - Microsoft MVP - C++ Metal Musings - Rex and my new metal blog
-
I wasn't really sure if this belonged to a particular message board so please no flaming! I am wondering what others habits are when it comes to programming. I just inherited a huge VB.NET program. In one VB page it has over 10k lines of code. One function within that has about half those lines. Do you have a "limit" on how big/small a function/method/class should be before breaking it into smaller parts? We just used the analyzer on this thing and found one case statement has over 70 routes to go. So I'm hoping to clean this puppy up. BTW...heat index is currently at 109 degrees. Hope everyone else is staying cool.
Try to keep a function to one page. A guide, not a rule however. Elaine :rose: The tigress is here :-D
-
Christian Graus wrote:
The book that covers all of this is called Code Complete
4 cups of coffee: $3.25 Bag of M&M peanuts: $2.49 Surfing Code Project: Free Don Miguel's comment on Code Complete[^]: Priceless There are some things money can't buy, for everything else there's Mastercard :-D
led mike wrote:
Don Miguel's comment on Code Complete[^]: Priceless
Please explain what is good about that. You've posted this as being some high-quality post-of-the-year a few times, but I fail to see anything that stands out about it.
Ryan
"Punctuality is only a virtue for those who aren't smart enough to think of good excuses for being late" John Nichol "Point Of Impact"
-
I wasn't really sure if this belonged to a particular message board so please no flaming! I am wondering what others habits are when it comes to programming. I just inherited a huge VB.NET program. In one VB page it has over 10k lines of code. One function within that has about half those lines. Do you have a "limit" on how big/small a function/method/class should be before breaking it into smaller parts? We just used the analyzer on this thing and found one case statement has over 70 routes to go. So I'm hoping to clean this puppy up. BTW...heat index is currently at 109 degrees. Hope everyone else is staying cool.
109 wow. It's only 68 for me but that's the beauty of A/C. Technology should read SD someday. I just saw you were a code-chick (slang intended for humor) that's cool!:cool: So what's a leckey anyways? I'd explain code-frog but it would take about 32 years as of July 18th.:-D
When I'm joking people take me seriously.
When I'm serious they think I'm joking.
I'm left to conclude my life must be a complete joke. :sigh: :laugh: :cool: -
Ennis Ray Lynch, Jr. wrote:
Switch to c#
Amen brother! I hate VB.
Ennis Ray Lynch, Jr. wrote:
In business applications readability and maintainability and self-documentation is more important that the absolute fastest code [in most cases].
Like I said, it's ugly code. Not much commenting. Old code just commented out. And it is definately not that fast. If it were me I'd rewrite the entire thing.
I wonder why VB gets such a bad name? Your boss could have very easily written the same progam in C#. Last time I checked there was nothing in C# that prevents a function from being a certain number of lines or that limits the branching in a switch statement. Judging by some of the questions in C# forum, I wouldn't be surprised if you will soon see C# applications built like this. Remember not everyone that is using C# right now has a C or java background.;P Mike Lasseter
-
109 wow. It's only 68 for me but that's the beauty of A/C. Technology should read SD someday. I just saw you were a code-chick (slang intended for humor) that's cool!:cool: So what's a leckey anyways? I'd explain code-frog but it would take about 32 years as of July 18th.:-D
When I'm joking people take me seriously.
When I'm serious they think I'm joking.
I'm left to conclude my life must be a complete joke. :sigh: :laugh: :cool:I prefer Geek Girl, but Code Chick works. When I ran into other computer science students around town they would always be like, "Oh I remember you! I had class xyz with you!" I guess I was pretty memorable being the only white chick in class.:cool: leckey be part of my last name. I was one of those girls who hyphenated.
-
I wasn't really sure if this belonged to a particular message board so please no flaming! I am wondering what others habits are when it comes to programming. I just inherited a huge VB.NET program. In one VB page it has over 10k lines of code. One function within that has about half those lines. Do you have a "limit" on how big/small a function/method/class should be before breaking it into smaller parts? We just used the analyzer on this thing and found one case statement has over 70 routes to go. So I'm hoping to clean this puppy up. BTW...heat index is currently at 109 degrees. Hope everyone else is staying cool.
Hi, I assume you talk about a program that has to be completed and you are faced with a lot of code from others who have worked on this program before. I think that you (forget the whole program you have at this moment), should make clear what the function is of this program, what it should do, what results are expected (who will work with it) and then compare all of this with what you have when you hit F5. When this result is acceptable regarding what the software should perorm, and the overall design is ok, i would go deeper into the code, otherwise i would suggest dropping what you have right now. When you look to modern software (corporate software) 99.9 % of the expected (Visual Studio based) software is all about inserting, editing and updating or removing data in a database that runs on the Windows Operating System and can be accesed via your Windows or Internet application, or be shared over a network, and is presented to the users in a logical working relating way,so these operations can be performed. 0.1% (my quess) are difficult calculations and so on, that need special attention. When you have the main things ok i would concentrate on this 1% with your whole team, and see how far you can go in realising the task these functions have ro perform. Good luck with your project:) With friendly greetings, Eric Goedhart Skype: eric-goedhart Deep in the fundamental heart of mind and Universe there is a reason. -Slartibartfast
-
109 wow. It's only 68 for me but that's the beauty of A/C. Technology should read SD someday. I just saw you were a code-chick (slang intended for humor) that's cool!:cool: So what's a leckey anyways? I'd explain code-frog but it would take about 32 years as of July 18th.:-D
When I'm joking people take me seriously.
When I'm serious they think I'm joking.
I'm left to conclude my life must be a complete joke. :sigh: :laugh: :cool:Belated birthday wishes Rex. I should've remembered it, especially because my birthday is also July 18th :)
Regards Senthil _____________________________ My Blog | My Articles | WinMacro
-
I wasn't really sure if this belonged to a particular message board so please no flaming! I am wondering what others habits are when it comes to programming. I just inherited a huge VB.NET program. In one VB page it has over 10k lines of code. One function within that has about half those lines. Do you have a "limit" on how big/small a function/method/class should be before breaking it into smaller parts? We just used the analyzer on this thing and found one case statement has over 70 routes to go. So I'm hoping to clean this puppy up. BTW...heat index is currently at 109 degrees. Hope everyone else is staying cool.
The one time we had a strictly enforced limit was when I developed safety critical code - and then we put a limit on the McCabe Complexity metric[^] per sub-program (15, I think). This means that there is a strict limit on the number of decision points in the function/procedure, making understanding of said code item, easier, as well as making unit testing cheaper. The only times you could argue against the limit was when you had case statements on big enumerated types (for example, our scheduler used an enumerated type as a sort of task id and switched on that to dispatch scheduled functions).
-
leckey wrote:
I know...I know. It does interact with another 3rd party product that uses FORTRAN.
:omg:
Whats with the :omg: - FORTRAN is a *real* engineers language....it's actually pretty good for engineers who aren't really software people - the language is simple enough that they can write their code to do what they want...and then you can wrap the horrible Fortran in something else (Ada 95 in my case - don't ask). Of course, these days, they're more likely to design the thing in Simulink[^] and autocode it using Real Time Workshop[^]. WHich isn't necessarily any better for the software person trying to interface to their code...
-
led mike wrote:
Don Miguel's comment on Code Complete[^]: Priceless
Please explain what is good about that. You've posted this as being some high-quality post-of-the-year a few times, but I fail to see anything that stands out about it.
Ryan
"Punctuality is only a virtue for those who aren't smart enough to think of good excuses for being late" John Nichol "Point Of Impact"
Ryan Binns wrote:
I fail to see anything that stands out about it.
Sorry I guess you have no sense of humor. I sent the OP and Don's reply to as email to devs in my group and it kills them all. People have it pinned up to their walls along with various Dilbert strips. For me it is the funniest post so far in the Lounge (Category: Development Related) :-D
-
I wasn't really sure if this belonged to a particular message board so please no flaming! I am wondering what others habits are when it comes to programming. I just inherited a huge VB.NET program. In one VB page it has over 10k lines of code. One function within that has about half those lines. Do you have a "limit" on how big/small a function/method/class should be before breaking it into smaller parts? We just used the analyzer on this thing and found one case statement has over 70 routes to go. So I'm hoping to clean this puppy up. BTW...heat index is currently at 109 degrees. Hope everyone else is staying cool.
leckey wrote:
I just inherited a huge VB.NET program. In one VB page it has over 10k lines of code. One function within that has about half those lines. Do you have a "limit" on how big/small a function/method/class should be before breaking it into smaller parts? We just used the analyzer on this thing and found one case statement has over 70 routes to go.
Thats is quite common for a generated parser.**
xacc.ide-0.2.0 preview - Now in 100% C# goodness
**
-
I wasn't really sure if this belonged to a particular message board so please no flaming! I am wondering what others habits are when it comes to programming. I just inherited a huge VB.NET program. In one VB page it has over 10k lines of code. One function within that has about half those lines. Do you have a "limit" on how big/small a function/method/class should be before breaking it into smaller parts? We just used the analyzer on this thing and found one case statement has over 70 routes to go. So I'm hoping to clean this puppy up. BTW...heat index is currently at 109 degrees. Hope everyone else is staying cool.
"Things should be made as simple as possible, but no simpler." -- Albert Einstein My oppinion is that good developer should not ever directly think about the number of lines , number of classes, and other such things. He should develop a good, effective, usable classes. If some function, to accomplish it's function has to be long, let it be, but only while that function's function is a single strict logical unit. One should consider refactoring if he founds that a function does more than it should, not that it is just longer or shorter than ordinary.
-
I wasn't really sure if this belonged to a particular message board so please no flaming! I am wondering what others habits are when it comes to programming. I just inherited a huge VB.NET program. In one VB page it has over 10k lines of code. One function within that has about half those lines. Do you have a "limit" on how big/small a function/method/class should be before breaking it into smaller parts? We just used the analyzer on this thing and found one case statement has over 70 routes to go. So I'm hoping to clean this puppy up. BTW...heat index is currently at 109 degrees. Hope everyone else is staying cool.
You need this book: http://www.amazon.com/gp/product/0201485672/102-8688208-6140902?v=glance&n=283155 Desparately. I'm surprised no one has mentioned it yet. It is exactly what you need. Read it. Live it.
-
Ennis Ray Lynch, Jr. wrote:
Switch to c#
Amen brother! I hate VB.
Ennis Ray Lynch, Jr. wrote:
In business applications readability and maintainability and self-documentation is more important that the absolute fastest code [in most cases].
Like I said, it's ugly code. Not much commenting. Old code just commented out. And it is definately not that fast. If it were me I'd rewrite the entire thing.
-
Ennis Ray Lynch, Jr. wrote:
Switch to c#
Amen brother! I hate VB.
Ennis Ray Lynch, Jr. wrote:
In business applications readability and maintainability and self-documentation is more important that the absolute fastest code [in most cases].
Like I said, it's ugly code. Not much commenting. Old code just commented out. And it is definately not that fast. If it were me I'd rewrite the entire thing.
-
If you inherited it from someone within the company, go and smack them on the head. No code file should be 10k lines. Some functions may tend to be long, but they should be refactored where-ever possible, ideally I would say 30 lines is a max for a function. The book that covers all of this is called Code Complete. If you don't own it, buy it. While I found I knew most of what it was saying, I still found it exciting to see it all in written form, with logical explanations for the benefit of people like the clown who wrote the code you're now looking at. I am however cautious about refactoring existing code, do it if you can, but beware, lest you go crazy and spend heaps of time chasing an ideal that may introduce bugs in code that already works. Christian Graus - Microsoft MVP - C++ Metal Musings - Rex and my new metal blog
applications should be decomposed so that each method or function performs one action. at the top level of the application, the action may be "populate_database". this method would then call lower level methods to perform each of the actions to populate a database where each lower level method performs a single action. strict line limitations on methods or functions is arbitrary. although i do go by the "the method should fit on my screen" guide. as far as code file size, i don't believe there is a need to limit the file size arbitrarily. we limit a file to a single class. for some of our abstract base classes, the class is pretty big to cover all the base methods. working with methods or functions that perform multiple tasks is a nightmare. in a previous life, i had the truly unfortunate nightmare of having to work with a code file that was over 80000 (yes that's eighty thousand) lines long. one function within this file was over 12000 lines long. to add insult to injury, there were goto statements in the code (it was C code) that were used for regular program execution (not error recovery). to make it even worse (which is a challenge) there was a macro with a goto statement in the file!! decomposition of the application into single actions is the way to go. schleprock