Au début de ton programme tu écris ça :
long valeur3 = 0 ;
Puis ensuite au milieu :
int valeur3=analogRead(capteur); //Relever la valeur3
Déjà un long et un int ce n'est pas la meme chose mais en plus tu as déjà déclare valeur3, il faut donc que tu écrives
valeur3=analogRead(capteur); //Relever la valeur3
Vu comment tu utilises la variable valeur3, un int est suffisent tu n'as pas besoin d'un long, pour plus d'explication va voir sur la doc de Arduino les différents types de variables mais la déclaration de ta variable devrait etre
int valeur3 = 0 ;