vendredi 25 janvier 2019

Doing the same thing inside various 'if' statements, can be improved?

I was coding when i faced this problem and i would like to know if this is correct or it can be improved for performance reasons:

            if (color == 0) {
                if (c.val[0] >= 0 && c.val[0] <= 64) {
                    //Black
                    Paint(cursor.x + x, cursor.y + y);
                }
            }
            else if (color == 1) {
                if (c.val[0] >= 64 && c.val[0] <= 128) {
                    //Grey
                    Paint(cursor.x + x, cursor.y + y);
                }
            }
            else if (color == 2) {
                if (c.val[0] >= 128 && c.val[0] <= 192) {
                    //White Gray
                    Paint(cursor.x + x, cursor.y + y);
                }
            }

As you can see im doing the exact same thing inside all IF statements and it looks kind weird, the program works as i want but i really think im missing something..

Thanks!

Aucun commentaire:

Enregistrer un commentaire