Reviewing code...

[Lost User]

You need the coding horrors forum I beleive. And I can't read the whole right hand side of your post anyways.

[Hosey]

I cannot deny that I have been guilty of such code sins, sometimes I write less than efficient code as I fire through a problem, but I dont think I've ever written redundant code like the example I found. Set a variable to a zero-length string, then on the very next line check it is a zero-length string... hahaha

[NormDroid]

I smell danger, I smell VB.

Two heads are better than one.

[leppie]

For writing ASP style code in ASP.NET, that code monkey would be shot!

xacc.ide
IronScheme - 1.0 RC 1 - out now!
((λ (x) `(,x ',x)) '(λ (x) `(,x ',x))) The Scheme Programming Language – Fourth Edition

[Dalek Dave]

I think it is sensible, you never know when the value of Zero changes! :)

------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave

[leppie]

It really struggles with the 16 spaced TAB characters :) The poor donkey probably have it setup so TAB's == 1 space in it's editor.

xacc.ide
IronScheme - 1.0 RC 1 - out now!
((λ (x) `(,x ',x)) '(λ (x) `(,x ',x))) The Scheme Programming Language – Fourth Edition

[Hosey]

Cheers for the pointer. Was only posted in the lounge as I was sharing general chit-chat we were having about it in the office. Will post correctly in future :)

[Dalek Dave]

Hanged! One Hangs Monkeys...see here[^]

------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave

[Hosey]

I agree with the sentiment, but it was a snippet from an ASP page, not ASP.NET :)

[Lost User]

Don't worry about it, it appears to have generated a lot of traffic so that would suggest it is perfectly adequete post for the lounge. My mistake.

[Hosey]

:thumbsup: :cool: all is well :)

[Hosey]

An old-skool legacy im trying to get rectified. But its a large code-base, and wont get changed overnight :) Personally use C# in all my "pet-projects" but have to stick to VB at the office. Worth sharing, just to make people cringe, but we still have a business critical application (used daily and directly linked to our revenues) that was written in VB 6 over 12 yrs ago... still going... Dont have the time to re-write it at the moment, and if it aint broke, dont fix it... :)

[Hosey]

HAHAH wasnt thinking outside the box like that... good man! ;P :-D

[NormDroid]

Hosey^ wrote:

and if it aint broke, dont fix it...

VB code is always broke ;)

Two heads are better than one.

[Pualee]

If the code was C and there were poorly managed global variables you could not otherwise control... I can understand why.

[Amar Chaudhary]

Have fun :)

List<List<urlKeeper>> PhraseResultSet = new List<List<urlKeeper>>();
c.Urls = u.AltavistaScrapper(c.Url);
for (int i = 0; i < uniqueKeywords.Count; i++)
{
PhraseResultSet.Add(u.AltavistaPhraseScrapper(uniqueKeywords[i]));

        // get ranking for comps 
        c.Competitors.FindAll(delegate(Competitors x)
        {
            return uniqueKeywords\[i\] == x.Phrase;
        }).ForEach(delegate(Competitors x)
        {
            PhraseResultSet\[i\].Sort(new Comparison<urlKeeper>(delegate(urlKeeper obj1, urlKeeper obj2)
            {
                return obj1.Rank.CompareTo(obj2.Rank);
            }));

            int index = PhraseResultSet\[i\].FindIndex(delegate(urlKeeper y)
            {
                return (y.HostName.ToUpper().Replace("HTTP://", "")
                                            .Replace("HTTPS://", "")
                                            .Replace("WWW.", "")
                    == x.Url.ToUpper().Replace("HTTP://", "")
                                      .Replace("HTTPS://", "")
                                      .Replace("WWW.", ""));
            });
            if (index >= 0)
                x.Rank = PhraseResultSet\[i\]\[index\].Rank;
            else
                x.Rank = -1;

        }); ;
    }

My Startup!!!!
Profile@Elance - feedback available too

[Lost User]

Often such strange things are the result of tweaking the code until it finally worked and not taking the time to clean up the debris from all this trial and error.

A while ago he asked me what he should have printed on my business cards. I said 'Wizard'. I read books which nobody else understand. Then I do something which nobody understands. After that the computer does something which nobody understands. When asked, I say things about the results which nobody understand. But everybody expects miracles from me on a regular basis. Looks to me like the classical definition of a wizard.

[George Carmichael]

I'm astounded that people will criticize code, but make spelling mistakes during their public diatribe ("embaressed").

GC

[Tomz_KV]

True. In many cases, if a page shows up without a significant delay, its code might not be looked at again.

TOMZ_KV

[Jason Christian]

I think my favorite was an Access program (yes, there are things worse than VB), where most of the logic was in nested IIfs within queries. After copying them into a text editor so I could separate them out enough to see what was going on, I discovered that the last 3 or 4 of the 10-15 nested IIfs all =0 for the true result. There were 4-5 query columns like this. So the last 3 or 4 conditions didn't need to be checked because they would all result in 0. If a, then 0, else if b, then 0, else if c then 0. Lovely code.