ciao
anche io non me la cavo bene con js, ma mutuando da php vedo che definisci una funzione
function valorelettera(lettera,value)
che accetta due valori
poi qundo la richiami (es)
var q1= valorelettera('q');
ne assegni uno solo, da cui l'alert che ti dice che uno è indefinito