Koden känns tämligen förvirrande, inte helt lyckat med klassnamnet value i sammanhanget kanske
Vad är det för span? Som allstars påpekar går det alldeles utmärkt att direkt hämta värdet från ett val och normalt sett brukar man använda change istället för click som event på en select. Känns som en del relevant html saknas i exemplet.