Качество кода

[iiiusion]

Всем добрый вечер. Интересует ваше мнение по поводу моего кода https://bitbucket.org/iiiusion/coloreffect. Хочу понять производящее впечатление. Если не сложно и спасибо)

[in4core]

Где смотореть? Ниче не понятно. Выкладывайте суда

[Aquahawk]

лажа номер один

import flash.display.StageQuality;
import flash.events.Event;
 
import gameplay.GP_Dead_Window;
 
import gameplay.MouseController;
 
import gifts.GiftBase;
 
import gifts.GiftsController;
 
import global.*;
	import gameplay.GamePlay;
 
import igine.IG_KBC;
 
import igine.IG_Pool;
 
import igine.IG_StatusInformer;
import igine.IG_GUI;
import igine.Igine;
    import assets.BaseAssets

Добавлено через 37 секунд
два

		////////////////////
		public static var debugMode:Boolean = true;
        public var command:String = "gameplay";
		private var childs:Array = [];
		public var blackBox:BlackBox;
        public static var _gameplay:GamePlay;

что-то не удобно с бакета кописпастить. короче форматирование точно кол, дальше ща гляну

Добавлено через 6 минут
GP_Inventory.as конструктор точно не здоровый

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

 private function greenBoxOverFunc1():void{
        baloon.x = canvas.greenAch1.x;
        baloon.y = canvas.greenAch1.y - canvas.greenAch1.height/2;
        baloon.titleText.text = Achieves.greenBoxText[0][0];
        baloon.infoText.text = Achieves.greenBoxText[0][1];
        if (Achieves.greenBox[0]) baloon.comletedMc.visible = true else baloon.comletedMc.visible = false;
        TweenMax.to(baloon,.2,{alpha:1});
        if(Achieves.greenBox[0] && canvas.greenAch1) canvas.greenAch1.gotoAndStop(2) else canvas.greenAch1.gotoAndStop(1);
    }
    private function greenBoxOverFunc2():void{
        baloon.x = canvas.greenAch2.x;
        baloon.y = canvas.greenAch2.y - canvas.greenAch2.height/2;
        baloon.titleText.text = Achieves.greenBoxText[1][0];
        baloon.infoText.text = Achieves.greenBoxText[1][1];
        if (Achieves.greenBox[1]) baloon.comletedMc.visible = true else baloon.comletedMc.visible = false;
        TweenMax.to(baloon,.2,{alpha:1});
        if(Achieves.greenBox[1] && canvas.greenAch2) canvas.greenAch2.gotoAndStop(2) else canvas.greenAch2.gotoAndStop(1);
    }
    private function greenBoxOverFunc3():void{
        baloon.x = canvas.greenAch3.x;
        baloon.y = canvas.greenAch3.y - canvas.greenAch3.height/2;
        baloon.titleText.text = Achieves.greenBoxText[2][0];
        baloon.infoText.text = Achieves.greenBoxText[2][1];
        if (Achieves.greenBox[2]) baloon.comletedMc.visible = true else baloon.comletedMc.visible = false;
        TweenMax.to(baloon,.2,{alpha:1});
        if(Achieves.greenBox[2] && canvas.greenAch3) canvas.greenAch3.gotoAndStop(2) else canvas.greenAch3.gotoAndStop(1);
    }
    private function prizewinnerBoxOverFunc1():void{
        baloon.x = canvas.prizewinnerAch1.x;
        baloon.y = canvas.prizewinnerAch1.y - canvas.prizewinnerAch1.height/2;
        baloon.titleText.text = Achieves.gifterBoxText[0][0];
        baloon.infoText.text = Achieves.gifterBoxText[0][1];
        if (Achieves.gifterBox[0]) baloon.comletedMc.visible = true else baloon.comletedMc.visible = false;
        TweenMax.to(baloon,.2,{alpha:1});
        if(Achieves.gifterBox[0] && canvas.prizewinnerAch1) canvas.prizewinnerAch1.gotoAndStop(2) else canvas.prizewinnerAch1.gotoAndStop(1);
    }
    private function prizewinnerBoxOverFunc2():void{
        baloon.x = canvas.prizewinnerAch2.x;
        baloon.y = canvas.prizewinnerAch2.y - canvas.prizewinnerAch2.height/2;
        baloon.titleText.text = Achieves.gifterBoxText[1][0];
        baloon.infoText.text = Achieves.gifterBoxText[1][1];
        if (Achieves.gifterBox[1]) baloon.comletedMc.visible = true else baloon.comletedMc.visible = false;
        TweenMax.to(baloon,.2,{alpha:1});
        if(Achieves.gifterBox[1] && canvas.prizewinnerAch2) canvas.prizewinnerAch2.gotoAndStop(2) else canvas.prizewinnerAch2.gotoAndStop(1);
    }
    private function prizewinnerBoxOverFunc3():void{
        baloon.x = canvas.prizewinnerAch3.x;
        baloon.y = canvas.prizewinnerAch3.y - canvas.prizewinnerAch3.height/2;
        baloon.titleText.text = Achieves.gifterBoxText[2][0];
        baloon.infoText.text = Achieves.gifterBoxText[2][1];
        if (Achieves.gifterBox[2]) baloon.comletedMc.visible = true else baloon.comletedMc.visible = false;
        TweenMax.to(baloon,.2,{alpha:1});
        if(Achieves.gifterBox[2] && canvas.prizewinnerAch3) canvas.prizewinnerAch3.gotoAndStop(2) else canvas.prizewinnerAch3.gotoAndStop(1);
    }
 
    private function sprinterBoxOverFunc1():void{
        baloon.x = canvas.sprinterAch1.x;
        baloon.y = canvas.sprinterAch1.y - canvas.sprinterAch1.height/2;
        baloon.titleText.text = Achieves.sprinterBoxText[0][0];
        baloon.infoText.text = Achieves.sprinterBoxText[0][1];
        if (Achieves.sprinterBox[0]) baloon.comletedMc.visible = true else baloon.comletedMc.visible = false;
        TweenMax.to(baloon,.2,{alpha:1});
        if(Achieves.sprinterBox[0] && canvas.sprinterAch1) canvas.sprinterAch1.gotoAndStop(2) else canvas.sprinterAch1.gotoAndStop(1);
    }
    private function sprinterBoxOverFunc2():void{
        baloon.x = canvas.sprinterAch2.x;
        baloon.y = canvas.sprinterAch2.y - canvas.sprinterAch2.height/2;
        baloon.titleText.text = Achieves.sprinterBoxText[1][0];
        baloon.infoText.text = Achieves.sprinterBoxText[1][1];
        if (Achieves.sprinterBox[1]) baloon.comletedMc.visible = true else baloon.comletedMc.visible = false;
        TweenMax.to(baloon,.2,{alpha:1});
        if(Achieves.sprinterBox[1] && canvas.sprinterAch2) canvas.sprinterAch2.gotoAndStop(2) else canvas.sprinterAch2.gotoAndStop(1);
    }

