yiisoft /
yii-demo
| Conditions | 1 |
| Paths | 1 |
| Total Lines | 71 |
| Code Lines | 46 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 6 | ||
| Bugs | 1 | Features | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 33 | public function __invoke(ContainerInterface $container) |
||
| 34 | { |
||
| 35 | $routes = [ |
||
| 36 | // Lonely pages of site |
||
| 37 | Route::get('/', [SiteController::class, 'index']) |
||
| 38 | ->name('site/index'), |
||
| 39 | Route::methods([Method::GET, Method::POST], '/contact', [ContactController::class, 'contact']) |
||
| 40 | ->name('site/contact'), |
||
| 41 | Route::methods([Method::GET, Method::POST], '/login', [AuthController::class, 'login']) |
||
| 42 | ->name('site/login'), |
||
| 43 | Route::get('/logout', [AuthController::class, 'logout']) |
||
| 44 | ->name('site/logout'), |
||
| 45 | |||
| 46 | // User |
||
| 47 | Group::create('/user', [ |
||
| 48 | // Index |
||
| 49 | Route::get('[/page-{page:\d+}]', [UserController::class, 'index']) |
||
| 50 | ->name('user/index'), |
||
| 51 | // Profile page |
||
| 52 | Route::get('/{login}', [UserController::class, 'profile']) |
||
| 53 | ->name('user/profile'), |
||
| 54 | ]), |
||
| 55 | |||
| 56 | // User |
||
| 57 | Group::create('/api', [ |
||
| 58 | Route::get('/info/v1', function (DataResponseFactoryInterface $responseFactory) { |
||
| 59 | return $responseFactory->createResponse(['version' => '1.0', 'author' => 'yiisoft']); |
||
| 60 | })->name('api/info/v1'), |
||
| 61 | Route::get('/info/v2', ApiInfo::class) |
||
| 62 | ->addMiddleware(FormatDataResponseAsJson::class) |
||
| 63 | ->name('api/info/v2'), |
||
| 64 | Route::get('/user', [ApiUserController::class, 'index']) |
||
| 65 | ->name('api/user/index'), |
||
| 66 | Route::get('/user/{login}', [ApiUserController::class, 'profile']) |
||
| 67 | ->addMiddleware(FormatDataResponseAsJson::class) |
||
| 68 | ->name('api/user/profile'), |
||
| 69 | ], $container)->addMiddleware(ApiDataWrapper::class)->addMiddleware(FormatDataResponseAsXml::class), |
||
| 70 | |||
| 71 | // Blog routes |
||
| 72 | Group::create('/blog', [ |
||
| 73 | // Index |
||
| 74 | Route::get('[/page{page:\d+}]', [BlogController::class, 'index']) |
||
| 75 | ->name('blog/index'), |
||
| 76 | // Post page |
||
| 77 | Route::get('/page/{slug}', [PostController::class, 'index']) |
||
| 78 | ->name('blog/post'), |
||
| 79 | // Tag page |
||
| 80 | Route::get('/tag/{label}[/page{page:\d+}]', [TagController::class, 'index']) |
||
| 81 | ->name('blog/tag'), |
||
| 82 | // Archive |
||
| 83 | Group::create('/blog', [ |
||
| 84 | // Index page |
||
| 85 | Route::get('', [ArchiveController::class, 'index']) |
||
| 86 | ->name('blog/archive/index'), |
||
| 87 | // Yearly page |
||
| 88 | Route::get('/{year:\d+}', [ArchiveController::class, 'yearlyArchive']) |
||
| 89 | ->name('blog/archive/year'), |
||
| 90 | // Monthly page |
||
| 91 | Route::get('/{year:\d+}-{month:\d+}[/page{page:\d+}]', [ArchiveController::class, 'monthlyArchive']) |
||
| 92 | ->name('blog/archive/month') |
||
| 93 | ]), |
||
| 94 | ]), |
||
| 95 | ]; |
||
| 96 | |||
| 97 | $collector = $container->get(RouteCollectorInterface::class); |
||
| 98 | $collector->addGroup( |
||
| 99 | Group::create(null, $routes) |
||
| 100 | ->addMiddleware(FormatDataResponse::class) |
||
| 101 | ); |
||
| 102 | |||
| 103 | return new UrlMatcher(new RouteCollection($collector)); |
||
| 104 | } |
||
| 106 |
