The Weird and The Wonderful

   

The Weird and The Wonderful forum is a place to post Coding Horrors, Worst Practices, and the occasional flash of brilliance.

We all come across code that simply boggles the mind. Lazy kludges, embarrassing mistakes, horrid workarounds and developers just not quite getting it. And then somedays we come across - or write - the truly sublime.

Post your Best, your worst, and your most interesting. But please - no programming questions . This forum is purely for amusement and discussions on code snippets. All actual programming questions will be removed.

FirstPrevNext

Re: Intellisense Strikes Again Kornfeld Eliyahu Peter17-Feb-16 1:40 Kornfeld Eliyahu Peter 17-Feb-16 1:40  You probably lost the 'joke' icon... I would like to know in what way the debugger and the intellisense is better? And VB is NOT case-insensitive - the IDE only fixes your cases after you wrote the name down the first time (and this happens also in C# now)... As for the properties... C# public int Number { get; private set; } Skipper: We'll fix it. Alex: Fix it? How you gonna fix this? Skipper: Grit, spit and a whole lotta duct tape. Re: Intellisense Strikes Again Sinisa Hajnal17-Feb-16 20:58 Sinisa Hajnal 17-Feb-16 20:58  See my comment above. Re: Intellisense Strikes Again Richard Deeming18-Feb-16 3:03 Richard Deeming 18-Feb-16 3:03  Or, with C# 6: C# public int Number { get; } = 42; // Or: public int Number { get; } public YourClass() { Number = 42; } "These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer From this week's mobile newsletter Richard MacCutchan21-Jan-16 5:09 Richard MacCutchan 21-Jan-16 5:09  Quote: In this post, we will demonstrate how you can use Google Sign-In with your backend. Really? Re: From this week's mobile newsletter F-ES Sitecore8-Feb-16 23:56 F-ES Sitecore 8-Feb-16 23:56  Well a lot of people use twitter with their backend... tricky C# literals Tomas Takac20-Jan-16 3:00 Tomas Takac 20-Jan-16 3:00  I have a code where I need to get an exclusive lock on a file. It goes something like this: C# try { // open file } catch(IOException ex) { if(ex.HResult == 0x80070020) { // retry } else { throw; } } Looked good so I tested and it didn't work at all. So I debugged it and what i saw was: ex.HResult 0x80070020 int ex.HResult == 0x80070020 false bool There is something wrong about the value 0x80070020. So I tried to declare a constant: C# const int lockedFileHResult = 0x80070020; And there it was: Cannot implicitly convert type 'uint' to 'int'. Of course because that is actually a negative number but the compiler doesn't know that so it cannot fit it into an integer. Next time I'll be extra careful with hex literals. Re: tricky C# literals PIEBALDconsult20-Jan-16 3:09 PIEBALDconsult 20-Jan-16 3:09  And how about 0x80070020u ? Re: tricky C# literals Tomas Takac20-Jan-16 3:46 Tomas Takac 20-Jan-16 3:46  It was an uint, and the problem was that I was comparing that to and integer. The integer value is -2147024864 but there is no way to declare that in hex. Hex value always translates to positive number which is 2147942432 in this case. The hex view in debugger fooled me. Edit: ok there is a way thanks John. Re: tricky C# literals John Torjo20-Jan-16 3:43 John Torjo 20-Jan-16 3:43  Exception.HResult Property (System)[^] MS suggests you do something like this: const int SecondLevelHResult = unchecked( (int)0x81234567 ); Best, John -- Log Wizard - a Log Viewer that is easy and fun to use! Re: tricky C# literals Tomas Takac20-Jan-16 3:50 Tomas Takac 20-Jan-16 3:50  That's nice trick with unchecked. Thanks for sharing. I eventually used decimal format, it's just a number after all. Re: tricky C# literals den2k8820-Jan-16 4:00 den2k88 20-Jan-16 4:00  My 2 cents: do not use that magic number, declare it as a constant and comment it with the meaning. Believe me, this is one of the very few good practices that is ALWAYS a good practice. Magic numbers are tempting because they are fast to code in... and a hell to figure out. GCS d--- s-/++ a- C++++ U+++ P- L- E-- W++ N++ o+ K- w+++ O? M-- V? PS+ PE- Y+ PGP t++ 5? X R++ tv-- b+ DI+++ D++ G e++>+++ h--- ++>+++ y+++*      Weapons extension: ma- k++ F+2 X If you think 'goto' is evil, try writing an Assembly program without JMP. -- TNCaver "When you have eliminated the JavaScript, whatever remains must be an empty page." -- Mike Hankey "just eat it, eat it"."They're out to mold, better eat while you can" -- HobbyProggy Re: tricky C# literals TheGreatAndPowerfulOz20-Jan-16 4:17 TheGreatAndPowerfulOz 20-Jan-16 4:17  While I agree about the magic number, the problem here was one of silent conversion with data loss. Decrease the belief in God, and you increase the numbers of those who wish to play at being God by being “society’s supervisors,” who deny the existence of divine standards, but are very serious about imposing their own standards on society.-Neal A. Maxwell You must accept 1 of 2 basic premises: Either we are alone in the universe or we are not alone. Either way, the implications are staggering!-Wernher von Braun Re: tricky C# literals den2k8820-Jan-16 4:34 den2k88 20-Jan-16 4:34  Yes I understood it - and encountered the problem frequently enough to be my first bet when I saw it. I also saw the solutions of those who commented before me. I just put my tip, also because usually when one declares a constant puts more attention to the type specification, so it is more probable to correctly define it as an unsigned form the start. GCS d--- s-/++ a- C++++ U+++ P- L- E-- W++ N++ o+ K- w+++ O? M-- V? PS+ PE- Y+ PGP t++ 5? X R++ tv-- b+ DI+++ D++ G e++>+++ h--- ++>+++ y+++*      Weapons extension: ma- k++ F+2 X If you think 'goto' is evil, try writing an Assembly program without JMP. -- TNCaver "When you have eliminated the JavaScript, whatever remains must be an empty page." -- Mike Hankey "just eat it, eat it"."They're out to mold, better eat while you can" -- HobbyProggy Re: tricky C# literals Tomas Takac20-Jan-16 4:29 Tomas Takac 20-Jan-16 4:29  I agree with the constant. I usually do this after I finish developing a method. I admit In this case I'd seen the problem immediately if I declared the constant upfront. Re: tricky C# literals den2k8820-Jan-16 4:36 den2k88 20-Jan-16 4:36  That's precisely why I suggested it I tripped over that problem and similar ones many times and usually delcaring all the data and constant firsthand helps me to avoid simple but time-consuming mistakes like this. GCS d--- s-/++ a- C++++ U+++ P- L- E-- W++ N++ o+ K- w+++ O? M-- V? PS+ PE- Y+ PGP t++ 5? X R++ tv-- b+ DI+++ D++ G e++>+++ h--- ++>+++ y+++*      Weapons extension: ma- k++ F+2 X If you think 'goto' is evil, try writing an Assembly program without JMP. -- TNCaver "When you have eliminated the JavaScript, whatever remains must be an empty page." -- Mike Hankey "just eat it, eat it"."They're out to mold, better eat while you can" -- HobbyProggy Re: tricky C# literals John Torjo20-Jan-16 9:10 John Torjo 20-Jan-16 9:10  While I totally agree with you, this still won't save you in the OP's case: C# const int a = 5; const uint b = 6; var c = a == b; This compiles. So long story short, if you have a comparison signed with unsigned, the compiler will compile the code (and at warning level 4 = default), won't issue any warning. Best, John -- Log Wizard - a Log Viewer that is easy and fun to use! Re: tricky C# literals den2k8820-Jan-16 20:50 den2k88 20-Jan-16 20:50  Strange - C++ does and as far as I remember C# is much more picky when it comes to warnings. To be clear I usually compile with Visual Studio 6 and the last C# I used is that of VS2008. GCS d--- s-/++ a- C++++ U+++ P- L- E-- W++ N++ o+ K- w+++ O? M-- V? PS+ PE- Y+ PGP t++ 5? X R++ tv-- b+ DI+++ D++ G e++>+++ h--- ++>+++ y+++*      Weapons extension: ma- k++ F+2 X If you think 'goto' is evil, try writing an Assembly program without JMP. -- TNCaver "When you have eliminated the JavaScript, whatever remains must be an empty page." -- Mike Hankey "just eat it, eat it"."They're out to mold, better eat while you can" -- HobbyProggy Re: tricky C# literals KarstenK20-Jan-16 6:23 KarstenK 20-Jan-16 6:23  Hasnt the compiler yelled about it Press F1 for help or google it. Greetings from Germany One of my own.... Duncan Edwards Jones14-Jan-16 0:11 Duncan Edwards Jones 14-Jan-16 0:11  VB.NET Protected Overridable Function getActivityMarketValue(ByVal act As IActivityRecord, ByVal brokerRecord As IST4Record) As Double Call getTradeMarketValue(act, brokerRecord) End Function I stepped through this about 3 times before I worked out why the activity market value kept being set to zero... Re: One of my own.... PIEBALDconsult14-Jan-16 12:23 PIEBALDconsult 14-Jan-16 12:23  ... Aaaand the reason is... ? Re: One of my own.... Duncan Edwards Jones14-Jan-16 20:37 Duncan Edwards Jones 14-Jan-16 20:37  I was calling the function - not returning it...so even though the right market value was calculated, it never left that function. Re: One of my own.... Stefan Küsters14-Jan-16 21:15 Stefan Küsters 14-Jan-16 21:15  Brilliant VB language definition... A function returns a default value if no return value is specified explicitly... Re: One of my own.... RugbyLeague14-Jan-16 21:47 RugbyLeague 14-Jan-16 21:47  It's not just VB which does that Re: One of my own.... Kornfeld Eliyahu Peter17-Jan-16 23:21 Kornfeld Eliyahu Peter 17-Jan-16 23:21  The beauty of VB... Skipper: We'll fix it. Alex: Fix it? How you gonna fix this? Skipper: Grit, spit and a whole lotta duct tape. Re: One of my own.... Nicholas Marty20-Jan-16 4:12 Nicholas Marty 20-Jan-16 4:12  Shouldn't that create a compiler error or at least warning as your function doesn't explicitly return a value?

Last Visit: 31-Dec-99 18:00     Last Update: 30-May-24 18:34 Refresh ᐊ Prev1...170171172173174175176177178179 Next ᐅ

General    News    Suggestion    Question    Bug    Answer    Joke    Praise    Rant    Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.