togli return in tutte e 4 le righe dove lo hai messo.

return è una parola chiave di javascript che si usa solo all' interno di funzioni e serve o a dare indicazione che a quel punto la funzione deve essere o abortita o terminata, oppure per restituire ("ritornare") un valore specifico come risultato dei calcoli che la funzione ha effettuato, esempio
return 10
return UnNomeDiVariabileQui
ecc...

Tui non hai funzioni lì, hai solo righe procedurali (if, else ecc) al di fuori di una funzione che le encapsuli

function nomeFunzione(){
//codice qui
}

Quando delle istruzioni sono incluse dentro una funzione, vengono messe diciamo così in "attesa" cioè non vengono eseguite finchè un qualche evento non invoca espressamente la esecuzione di quella funzione e dei codici che vi sono incapsulati dentro.

Quando invece delle istruzioni non sono incluse dentro una funzione (ed è il tuo caso) vengono eseguite immediatamente non appena lette dal browser, senza alcuna attesa.
In tali ultime espressioni, la parola chiave return non è ammessa: essa è riservata per i codici incapsulati dentro delle funzioni.