You may like to keep a boolean state for each LED, then the code becomes even simpler...
if (digitalRead (redled2Pin) == LOW)
digitalWrite (redled2Pin, HIGH);
else
digitalWrite (redled2Pin, LOW);
BECOMES
{global - up top}
bool redState;
{local - to your function}
redState = !redState;
digitalWrite (redled2Pin, redState);