Click here to Skip to main content
15,886,873 members

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.

 
GeneralAh the joys of working in a team Pin
Argonia21-Jan-14 1:49
professionalArgonia21-Jan-14 1:49 
GeneralRe: Ah the joys of working in a team Pin
JMK8921-Jan-14 2:03
professionalJMK8921-Jan-14 2:03 
GeneralRe: Ah the joys of working in a team Pin
Argonia21-Jan-14 2:14
professionalArgonia21-Jan-14 2:14 
GeneralRe: Ah the joys of working in a team Pin
Freak3022-Jan-14 1:25
Freak3022-Jan-14 1:25 
GeneralRe: Ah the joys of working in a team Pin
Ravi Bhavnani21-Jan-14 2:37
professionalRavi Bhavnani21-Jan-14 2:37 
GeneralRe: Ah the joys of working in a team Pin
Shameel21-Jan-14 3:23
professionalShameel21-Jan-14 3:23 
GeneralRe: Ah the joys of working in a team Pin
Ravi Bhavnani21-Jan-14 4:58
professionalRavi Bhavnani21-Jan-14 4:58 
GeneralAnother gem from the VB6 code I'm "porting" Pin
Rob Grainger16-Jan-14 23:25
Rob Grainger16-Jan-14 23:25 
In one function, the following is used to call SomeFunc (names changed to protect the guilty):

VB
If UCase(SomeFunc("" & int_var, bool_var)) = "YES" Then
    F.chkBox.Value = 1
Else 
    F.chkBox.Value = 0
End If


(with two more calls to the same function further down).

Where F is a form variable (because frm would be too hard to type)
Looking at the definition...

VB
Private Function SomeFunc(Result As Long, Optional cond As Boolean) As String
    If cond = True Then
        Select Case (Result)
            Case Is < 3
                SomeFunc = "No"
            Case Is > 2
                SomeFunc = "Yes"
        End Select
        
    Else
        Select Case (Result)
            Case 1
                SomeFunc = "No"
            Case Is >= 2
                SomeFunc = "Yes"
        End Select
    End If
End Function


Why use a Boolean when a string comparison, case conversion and check for null string will do?
Why is that Boolean argument optional, when it is always supplied?

(before someone says, predictably, that the real WTF is VB, I'd like to point out I've seen code just as horrible in most languages.
The real real WTF is that people who produce such garbage are still employed.
"If you don't fail at least 90 percent of the time, you're not aiming high enough."
Alan Kay.

GeneralRe: Another gem from the VB6 code I'm "porting" Pin
NormDroid16-Jan-14 23:51
professionalNormDroid16-Jan-14 23:51 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Rob Grainger17-Jan-14 0:01
Rob Grainger17-Jan-14 0:01 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Chris Quinn17-Jan-14 3:11
Chris Quinn17-Jan-14 3:11 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Rob Grainger17-Jan-14 3:16
Rob Grainger17-Jan-14 3:16 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Richard Deeming17-Jan-14 1:33
mveRichard Deeming17-Jan-14 1:33 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Rob Grainger17-Jan-14 3:10
Rob Grainger17-Jan-14 3:10 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Richard Deeming17-Jan-14 3:35
mveRichard Deeming17-Jan-14 3:35 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Ron Beyer17-Jan-14 4:48
professionalRon Beyer17-Jan-14 4:48 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Freak3019-Jan-14 23:41
Freak3019-Jan-14 23:41 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Sentenryu20-Jan-14 4:00
Sentenryu20-Jan-14 4:00 
GeneralRe: Another gem from the VB6 code I'm "porting" Pin
Marc Clifton21-Jan-14 2:12
mvaMarc Clifton21-Jan-14 2:12 
GeneralA backslash in C++ Pin
Tsuda Kageyu14-Jan-14 20:56
Tsuda Kageyu14-Jan-14 20:56 
GeneralRe: A backslash in C++ Pin
Duncan Edwards Jones14-Jan-14 21:02
professionalDuncan Edwards Jones14-Jan-14 21:02 
GeneralRe: A backslash in C++ Pin
Bernhard Hiller14-Jan-14 21:04
Bernhard Hiller14-Jan-14 21:04 
GeneralRe: A backslash in C++ Pin
Tsuda Kageyu14-Jan-14 21:13
Tsuda Kageyu14-Jan-14 21:13 
GeneralRe: A backslash in C++ Pin
Bernhard Hiller14-Jan-14 21:44
Bernhard Hiller14-Jan-14 21:44 
GeneralRe: A backslash in C++ Pin
Nicholas Marty15-Jan-14 0:43
professionalNicholas Marty15-Jan-14 0:43 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

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