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

    −87

    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
    /src/com/<секрет>/client/view/components/movicons/ImageHolderView.mxml
 			protected function onFlipRollOver(event:MouseEvent):void
 			{
-				if ( (track && track.slide &&(track.slide.selectedTrack==null || (holderData && track.slide.selectedTrack.sprite!=holderData)))
-					|| (  holderData && holderData.sourceUrl && holderData.getName()==ImageSpriteVO.EMPTY_IMAGE_NAME) 
+				if (((track && track.slide && (track.slide.selectedTrack==null || (holderData && track.slide.selectedTrack.sprite!=holderData)))
+					|| (  holderData && holderData.sourceUrl && holderData.getName()==ImageSpriteVO.EMPTY_IMAGE_NAME)) 
 					&& <секрет>.slidesMode 
 					&& !<секрет>.fullScreenMode)
 				{

/src/com/<секрет>/client/view/components/movicons/ImageView.mxml
 			protected function onFlipRollOver(event:MouseEvent):void
 			{
-				if ( (track && track.slide &&(track.slide.selectedTrack==null || (holderData && track.slide.selectedTrack.sprite!=holderData)))
-					|| (  holderData && holderData.sourceUrl && holderData.getName()==ImageSpriteVO.EMPTY_IMAGE_NAME)
+				if ( ((track && track.slide &&(track.slide.selectedTrack==null || (holderData && track.slide.selectedTrack.sprite!=holderData)))
+					|| (  holderData && holderData.sourceUrl && holderData.getName()==ImageSpriteVO.EMPTY_IMAGE_NAME))
 					&& <секрет>.slidesMode 
 					&& !<секрет>.fullScreenMode)
 				{

    Тяжело было прочитать, попросили поправить, поправил.

    Запостил: wvxvw, 23 Сентября 2014

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

    • ava Vasiliy 23.09.2014 16:04 # 0

      track && track.slide в отдельную ветку, holderData вынести за скобки Вот эту байду (track.slide.selectedTrack==null || (holderData && track.slide.selectedTrack.sprite!=holder Data) в отдельную переменную и назвать осмысленно. Вот к чему приводят неявные состояния.
      Ответить

      • ava wvxvw 23.09.2014 16:12 # 0

        Тут как бы смысл в бессмысленной правке: человек просто добавил пару лишних скобок, чтобы "добавить читабельности".
        Я более чем уверен, что для того, чтобы исправить вышенаписаное нужно было бы разделить компонент на несколько состояний, чтобы не было такой кучи повторяющихся проверок, а не заниматься группировкой этих проверок внутри if'а. Уже сам факт того, что эти проверки повторяются (тем более в разных компонентах!) говорит о том, что они неправильно спроектированы.
        Ответить

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