1. Ничего не значащие имена переменных (назовите их нормально). Вместо "t", что-то типа updateTimer, хотя не понятно что он вообще делает в этом классе.
2. "Магические числа". Заведите константы для состояний и не сравнивайте числа как строки.
3. Постоянные обращения к вышестоящим объектам. Это нарушение принципа инкапсуляции.
4. Объект сам себя удаляет.
Цитата:
|
Подскажите, как я должен был поступить?
|
Нет никакой необходимости дергать родителя напрямую. game state и artefact можно было передать в конструктор объекта, а checkCrush должен проверять вышестоящий (или специальный) класс.