Zamysł jak najbardziej słuszny, niepowiem, ale... ;)
- wątpię czy gdziekolwiek indziej będzie używane jeszcze coś w stylu splash activity
- nazwa klasy ni jak nie wskazuje na to, że tam dzieje się coś na wątkach i że odpalane są jakieś inne activity. Nie wiem czy ktoś pomyślałby o podziedziczeniu z tej klasy w takim celu
- nie jestem pewien, czy klasa daje na tyle dużo możliwości, żeby z niej dziedziczyć - jest dość analogiczna do AsyncTask, jednak nigdzie nie zwraca wyniku, nie przyjmuje parametrów, nie daje możliwości śledzenia postępu wątku pobocznego itp. To nie są zarzuty, że klasa jest zła, chodzi tylko o to, że w przypadku dziedziczenia trzeba by napisać samemu w klasie pochodnej większość rzeczy która może być potrzebna.
- idąc tropem umożliwiania łatwego dziedziczenia powinniśmy też dać klasie pochodnej możliwość chociażby zmiany minimalnego timeoutu wyświetlania, co teraz nie jest możliwe. Skoro i tak trzeba by było zmieniać tę klasę, żeby to osiągnąć, czy wtedy nie będzie się dało też łatwo zmienić metod o których rozmawiamy?

Nie zrozum mnie źle - bardzo fajnie że nie wrzucasz całego kodu do jednej metody, ale wyłączasz go do innych metod, że myślisz o reużywalności, dziedziczeniu i całej reszcie ;) Chodzi mi tylko o to, że w tym konkretnym przypadku jest to dla mnie bardzo nadmiarowe, a utrudnia czytelność. Z własnego doświadczenia wiem, że jak zostawiasz nadmiarowe metody, które może kiedyś się przydadzą, to prawie na 100% tak nie będzie, więc chcę Cię od tego ustrzec :)