за копипастинг 5 за код кол

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

Mike Zadorin committed 4854ce1
2014-01-05

чудо правки

коммент ок, понятно что и зачем, всегда мечтаю увидеть такие от коллег

Добавлено через 15 минут
всё, спать пойду. В итоге очень странно, вроде какие-то попытки выделить вьюхи и контроллеры отдельно и данные где-то должны быть но с ходу не нашёл. Но то как оно написано... Я бы сказал что такое форматирование и копипастинг следствие карайней малоопытности. Короче имхо не ок.

Добавлено через 20 минут
Мешанина из CamelCase и under_score

[iiiusion]

прошу не смотреть на коменты) писал их для себя, да и вся репа не предполагалась изначально быть публичной) согласен код местами не фонтан. никаких вьюх нету тут, ну в привычном виде ибо это не mvc) хотелось бы помимо очевидных ляпов(понятно они есть)) чёнить поглубже копнуть)

[ZackMercury]

PHP код:

местами 


Где вы вообще нашли место, где этот код фонтан?)
Эт вообще код не программиста, а аниматора какого-то, который впервые увидел код AS3, ну или перешёл с AS2.

[caseyryan]

 public static var _gameplay:GamePlay;
        public static var _dead:GP_Dead_Window;
        public static var _main:Main;
        public static var _mainmenu:MainMenu;
        public static var _gamesaver:GameSaver;
        public static var _mouseController:MouseController;
        public static var _stage:Stage;

Ужас. Во-первых зачем они объявлены как public static? А во-вторых, по конвенциям as3 с андерскора должны начинаться названия приватных свойств

Названия некоторых классов жесть

MM_Logo_Blink.as
MM_PseudoHero.as

import global.Layers;

Названия классов принято писать в единственном числе.

В общем, тут можно миллион минусов накопать. Просто в лом все пересматривать

п.с. Меня вот интересует, а зачем такое выкладывать на оценку общественности?

[Wolsh]

if (Achieves.greenBox[1]) baloon.comletedMc.visible = true else baloon.comletedMc.visible = false;

То есть

baloon.comletedMc.visible = Achieves.greenBox[1];

. я правильно понимаю?

[elder_Nosferatu]

@Wolsh
Возможно

baloon.comletedMc.visible = Boolean(Achieves.greenBox[1]);

Мало что он в зеленый ящик запихал. Ифу же пофиг, он на все булевскими глазами смотрит.

[Aquahawk]

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

Вот это и признак *****кода. Или фонтан или нет. http://ru.wikipedia.org/wiki/Теория_разбитых_окон почитайте

Добавлено через 2 минуты
на лурке тоже круто написано http://lurkmore.to/Теория_разбитых_окон

[iiiusion]

вобщем это и хотел узнать) код <убрано цензурой> написан, лучше такое <убрано цензурой> <убрано цензурой> никому и никогда не показывать) всем спасибо)