Bug of the day

[Chris Maunder]

That's fine. I'm off to play with the reputation table. <Later on...> Oopsie! Looks like my counting skills still aren't up to scratch.

cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP

[Lost User]

Wrong forum, take it to DB. ;P

============================== Nothing to say.

[Pete OHanlon]

Just remember who sucked updefended to you ;)

Forgive your enemies - it messes with their heads

"Mind bleach! Send me mind bleach!" - Nagy Vilmos

My blog | My articles | MoXAML PowerToys | Mole 2010 - debugging made easier - my favourite utility

[bVagadishnu]

You forgot to drop trou while counting? :confused:

Schenectady? What am I doing in Schenectady?

[Lost User]

You're not married and therefore your member is not suppressed.

[Single Step Debugger]

Using all caps for the Transact SQL operators is SO last century. :)

There is only one Vera Farmiga and Salma Hayek is her prophet! Advertise here – minimum three posts per day are guaranteed.

[Chris Maunder]

I'm old school

cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP

[Mel Padden]

Is there a good reason why you declared it only 20 chars long? Just checking... My auto-default for in-proc literals is nvarchar(255), just cos I have to deal with localised strings and 255 is generally long enough that I don't have to worry about catching these kinds of bugs.

Beautiful is better than ugly. Explicit is better than implicit. Simple is better than complex. Complex is better than complicated. Flat is better than nested. Sparse is better than dense. In the face of ambiguity, refuse the temptation to guess.

[Peter Mulholland]

assignment instead of equality check? Shoulda used yoda code. [edit] Ok, so after looking at the other responses i was way off, I guess. Still, is that not stringly typed code? [/edit]

Pete

[Chris Maunder]

It was set at a value guaranteed to be longer than what I would need. You can see the irony, of course.

cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP

[wizardzz]

Ah, suppressing members now? We are the 99% you know. ;)

"I have a theory that the truth is never told during the nine-to-five hours. " — Hunter S. Thompson

[ekolis]

Help! I'm being suppressed! *gets thrown into a wheelbarrow* I'm not dead yet! *KATHWACKLE* Now he is...

[James H]

21 char string literal into 20 char varchar = 'SkipSuppressedMember' losing the s

[Steven Nicholas]

Well, males do have an extra digit. :sigh:

[edmurphy99]

well I see this has already been answered but I like to use print to see what variables really are

DECLARE @SuppressionMethod varchar(20)

-- ... lots of code...

SET @SuppressionMethod = 'SkipSuppressedMembers'
print @SuppressionMethod
--returns SkipSuppressedMember

[YvesDaoust]

In this case, blame can be put on the programming language: it should have warned you of the string overflow. Computers are ill-mannered when they do not tell us about our mistakes that they have spotted.

[Fabio Franco]

It's to mask the fact they are worse on everything else.

"To alcohol! The cause of, and solution to, all of life's problems" - Homer Simpson

[Victor Ulloa]

SkipSuppressedMembers has 21 chars, not 20. You're comparing lower('SkipSuppressedMember') against skipsuppressedmembers

[Kabwla Phone]

To feed the trolls and have the string fit in the variable, you might consider increasing the length of said troll variable. Other problems in the code, using 'lower' to do a string comparission in SQL, which is case insensitive (unless you start toying with the collation).

[agolddog]

How about varchar(more)?