Click here to Skip to main content
13,201,050 members (54,411 online)

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.

 
GeneralRe: Statics Pin
ekolis10-May-12 13:33
memberekolis10-May-12 13:33 
GeneralRe: Statics Pin
CIDev10-May-12 13:40
memberCIDev10-May-12 13:40 
GeneralRe: Statics Pin
Bernhard Hiller10-May-12 22:35
memberBernhard Hiller10-May-12 22:35 
GeneralRe: Statics Pin
PIEBALDconsult11-May-12 5:13
memberPIEBALDconsult11-May-12 5:13 
JokeRe: Statics Pin
Rahul Rajat Singh14-May-12 2:32
memberRahul Rajat Singh14-May-12 2:32 
GeneralRe: Statics Pin
Naerling17-May-12 12:31
memberNaerling17-May-12 12:31 
GeneralRe: Statics Pin
Zac Greve23-May-12 13:08
memberZac Greve23-May-12 13:08 
GeneralProblem? Pin
glhrmbhnrt10-May-12 2:57
memberglhrmbhnrt10-May-12 2: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?

Look.

/*
   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><br/>" +
@"   <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><br/><br/>" +
           "  </td>" +
           " </tr>" +
          "</table>" +
      "</td>";
 
    cnt++;
}
html += "</tr></table>";
 
divHtml.InnerHtml = html;


That's my bro, there's an classic asp tag to use in asp.net? 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? Pin
BobJanova11-May-12 3:55
memberBobJanova11-May-12 3:55 
GeneralReally??? Pin
KP Lee9-May-12 20:36
memberKP Lee9-May-12 20:36 
GeneralRe: Really??? Pin
glhrmbhnrt10-May-12 2:24
memberglhrmbhnrt10-May-12 2:24 
GeneralRe: Really??? Pin
CIDev10-May-12 4:08
memberCIDev10-May-12 4:08 
GeneralKinda guy Pin
glhrmbhnrt9-May-12 8:02
memberglhrmbhnrt9-May-12 8:02 
GeneralRe: Kinda guy Pin
CDP18029-May-12 21:08
memberCDP18029-May-12 21:08 
GeneralRe: Kinda guy Pin
Mohibur Rashid15-May-12 20:08
memberMohibur Rashid15-May-12 20:08 
GeneralRe: Kinda guy Pin
BobJanova9-May-12 23:02
memberBobJanova9-May-12 23:02 
GeneralRe: Kinda guy Pin
CDP18029-May-12 23:14
memberCDP18029-May-12 23:14 
GeneralRe: Kinda guy Pin
CIDev10-May-12 4:05
memberCIDev10-May-12 4:05 
GeneralRe: Kinda guy Pin
PIEBALDconsult10-May-12 6:19
memberPIEBALDconsult10-May-12 6:19 
GeneralConstant Headache Pin
AspDotNetDev7-May-12 15:04
protectorAspDotNetDev7-May-12 15:04 
GeneralRe: Constant Headache Pin
ekolis9-May-12 6:26
memberekolis9-May-12 6:26 
GeneralRe: Constant Headache Pin
AspDotNetDev9-May-12 7:12
protectorAspDotNetDev9-May-12 7:12 
GeneralRe: Constant Headache Pin
ryanb3110-May-12 6:39
memberryanb3110-May-12 6:39 
GeneralRe: Constant Headache Pin
AspDotNetDev10-May-12 7:10
protectorAspDotNetDev10-May-12 7:10 
GeneralRe: Constant Headache Pin
Jörgen Andersson11-May-12 8:44
memberJörgen Andersson11-May-12 8:44 

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.


Advertise | Privacy |
Web01 | 2.8.171020.1 | Last Updated 23 Oct 2017
Copyright © CodeProject, 1999-2017
All Rights Reserved. Terms of Service
Layout: fixed | fluid