Safe Division

[dandymcgee]

Saw this today. I don't even.. I'm speechless.

private double SafeDivide(double p_fVal1, double p_fVal2)
{//will not fail if dividing by zero

    double fReturn = 0;

    try
    {

        if ((p_fVal1 == 0) || (p_fVal2 == 0))
        {
            fReturn = 0;
        }

        fReturn = p_fVal1 / p_fVal2;

    }
    catch
    {
    }
    return fReturn;
}

[bbguimaraes]

The best part is "p_fVal1 == 0". Gotta save those FPU cycles.

[Falco Girgis]

I don't want to live on this planet anymore...

[Tim Wilson]

Oh dear...

[Rebornxeno]

I've been looking for just this! Thanks!

[dandymcgee]

I've been looking for just this! Thanks!

No problem, just don't forget to add to the end:

system("pause");

[Falco Girgis]

Don't forget a few of these as well...

Code: Select all

__asm__ __volatile__ ("nop");

[YourNerdyJoe]

Today was a good day, then I saw this.

[dandymcgee]

Found this same function copy/pasted in another project today. *dies a little bit more inside*

[YourNerdyJoe]

Found this same function copy/pasted in another project today. *dies a little bit more inside*

lol probably just thought "hey it has 'safe' in the name so it must be good"

[guding12]

Why set fReturn to 0 when it's going to divide by zero anyways? I rate this 0/0.

[bbguimaraes]

Why set fReturn to 0 when it's going to divide by zero anyways? I rate this

Floating point exception (core dumped)

[ChrisGorsky]

That just made my night...

Saw this today. I don't even.. I'm speechless.

private double SafeDivide(double p_fVal1, double p_fVal2)
{//will not fail if dividing by zero

    double fReturn = 0;

    try
    {

        if ((p_fVal1 == 0) || (p_fVal2 == 0))
        {
            fReturn = 0;
        }

        fReturn = p_fVal1 / p_fVal2;

    }
    catch
    {
    }
    return fReturn;
}