Разделение программы на классы

[Wolsh]

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

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

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

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

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

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

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

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

[zzSpirit]

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

[gagaga]

Сообщение от samana

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

		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

[zzSpirit]

Сообщение от gagaga

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

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

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

[TWETTI]

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

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

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

Да.

Сообщение от zzSpirit

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

if (down && up)
{
    trace("Нажата клавиша вниз и вверх");
}

[zzSpirit]

Сообщение от TWETTI

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

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

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

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

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

Сообщение от TWETTI

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

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

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

Сообщение от TWETTI

if (down && up)
{
    trace("Нажата клавиша вниз и вверх");
}

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

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

		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;
			}
		}

[TWETTI]

Сообщение от zzSpirit

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

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

[zzSpirit]

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

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

Сообщение от TWETTI

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

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

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

[TWETTI]

Сообщение от zzSpirit

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

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

[zzSpirit]

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

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