Форум Flasher.ru - Разделение программы на классы

Форум Flasher.ru (http://www.flasher.ru/forum/index.php)

- ActionScript 3.0 (http://www.flasher.ru/forum/forumdisplay.php?f=83)

- - Разделение программы на классы (http://www.flasher.ru/forum/showthread.php?t=191788)

Цитата:

Класс Main - это главный контроллер

Контроллер — это класс Keys. Не используйте слово "контроллер" в смысле "центр логики". Контроллер не более чем драйвер, переводящий некие действия пользователя в понятные приложению данные.

Цитата:

Если нет, то зачем он нужен?

Для отображения персонажа, вестимо. Для управления его анимацией, поворотами, ходьбой, прыжками и прочими визуальными эффектами. А что он, по-вашему, должен хранить? Если бы персонажей было сто, был бы смысл каждому из них хранить свою "скорость". А когда он один, достаточно карте иметь одно приватное свойство "скорость персонажа". И то только если эта скорость на самом деле переменная.

Цитата:

в классе Beatle должны быть свойства + методы

Безусловно. Свойства и методы, относящиеся к данному экземпляру. Методы "заплакать", "подпрыгнуть", "присесть" и т.п., то есть "перейти на такой-то кадр анимации". И свойства типа ширина и высота. Потому что даже х и у нельзя считать его собственными свойствами "в вакууме". Это его свойства в системе координат его контейнера (карты). Отсюда и методы типа moveDown выглядят как-то смешно. С таким же успехом можно написать beatle.y += persSpeed;. Вот только логики здесь будет гораздо больше: система изменяет свои части, а не части изменяют систему. Это называется словом Иерархия. А его противоположность — Хаос, когда шашки сами решают куда им ходить.

Цитата:

Что посоветуете изучить/прочитать, чтобы усвоить проектирование ООП приложений?

Ну хотя бы ActionScript 3.0. Шаблоны проектирования.

Ура! Я понял о чем Вы говорили и написал передвижение своего жука по всем правилам. Прочитал Ваш последний пост и всё сходится. Теперь намного лучше понимаю ООП.

Цитата:

Сообщение от samana (Сообщение 1114067)

Если вы хотите сделать управление внутри класса Beatle, то это может выглядеть примерно так (простой вариант):

Код AS3:

                private function stage_keyUp(e:KeyboardEvent):void 

                {

                        if (e.keyCode == 37) keyLeft = false;

                        if (e.keyCode == 39) keyRight = false;

                }

 

                private function stage_keyDown(e:KeyboardEvent):void 

                {

                        if (e.keyCode == 37) keyLeft = true;

                        if (e.keyCode == 39) keyRight = true;

                }

Тогда в Маин классе остаётся только добавить экземпляр класса Beatle на сцену.

это не правильно... 1 кейкод не может быть одновременно равен 2-м значениям, нужно использовать конструкцию switch или хотябы if else if

Цитата:

Сообщение от gagaga (Сообщение 1114276)

Хм... В моем примере также сделано. Но где тут кейкод равняется двум значениям? Такого нету.
А если if else if или switch использовать - то нажатия двух клавиш одновременно не будут фиксироваться.
По-моему пример юзера samana правильный.

Добавлено через 5 минут
Упс, а хотя да. e.keyCode не может быть равен двум значениям, нужно использовать switch.
Только не совсем понимаю, если нажаты 2 и более клавиш, то через кадр по очереди будут разные коды в e.keyCode? Как тогда определяется нажатие двух клавиш, ведь действительно e.keyCode не может равняться двум значениям, если он не массив...

В вашем классе Keys функция onKeyDown и onKeyUp должна выглядеть так:

Код AS3:

private function onKeyDown(event:KeyboardEvent):void

{

        if (event.keyCode == 38)

        {

                up = true;

        }

        else if (event.keyCode == 40)

        {

                down = true;

        }

        else if (event.keyCode == 37)

        {

                left = true;

        }

        else if (event.keyCode == 39)

        {

                right = true;

        }

}

Одновременно клавиши нажиматься не могут. Нажали вниз, произошло события KeyboardEvent.KEY_DOWN, down стал равен true, затем через несколько миллисекунд нажали вправо, произошло тоже самое событие, теперь down и right равны true, остальные переменные false. При отпускании этих кнопок произойдут два события KeyboardEvent.KEY_UP и по очереди down и up будут вновь равны false.

Добавлено через 3 минуты

Цитата:

Сообщение от zzSpirit (Сообщение 1114285)

Только не совсем понимаю, если нажаты 2 и более клавиш, то через кадр по очереди будут разные коды в e.keyCode?

Да.

Цитата:

Сообщение от zzSpirit (Сообщение 1114285)

Как тогда определяется нажатие двух клавиш, ведь действительно e.keyCode не может равняться двум значениям, если он не массив...

Код AS3:

if (down && up)

{

    trace("Нажата клавиша вниз и вверх");

}

Цитата:

Сообщение от TWETTI (Сообщение 1114289)

Только что проверил, запустил и нажал сразу две клавиши. Жук двигался по диагонали, output такой:

Код AS3:

keyCode: 40

keyCode: 39

keyCode: 39

keyCode: 39

keyCode: 39

keyCode: 39

и так далее... Дальше всегда 39

Получается, клавиша вниз всегда оставалась true, даже тогда, когда аутпут показывал только клавишу влево. Ну, все правильно, хотя и не очень понятно, как в следующих кадрах АС3 знал, что все еще нажата клавиша вниз, если аутпут всегда выдавал только 39 на каждом кадре. Загадка.

Добавлено через 5 минут

Цитата:

Сообщение от TWETTI (Сообщение 1114289)

В вашем классе Keys функция onKeyDown и onKeyUp должна выглядеть так:

Изменил. понимаю, что так правильнее, хотя разницы и нету. По идее, быстрее должно работать.

Добавлено через 7 минут

Цитата:

Сообщение от TWETTI (Сообщение 1114289)

Код AS3:

if (down && up)

{

    trace("Нажата клавиша вниз и вверх");

}

Нет, я имел ввиду не как определить что нажаты две клавиши, а как AS3 определяет, что нажаты 2 клавиши. Посмотрите output. Там кейкод клавиши которая первая была нажата, а потом уже идет только кейкод клавиши, которая была нажата второй. Они идут не по очереди.

Добавлено через 14 минут
Изменил ИФы на Свитчи. И код лучше читается и меньше по размеру и правильней. Теперь там такие конструкции:

Код AS3:

                private function onKeyDown(event:KeyboardEvent):void

                {

                        switch(event.keyCode)

                        {

                                case 38 : up = true; break;

                                case 40 : down = true; break;

                                case 37 : left = true; break;

                                case 39 : right = true; break;

                        }

                }

Цитата:

Сообщение от zzSpirit (Сообщение 1114296)

AS3 никак не определяет что две клавиши нажаты. Это должны делать вы сами.

Обновленный архив с проектом загрузил по той же ссылке.

Добавлено через 1 минуту

Цитата:

Сообщение от TWETTI (Сообщение 1114302)

AS3 никак не определяет что две клавиши нажаты. Это должны делать вы сами.

Но ведь если 2 клавиши нажаты, он в output пишет сначала кейкод первой клавиши, а потом постоянно кей-код другой клавиши. (мой output написал выше)
При этом обе клавиши равны true. Хотя кейкод первой клавиши он больше не получал.

Добавлено через 4 минуты
Ой, опять туплю. Всё правильно. Первая клавиша не должна стать false, если не поступило команды о ее отпускании. Извиняюсь.

Цитата:

Сообщение от zzSpirit (Сообщение 1114303)

Но ведь если 2 клавиши нажаты, он в output пишет сначала один кейкод, а потом постоянно кей-код другой. (мой output написал выше)
При этом обе клавиши равны true.

Все правильно. В output выводится информация о том, какая клавиша сейчас нажата. Обе клавишы равны true потому, что вы в коде это учли.

Да. Уже написал выше. Всё правильно. Не было команды false для первой клавиши, поэтому она всегда была true, а в output выводился кейкод второй клавиши.

Добавлено через 1 час 24 минуты
Wolsh, Благодарю за книгу. Буду ее очень внимательно изучать. Благодаря Вашим подробным обьяснениям, многое понял. Большое Вам спасибо!