The Weird and The Wonderful

try {
    if (GetLimitMessage(a,b) != null && GetLimitMessage(a,b).Equals(GetLimitMessage(1, -1))) {
        if (!string.IsNullOrEmpty(GetLimitMessage(a,b)) {
            try {
                MessageBox.Show(GetLimitMessage(a,b));
            }
            catch (Exception ex) {
                throw;
            }
        }
    }
}
catch (Exception ex) {
    MessageBox.Show("The monkey's dead!");
}

if(Helper.GetLimitMessage(total, 100)) //Yep, 100 is harcoded on every line it's used...
{
    ViewBag.LimitMessage = Helper.GetLimitMessage(total, 100);
    ViewData["LimitMessage"] = Helper.GetLimitMessage(total, 100);
}

@if(ViewBag.LimitMessage != "")
{
     @Helper.GetLimitMessage(total, 100)
}
else if (ViewData["LimitMessage"] != null)
{
     @ViewData["LimitMessage"]
}

Manfred R. Bihy wrote:

And what with the uninitialised statusMutex thingy?

Marco Bertschi wrote:

It is Qt code, where objects can be initialized by just writing "ObjectType variablename;".

Marco Bertschi wrote:

Not initialized variables are bad!

class  MutexLocker {
    QMutex &m;
public:
    MutexLocker(QMutex &qm) : m(qm) { m.lock(); }
    ~MutexLocker() { m.unlock(); }
}; // class

int GetStatus(){
   MutexLocker tmp(statusMutex);
   return status;
   // destructor unlocks 
}

Manfred R. Bihy wrote:

The compiler should issue a warning or unreachable code. Did it?

GuyThiebaut wrote:

I should have coded:

C#

if (item.Substring(0, 3).ToLower() != "PF_".ToLower())

if (string.Compare(item, 0, "PF_", 0, 3, StringComparison.OrdinalIgnoreCase) != 0)

if(item.StartsWith("PF_", StringComparison.OrdinalIgnoreCase)) ...

if(1600548864 == (*(int*)&item.ToLower()) << 8) ...

if (((BitConverter.ToInt32(Encoding.ASCII.GetBytes(item), 0) & 6250335) ^ 6243920) == 0)

((xum 567 487) (map item (lambda x (find (xor x (and y z)) (encode pi 49)))

CDP1802 wrote:

((xum 567 487) (map item (lambda x (find (xor x (and y z)) (encode pi 49)))

The language is JavaScript. that of Mordor, which I will not utter here