Uhm creo que te seria mas facil haciendolo asi:
Código javascript
:
Ver original<script>
function valuecheck(check)
{
alert('El check '+check.name+' tiene el valor '+check.value);
}
</script>
Código HTML:
<input type="checkbox" name="check1" value="1" onclick="valuecheck(this)"/>
<input type="checkbox" name="check2" value="2" onclick="valuecheck(this)"/>
<input type="checkbox" name="check3" value="3" onclick="valuecheck(this)"/>
Lo que pasa con tu codigo es que no puedes acceder asi como lo tienes a los valores del checkbox, al enviarle 'this' como parametro, le envias el checkbos y nada mas accesas a sus propiedades con .value o .name etc.
PD: tus checkbox tienen 2 atrivutos value, eso no se puede xP