|
I saw an equivalnet at a client site ages ago where the main form had several hundred labels on the bottom, each with "Visible=False".
Turned out the dev was using them as "variables that have a change event" 
|
|
|
|
|
I came across something similar when I was hired at my current company. I was porting a VB.NET application to C# and noticed that the original developer was using DataGrids as business logic class members to hold data. I guess he wasn't familiar with the System.Data namespace.
|
|
|
|
|
Let me start with one of my own from my days when I either didn't understand or didn't trust VBScript
Function TestBit(Value, Bit)
TestBit = ((CLng(Value) And CLng(Bit)) > 0)
End Function
All I'll say is that I was young and foolish.
cheers,
Chris Maunder
CodeProject.com : C++ MVP
|
|
|
|
|
Chris Maunder wrote: didn't understand
Most likely since I've seen this code in C, C#, VB...
|
|
|
|
|
Chris Maunder wrote: didn't trust VBScript
Can hardly blame you there... damn context-sensitive "and"s...
----
It appears that everybody is under the impression that I approve of the documentation. You probably also blame Ken Burns for supporting slavery.
--Raymond Chen on MSDN
|
|
|
|
|
That reminds me of a previous boss who made us initialize all C++ pointers to null TWICE (just to make sure)
|
|
|
|
|
"pointer = null; pointer = null;"?
Well... very interesting. However, I suppose it doesn't make the problem solved.
I would rather use:
"while (pointer != null) pointer = null;"
|
|
|
|
|
When I was first learning C#, I had an attitude about it. I was hardcore C++ before that. Now, I love C#, but anyhow...
So, I was writing a unit test, and I needed to have every third item do something in a list. I forget what. I knew about Math., and I looked for Math.Mod. Obviously, it's not there, it's the % operator. Which I should have known. But, I was having a bad day, and, cursing C#, I wrote my own mod function.
A year or so later, this code was found by someone and sent to the entire company on our social mailing list. I knew right away what I'd done, and I'm glad it happened ( what I did was atypical at the time, but now I'd all the more remember it and dig deeper for obvious stuff that isn't where I first look for it ), but I sure did feel a goose when I saw it and realised that what I wrote was superfluous.
C# was my first major language change ( I'd done Python and other stuff, but always C++ at the core, C# was my first change of core language ). I suspect I was a case study for how NOT to approach such a major change. Hopefully the next time things change like that, I'll take it better.
I guess the other lesson to be learned out of this is the importance of code reviews, which this particular company does not do, to this day ( I floated it once or twice and got laughed to scorn ).
Christian Graus - Microsoft MVP - C++
Metal Musings - Rex and my new metal blog
"I am working on a project that will convert a FORTRAN code to corresponding C++ code.I am not aware of FORTRAN syntax" ( spotted in the C++/CLI forum )
|
|
|
|
|
Christian Graus wrote: the importance of code reviews, which this particular company does not do, to this day ( I floated it once or twice and got laughed to scorn ).
I've floated that idea and the response has been it would be wonderful to do, but we don't have the time.
|
|
|
|
|
Colin Angus Mackay wrote: I've floated that idea and the response has been it would be wonderful to do, but we don't have the time.
The response back should be:
How much time have we spent doing things over because they weren't done right the first time? A code review would have prevented that.
-----------------------------
In just two days, tomorrow will be yesterday.
|
|
|
|
|
Colin Angus Mackay wrote: ...but we don't have the time.
It's amazing how many companies find time to redo something, though.
"We don't have time to do it correct up front, but we have plenty of time to fix it on the backend."
"Approved Workmen Are Not Ashamed" - 2 Timothy 2:15
"Judge not by the eye but by the heart." - Native American Proverb
|
|
|
|
|
Code Complete has real-world statistics on which techniques spot the most defects. Code Reviews win (with the suggestion that just reading code by different people do the trick for defect finding). And empirical evidence that Increasing QA cost does usually not increase overall cost.
Developers, Developers, Developers, Developers, Developers, Developers, Velopers, Develprs, Developers!
We are a big screwed up dysfunctional psychotic happy family - some more screwed up, others more happy, but everybody's psychotic joint venture definition of CP
Linkify!|Fold With Us!
|
|
|
|
|
Having been through several code reviews over the past few years, they do help so long as the people involved in the review understand the objective of the code and its context. Sometimes, major problems are avoided by this process.
However, there will be one or more persons involved in the review who consider formatting more important than the functionality.
Phil
|
|
|
|
|
I heard peterchen used vbscript recently. 
"Great job, team. Head back to base for debriefing and cocktails."
(Spottswoode "Team America")
|
|
|
|
|
Well, I wanted first post on a forum at least once in life 
Developers, Developers, Developers, Developers, Developers, Developers, Velopers, Develprs, Developers!
We are a big screwed up dysfunctional psychotic happy family - some more screwed up, others more happy, but everybody's psychotic joint venture definition of CP
Linkify!|Fold With Us!
|
|
|
|
|
|
If just I knew what to post....
Oh yeah, I recently used VBScript. Don't ask, just pity me.
Developers, Developers, Developers, Developers, Developers, Developers, Velopers, Develprs, Developers!
We are a big screwed up dysfunctional psychotic happy family - some more screwed up, others more happy, but everybody's psychotic joint venture definition of CP
Linkify!|Fold With Us!
|
|
|
|
|
Great...Keep it up....
why not now use manuscript
Recently I heard a babyscript developed too... How i wish I'd one copy of it. 
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
