1. А зачем их делать полями класса, если они нигде, кроме как в onCreate не используются?
2. Нет проверки на то, что findViewById вернет null.