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

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

新規登録して質問してみよう
ただいま回答率
85.48%
PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

解決済

2回答

10791閲覧

画像をアップロードするときにエラーが出てしまいます。

Kuriaki

総合スコア85

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

0グッド

0クリップ

投稿2017/01/17 07:35

編集2017/01/17 07:37

コードにエラーチェックが書いてあり、ファイルサイズが0のときにはエラー文が表記するようになってるですが、先ほどテストをしてるときに、画像によってファイルが0になる場合があります。

###エラー文はこちらです↓↓
Warning: getimagesize(): Filename cannot be empty in C:\xampp\htdocs\up\resize.php on line 17

###エラー文の17行は$get = getimagesize($tmp_name)の行です。

ちなみに私はphp.iniのmaxfilesizeを5Mにしています。
画像のサイズが小さいときは普通に動きますが、4Mの画像や5Mを超える画像ですと、エラー文のように、画像ファイルが入ってないと、エラー分が出てしまいます。
よろしくお願いいたします。

html

1<!DOCTYPE html> 2<html> 3<head> 4 5 <meta charset="utf-8"> 6 <title>php</title> 7</head> 8<body> 9<form action="resize.php" method="post" name="form" enctype="multipart/form-data"> 10 11 12<input type="hidden" name="MAX_FILE_SIZE" value="524288"> 13<p> 14<input type="file" name="upfile" size="30" id="upload"> 15</p> 16 17<p> 18<input type="submit" value="送信する"> 19</p> 20</form> 21</body> 22 </html>

php

1<?php 2$file = $_FILES['upfile']; 3$tmp_name = $file['tmp_name']; // 一時ファイルのパス 4if($_FILES['upfile']['size'] === 0) { 5 echo '<FONT COLOR="RED">ファイルを選択して下さい!</FONT>'; 6}else{ 7 echo '送信完了'; 8} 9if ($_FILES['upfile']['error'] === 2) { 10 echo '<FONT COLOR="RED">ファイルサイズを小さくしてください!</FONT>'; 11 } 12if (! preg_match('/\.jpg$|\.jpeg$/i', $tmp_name)) { 13 echo ""; 14} else{ 15 echo '<FONT COLOR="RED">JPEGファイルを選択してください!</FONT>'; 16} 17$get = getimagesize($tmp_name); 18if ($get["0"]>1000) { 19 echo '<FONT COLOR="RED">横幅を小さくしてください!</FONT>'; 20} 21else{ 22 echo ""; 23} 24if ($get["1"]>1000) { 25 echo '<FONT COLOR="RED">縦幅を小さくしてください!</FONT>'; 26 27}else{ 28 echo ""; 29} 30?> 31

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

既に解決済みではありますが、、

$_FILES['upfile']['size'] === 0で、ファイルがアップロードされたか否かを判断するべきではありません。

なぜなら、私の環境(※)で試した限りでは、アップロードされたファイルが指定したファイルサイズを超えた場合にも、ファイルをアップロードしなかった場合と同様に$_FILES['upfile']['size']に 0 がセットされたからです。
※ Mac OSX 10.9, Chrome v55.0, PHP v5.4

そのため、まず最初に$_FILES['upfile']['error']を見て、何が起きたかを判断すべきです。

例えば、以下のような感じです。

php

1<?php 2ini_set('display_errors', '1'); 3error_reporting(E_ALL); 4 5if (!isset($_FILES['upfile'])) { 6 echo '<FONT COLOR="RED">恐らくアップロードページのHTMLに記述ミス有り!</FONT>'; 7 return; 8} 9 10$file = $_FILES['upfile']; 11$tmp_name = $file['tmp_name']; // 一時ファイルのパス 12 13switch ($file['error']) { 14 case UPLOAD_ERR_OK: 15 break; 16 case UPLOAD_ERR_INI_SIZE: 17 case UPLOAD_ERR_FORM_SIZE: 18 echo '<FONT COLOR="RED">ファイルサイズを小さくしてください!</FONT>'; 19 return; 20 case UPLOAD_ERR_PARTIAL: 21 echo '<FONT COLOR="RED">もう一度実行してみてください!</FONT>'; 22 return; 23 case UPLOAD_ERR_NO_FILE: 24 echo '<FONT COLOR="RED">ファイルを選択して下さい!</FONT>'; 25 return; 26 case UPLOAD_ERR_NO_TMP_DIR: 27 case UPLOAD_ERR_CANT_WRITE: 28 case UPLOAD_ERR_EXTENSION: 29 echo '<FONT COLOR="RED">サーバ内部で発生したエラー!</FONT>'; 30 return; 31 default: 32 echo '<FONT COLOR="RED">何が起きたか分からない!</FONT>'; 33 return; 34} 35 36$finfo = new finfo(FILEINFO_MIME_TYPE); 37if (!preg_match('/\.jpg$|\.jpeg$/i', $file['name']) 38 || $finfo->file($tmp_name) !== 'image/jpeg') { 39 echo '<FONT COLOR="RED">JPEGファイルを選択してください!</FONT>'; 40 return; 41} 42 43$result = true; 44$get = getimagesize($tmp_name); 45 46if ($get["0"]>1000) { 47 $result = false; 48 echo '<FONT COLOR="RED">横幅を小さくしてください!</FONT>'; 49} 50if ($get["1"]>1000) { 51 $result = false; 52 echo '<FONT COLOR="RED">縦幅を小さくしてください!</FONT>'; 53} 54 55if ($result) { 56 echo '送信完了'; 57}

UPLOAD_ERR_OK などのエラーコードの詳細は、以下を参照してください。
http://php.net/manual/ja/features.file-upload.errors.php

投稿2017/01/17 08:45

編集2017/01/17 09:15
KiyoshiMotoki

総合スコア4791

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

0

ベストアンサー

できるだけ、元のソースで。

php

1<?php 2 3$file = $_FILES['upfile']; 4$tmp_name = $file['tmp_name']; // 一時ファイルのパス 5if ($_FILES['upfile']['size'] === 0) { 6 echo '<FONT COLOR="RED">ファイルを選択して下さい!</FONT>'; 7} else { 8 echo '送信完了'; 9 10 $get = getimagesize($tmp_name); 11 if ($get["0"] > 1000) { 12 echo '<FONT COLOR="RED">横幅を小さくしてください!</FONT>'; 13 } else { 14 echo ""; 15 } 16 if ($get["1"] > 1000) { 17 echo '<FONT COLOR="RED">縦幅を小さくしてください!</FONT>'; 18 } else { 19 echo ""; 20 } 21} 22if ($_FILES['upfile']['error'] === 2) { 23 echo '<FONT COLOR="RED">ファイルサイズを小さくしてください!</FONT>'; 24} 25if (!preg_match('/\.jpg$|\.jpeg$/i', $tmp_name)) { 26 echo ""; 27} else { 28 echo '<FONT COLOR="RED">JPEGファイルを選択してください!</FONT>'; 29}

投稿2017/01/17 07:39

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Kuriaki

2017/01/17 07:50

回答ありがとうございます。 位置を変えることでエラーはでないのでしょうか?
退会済みユーザー

退会済みユーザー

2017/01/17 07:51

試せばわかることなので、回答は差し控えます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問