Хорошее MVC

[in4core]

Ни в коем случае, я не обижаюсь, ты сказал все верно, я уже давно не развиваюсь в этой сфере, дабы не испытываю большой нужды, у меня четко поставленная концепция разработки программ на МВС, лишнего мне не надо. А вот по поводу - пишу на нем потому что МОДНО - тут не прав, я пишу, потому что это очень УДОБНО, удобнее, пока ничего не встречал, как в плане дописывания модулей, так и в плане быстрой организации любого приложения.
Psycho Tiger. Давай тогда поставим вопрос ребром. ( не знаю как у тебя, у меня программирование это второй заработок , скорее больше хобби ). У меня нет ОГРОМНОГО ( подчеркну это слово, дабы тем не менее всегда узнаешь, что то новое) желания развития дальше в этой сфере
а) Все, что я знаю - достаточно для того, чтобы писать хороший, чистый код, который будет выполнять то, что от него требуется
б) У меня нет ни одного приложения, которое не работает или работает с глюками.
в) У меня стабильная клиентура, которая мной полностью довольна, мне не нужно рыпаться по фриланс сайтам ища работу за копейки.
г) При том, что это мое хобби, так скажем, мой заработок составляет в среднем 3000$ в месяц.
И так, я имею все, что нужно, а так же высокий ЗП, даже по МСК, если считать. О каком саморазвитии мне нужно подумать, скажи мне, студент? Сидеть, читать книжки по паттернам, статьи на форумах англоязычных и т.п., ради лишь того, чтобы сидеть тут с тобой и мерится письками, кто лучше понимает МВС? Ну это просто смешно.
Да несомненно, это может еще и пригодится работая в команде флешеров, но меня данная стезя не привлекает , подстариваться под кого то, соблюдать конвенции принятые только в данном офисе и т.п. - фу и фу.
На самом деле ты еще просто молод, вот и прет у тебя юношеский максимализм, типа я круче всех, могу со всеми поспорить и т.п. - ну это не плохо и не хорошо, это в порядке вещей, с возрастом придет осознание, и естественно ты забудешь, о том, как сидеть и пояснять всем , что такое MVP, а что MVVM , у тебя будут слегка другие приоритеты в жизни Удачи.
И я думаю далее, лучше помоги человеку, который задал вопрос, дабы тема вроде как не для холивара, а не спорь со мной, все равно не переспоришь)))

[Babylon]

http://jessewarden.com/2012/08/backb...evelopers.html

[Akopalipsis]

Прежде чем начать оправдываться, хочу поблагодарить in4core за его труд!
Если бы не он, то возможно я бы и не получил ответа, на свой глупый вопрос.
Такой "код".. я даже не буду говорить от куда он взялся, лучше его раз и навсегда забыть.
Статью Psycho Tiger я не один раз прочёл, но возможно и вправду что то не допонял. Ещё есть догадка, что меня не понимают из-за того, что я хочу сделать так, как в жизни не когда не делают.

[Dukobpa3]

Примерно так.
Контроллер инитит модель и вью.
Вью подписывается на модель.
Контроллер подписывается на вью.
Когда из вью что-то прилетает в контроллер - он может дернуть модель и что-то в ней поменять. А модель в свою очередь собщит об изменении наружу.
Так что всё грубо говоря верно, только кода мало, думаю когда будет больше - появятся еще вопросы.

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

Т.е. именно эту ситуацию лучше было решить без контроллера вообще.
В модели лежит словарь:

{ 0:0xF5F5D6, 1:0xDBF5D6}

А вью при клике просто лезет в модель и смотрит нужный цвет.

//==========================

Второй вариант.

Так как сделал ты. Но в модели есть переменная : currentColor
И в методе в котором у тебя свитч и диспатч пачки событий наружу - просто установка значения в эту переменную, и диспатч ОДНОГО события CHANGE вконце.

Вью получает событие, и рисует model.currentColor

//====================

Короче идея в том что в модели не только логика. Но и данные.
Т.е. логика которая не меняет данных - модели не очень нужна. Хотя туда можно добавить какие-то методы типа:

function getCurrentColorByButtonIndex(id:int):uint //(который по сути геттер, и дергается он не из контроллера, а из вью)

[Akopalipsis]

Dukobpa3 Спасибо Вам за помощь!
я удалил предыдущий код, добавил изменённый + новый класс -...

думаю когда будет больше - появятся еще вопросы.

Так и есть, я просто сначала убедиться хочу, что самое минимальное понимаю, а потом уже продвигаться к тому, что я вообще не знаю как сделать. По этому ушёл делать пример дальше, чтобы Вам понятней было.

