Coding Horrors | Forum profile

I cam across this C++ gem today in some code that a co-worker wrote. In his defense, the bulk of his programming experience is in OS/390 Assembler. char *ShowDrive( TAPEDRIVE *DRV, char *out ) { int status; char *msg; status = 0x2D2D; if ( DRV->stat & POWERON ) status = 0x4E4F; if ( DRV->stat & MOUNTED ) status = 0x544D; if ( DRV...

Assume that pTimeText is containing 2 dates seperated by space. Like pTimeText = "31/12/2008 15:36 29/12/2008 15:36" this.txtTime.Text = Convert.ToDateTime(pTimeText.Substring(0, pTimeText.IndexOf(":") + 3)).ToString(Logic.DateConfigurer.DateFormatWithTi me) + Environment.NewLine + Convert.ToDateTime(pTimeText.Substring(pTimeText.I ndexOf(":") + 3)).ToString(Logic.DateConfigurer....

Recently, I stumbled across this little gem. I don't have the exact code handy, but the gist of it is: nErrorCode = cFtpConn.SetHost(HOST); if (nErrorCode == 0) { nErrorCode = cFtpConn.SetUser(USERNAME); if (nErrorCode == 0) { nErrorCode = cFtpConn.SetPassword(PASSWORD); if (nErrorCode == 0) { nErrorCode = cFtpConn.SetPath(PATH); if (nErrorCode...

My 5, that code is pure Horror. Anyway (I made a test) the goto statement it is ininfluent. BTW I didn't know C# has goto statement, is there a reason to have it in such a language?

Working on an old vb6 project I found this treasure ... Function conv8(number) As String If number = "" Then number = 0 End If conv8 = Hex(Numero) Select Case Len(conv8) Case 1 conv8 = 0 & 0 & 0 & 0 & 0 & 0 & 0 & conv8 Case 2 conv8 = 0 & 0 & 0 & 0 & 0 & 0 & conv8 Case 3 conv8 = 0 & 0 & 0 & 0 & 0 & conv8 Case 4 ...

DECLARE p_cursor CURSOR FOR SELECT TOP 1 SortDateStart AS DateStart, SortDateEnd AS DateEnd ... This code was in a trigger...

a friend of mine - many years ago - was asked to port a physics simulation program from one platform to another. The software performed beautifully, was efficient, and had won awards for educational excellence etc. the source code was several thousand lines of assembly, which was large but not too large for that platform and era. The program had two comments in it, one word each. They were not...

I was going through our documentation and noticed this gem. We had a senior developer who created our development standards. This was part of the document he wrote for us. We had the .NET 2.0 framework available to us (hint: Generics) when this was written. I hope you find this as entertaining as I did. On a side note, when the efficiencies were brought to his attention and refuted with a ...

Well, you know, exception handling automatically-magically-definitely solves the goto-spaghetti-code nightmare...

bool foo(int n) { if (n > 16) return true; else if (n <= 16) return false; else return false; } i love this one!

I cam across this C++ gem today in some code that a co-worker wrote. In his defense, the bulk of his programming experience is in OS/390 Assembler. char *ShowDrive( TAPEDRIVE *DRV, char *out ) { int status; char *msg; status = 0x2D2D; if ( DRV->stat & POWERON ) status = 0x4E4F; if ( DRV->stat & MOUNTED ) status = 0x544D; if ( DRV...