|
Hm, i am totally sure you pasted something very similar to this one. Is this from the same author. I want a autograph please.
Microsoft ... the only place where VARIANT_TRUE != true
|
|
|
|
|
|
At least did he try to keep this thing readable: Note the extra spaces before UPDATE ind in the WHERE clause.
|
|
|
|
|
Bernhard Hiller wrote: At least did he Visual Studio try to keep this thing readable: Note the extra spaces before UPDATE ind in the WHERE clause.
FTFY!
|
|
|
|
|
Ah, I think I see the problem - it's VB! 
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
I think this time VB has nothing to do with it!
Reality is an illusion caused by a lack of alcohol
"Nagy, you have won the internets." - Keith Barrow
|
|
|
|
|
OriginalGriff wrote: UPDATE ALLUGTAMIL
You failed to see the actual problem: the textboxes contain text in TAMIL characters, but in the database varchar fields were used instead of nvarchar... 
|
|
|
|
|
Note to self: remove tin-foil hat before answering questions...
This message is manufactured from fully recyclable noughts and ones. To recycle this message, please separate into two tidy piles, and take them to your nearest local recycling centre.
Please note that in some areas noughts are always replaced with zeros by law, and many facilities cannot recycle zeroes - in this case, please bury them in your back garden and water frequently.
|
|
|
|
|
OriginalGriff wrote: I hope he's a beginner
OriginalGriff wrote: no breaks
Yes, well everyone knows that only senior devs are allowed to use the line continuation character. If we let all VB devs use it then the hierarchy would break down.
<ExcuseMe>I just wanted to underscore that.</ExcuseMe>
|
|
|
|
|
Whoo hoo! That means I'm a Senior Developer! Does that mean I get a pay rise?
This message is manufactured from fully recyclable noughts and ones. To recycle this message, please separate into two tidy piles, and take them to your nearest local recycling centre.
Please note that in some areas noughts are always replaced with zeros by law, and many facilities cannot recycle zeroes - in this case, please bury them in your back garden and water frequently.
|
|
|
|
|
OriginalGriff wrote: Does that mean I get a pay rise?
You already did.
|
|
|
|
|
The only thing I see wrong with that UPDATE statement is the naming convention of the textboxes.
|
|
|
|
|
|
Ok I wanna know just exactly how you broken into our TFS and downloaded the code from one of our files!?
|
|
|
|
|
Just stumbled across this:
int page = 1;
while (true) {
if (page > pageCount) {
break;
}
page++;
}
Considering how common it is to use an index in a loop, you'd think the language designers would have a better way of dealing with this... 
EDIT: even better, in 2 of the 3 places using page, it uses page - 1 :facepalm:
|
|
|
|
|
If I use ILASM to disassemble .NET coe, it frequently decodes for-loops like this.
Maybe the original developer appropriated (ie. stole) some code using similar techniques without checking the output carefully.
(Personally, I use it to check correct usage of poorly documented APIs, but I'm sure many are less scrupulous).
"If you don't fail at least 90 percent of the time, you're not aiming high enough."
Alan Kay.
|
|
|
|
|
Rob Grainger wrote: it frequently decodes for-loops like this
Of course, the simplest way to write a loop in an assembly-like language is something like:
[assignment]
start_loop:
if ![condition] goto end_loop
[body]
[increment]
goto start_loop
end_loop:
That doesn't mean it should be written that way in a high-level language though, even if you technically can.
|
|
|
|
|
Rob's suggesting the duhveloper just stole recompiled code out of reflector without thinking; and building in logic to detect all 90 bazillion high level constructs and correctly promote the result of reflection to them instead of the simplest valid code is 99.9999% of the work involved in writing a reflection tool. The length of the long tail means that it's a job that will probably never be done.
Did you ever see history portrayed as an old man with a wise brow and pulseless heart, waging all things in the balance of reason?
Is not rather the genius of history like an eternal, imploring maiden, full of fire, with a burning heart and flaming soul, humanly warm and humanly beautiful?
--Zachris Topelius
Training a telescope on one’s own belly button will only reveal lint. You like that? You go right on staring at it. I prefer looking at galaxies.
-- Sarah Hoyt
|
|
|
|
|
Thank you for saving me the effort, exactly my point.
"If you don't fail at least 90 percent of the time, you're not aiming high enough."
Alan Kay.
|
|
|
|
|
while(true) {
thingie.Open();
if (thingie.Stale() && YesUserReallyWantsToUpgrade())
{
thingie.Close();
continue;
}
break;
}
On the plus side, I've had secret santa's that were worse. On the down side, that would look better with a goto, which doesn't sound exactly good.
|
|
|
|
|
We can hide goto[^] even better:
bread.Open();
try
{
if (bread.Stale() && Banana())
{
throw DamnUserSeeksDooom();
}
}
catch (DamnUserSeeksDooom DUST)
{
dust.Sweep();
}
finally
{
bread.close();
}
Teez javaOOP style. The more blue words the more proffesional code looks, you know. 
Greetings - Jacek
|
|
|
|
|
Jacek Gajek wrote: catch (DamnUserSeeksDooom DUST)
{
dust.Sweep();
}
i guess you swept your DUST before the actual call and it became smaler dust 
P.S sleeping only 5 hours isn't working well with my spelling skills.
Microsoft ... the only place where VARIANT_TRUE != true
|
|
|
|
|
So the thingie refuses to upgrade, because it does nothing as long as the user wants it to upgrade? What does it do if the user doesn't want it to upgrade? Does it upgrade in this case?
The good thing about pessimism is, that you are always either right or pleasently surprised.
|
|
|
|
|
That's a bit mroe complex than this simplified example of course (surprisingly, I don't have variales named "thingie" in there) - depending on the forma,t it's read only access or no access at all, indeed.
Since the conversion runs in a separate process, I'm in the middle of a transaction, and I have to be careful to throw more milliseconds into that piece of code, tat "close, convert and retry" seems to be the best approach.
Alternatively, I thought of an inner function that returns either the newly initialized session, or information about how to upgrade, but that code doesn#t look much better either.
|
|
|
|
|
Inheriting the maintenance of some 3rd party code today!
Mused over it, looking at the compilation warningS....
I did find this piece of code, aw... I should say, while I know it's valid C#, it displays a creativity beyond my own!!!
public class DesignRegistrationSearchCriteria
{
public string Manufacturer { get; set; }
public string manufacturer { get; set; }
}
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