package 
{
	import flash.display.Sprite;
 
	public class Main extends Sprite 
	{
 
		public function Main() 
		{
			new BaseController(this);
		}
 
	}
 
}

package 
{
	import flash.display.DisplayObjectContainer;
	import flash.events.DataEvent;
 
	public class BaseController 
	{
		private var _doc:DisplayObjectContainer;
 
		private var _model:Model;
		private var _view:View;
 
		private var _storageObject:Object =
		{
 
		}
 
		public function BaseController($doc:DisplayObjectContainer) 
		{
			super();
 
			_doc = $doc;
 
			_model = new Model();
			_model.DOCSize = [_doc.root.parent.stage.stageWidth, _doc.root.parent.stage.stageHeight];
			trace(_doc.root.parent)
			_view = new View(_model);
			_view.addEventListener('MENU', view_viewClickHandler);
 
			_doc.addChild(_view);
		}
 
		private function view_viewClickHandler(event:DataEvent):void 
		{
			_model.accessStatus(event.data);
		}
 
	}
 
}

package 
{
	import adobe.utils.CustomActions;
	import flash.events.Event;
	import flash.events.EventDispatcher;
 
	public class Model extends EventDispatcher implements IModel 
	{
		private var _DOCSize:Array = [];
		private var _getColor:uint ;
 
		private var _menuStatus:String;
 
		public function Model()
		{
			super();
 
		}
		public function accessStatus($button:String):void
		{
			if ($button == '0')
			{
				if (_menuStatus == $button) return;
				_menuStatus = $button;trace(';;;;;;;;;;;')
				_getColor = 0xA8F2ED;
				dispatchEvent(new Event('MODEL_MENU'));
			}
			else
			if ($button == '1')
			{
				if (_menuStatus == $button) return;
				_menuStatus = $button;
				_getColor = 0xEBF2A8;
				dispatchEvent(new Event('MODEL_MENU'));
			}
			else
			if ($button == '2')
			{
				return;
			}
			else
			{
				throw new Error('ERROR');
			}
		}
 
		public function get DOCSize():Array 
		{
			return _DOCSize;
		}
 
		public function set DOCSize(value:Array):void 
		{
			_DOCSize = value;
		}
 
		public function get getColor():uint 
		{
			return _getColor;
		}
 
		public function set getColor(value:uint):void 
		{
			_getColor = value;
		}
 
	}
 
}

package 
{
	import flash.events.IEventDispatcher;
 
	public interface IModel extends IEventDispatcher
	{
		function accessStatus($button:String):void;
		function get DOCSize():Array; 
		function get getColor():uint ;
	}
 
}

package 
{
	import flash.display.Sprite;
	import flash.events.DataEvent;
	import flash.events.Event;
	import flash.events.MouseEvent;
 
	public class View extends Sprite 
	{
		private var _model:IModel;
 
		private var _button:Button;
		private var _menuContainer:Sprite;
		private var _backgraund:Sprite;
 
		public function View($model:IModel) 
		{
			super();
 
			this.mouseEnabled = false;
 
			_model = $model;
			_model.addEventListener('MODEL_MENU', model_modelMenuHandler);
 
			_backgraund = new Sprite();
			_backgraund.mouseChildren = false;
			_backgraund.mouseEnabled = false;
			_backgraund.graphics.beginFill(0xF7E1D5);
			_backgraund.graphics.drawRect(0, 0, _model.DOCSize[0], _model.DOCSize[1]);
			_backgraund.graphics.endFill();
			super.addChild(_backgraund);
 
 
			this.addEventListener(MouseEvent.MOUSE_DOWN, this_mouseDownHandler);
 
 
			var id:int = 0;
			_menuContainer = new Sprite();
			_menuContainer.name = 'menu';
			super.addChild(_menuContainer);
 
			for (var i:int = 0; i < 3; i++) 
			{
				_button = new Button();
				_button.id = id++;
				_button.x += 72 * i;
				_menuContainer.addChild(_button);
			}
 
 
		}
 
		private function model_modelMenuHandler(event:Event):void 
		{
			_backgraund.graphics.clear();
			_backgraund.graphics.beginFill(uint(_model.getColor));
			_backgraund.graphics.drawRect(0, 0, _model.DOCSize[0], _model.DOCSize[1]);
			_backgraund.graphics.endFill();
		}
 
		private function this_mouseDownHandler(event:MouseEvent):void 
		{
			super.dispatchEvent(new DataEvent('MENU',false,false,event.target.id));
		}
 
	}
 
}

