Click here to Skip to main content

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, embarrasing 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.

GeneralRe: Statics Pinmemberekolis10-May-12 14:33 
GeneralRe: Statics PinmemberCIDev10-May-12 14:40 
GeneralRe: Statics PinmemberBernhard Hiller10-May-12 23:35 
GeneralRe: Statics PinmemberPIEBALDconsult11-May-12 6:13 
JokeRe: Statics PinmemberRahul Rajat Singh14-May-12 3:32 
GeneralRe: Statics PinmemberNaerling17-May-12 13:31 
GeneralRe: Statics PinmemberZac Greve23-May-12 14:08 
GeneralProblem? Pinmemberglhrmbhnrt10-May-12 3:57 
Why use repeaters/datalists/grids when you can do it via a foreach loop and throw the whole string on a div.innerHTML? D'Oh! | :doh:
The item template never changes, and the guy keep doing it via code. Cry | :((
Am i unlucky?
   some code done here....
   database select, some ifs...
string html = "<table><tr>";
int cnt = 1;
foreach (DataRow reg in Data.Rows)
   if (cnt > 3)
       html+= "</tr><tr>";
       cnt = 1;
   html += "<td height='210px' valign='top' align='left'>" +
         "<table align='left' height='100%' width='239px'>" +
           " <tr>" +
            "  <td valign='top'>" +
           @"   <a class="Registroclick" href="javascript: void(0)" idseg="0" idcat=" + reg["some reg"].ToString() + "  endereco='" + destpage+ "?idcat=" + reg["some reg"].ToString() + "'><img src='" + vid+ "destaque/" + reg["some reg"].ToString() + "' width='235' height='136' border='0'/></a>
" +
@"   <a class="Registroclick" href="javascript: void(0)" idseg="0" idcat=" + reg["some reg"].ToString() + "  endereco='" + destpage+ "?idcat=" + reg["some reg"].ToString() + "'>" + reg["some reg"] + "</a>
" +
           "  </td>" +
           " </tr>" +
          "</table>" +
html += "</tr></table>";
divHtml.InnerHtml = html;
That's my bro, there's an classic asp tag to use in Maybe he needs it.
It's a ascx with 150 lines. 100 lines on this method.
Also StringBuilder() and String.Format() seems to be useless.
Now imagine out how to update this layout;
return true;

GeneralRe: Problem? PinmemberBobJanova11-May-12 4:55 
GeneralReally??? PinmemberKP Lee9-May-12 21:36 
GeneralRe: Really??? Pinmemberglhrmbhnrt10-May-12 3:24 
GeneralRe: Really??? PinmemberCIDev10-May-12 5:08 
GeneralKinda guy Pinmemberglhrmbhnrt9-May-12 9:02 
GeneralRe: Kinda guy [modified] PinmemberCDP18029-May-12 22:08 
GeneralRe: Kinda guy PinmemberMohibur Rashid15-May-12 21:08 
GeneralRe: Kinda guy PinmemberBobJanova10-May-12 0:02 
GeneralRe: Kinda guy PinmemberCDP180210-May-12 0:14 
GeneralRe: Kinda guy PinmemberCIDev10-May-12 5:05 
GeneralRe: Kinda guy PinmemberPIEBALDconsult10-May-12 7:19 
GeneralConstant Headache PinprotectorAspDotNetDev7-May-12 16:04 
GeneralRe: Constant Headache Pinmemberekolis9-May-12 7:26 
GeneralRe: Constant Headache PinprotectorAspDotNetDev9-May-12 8:12 
GeneralRe: Constant Headache Pinmemberryanb3110-May-12 7:39 
GeneralRe: Constant Headache PinprotectorAspDotNetDev10-May-12 8:10 
GeneralRe: Constant Headache PinmemberJörgen Andersson11-May-12 9:44 

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

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

Advertise | Privacy | Mobile
Web01 | 2.8.141223.1 | Last Updated 25 Dec 2014
Copyright © CodeProject, 1999-2014
All Rights Reserved. Terms of Service
Layout: fixed | fluid