Recherche:

Le listener était défini dans le premier code que tu as donnés , puis ne l'était plus dans le second.

Fonctionne par étape , fait fonctionner ton bouton , puis rajoute le spinner (ou inversement).

A aucun moment tu ne défini de listener pour ton button1 , dans ton cas dans le onCreate rajoute :

button1.setOnClickListener(this);

Ton code lié à ton spinner est mal placé. Actuellement il est dans le onclick et non conditionné , ce qui veux dire qu'à chaque élément de ton interface qui va générer un onClick le code du spinner...