複数処理のスマートな書き方を教えてください

スマートな書き方を教えてください。
フォームの処理について、自分への送信、相手方への送信、フォルダへのテキスト保存
3つの処理を行っています。
全ての処理ごとにエラーがあれば止めて、全部通れば成功としたいです。

今書いている方法が単体で調べたのをつなげているだけです。

php
1$aaa = sendmail(/*自分への送信処理*/);
2if(aaa){
3　$bbb = sendmail(/*相手方への送信処理*/);
4  if(bbb){
5    /*フォルダへのテキスト保存*/
6    if(成功){
7      echo 'OK';
8    }else{
9      echo 'NG3';
10    }
11  }else{
12    echo 'NG2';
13  }
14}else{
15  echo 'NG';
16}

これをもっと可読性、更新世を高くなるように書き換えたいのですが
スマートな方法が分かりません、教えて欲しいです。

nullpon

2018/02/02 23:27

コードを```で括れば、回答者が読みやすくなります。

yumikori

2018/02/02 23:46

ご指摘ありがとうございます、こちらで大丈夫でしょうか？」

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

回答1件

ベストアンサー

無理だと思いますので、せめてテキスト保存を最初にすると良いと思います。それならテキスト保存に失敗したらメールは送らなくてすみますので。
今の流れだとメールが絶対飛んでしまい問題があると思います。

あとは相手への送信を最後にすれば、各処理が失敗した時に中断してもさほど問題はないような気がします。

中断する方法としては、例えば提示していただいたプログラムを全部関数の中に入れ、失敗したらそこでreturnすれば中断できます。

追記

全部関数にして、下記のようにすれば入れ子ではなくなります。

php
1function execute()
2{
3    if (!save()) {
4        return false;
5    }
6    if (!mailToAdmin()) {
7        return false;
8    }
9    if (!mailToCustomer()) {
10        return false;
11    }
12    return true;
13}