質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.95%

【PHP】1つのインスタンスで同じような処理を三回繰り返したい。

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,041

score 41

それぞれ異なるnameを持ったファイルタグからファイルをアップロードして、①ファイルがアップロードされたか、②拡張子がJPG,PED,PNGか③ファイルサイズが2MB以下か、以上をクリアしたら正しくアップロード完了というような判定をして、できなければエラーを返すというような処理を実装したいです。

1つのインスタンスで同じ処理を三回繰り返したいのですが、画面に反映させたところうまく文字が出力されません・・・・
バリデーションは正しくできていると思います。
うまくそれぞれのnameに合わせた処理が出力させるような使用にしたいです。

よろしくお願いします。

コード

<body>
<p>テスト</p>
<p>引数に<?php echo $filename[0]; ?>を指定した場合の処理結果は<?php echo $Result->check($filename[0]);?></p>
<p>引数に<?php echo $filename[1]; ?>を指定した場合の処理結果は<?php echo $Result->check($filename[1]);?></p>
<p>引数に<?php echo $filename[2]; ?>を指定した場合の処理結果は<?php echo $Result->check($filename[2]);?></p>

</body>
コード
<?php

$Result = new Result;
$errData = $Result->getErrData();
$filename = array('certification','cert_seal','citizen');


class Result{
  public $errData;
  //public $filename = array('certification','cert_seal','citizen');

  public function first(){
    $this->check($filename[0]);
    $this->check($filename[1]);
    $this->check($filename[2]);
  }


  function check($_filename){
    $ret = false;//一番最後にいかない限りfalseを返す
    if(is_uploaded_file($_FILES[$_filename]["tmp_name"])){
      if(preg_match("(.+\.(jpg|pdf|png))",$_FILES[$_filename]["name"])){
        $filesize = filesize($_FILES[$_filename]['tmp_name']);
        if($filesize <= 2097152){
          $this->errData="アップロード成功!";
          $ret=true;
        } else {
          $this->errData="ファイルサイズが大きすぎますので2MB以下にしてください。";
        }
      } else {
        $this->errData="jpg,pdf,png の形式ではありません。jpg,pdf,png形式のファイルのみアップロードできます。";
      }
    } else {
      $this->errData="ファイルが参照されていません。";
    }
    return $ret;
  }

    function getErrData(){
      $errData['certification_err']="テスト1";
      $errData['cert_seal_err']="テスト2";
      $errData['citizen_err']="テスト3";
    return $this->errData;
  }


}


?>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

+2

「function check」の戻り値をメッセージ文字列にすれば出力できるのでは?
たとえば

function check($_filename){
  // $ret = false;//一番最後にいかない限りfalseを返す
  if(is_uploaded_file($_FILES[$_filename]["tmp_name"])){
    if(preg_match("(.+\.(jpg|pdf|png))",$_FILES[$_filename]["name"])){
      $filesize = filesize($_FILES[$_filename]['tmp_name']);
      if($filesize <= 2097152){
        return "アップロード成功!";
        // $ret=true;
      } else {
        return "ファイルサイズが大きすぎますので2MB以下にしてください。";
      }
    } else {
      return "jpg,pdf,png の形式ではありません。jpg,pdf,png形式のファイルのみアップロードできます。";
    }
  } else {
    return "ファイルが参照されていません。";
  }
  //return $ret;
}


こんなふうに。

■追記

「『function check』の戻り値はbooleanにしたい」とか言われてしまったので、質問者ソースを部分的に添削してみます。
まずはphp側:class Result

class Result{
  public $errData;

  const success = "アップロード成功!"; // 定数として成功時メッセージを定義

  //使わない変数は邪魔です。消しましょう。
  //public $filename = array('certification','cert_seal','citizen');

  /* 使ってないメソッドは邪魔です。消しましょう。
  public function first(){
    $this->check($filename[0]);
    $this->check($filename[1]);
    $this->check($filename[2]);
  }
  */

  function check($_filename){
    // $ret = false;//一番最後にいかない限りfalseを返す // ←これ実はいらないでしょ?
    if(is_uploaded_file($_FILES[$_filename]["tmp_name"])){
      if(preg_match("(.+\.(jpg|pdf|png))",$_FILES[$_filename]["name"])){
        $filesize = filesize($_FILES[$_filename]['tmp_name']);
        if($filesize <= 2097152){
          // $this->errData="アップロード成功!"; // 定数で持つので不要。
          // $ret=true; // ↓これでいいでしょ?true返すのはこのブロックだけなんだから。
          return true;
        } else {
          $this->errData="ファイルサイズが大きすぎますので2MB以下にしてください。";
        }
      } else {
        $this->errData="jpg,pdf,png の形式ではありません。jpg,pdf,png形式のファイルのみアップロードできます。";
      }
    } else {
      $this->errData="ファイルが参照されていません。";
    }
    // return $ret; // ↓これでいいでしょ?
    return false;
  }

  function getErrData(){
    /* ここのコードはゴミでしょ?消しましょう。
    $errData['certification_err']="テスト1";
    $errData['cert_seal_err']="テスト2";
    $errData['citizen_err']="テスト3";
    */
    return $this->errData;
  }
}


で、function checkの戻り値(boolean)を有効活用するならhtml側ではこんな感じ?

<p>引数に<?php echo $filename[0]; ?>を指定した場合の処理結果は<?php echo $Result->check($filename[0]) ? $Result->success : $Result->getErrData(); ?></p>

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/04/21 11:03

    ありがとうございます!!上記のコードでうまく処理できましたが、戻り値にはブーリアン型のtrue か falseを返すような値にはしておきたいです。
    他に有効なのがあればご教授お願いします・・・!

    キャンセル

  • 2016/04/21 11:11

    戻り値を何に使うのですか?
    質問者ソースでは「function check」の戻り値をまったく利用していないように思えます。
    成功したら「アップロード成功!」と固定文字を出す?
    だったらこのfunctionの中で「$this->errData="アップロード成功!"」とやってるのは意味がない処理ですね。

    キャンセル

  • 2016/04/21 11:35

    function checkの有効利用例を追記してみました。

    キャンセル

  • 2016/04/21 12:28

    かなり厳しい添削でしたが、ためになりました!!
    こんなに言われますとかなり気持ちいいですね(笑)

    ありがとうございます!!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.95%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る