en que falla mi consulta?

[ZeroZ]

Quiero actualizar grupo, disco, formato y localizacion mediante un formulario donde se le pasa el id.
Esta consulta la uso para actualizar, está bien?

$cons="UPDATE discos SET grupo=$grupo, disco=$disco, formato=$formato, localizacion=$localizacion WHERE id='$id'";