Need help writing more efficient code (total newbie)

Projects Programming
3

Possible skeleton code -

// BUG CORRECTED

#define ARRAY_SIZEOF(ARRAY) (sizeof(ARRAY) / sizeof(ARRAY[0]))

const uint8_t   pinLED0     =  3;
const uint8_t   pinLED1     =  5;
const uint8_t   pinLED2     =  6;
const uint8_t   pinLED3     =  9;
const uint8_t   pinLED4     = 10;
const uint8_t   pinLED5     = 11;
const uint8_t   pinLED6     = 13;

const uint8_t   pinsLED[]   = { pinLED0, pinLED1, pinLED2, pinLED3, pinLED4, pinLED5, pinLED6 };

void loop()
{
    for ( int i = 0; i < ARRAY_SIZEOF(pinsLED); i++ )
    {
        int value = <some calculations to derive value>;

        analogWrite(pinsLED[i], value);
        delay(100);
    }
}


void setup()
{
    for ( int i = ARRAY_SIZEOF(pinsLED); i--; )
    {
        pinMode(pinsLED[i], OUTPUT);
    }
}