Если делаете переключатель между тремя темами оформления, то проще сделать так:
CSS:
/* общие для всех тем */
.navigation { /* style */ }
.sect-one { /* style */ }
.project-title { /* style */ }
/* конкретные для темы evil вдобавок к общим */
body.theme-evil .navigation { /* style */ }
body.theme-evil .sect-one { /* style */ }
body.theme-evil .project-title { /* style */ }
/* конкретные для темы hacker вдобавок к общим */
body.theme-hacker .navigation { /* style */ }
body.theme-hacker .sect-one { /* style */ }
body.theme-hacker .project-title { /* style */ }
/* ..... */
Тогда JS вообще легко уменьшится до нескольких строк:
$(document).ready(function() {
var themes = ['evil', 'hacker', 'basic'];
var oldTheme = 'basic';
themes.forEach(function(theme) {
$('#' + theme).click(function() {
$('body').removeClass('theme-' + oldTheme);
$('body').addClass('theme-' + theme);
oldTheme = theme;
});
});
});
