Форум 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=207904)

iiiusion 16.05.2014 02:06

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

in4core 16.05.2014 02:14

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

Aquahawk 16.05.2014 02:26

лажа номер один
Код AS3:

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 секунд
два
Код AS3:

                ////////////////////
                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 минут
Код AS3:

 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 16.05.2014 03:29

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

ZackMercury 16.05.2014 04:40

PHP код:

местами 

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

caseyryan 16.05.2014 08:19

Код AS3:

 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 с андерскора должны начинаться названия приватных свойств

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

MM_Logo_Blink.as
MM_PseudoHero.as


Код AS3:

import global.Layers;

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

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

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

Wolsh 16.05.2014 09:00

Код AS3:

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

То есть
Код AS3:

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

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

elder_Nosferatu 16.05.2014 10:46

@Wolsh
Возможно
Код AS3:

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

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

Aquahawk 16.05.2014 11:45

Цитата:

прошу не смотреть на коменты) писал их для себя, да и вся репа не предполагалась изначально быть публичной) согласен код местами не фонтан.
Вот это и признак *****кода. Или фонтан или нет. http://ru.wikipedia.org/wiki/Теория_разбитых_окон почитайте

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

iiiusion 16.05.2014 15:14

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


Часовой пояс GMT +4, время: 10:17.

Copyright © 1999-2008 Flasher.ru. All rights reserved.
Работает на vBulletin®. Copyright ©2000 - 2026, Jelsoft Enterprises Ltd. Перевод: zCarot
Администрация сайта не несёт ответственности за любую предоставленную посетителями информацию. Подробнее см. Правила.