0

Написал код на остановку,но почему выполняется ТОЛЬКО последнее условие?

for(Point p : level.brickCoords) { 
        if(new Rectangle((int) (newX), (int) (newY), SPRITE_SCALE + 14, SPRITE_SCALE + 14).intersects(new Rectangle(p.x, p.y, level.SCALE,level.SCALE)) && rotate == rotate.NORTH) {
           speed = 0;

        if(rotate == rotate.EAST || rotate == rotate.WEST || rotate == rotate.SOUTH) { 
           speed = 2.1f;        
         }  

        }else if(new Rectangle((int) (newX), (int) (newY),SPRITE_SCALE + 14, SPRITE_SCALE + 14).intersects(new Rectangle(p.x, p.y, level.SCALE,level.SCALE)) && rotate == rotate.SOUTH) {
           speed = 0;      
        if(rotate == rotate.EAST || rotate == rotate.WEST || rotate == rotate.NORTH) { 
           speed = 2.1f;       

        }   

        }else if(new Rectangle((int) (newX), (int) (newY),SPRITE_SCALE + 14, SPRITE_SCALE + 14).intersects(new Rectangle(p.x, p.y, level.SCALE,level.SCALE)) && rotate == rotate.EAST) {
           speed = 0;  
        if(rotate == rotate.EAST || rotate == rotate.NORTH || rotate == rotate.SOUTH) { 
           speed = 2.1f;       

        }    
        }else if(new Rectangle((int) (newX), (int) (newY),SPRITE_SCALE + 14, SPRITE_SCALE + 14).intersects(new Rectangle(p.x, p.y, level.SCALE,level.SCALE)) && rotate == rotate.WEST) {
          speed = 0;
         }  
        if(rotate == rotate.NORTH || rotate == rotate.EAST || rotate == rotate.SOUTH) { 
          speed = 2.1f;    

         }
3
  • Скажите, Вам знакомо слово рефакторинг? У вас гора дублирующего кода. Избавьтесь от него для начала, глядишь, все и заработает... 9 мар 2019 в 16:43
  • Скажите, "специалист", что обозначает гора в вашем понимание? все что повторяется это 'new Rectangle((int) (newX), (int) (newY),SPRITE_SCALE + 14, SPRITE_SCALE + 14).intersects(new Rectangle(p.x, p.y, level.SCALE,level.SCALE))', конечно можно сделать так 'if(new Rectangle((int) (newX), (int) (newY),SPRITE_SCALE + 14, SPRITE_SCALE + 14).intersects(new Rectangle(p.x, p.y, level.SCALE,level.SCALE)) && rotate == rotate.NORTH и т.д)', но мне кажется, со всем капельку, что он прилипнет к кубику, пожалуйста разберитесь в коде и потом лишь пишите,рефакторинг тут никак не поможет(да он и не нужен) 10 мар 2019 в 7:34
  • Ну нельзя, так нельзя... Но вы все равно посмотрите, что такое полиморфизм. Если на ваш взгляд 6 условных операторов, 4 из которых вложенных , помещенных в цикл, не требуют рефакторинга, тогда пишите и дальше в таком же духе. С другой стороны вы правы. Этот код не требует рефакторинга. Он настолько плохой, что рефакторингу не поддается. Его нужно просто удалить и написать заново. Вы можете найти здесь ошибку, вопрос в том, что вы будете делать, когда сложность задачи возрастет в сотни раз (а в реальной разработке так и есть) 10 мар 2019 в 9:46

0

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Посмотрите другие вопросы с метками или задайте свой вопрос.