プログラムにおける共通処理について

it新人のエンジニアです。お手柔らかにお願いします。

###前提・実現したいこと

①共通処理を関数化したいです。
②上記にあたり神クラスを使っていいのかがわかりません。

全体の流れ

電話番号やメールアドレスやホームページアドレスに対し、下記のような正規表現をかけるプログラムを書きました。しかし、正規表現は随所で使われており、ライブラリに神クラスを作りそこにstaticメソッドとしてまとめたいなと考えました。その関数化の方法がいまいちわかりません。(そもそも神クラスはダメだなどという記事をよく見ますが・・・)

それぞれの正規表現で、マッチした場合はそのマッチした値を、マッチしなかった場合はエラー文を$err_data配列に格納しその後の処理は飛ばし、次のループに行くようにしています。

このような書き方を見て、こうするべきという書き方などあれば、ご教授いただければ幸いです。

###該当のソースコード

php
1
2        // csvファイル読み込み処理(略)
3        
4        foreach ($file as $key => $row) {
5            list(
6                $company_id,
7                $employment_site_id, 
8                $company_name,
9                $phone_number,
10                $mail_address,
11                $home_page_url,
12            ) = $row;
13
14            // 電話番号の存在確認
15            if ($phone_number) {
16                // 電話番号は正しい形式か？
17                if (preg_match('/([0-9]{2,4}-[0-9]{2,4}-[0-9]{3,4})/', $phone_number, $match)) {
18                    $phone_number = $match[0];
19                } else {
20                    $err_data[] = "会社ID ${company_id} の電話番号 ${phone_number} の形式が違います。確認してください";
21                    continue;
22                }
23            }
24
25            // メールアドレスの存在確認
26            if ($mail_address) {
27                // メールアドレスは正しい形式か？
28                if (preg_match('/[\w.\-]+@[\w\-]+.[\w.\-]+/', $mail_address, $match)) {
29                    $mail_address = $match[0];
30                } else {
31                    $err_data[] = "会社ID ${company_id} のメールアドレス ${mail_address} の形式が違います。確認してください";
32                    continue;
33                }
34            }
35
36            // ホームページアドレスの存在確認
37            if ($home_page_url) {
38                // ホームページアドレスが正しい形式か？
39                if (preg_match('/(http|https|ftp)://([A-Z0-9][A-Z0-9_-]*(?:.[A-Z0-9][A-Z0-9_-]*)+):?(\d+)?/?/i', $home_page_url, $match)) {
40                    $home_page_url = $match[0];
41                } else {
42                    $err_data[] = "会社ID ${company_id} のホームページアドレス ${home_page_url} の形式が違います。確認してください";
43                    continue;
44                }
45             }
46
47        //DBインサート処理(略)
48

###補足情報(言語/FW/ツール等のバージョンなど)

開発環境：Vagrant + VirtualBox / Homestead
本番環境：CentOS7 nginx MySQL PHP7.2
言語：PHP 7.2.3 (cli)(NTS)
フレームワーク：Laravel5.5 (LTS)
サーバー：nginx
DB：MySQL

m.ts10806

2018/05/09 04:24 編集

Laravel 5の機能でできることが沢山あるように思うのですが、LaravelのValidationについては確認してみたのでしょうか（ドキュメントに結構ありますよ）

T.FUJII

2018/05/09 08:03

ご回答ありがとうございます。Validationについて使い方がわかっていなかったので確認しました。

行動規範の内容に同意します

回答4件

ベストアンサー

そもそも、Laravel5.5を使っているんですよね？
そうであれば、その様な神クラスはいらないと思います。
例えば、E-Mailのチェックであれば

php
1$this->validate($request, [
2            'email'     => 'required|email',
3        ]);

で十分でしょう。
CSVファイルのチェックとして使うのであれば、

php
1foreach ($fileContents as $fileContent){
2            $validator = Validator::make($fileContent, $rules);
3            $validator->validate();
4        }

としても良いです。
どの様なValidationが出来るかはこちらのサイトが参考になります。

標準にない場合も、カスタムバリデーションを作れるのでそれを使うようにするのがおススメです。

投稿2018/05/09 04:20

motuo

総合スコア3027

T.FUJII

2018/05/09 08:13

すみません読み足りませんでした。Validator::makeのやり方ご教授いただきありがとうございます。

行動規範の内容に同意します

よほど例外的なものがないかぎりバリデートであれば標準関数で良いような気がします

PHP
1filter_var($mailaddress, FILTER_VALIDATE_EMAIL);
2filter_var($url, FILTER_VALIDATE_URL, FILTER_FLAG_PATH_REQUIRED);

また電話番号のパターン「/([0-9]{2,4}-[0-9]{2,4}-[0-9]{3,4})/」は、
間違ってると思うので、きちんと調べられないならヤラないほうがマシだと思います
（もしくは数字と-だけチェックするとかそのレベルで）

投稿2018/05/09 03:46

yambejp

総合スコア114771

maisumakun

2018/05/09 03:51

たしかに、これでは市外局番が5桁の地域は通りませんね。

T.FUJII

2018/05/09 03:55

ご回答ありがとうございます。正規表現の知識があまりなく、どこかのサイトに転がっていたものを流用しました。５桁の地域があるのですね・・・参考にさせていただきます。

m.ts10806

2018/05/09 06:22

5桁については昔は郡部などでよく使われていましたが、市町村合併諸々でほとんどなくなりましたね。参考：日本における市外局番の変更#現存する5桁市外局番 https://ja.wikipedia.org/wiki/%E6%97%A5%E6%9C%AC%E3%81%AB%E3%81%8A%E3%81%91%E3%82%8B%E5%B8%82%E5%A4%96%E5%B1%80%E7%95%AA%E3%81%AE%E5%A4%89%E6%9B%B4#%E7%8F%BE%E5%AD%98%E3%81%99%E3%82%8B5%E6%A1%81%E5%B8%82%E5%A4%96%E5%B1%80%E7%95%AA

yambejp

2018/05/09 07:04

入力の制限については、現有の市外局番をチェックすべきですが統合などで変更になった場合柔軟に対応できるか課題があります {2,4}{2,4}{3,4}だと最短で2+2+3、最長で4+4+4になるのでこれもバリデートとしては微妙結局なにを弾きたいのかというのが目的になるので数値とハイフン以外の文字はNG、最大でハイフン込みで13桁までくらいが落とし所でしょう。（最初は0から始まるとかくらいは気にしてもよい）

motuo

2018/05/09 08:02

使ったことないですが、Laravelではお手軽電話番号チェックのパッケージもありますね。 https://github.com/Propaganistas/Laravel-Phone

T.FUJII

2018/05/09 08:17

みなさま >> ご回答ありがとうございます。正規表現って奥が深くて難しいですね。転がっていた正規表現もいろいろな基準で書かれたものだと思うので、むやみにコピペして使うのはダメでした。

m.ts10806

2018/05/09 08:19

正規表現チェッカー　というサイトがあるのでそちらをご活用されては？今回はいずれにしてもLaravelの機能を使う方が賢明だと思います。

T.FUJII

2018/05/09 08:27

mts10806様 > 承知しました。ご教授ありがとうございます！