Ваше утверждение для создания и показа Тоста является правильным. Вероятно, у вас логическая ошибка. Вы абсолютно уверены, что ваш оператор switch использует случай 8? Было бы неплохо поместить в каждый случай несколько операторов журнала, которые распечатывали бы какой это случай, чтобы вам было легче отлаживать происходящее.
Я не уверен, как выглядит остальная часть вашей деятельности, но, похоже, это какой-то метод обратного вызова прослушивателя щелчков. Если это так, у вас не должно быть
TextView color = (TextView) findViewById(R.id.text);
Внутри обратного вызова. Это означает, что findViewById () будет вызываться каждый раз при щелчке по вашему представлению. Поскольку это относительно дорогой метод, вызывать его более уместно, вызывая его один раз в вашем onCreate и просто сохраняя ссылку, которую вы получите, в остальное время, когда вам это понадобится.