How much time should be spent on the little things
-
I've always preferred the least spaces except after the comma: function_name(param1, param2) { }
"This perpetual motion machine she made is a joke. It just keeps going faster and faster. Lisa, get in here! In this house, we obey the laws of thermodynamics!" - Homer Simpson Web - Blog - RSS - Math - LinkedIn - BM
That's my style too - I use white space where it will aid legibility, but not to excess. :)
Anna :rose: Linting the day away :cool: Anna's Place | Tears and Laughter "If mushy peas are the food of the devil, the stotty cake is the frisbee of God"
-
I put out a programming style guide and the boss overruled a few things. For example I had call to a function as function_name( param1, param2 ); and my boss changed it to function_name (param1, param2); Honestly, to me it doesn't matter which way or even if both ways are used as long as an individual is consistent. However, I've written 8K SLOC in my style (before the guide came out) and now my boss wants me to "take however long it has to" to meet his style. BTW, no actual schedule relief is allowed and I'm supposed to be done this Monday. I can get the code working by COB Monday without his style guide changes to it. I may be another week of mind numbing work to change the spacing. So how much time should be spent doing the little things such as changing style? (just to let you know, the boss was going to put out the style guide until it was 4 months late then gave me the task) Thanks Joe Q
You spend as much time on it as the boss want you to, then you document what you were working on. When he asks you why you aren't writing any new code, you bring out your documents and say, "Here, that's why." A good way to fix the style of lots of code is to use Visual Studio's auto-format feature. You may have to play with the setting a bit to get the exact style you want, but after that, you can open all your files, hit Ctrl-K-D on each one, and yer done. Really, you shouldn't be spending any time on that kind of crap. Old code should be 'grandfathered in' when a new style rule comes out, and should not be fixed until it's re-written. Also, the change you highlighted is totally stupid, and still wrong. It still has an extra space in it. Who is this fascist anyway?
"Quality Software since 1983!"
http://www.smoothjazzy.com/ - see the "Programming" section for (freeware) JazzySiteMaps, a simple application to generate .Net and Google-style sitemaps! -
No visual studio here. It's National Instruments LabWindows CVI. I am doing that type of replace. Thanks
NI CVI has extensive capability to do what you want, including regular expressions and multiple files. Open any source file, then select Edit/Replace. In that window you can enter what you want to change, how you want to change it, what files you want to act on, etc. Hope this allows you more time with your family. Phil Shell
-
-
2010 is less than 3 years 2 months away. I'm already using my 2020 to-do list.
-
Joe Q wrote:
So how much time should be spent doing the little things such as changing style?
Is you're boss a dev or wants to pretend he's one (I've seen that before too). If he has no real reason other than he likes his style better, then welcome to micro management hell.
Jeremy Falcon A multithreaded, OpenGL-enabled application.[^]
-
Entirely superficial changes like whitespace...you shouldn't spend more than a few seconds changing from one style to another. Find a beautifier that does that for you?
It's frustrating being a genius and living the life of a moron!!!
Hockey wrote:
Find a beautifier that does that for you?
Do you have a suggestion? I've tried one on another (I forget the name)project. It was VERY flexable but hard to get the right set of attributes to make the code like the style guide (I never got it perfect)
-
You're using CVI? You have my sympathy. :rose: I used to work on some of Racal Instruments instrumentation runtime systems and virtual instrument drivers, BTW. It was interesting stuff, and far more capable in its field than anything CVI was ever capable of.
Anna :rose: Linting the day away :cool: Anna's Place | Tears and Laughter "If mushy peas are the food of the devil, the stotty cake is the frisbee of God"
-
You spend as much time on it as the boss want you to, then you document what you were working on. When he asks you why you aren't writing any new code, you bring out your documents and say, "Here, that's why." A good way to fix the style of lots of code is to use Visual Studio's auto-format feature. You may have to play with the setting a bit to get the exact style you want, but after that, you can open all your files, hit Ctrl-K-D on each one, and yer done. Really, you shouldn't be spending any time on that kind of crap. Old code should be 'grandfathered in' when a new style rule comes out, and should not be fixed until it's re-written. Also, the change you highlighted is totally stupid, and still wrong. It still has an extra space in it. Who is this fascist anyway?
"Quality Software since 1983!"
http://www.smoothjazzy.com/ - see the "Programming" section for (freeware) JazzySiteMaps, a simple application to generate .Net and Google-style sitemaps!Jasmine2501 wrote:
You spend as much time on it as the boss want you to, then you document what you were working on. When he asks you why you aren't writing any new code, you bring out your documents and say, "Here, that's why."
I've tried things like that before and I usually just get dinged for not completing everything. Around here, it seems many managers have no concept of time. I tell them I have 2 tasks and enough time to do one and ask which ONE should I do? The answer around here is BOTH. When I try to explain about the concept of linear time that doesn't expand or contract (except in meetings) they just get upset with me.
-
NI CVI has extensive capability to do what you want, including regular expressions and multiple files. Open any source file, then select Edit/Replace. In that window you can enter what you want to change, how you want to change it, what files you want to act on, etc. Hope this allows you more time with your family. Phil Shell
-
I think it might be that it's something he can control where there are many, many things on there he has no control over.
-
Jasmine2501 wrote:
You spend as much time on it as the boss want you to, then you document what you were working on. When he asks you why you aren't writing any new code, you bring out your documents and say, "Here, that's why."
I've tried things like that before and I usually just get dinged for not completing everything. Around here, it seems many managers have no concept of time. I tell them I have 2 tasks and enough time to do one and ask which ONE should I do? The answer around here is BOTH. When I try to explain about the concept of linear time that doesn't expand or contract (except in meetings) they just get upset with me.
Sounds like you need to make sure your resume is up to date. Programmers are in demand right now, you don't have to put up with that treatment.
"Quality Software since 1983!"
http://www.smoothjazzy.com/ - see the "Programming" section for (freeware) JazzySiteMaps, a simple application to generate .Net and Google-style sitemaps! -
Wow! most people have never heard of CVI! We're using a Racal switch and using there drivers. Most instruments have IVI drivers which are very complex and at times confusing.
-
I'm using CVI for 7 years too (we're developing instrument drivers, also for LabVIEW). BTW goto is used as replacement of try...catch statementm which you don't have in ANSI C. Every IVI instrument driver uses it.
jiri wrote:
BTW goto is used as replacement of try...catch statementm which you don't have in ANSI C. Every IVI instrument driver uses it.
I've been using CVI about 5 now and I've noticed the goto's in the error macro's. Also, when you compile a DLL it put's all your functions usable instead of just the one's you mark with __declspec(dllexport). It's a hassle to see 50 functions when only 2 are to be used by the user.
-
Joe Q wrote:
Then, in a meeting with the customer he announced I was the software lead.
Arf arf arf!!!! :laugh: :laugh: It's like everybody knows you're in charge, except ... you!
Like that scene from Apocalypse Now... Who's your commanding officer .... Aren't you?
-
jiri wrote:
BTW goto is used as replacement of try...catch statementm which you don't have in ANSI C. Every IVI instrument driver uses it.
I've been using CVI about 5 now and I've noticed the goto's in the error macro's. Also, when you compile a DLL it put's all your functions usable instead of just the one's you mark with __declspec(dllexport). It's a hassle to see 50 functions when only 2 are to be used by the user.
-
Go to Build->Target Setttings. There's a box called Exports. Change that to nothing. By default for DLL there's a header and then CVI exports everything what is in the header.
jiri wrote:
Go to Build->Target Setttings. There's a box called Exports. Change that to nothing. By default for DLL there's a header and then CVI exports everything what is in the header.
In CVI 8.0 in the Exports box there's a "Change..." When that button is selected it brings up a "DLL Export Options" dialog box. If you say to use the header file (include file symbols), in CVI 8.0 but not earlier version, it returns an error with the prototypes. NI's solution (and the only one that seemed to work) was to change it to symbols marked for export. This seems to export the ones NOT marked for export, too.
-
I put out a programming style guide and the boss overruled a few things. For example I had call to a function as function_name( param1, param2 ); and my boss changed it to function_name (param1, param2); Honestly, to me it doesn't matter which way or even if both ways are used as long as an individual is consistent. However, I've written 8K SLOC in my style (before the guide came out) and now my boss wants me to "take however long it has to" to meet his style. BTW, no actual schedule relief is allowed and I'm supposed to be done this Monday. I can get the code working by COB Monday without his style guide changes to it. I may be another week of mind numbing work to change the spacing. So how much time should be spent doing the little things such as changing style? (just to let you know, the boss was going to put out the style guide until it was 4 months late then gave me the task) Thanks Joe Q
All you have to do is search out the "( " and " )" and to a replace. Even with that many lines of code it wouldn't take more than a few hours. I say the boss is the boss and his style should be adhered to. There really isn't a debate is there? Are you asking whether you should ignore it? Good luck and check out Monster or Dice. ;)
What's in a sig? This statement is false. Build a bridge and get over it. ~ Chris Maunder