package 
{
	import flash.display.Sprite;
 
	public class Button extends Sprite 
	{
		private var _id:int;
 
		public function Button() 
		{
			this.graphics.beginFill(0xA9B6F1);
			this.graphics.drawRect(0, 0, 70, 45);
			this.graphics.endFill();
		}
 
		public function get id():int 
		{
			return _id;
		}
 
		public function set id(value:int):void 
		{
			_id = value;
		}
 
	}
 
}

[in4core]

_model.DOCSize = [_doc.root.parent.stage.stageWidth, _doc.root.parent.stage.stageHeight];

_model.DOCSize = [_doc.stage.stageWidth, _doc.stage.stageHeight];

_view.addEventListener('MENU', view_viewClickHandler);

Тут лучше делать кастом, типа ViewEvents - это хорошая практика. Так же стоит называть константы логично, например MENU_CLICK_EVENT!!

$button:String)

Аналогично - называйте логично, кнопка не может быть строкой, сам же потом запутаетесь. В моделе так же создать кастом ModelEvents - И делать адекватные имена MODEL_CUSTOM_EVENT.

А вот с интерфейсами спешить не надо, просто так шмалять ими не стоит, лучше на первых порах и в небольших проектах их не использовать, толку они вам не сделают, только мозг засорят на начальной стадии. В любом случае со временем вы сами начнете понимать когда они действительно пригодятся.

public function View($model:IModel)

Я бы посоветовал привыкать к
view.model = someModel. То есть к сеттерам , а не передаче в конструктор. Потому, что если моделей надо будет больше пихать их в конструктор, ну не очень красиво, да и на момент создания конструктора они могут быть Null.

super.dispatchEvent(new DataEvent('MENU',false,false,event.target.id));

Нужно так :

dispatchEvent(new DataEvent(DataEvent.MENU_CUSTOM_EVENT, target.id));

Добавлено через 2 минуты
Ну и наконец, зачем вам все это надо то? Что вы хотите получить? Какое то приложение делаете или что? Если свой сайт решили писать, я могу показать, как это будет в MVC эквиваленте, конечно не идеально, но похожеее на правду

[Akopalipsis]

in4core Спасибо за учение, я естественно всё учел и ещё у меня есть один вопрос - вот там где Вы писали про кнопку, что она не может быть строкой... я вчера когда делал события, то первым делом вспомнил, что желательнее использовать события нативные, так как такая практика в будущем поможет делать легко переносимые модули. Но когда я дошел до кода с кнопкой, то сразу пришёл к выводу, что лучше сделать свое событие, куда добавить пункт самого обьекта-target и уже полноценно считывать его id. И ещё одним аргументом в пользу собственного события было сообщение, когда говорили, что вот по таким строковым сравнениям, как раз можно попасть в неприятную ситуацию. По этому - как лучше делать, нативными сообщениями со строкой или собственное событие с передачей target?

Ну и наконец, зачем вам все это надо то? Что вы хотите получить?

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

[in4core]

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

Всегда, кто нибудь да что то скажет - это интернет. Не бывает хорошего МВС, вот с чего надо начинать. Бывают разные реализации.
Кто вам сказал что нативные события лучше испольщовать чем кастомные? Ничего подобного - делать надо так, как удобоваримо и располагает ситуация. Чтобы ответь на вопрос передачи таргета,строк или еще чего, надо сначала сформулировать задачу, типа хочу чтобы по клику на меню создавался модуль Б и т.п. Тогда вам подскажут - а пока говорить неочем

[Akopalipsis]

Не знаю когда закралась ко мне в голову эта мысль, но точно знаю, когда я решил, что её уже не оставлю без внимания - этот момент наступил после этого поста. Но я не думал, что у меня появится ещё одна мысль, которая побудит меня на столь раннее задание этого вопроса.
Вот все говорят, что вью не имеет ссылки на контроллер ( и как считаю я, только по этому не рассматривают ПЛЮСЫ от этого ), но если немного отдалится от этого и забыть о EventDispatcher, то как бы она могла передать массив данных требуемые в модели? Или вью, которая посылая события EventDispatcher ( по сути калбеки ), разве не имеет ссылку на контроллер? Ведь это одно и тоже... В чем я не прав?
И хоть застрелите, но я тоже подсел на эту тему, не могу не чего с собой поделать))

[Psycho Tiger]

Вью имеет ссылку на контроллер на уровне приложения, но не на уровне кода, с которым ты работаешь.
Ядро может быть сколь-угодно-криво-написанное, но если оно имеет хорошее API и достаточный функционал – тебе никогда не придется лезть в это ядро, а сам ты сможешь писать хороший код, который легко поддерживать.