Afficheur 7 segments

Oui c'est un afficheur à cathode commune. J'ai mis une résistante a chaque sortie de l'afficheur puis je les ai relié aux différentes entrées de l'arduino. Ce n'est pas ce qu'il fallait faire?