deleted 33 characters in body; edited title

Source Link

edited Aug 15, 2015 at 18:59

34.9k
13
133
237

Are the following IF statements acceptable? Checking opcode values

I haveWhat troubles me are the following code:empty /**/ brackets. Is there a cleaner way of handling this logic?

if (opcode == OP_0) { /* continue */}
    else if (opcode == OP_1NEGATE) { /* continue */}
    else if (opcode >= OP_1 && opcode <= OP_16) { /* continue */}
    else { throw new Exception("decodeFromOpN called on non OP_N opcode"); }

What troubles me are the empty /**/ brackets.

Is there a cleaner way of handling this logic?

Are the following IF statements acceptable?

I have the following code:

if (opcode == OP_0) { /* continue */}
    else if (opcode == OP_1NEGATE) { /* continue */}
    else if (opcode >= OP_1 && opcode <= OP_16) { /* continue */}
    else { throw new Exception("decodeFromOpN called on non OP_N opcode"); }

What troubles me are the empty /**/ brackets.

Is there a cleaner way of handling this logic?

Checking opcode values

What troubles me are the empty /**/ brackets. Is there a cleaner way of handling this logic?

if (opcode == OP_0) { /* continue */}
    else if (opcode == OP_1NEGATE) { /* continue */}
    else if (opcode >= OP_1 && opcode <= OP_16) { /* continue */}
    else { throw new Exception("decodeFromOpN called on non OP_N opcode"); }

Question Protected by rolfl

occurred Mar 11, 2014 at 21:59

deleted 17 characters in body; edited tags

Source Link

edited Mar 11, 2014 at 1:37

Jamal

34.9k
13
133
237

I have the following code:

    if (opcode == OP_0) { /* continue */}
        else if (opcode == OP_1NEGATE) { /* continue */}
        else if (opcode >= OP_1 && opcode <= OP_16) { /* continue */}
        else { throw new Exception("decodeFromOpN called on non OP_N opcode"); }

What troubles me isare the empty /**//**/ brackets.

Is there a cleaner way of handling the this logic?

I have the following code:

    if (opcode == OP_0) { /* continue */}
        else if (opcode == OP_1NEGATE) { /* continue */}
        else if (opcode >= OP_1 && opcode <= OP_16) { /* continue */}
        else { throw new Exception("decodeFromOpN called on non OP_N opcode"); }

What troubles me is the empty /**/ brackets.

Is there a cleaner way of handling the this logic?

I have the following code:

if (opcode == OP_0) { /* continue */}
    else if (opcode == OP_1NEGATE) { /* continue */}
    else if (opcode >= OP_1 && opcode <= OP_16) { /* continue */}
    else { throw new Exception("decodeFromOpN called on non OP_N opcode"); }

What troubles me are the empty /**/ brackets.

Is there a cleaner way of handling this logic?

Tweeted twitter.com/#!/StackCodeReview/status/443154982347739136

occurred Mar 10, 2014 at 22:42

Source Link

asked Mar 10, 2014 at 21:12

makerofthings7

713
2
8
14

Are the following IF statements acceptable?

I have the following code:

    if (opcode == OP_0) { /* continue */}
        else if (opcode == OP_1NEGATE) { /* continue */}
        else if (opcode >= OP_1 && opcode <= OP_16) { /* continue */}
        else { throw new Exception("decodeFromOpN called on non OP_N opcode"); }

What troubles me is the empty /**/ brackets.

Is there a cleaner way of handling the this logic?

c#

Stack Exchange Network

Return to Question

Are the following IF statements acceptable? Checking opcode values

Are the following IF statements acceptable?

Checking opcode values

Are the following IF statements acceptable?