search

GurSergey / altarix_homework

homework
0 stars 0 forks source link

DRY - уменьшение копипасты #5

Open ravilASH opened 5 years ago

ravilASH commented 5 years ago

Например так 

            case self::SHOOT_PLAYER_ONE:
            case 'shootPlayer2':
......
                $player = ( $command == self::SHOOT_PLAYER_ONE) ? Game::PLAYER_ONE : Game::PLAYER_TWO ;
                $hit = $model->shootPlayer($x, $y, $id, $player);
......
                break;

Одновременно снизится количество методов у игры, что на пользу задаче упрощения АПИ класса игры

https://github.com/GurSergey/altarix_homework/blob/e0937efe855a009a8e29b0ce601218f472d772c4/battleship/src/Controller.php#L93

GurSergey commented 5 years ago

30 минут, 01.12.18

GurSergey commented 5 years ago

Заняло 20 минут. коммит 02.12.18