ci sono apici singoli all'interno delle stringhe devi anteporre il carattere \ se vuoi considerarle come parte delle stringhe stesse e non il carattere di fine stringa, se usi un editor di testo decente di solito vedi le stringhe in colore diverso ed è facile accorgersi del problema... inoltre se non usi le graffe nell'if js considererà solo la prima linea di codice sottoposta a tale verifica, mentre le restanti righe di codice saranno sempre eseguite.

p.s. non vedo nessun ciclo nel codice che hai postato