ДействиеСценарий / Говнокод #16753 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 wvxvw, (Updated )

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

  • ava Vasiliy # Ссылка на оригинал 0

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

    • ava wvxvw # Ссылка на оригинал 0 ⇈

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

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