no uses variables, usa un array. recorre el array con el ciclo, pero en lugar de la declaración (18) usa el largo del array (como te dice mycol)
por otro lado, deberás usar una función anónima y para pasarle el indice
Cita: for (var i = 0; i < color.length; i++) {
(function(i) {
$("#modules-colors-nav-"+i).click(function(event){
$("#nav").css({ 'background', color[i]});
});
})(i);
}