1. ActionScript / Говнокод #15702

    −140

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    private function setHandlesVisiablity(isTextBox:Boolean, isSpecialMode:Boolean = false):void
{
	if (isSpecialMode)
	{
		(_handles[0].handle as Sprite).visible = false; 				  //center
		(_handles[1].handle as Sprite).visible = false;     		      //top
		(_handles[2].handle as Sprite).visible = false;	  			 	  //right
		(_handles[3].handle as Sprite).visible = false;		      	   	  //bottom
		(_handles[4].handle as Sprite).visible = false; 			  	  //left
		(_handles[5].handle as Sprite).visible = false;		      		  //topLeft
		(_handles[6].handle as Sprite).visible = false;		    	      //topRight
		(_handles[7].handle as Sprite).visible = false;		    	      //bottomRight
		(_handles[8].handle as Sprite).visible = false;		   		      //bottomLeft
	}
	else
	{
		(_handles[0].handle as Sprite).visible = true; 					  //center
		(_handles[1].handle as Sprite).visible = !isTextBox;		      //top
		(_handles[2].handle as Sprite).visible = true;	  				  //right
		(_handles[3].handle as Sprite).visible = !isTextBox;		      //bottom
		(_handles[4].handle as Sprite).visible = true; 			  		  //left
		(_handles[5].handle as Sprite).visible = !isTextBox;		      //topLeft
		(_handles[6].handle as Sprite).visible = !isTextBox;		      //topRight
		(_handles[7].handle as Sprite).visible = !isTextBox;		      //bottomRight
		(_handles[8].handle as Sprite).visible = !isTextBox;		      //bottomLeft
	}
}

    Нужно отдельно объяснить, что выключка у коментариев не сложилась потому, что уже два разных программиста отрефакторили этот код. Изначально коментарии были каждый на своей строчке и равнялись разными людьми использующими разные настройки ИДЕ.

    Запостил: wvxvw, 08 Апреля 2014

    Комментарии (17) RSS

    • ava wvxvw 08.04.2014 10:48 # +2

      Фак, визияблити, только сейчас заметил.
      Ответить

    • ava bormand 08.04.2014 11:03 # 0

      А если поменять handles[5] на handles.topLeft, то комменты не понадобятся ;)
      Ответить

      • ava wvxvw 08.04.2014 11:06 # 0

        А можно было еще и создать группы кнопок (это такие маленькие квадратики, которые показывают вокруг редактируемой картинки, чтобы ее растянуть или покрутить).
        Ответить

        • ava bormand 08.04.2014 11:23 # 0

          В одну группу поместить center, left и right, а во вторую - все остальное? Ну да, вполне логично.
          Ответить

    • ava bormand 08.04.2014 11:25 # 0

      P.S. <пробелосрач>
      > разными людьми использующими разные настройки ИДЕ
      Решить эту проблему помогает стиль indent with tabs, align with spaces, при этом ширина таба должна быть равна ширине индента.
      </пробелосрач>
      Ответить

      • ava guest 08.04.2014 17:30 # 0

        Разные отступы есть даже в стандартной библиотеке жавы.
        Ответить

      • ava 1024-- 08.04.2014 17:48 # +1

        <пробелосрач>
        Это же пробелосрач? Так и использовать надо п р о б е л ы.
        С пробелами ваши волосы становятся гладкими и шелковистыми, а код вписывается в положенные 80 символов, а не в 70 + randomGauss(0.0, 3.0)
        </пробелосрач>
        Ответить

        • ava wvxvw 08.04.2014 18:47 # +1

          Мне в этом отношении очень нравится позция Эмакса (прадва, неожидано?): кнопка "таб" не вставляет табуляцию, а правит отступ у текущей строки так, чтобы он был консистентным с настойками и контекстом, типа как в клипсе Ctrl+i. Так у всяких умельцев отпадает желание вручную украсить код произвольными пробельными элементами.
          Аналогично - выравнивание в одну колонку, тоже делается не печатанием / копированием пробелов.

          ЗЫ: http://xkcd.com/1301/
          Ответить

          • ava guest 08.04.2014 19:19 # +1

            Все иде позволяют конвертировать одно в другое.
            Ответить

            • ava wvxvw 08.04.2014 19:40 # 0

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

              • ava guest 08.04.2014 19:56 # 0

                А в меню залезть - это не легкодоступные?
                Ответить

                • ava wvxvw 08.04.2014 21:26 # +2

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

                  • ava guest 08.04.2014 23:24 # 0

                    мы злдесь обсуждаем людей, не запускавших иде?
                    Ответить

                    • ava wvxvw 09.04.2014 00:40 # 0

                      Мы тут обсуждаем людей, у которых все запущено, не только ИдЕ.
                      Ответить

      • ava eth0 08.04.2014 20:32 # +1

        Бьютификатор с настройками стиля. И тут пох на жалкие попытки, можно хоть тарасоформатирование забабахать.
        Ответить

        • ava bormand 08.04.2014 20:39 # 0

          С системами контроля версий могут быть проблемки. Но вроде решаемо, если оно перед коммитом будет сводить все к одному стандарту, описанному в корпоративном стайлгайде.
          Ответить

    • ava guest 08.04.2014 17:32 # 0

      Толсто.
      Ответить

    Добавить комментарий