| Conditions | 3 |
| Paths | 52 |
| Total Lines | 83 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 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 |
||
| 152 | public function shipping_出荷CSV登録(\AcceptanceTester $I) |
||
| 153 | { |
||
| 154 | $I->wantTo('EA0903-UC04-T01 出荷CSV登録'); |
||
| 155 | |||
| 156 | $entityManager = Fixtures::get('entityManager'); |
||
| 157 | /* @var Customer $Customer */ |
||
| 158 | $Customer = (Fixtures::get('createCustomer'))(); |
||
| 159 | /* @var Order[] $Orders */ |
||
| 160 | $Orders = (Fixtures::get('createOrders'))($Customer, 3); |
||
| 161 | // 入金済みに更新しておく |
||
| 162 | $Status = $entityManager->getRepository('Eccube\Entity\Master\OrderStatus')->find(OrderStatus::PAID); |
||
| 163 | foreach ($Orders as $newOrder) { |
||
| 164 | $newOrder->setOrderStatus($Status); |
||
| 165 | } |
||
| 166 | $entityManager->flush(); |
||
| 167 | |||
| 168 | /* |
||
| 169 | * 出荷再検索 出荷日/伝票番号が登録されていないことを確認 |
||
| 170 | */ |
||
| 171 | |||
| 172 | $OrderManagePage = OrderManagePage::go($I) |
||
| 173 | ->詳細検索設定() |
||
| 174 | ->入力_ご注文者お名前($Customer->getName01().$Customer->getName02()) |
||
| 175 | ->入力_ご注文者お名前フリガナ($Customer->getKana01().$Customer->getKana02()) |
||
| 176 | ->検索(); |
||
| 177 | |||
| 178 | $I->see('検索結果:3件が該当しました', OrderManagePage::$検索結果_メッセージ); |
||
| 179 | |||
| 180 | $I->assertEmpty($OrderManagePage->取得_出荷伝票番号(1)); |
||
| 181 | $I->assertEmpty($OrderManagePage->取得_出荷伝票番号(2)); |
||
| 182 | $I->assertEmpty($OrderManagePage->取得_出荷伝票番号(3)); |
||
| 183 | $I->assertEquals('未発送', $OrderManagePage->取得_出荷日(1)); |
||
| 184 | $I->assertEquals('未発送', $OrderManagePage->取得_出荷日(2)); |
||
| 185 | $I->assertEquals('未発送', $OrderManagePage->取得_出荷日(3)); |
||
| 186 | |||
| 187 | /* |
||
| 188 | * 出荷CSV登録 |
||
| 189 | */ |
||
| 190 | |||
| 191 | $csv = implode(PHP_EOL, [ |
||
| 192 | '出荷ID,出荷伝票番号,出荷日', |
||
| 193 | $Orders[0]->getShippings()[0]->getId().',00001,2018-01-01', |
||
| 194 | $Orders[1]->getShippings()[0]->getId().',00002,2018-02-02', |
||
| 195 | $Orders[2]->getShippings()[0]->getId().',00003,2018-03-03', |
||
| 196 | ]); |
||
| 197 | |||
| 198 | $csvFileName = codecept_data_dir().'/shipping.csv'; |
||
| 199 | file_put_contents($csvFileName, $csv); |
||
| 200 | |||
| 201 | try { |
||
| 202 | ShippingCsvUploadPage::go($I) |
||
| 203 | ->入力_CSVファイル('shipping.csv') |
||
| 204 | ->CSVアップロード(); |
||
| 205 | |||
| 206 | $I->see('出荷登録CSVファイルをアップロードしました。', ShippingCsvUploadPage::$完了メッセージ); |
||
| 207 | |||
| 208 | /* |
||
| 209 | * 出荷再検索 出荷日/伝票番号が登録されたことを確認 |
||
| 210 | */ |
||
| 211 | |||
| 212 | $OrderManagePage = OrderManagePage::go($I) |
||
| 213 | ->詳細検索設定() |
||
| 214 | ->入力_ご注文者お名前($Customer->getName01().$Customer->getName02()) |
||
| 215 | ->入力_ご注文者お名前フリガナ($Customer->getKana01().$Customer->getKana02()) |
||
| 216 | ->検索(); |
||
| 217 | |||
| 218 | $I->see('検索結果:3件が該当しました', OrderManagePage::$検索結果_メッセージ); |
||
| 219 | |||
| 220 | $I->assertEquals('00003', $OrderManagePage->取得_出荷伝票番号(1)); |
||
| 221 | $I->assertEquals('00002', $OrderManagePage->取得_出荷伝票番号(2)); |
||
| 222 | $I->assertEquals('00001', $OrderManagePage->取得_出荷伝票番号(3)); |
||
| 223 | $I->assertEquals('2018/03/03', $OrderManagePage->取得_出荷日(1)); |
||
| 224 | $I->assertEquals('2018/02/02', $OrderManagePage->取得_出荷日(2)); |
||
| 225 | $I->assertEquals('2018/01/01', $OrderManagePage->取得_出荷日(3)); |
||
| 226 | $I->assertEquals('発送済み', $OrderManagePage->取得_ステータス(1)); |
||
| 227 | $I->assertEquals('発送済み', $OrderManagePage->取得_ステータス(2)); |
||
| 228 | $I->assertEquals('発送済み', $OrderManagePage->取得_ステータス(3)); |
||
| 229 | } finally { |
||
| 230 | if (file_exists($csvFileName)) { |
||
| 231 | unlink($csvFileName); |
||
| 232 | } |
||
| 233 | } |
||
| 234 | } |
||
| 235 | |||
| 312 |
You can fix this by adding a namespace to your class:
When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries.