- 1
- 2
- 3
this.editable = true;
this.itemEditor = new ClassFactory(UIDEditor);
this.disableInEdit = true;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 128
−114
this.editable = true;
this.itemEditor = new ClassFactory(UIDEditor);
this.disableInEdit = true;
Чудесное сочетание значений при котором мы не можем редактировать запись но при этом видеть в окошке редактирования...
Это только два месяца разработки, интересно, костыли какой кудрявости я увижу через год?
−133
protected function filterGroupResizedHandler(event:Event):void {
var label1Width:Number = subform1.width-viewInput.width;
var label2Width:Number = subform2.width-statusSelect.width;
var label3Width:Number = subform3.width-functionSelect.width;
var label4Width:Number = subform4.width-legalEntitySelect.width;
var label5Width:Number = subform5.width-primaryTraderSelect.width;
var averageFieldWidth:int = (filterGroup.width-(label1Width+label2Width+label3Width+label4Width+label5Width)-filterGroup.gap*6)/5;
subform1.width = label1Width+averageFieldWidth;
subform2.width = label2Width+averageFieldWidth;
subform3.width = label3Width+averageFieldWidth;
subform4.width = label4Width+averageFieldWidth;
subform5.width = label5Width+averageFieldWidth;
}
−151
internal final function setResponseObject (response : Object) : void
{
if (response != null)
{
this.parseResponse(response);
}
}
protected function parseResponse (response : Object) : void
{
this.setResult(response);
}
protected final function setResult (result : Object) : void
{
this._result = result;
}
−145
// это дорога
/**
* Records status changed handler. This happens when user tries to change
* status of one or more records
*/
private function recordsStatusChangedHandler (e : TableBasedModelEvent) : void
{
this.makeNewSearchAfterRecordsChanged();
}
// это мост
/**
* @private event handler
* Here we prevent default inline editing
*/
private function cellEditBeginHandler (e : AdvancedDataGridEvent) : void
{
e.preventDefault();
}
Попросила откомментировать тонкие моменты в коде.
О том что события глушатся потому что данные о том надо ли редактировать конкретное поле мы смотрим в колонке в которой включено редактирование, но редактируем не инлайн в табличке а по нажатию кнупки в окошке, никому конечно знать не надо...
−181
private function yesClickHandler(e:MouseEvent):void
{
this.hide();
try{
this.yesHandler();
}catch(e:Error){};
}
private function noClickHandler(e:MouseEvent):void
{
this.show();
try{
this.noHandler();
}catch(e:Error){};
}
Гадай, любимый коллега, чего у тебя там не работает после нажатия кнопки ок.
Или может это без злого умысла и временно забыл что можно использовать if(noHandler) чтобы проверить наличие каллбека?
−173
private function loginSuccessHandler (e : LoginEvent) : void
{
...
if (this.loggedInUser != null)
{
this.applicationView.showLoggedInUserInfo("Welcome " + this.loggedInUser.email);
}
else
{
this.applicationView.showLoggedInUserInfo("No logged in user");
}
}
private function get loggedInUser () : User
{
return this.usersModel.loggedInUser;
}
private function get usersModel () : IUsersModel
{
return UsersModel.instance;
}
Прячем точечки за оопешечкой.
−172
package com.db.prds.modules.application.presenter.commands
{
import ***.modules.application.presenter.ApplicationPresenter;
import ***.modules.application.presenter.IApplicationPresenter;
public class GoAboutCommand extends MainMenuCommand
{
public function GoAboutCommand (appPresenter : IApplicationPresenter)
{
super(appPresenter);
}
override public function execute () : void
{
this.appView.showModuleView(ApplicationPresenter(this.appPresenter).aboutPresenter.view);
super.execute();
}
}
}
Вы все еще не пользуетесь ооп? Тогда мы идем к вам!
−104
private function get prodComplRatingDict():DBDictionary
{
return this.dictModel.getDict(Dictionaries.PROD_COMPL_RATING_DICT);
}
private function get dictModel():IDictionariesModel
{
return DictionariesModel.instance;
}
−181
package ***.modules.login.view
{
import ***.modules.login.view.skins.LoginViewSkin;
import ***.uni.mvp.View;
import spark.components.Button;
import spark.components.TextInput;
public class LoginView extends View implements ILoginView
{
/************************************************************************************
* SKIN PARTS *
************************************************************************************/
[SkinPart(required="true")]
public var emailInput:TextInput;
[SkinPart(required="true")]
public var loginButton:Button;
/************************************************************************************
* VARS *
************************************************************************************/
public function LoginView()
{
super();
}
public function get emailInputWidget():TextInput
{
return this.emailInput;
}
public function get loginButtonWidget():Button
{
return this.loginButton;
}
protected override function configureView():void
{
this.setStyle("skinClass", LoginViewSkin);
}
}
}
Наклали в пространство имен...
−155
override public function execute():void
{
var maintainProductAttributePresenter:ITableBasedPresentrer = this.appPresenter.maintainProductAttributePresenter;
var maintainProductAttributeView:View = maintainProductAttributePresenter.view;
this.appView.showModuleView(maintainProductAttributeView);
this.appView.showModuleHeader(maintainProductAttributePresenter.getModuleTitle());
maintainProductAttributePresenter.updateData();
}
Сижу, копипазжу... Специально не стала выбиваться из стандарта... Может хоть на секунду мысль о том что мы занимаемся маразмом кому нибудь в голову придет?