any help please

Projects Programming
2 
    shiftOut(redData, clockPin, MSBFIRST, 0);
    shiftOut(redData, clockPin, MSBFIRST, 0);
    shiftOut(redData, clockPin, MSBFIRST, 0);
    shiftOut(redData, clockPin, MSBFIRST, 0);
    shiftOut(redData, clockPin, MSBFIRST, 0);
    shiftOut(redData, clockPin, MSBFIRST, 0);
    shiftOut(redData, clockPin, MSBFIRST, 0);
    shiftOut(redData, clockPin, MSBFIRST, 0);
    shiftOut(redData, clockPin, MSBFIRST, 0);
    shiftOut(redData, clockPin, MSBFIRST, 0);

Good thing you didn't have to do that 1000 times.

Get rid of the commented out code. Tell us what the code ACTUALLY does.