|
I failed to mention that this application was developed over the past 4 years by this guy. He is 50+ years old and is a part-time ITT Tech teacher; a job that he was doing while he worked here and I think still is doing. I don't know what he teaches but I think he has a CS degree.
I don't know how many years experience he has in the industry. It could be that this was his first C++ app though. He obviously is or was a C programmer at some point in time.
This week I removed 20,000+ lines of redundant and unnecessary code from the program. CODE, not comments. And I am just getting started. I have to add functionality to it now.
I spend half my time writing new C# code and the other half maintaining the legacy stuff. It makes me cry every time I close Visual Studio 2008 and open Borland Studio 2006 .
|
|
|
|
|
Any chance of being able to pull the Borland libraries into VS2k8, or does the app take excess advantage of Borlands proprietary language extensions?
Today's lesson is brought to you by the word "niggardly". Remember kids, don't attribute to racism what can be explained by Scandinavian language roots.
-- Robert Royall
|
|
|
|
|
"He wasn't let go for incompetance."
Try explaining why all the stuff you laid out is incompetent to labour courts. You also find that most of these lecturers really don't have a clue of real-world stuff (most, not all). One of my lectures recently wrote a book on design patterns in C#: she made an entire chapter on the observer pattern. Yep, freaken observer pattern in C# - I'll stick to events thank you very much and save myself a few classes every time I want to handle a click event. And she was the head of department. And the book was about C# 3.0 (you would think they would catch a wiff of the multicast delegate by that stage!).
By the way, a lot of the more influential folks in CS are giving universities a wrap lately (don't have any specific references.) for pushing out incompetant graduates: I left my first university after getting 70% plus for all my CS subjects without ever attending classes or studying (guess where that lecturer lectured?).
He who asks a question is a fool for five minutes. He who does not ask a question remains a fool forever. [Chineese Proverb]
Jonathan C Dickinson (C# Software Engineer)
|
|
|
|
|
Wow! Now that is a real coding horror! Five.
To me it sounds like an in-house app. I would really consider migrating the existing database to standard XML format, to be able to use a standard parser.
maizhiming wrote: There isn't a single pass by reference in the entire application. It is all pointers.
Also, like the guy above, I do not get the big problem od refrences vs. pointers. Of course const references are the most proper way in many cases, but IMO using pointers is not a horror, compared to the rest of the stuff you mention.
_____________________________________
Action without thought is not action
Action without emotion is not life
|
|
|
|
|
LOL Well, I graduated from ITT Tech in Seattle, WA in 1997. Hopefully, the code you're talking about above did not come from anyone tought by my professor. He was very strict on proper code practice, and placing things where they truly belong. However, I seemed to be the only guy there, besides my professor, who seemed to get it, so your post doesn't surprise me in the slightest.
Kyosa Jamie Nordmeyer - Taekwondo Yi (2nd) Dan
Portland, Oregon, USA
|
|
|
|
|
At my last job I had to work a such a mess except it was in VB6. The original programmer was clueless. It had Global variables all over the place. There were no arguments passed to any methods ever! Everything was a variable in a larger scope. And this was the core software for the entire business. 100 people used this every day.
I remember one section of code that had to import a bunch of records. It read a file line by line and inserted the records into the table with SQL (of course it didn't reference the columns so you can never add any columns to the table). On top of that it execute a select count(*) from tbl after each insert to display the count on the screen. It would take forever and the users would be complaining about the slow speed of the program.
I started looking for another job on my first day of work because I got 40 emails about problems with the program. Fortunately I was only there for about 6 weeks. What's funny is that I purposely did not put VB6 anywhere on my resume, not did I mention it in the interview because I HATE programming VB6. Yet they expected me to work on a VB6 application.
I didn't get any requirements for the signature
|
|
|
|
|
I used to work at a place where many of those things were done and no one saw anything wrong with it.
I switched to a better job.
Bill W
|
|
|
|
|
|
Any database table with a 1,000 columns should be flagged for restructuring. That is indeed a horror.
"The clue train passed his station without stopping." - John Simmons / outlaw programmer
"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon
|
|
|
|
|
Any database table with a 1,000 columns should be flagged for restructuring deletion.
Simon
|
|
|
|
|
Yes, and also terminate the database architect who came up with the idea.
"The clue train passed his station without stopping." - John Simmons / outlaw programmer
"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon
|
|
|
|
|
This is surely worse than the database of 991 tables that I am looking at....
|
|
|
|
|
darkelv wrote: the database of 991 tables that I am looking at....
Wow, there is also a summary table, I suppose.
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke
[My articles]
|
|
|
|
|
In my system, each participant is a row, and each response is a column.
I think he confuses relational databases with excel sheets
modified 12-Sep-18 21:01pm.
|
|
|
|
|
What's the difference?
|
|
|
|
|
Excel spreadsheets are prettier. Eventually, they'll upgrade their Oracle database to Excel.
|
|
|
|
|
hahaha !! Good 1 . My comment makes no sense but I just couldnt stop laughing lmao
|
|
|
|
|
was the title of this supposed to be a play on the old bgates joke of only ever needing 640k
|
|
|
|
|
neilmajithia wrote: was the title of this supposed to be a play on the old bgates joke of only ever needing 640
No, the title was a play on the Gates thing.
|
|
|
|
|
if i ever come across a table with sover 4000 columns id pack my desk, leave by the nearest fire exit and email in my resigation when i got home.
|
|
|
|
|
I felt dirty hitting Access's 255 column limit.
|
|
|
|
|
(again on the Gates thing...)
Well, 255 columns should be enough for anyone!
To hell with circumstances; I create opportunities.
|
|
|
|
|
TRCH (The Real Coding Horror) is in my opinion the database design.
As comparison... For performance reasons we have a separate database intended for read-access, to which we copy the data in a less relational manner. The result is a few flat tables with a horrible number of columns. Still the worst one only has 160 columns...
Despite everything, the person most likely to be fooling you next is yourself.
|
|
|
|
|
I am sorry to tell you that limitations like this exist in many places.
For example you can not display this data since DataGrids have a limitation of 65000 pixel width.
As you can see I marked this as news
Natza Mitzi
|
|
|
|
|