|
I think he should be informed of his mistake, encouragement might help him to overcome his deficiencies.
A.A.
asif
|
|
|
|
|
|
So what is wrong with it?
Use of private before a return declaration?
I'm unfamiliar with this syntax. In C++, private and public are used in the class declaration, not the function definitions.
|
|
|
|
|
eunderwo00 wrote: private integer inc(integer int_i)
{
integer int_j=int_i+1;
return int_j;
}
Ok this is an unnecessary function but what is the reason you let him go?
CleaKO
"I think you'll be okay here, they have a thin candy shell. 'Surprised you didn't know that.'" - Tommy (Tommy Boy)
"Fill it up again! Fill it up again! Once it hits your lips, it's so good!" - Frank the Tank (Old School)
|
|
|
|
|
I'm wondering if he wrote additional functions for the remaining operators.
Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007
|
|
|
|
|
I can only think that int_j should be explicitly assigned a value at declaration, before actual use in case int_i is passed unassigned, hence :
private integer inc(integer int_i)
{
integer int_j=0;
int_j=int_i+1;
return int_j;
}
|
|
|
|
|
sprint_sport wrote: I can only think that int_j should be explicitly assigned a value at declaration, before actual use in case int_i is passed unassigned, hence :
private integer inc(integer int_i)
{
integer int_j=0;
int_j=int_i+1;
return int_j;
}
Integers by default are 0 when declared just as booleans are false, strings are nothing, etc...
CleaKO
"I think you'll be okay here, they have a thin candy shell. 'Surprised you didn't know that.'" - Tommy (Tommy Boy)
"Fill it up again! Fill it up again! Once it hits your lips, it's so good!" - Frank the Tank (Old School)
|
|
|
|
|
It was about to become really costful, as he was paid on the number of lines he was writing. He even managed to write wrappers over each native Win32 API, even the ones he was not using, just to charge more... And above this, he planned to rewrite Windows in a VM !
That's an explanation, find more 
Kochise
In Code we trust !
|
|
|
|
|
|
Kochise wrote: he was paid on the number of lines he was writing
Oh, no! You're working in a place that measures code by the yard, and then complain that somebody fresh out of the egg is - well let's say a little uncertain in their grasp of fundamentals. Get a grip!
Frankly, I'd have fired others, starting (in the very first femtosecond) with the guy who instigated the fatuous "quantity rather than quality" regime, continuing with whoever did the hiring (since this job apparently called for an accurate, experienced programmer), and possibly with whoever did the firing (though that may have been for factors other than the predictable howlers from a coding tyro).
Kochise wrote: He even managed to write wrappers over each native Win32 API, even the ones he was not using
Did they work? That's moderately sophisticated for a kid straight out of school. This sounds to me like someone who, with a modicum of sympathetic mentoring obviously not available in your shop, could well turn into a real code warrior.
Oh, and what was the kid's supervisor doing while he did all this apparently unnecessary work? Think I'll add him to the firing line as well.
|
|
|
|
|
Regarding the language used by the programmer. I don't see anything bad with the code, at least he had the concept of private. Also this in reality doesn't speak so good from you and not from the "programmer right out from school".
Are many thing to take in consideration. He probably thought it will impress you doing this, or he didn't realize what he did. Also one of the obligation of seasoned Developers, is help the new generations to going the right direction.
Are thousands more other aspects that I take in consideration to say if someone it's or not a good programmer, than simple review his code.
|
|
|
|
|
GTuritto wrote: I don't see anything bad with the code,
Can you name just a single use for this function that isn't covered by the "+" operator?
Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007
|
|
|
|
|
None. It could be solved with ++ operator. But you will surprise to know how many Developers, (and some of them are really good) that I personally know that they never in their's life are used ++ and they had no idea about the ++ operator.
Now has I say before, if instead to get fuming and kick the guy out we drop our Ego has the greatest Developer's, we will realize that the idea even if it's stupid it's not bad. And before any one goes crazy here, let me explain my self. I have no idea in what context this method was, then I can't talk about this method it self.
If it was me first I will be say "What a F...", then I will have a chat with the guy asking why he did that, third I will lecture him why the method is bad, and showing to him the ++ operator, fourth I will see if we can improve the idea, or tell to the programmer to fix the little issue, fifth I will check again to see if he fixed in the right way. And last but not least, I will never, never laugh at someone in this way. All of us we did some stupid mistake like that.
Laugh at your self, not to a kid that just came out from school.
|
|
|
|
|
Not ++ it's just a +.
I now wonder why it wasn't written to take a second parameter to specify the size of the increment.
|
|
|
|
|
GTuritto wrote: It could be solved with ++ operator. But you will surprise to know how many Developers, (and some of them are really good) that I personally know that they never in their's life are used ++ and they had no idea about the ++ operator.
I, thankfully, am not one of them. I for one, as are most people, am completely satisfied with i=i+1 in the event that a ++ operator doesn't exist, in any language I use. I the consequences and performance benefits/costs of making a call to a function along with the optimizations available in todays compilers.
GTuritto wrote: If it was me first I will be say "What a F...", then I will have a chat with the guy asking why he did that, third I will lecture him why the method is bad, and showing to him the ++ operator, fourth I will see if we can improve the idea, or tell to the programmer to fix the little issue, fifth I will check again to see if he fixed in the right way.
Reading the original post, it's entirely possible that this DID occur, and he still doesn't work for the company. Maybe this person did all kinds of stupid things in his code, maybe not. We simply don't know. I know, it's no reason to be critical of it. But, we also don't know what kind of skill level assumption he was hired under either. If they were expecting higher quality work and he just couldn't deliver past high-school code, out the door he goes.
GTuritto wrote: All of us we did some stupid mistake like that.
There's a huge difference between making a stupid mistake like that and turning it in as production code. I for one, have never turned in a stupid mistake, no, let me change that, a stupid rationalisation like that! What on earth was this guy thinking when he wrote this? What's the rationale behind writing a function like this? Having said that, what's going through this guys head with the rest of the code he turned in, or will turn in in the future?
How much work are we going to now have to invest in him AND cleaning up his code. This guy wasn't ready for a full-time coding job, plain and simple. School, or his own study habits, it seems, has failed to prepare him for a real world coding job. In this cut-throat business, I don't know of too many internships that could spare the time or the resources to teach him what he should have learnt in class.
I know there's a certain minimum level of expertise you should expect of someone comming out of school, but this code sample fell WAY short of that.
Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007
|
|
|
|
|
Even then, why write a function to do it, he could just easily call +1 on the integer.
|
|
|
|
|
Exactly my point!
Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007
|
|
|
|
|
Better still - I say get rid of the person who hired him in the
first place. At least that way you guarantee such a disaster wont
enter "the" organisation again. 
|
|
|
|
|
Arash Partow wrote: I say get rid of the person who hired him in the
first place. At least that way you guarantee such a disaster wont
enter "the" organisation again.
I agree. The OP should have at least mentored him with the tech review, and if it happened again, then fire the programmer. Doesn't sound like the person was given much of a chance...
|
|
|
|
|
eunderwo00 wrote: He no longer works for the company.......
That's pretty lame of you to can him over stupid code like that. You could have at least maybe mentored him with Code Complete 
|
|
|
|
|
I need sample code for Paging in ASP 3.0 for Page Break while printing the Page(Report Page developed with HTML Tables) and continuing with the Page Heading and Remaining lines in the next Page and so on for the Oncoming pages if the no of Records are more.
|
|
|
|
|
I will study the source data carefully when testing my UI!
I've just laid out a data bound WinForms form three times, the first two discarded with disgust because the databinding provided by the VS2005 wizard sucked. I was pressing back and forward on the BindingNavigator much better but the record detail was staying the same. Something was wrong, so I deleted the form and tried again from scratch. Same thing, so I try and create the whole data source again.
Then, on the third form, I notice that most of the records are for companies, who are allocated a range of phone numbers, and I have one record for each phone number, with ALL the other field values identical. then I noticed that clicking back and forward on the navigator is indeed changing records by my observation of the last digit of the phone number.
|
|
|
|
|
Thou Shalt Know Thy Incoming Data.
|
|
|
|
|
It is the little things that get you after a while! 
INTP
"Program testing can be used to show the presence of bugs, but never to show their absence."Edsger Dijkstra
|
|
|
|
|
A few years back, I started a consulting engagement. We took over support of all client/server programs. Get this, it was a public utility and all internal databases were in Access. Anyway, there was a large complicated program written by a non-programmer. How bad was the code?
Well, if there was more than one way to connect to a database, the "programmer" used each type. If there was more than one way to do ANYTHING, they used every method. Then, to top it off, whenever we would make a change in one section of the program (tons of SQL) then another part would break. Each time a bug would be raised, it would be a few hours for fixing, testing, and fixing the sections that should not have broke but did. We stopped answering our phones at 4pm because we wanted to go home at 5. Otherwise, we'd work into dinner time.
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
