why do you have spinner = new Spinner(this)
in the set-up ?
surely you already have a Spinner in the XML of your layout, then you simply do spinner = (Spinner) findViewById(R.id.WHATEVER_THE_ID_IS_IN_THE_XML);
so you don't need a new
P.S. this is how I define a Spinner in an XML layout