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

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

ただいまの
回答率

91.35%

  • Python

    3828questions

    Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

  • Flask

    81questions

    FlaskはPython用のマイクロフレームワークであり、Werkzeug・Jinja 2・good intentionsをベースにしています。

  • Jinja

    17questions

Python: request.filesで取得した値の場合分け

解決済

回答 1

投稿 2017/11/27 12:05

  • 評価
  • クリップ 0
  • VIEW 57

gomakichi

score 4

 困っていること

Pythonでユーザー画像の変更ができるようにフォームを作成しています。
その際に、pngやjpegなどの画像拡張子に変更しているダミー画像(拡張子は画像だが中身は別ファイル)を弾くようにしたいと考えています。

自分で調べて見たところ、imghdrを使うと中身を見てくれることがわかりました。
また、アップロードした画像の中身が画像ファイルでなかった時Noneを返すという性質があることもわかりました。

そこで、Noneが返ってこなかったら正常な処理を、Noneだったらエラーメッセージを表示というように処理を書いています。

しかし、これだと特に画像を変更をせず変更完了ボタンを押した際にもNoneを返すため、エラーメッセージを表示してしまいます。

そこで、imghdrで中身を見る前に、そもそも画像が選択されているのかを判定したいのですが、どうif文を書いたらいいかわからず困っています。

 現在書いているコード

image = request.files['image']
print(image)
imageDecision = imghdr.what(image)
print(imageDecision)

if imageDecision is not None:
~~~

 print(image)の処理結果

  1. 何も選択せずに変更完了ボタンを押す
    <FileStorage: '' ('application/octet-stream')>

  2. ダミー画像を選択して変更完了ボタンを押す
    <FileStorage: 'ダミー.png' ('image/png')>

 print(imageDecision)の処理結果

  1. 何も選択せずに変更完了ボタンを押す
    None

  2. ダミー画像を選択して変更完了ボタンを押す
    None

FileStorageが空だったらみたいに書けばいいのかなとは思うのですが、具体的にどう指定したら良いのかわからず困っています。。。。
以上です。よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

プロフィール変更画面でしょうか?

もし画像のアップロードが必須ではない(ニックネームやパスワードだけ変更する等)のなら、"もし画像がアップロードされたら" チェック処理とユーザー画像の変更処理を行うという流れにするのが自然かなと思います。

request.files['image'] からわかるように、アップロードされたファイルは request.files にリストとして格納されています。Python ではリストに特定の要素が含まれているかを <要素> in <リスト> で検索できるので、その判定を入れてあげればよいと思います。

"""プロフィール画像がアップロードされたら差し替える"""
if 'image' in request.files:
    image = request.files['image']
    imageDecision = imghdr.what(image)
    if imageDecision is None:
        flash('ファイルは画像ではないか壊れています', 'error' )
        return redirect(request.url)

またユーザー画像変更だけをするページであれば、ファイルをアップロードしないのはそもそもユーザー側の操作ミスなので、即エラーとするのがよいですね。

if 'image' in request.files:
    flash('ファイルを選択して下さい', 'error' )
    return redirect(request.url)

image = request.files['image']
imageDecision = imghdr.what(image)

if imageDecision is None:
    flash('ファイルは画像ではないか壊れています', 'error' )
    return redirect(request.url)

余談ですがHTMLのフォーム側にも適切なバリデーションを入れてあげると親切です。

たとえば、画像の選択を必須にしたいのならば required 属性を付与したり、ファイル形式を限定したいのであれば accept 属性を付与します。

<form>
    <input type="file" accept=".jpg,.gif,.png" required>
</form>

ブラウザによっては画像だけピックアップしてくれたりとユーザーの利便性が向上します。

投稿 2017/11/28 10:57

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/30 11:17

    遅くなってすみません。上記の内容で無事解決いたしました。ありがとうございます。

    キャンセル

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

ただいまの回答率

91.35%

関連した質問

  • 解決済

    pythonでのファイル検索、読み込みについて

    例えば A/B/C/D/ の中にEというディレクトリとFというディレクトリがあったとします。 EとFにはそれぞれe001.txt〜e100.txtとf001.txt~f100

  • 解決済

    Python - バイナリデータで読み込んだ画像を保存せずにイロイロ

    Python3です  やりたいこと 画像URLから画像を引っ張ってイチャコラするときに、一旦保存して編集しているプロセスをすっ飛ばしたい。  具体的に import

  • 解決済

    ボタンを押してデータの書き換えをしたい

    https://teratail.com/questions/73935 これの続きです。 評価されてない画像データを1つだけ表示して、ボタンで画像の種類を分類したいです。

  • 解決済

    pythonの画像の読み込みについて

    pythonの勉強をしています。ある画像を読み込み、それを配列化してテキストデータとしてファイルに保存したいです。 下記のようなコードを描いたのですが、これが正しく出力がされ

  • 受付中

    Pythonで複数画像データを配列化して保存する方法

    Pythonで複数画像データを数値化して配列に保存することを考えています。 1枚の画像データはflatten()関数で1行にして、それを画像枚数分だけ配列に保存したいです。

  • 解決済

    配列の中身を文字列として取得させたい

    配列の中身を文字列として取得させたいです。 image配列に [2, 5, 5, 2] という要素が入っています。 これを、 2 5 5 2 というように2行2列で

  • 受付中

    大量のCSVデータの処理方法

    100個ほどCSVファイルがあり、 それぞれのファイルで、30×30のセルに数値が入力されています。 この100個のCSVファイルに対して、同じセル座標について最大値最小値を計算し

  • 解決済

    pythonのスライスについて

    a = b[:,0] このようなコードがあったとき、どのようなことがおこなわれますか? bはこのコードがなりたつ何かだとすると なにだったら成り立ちますか? すみません、間

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

  • Python

    3828questions

    Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

  • Flask

    81questions

    FlaskはPython用のマイクロフレームワークであり、Werkzeug・Jinja 2・good intentionsをベースにしています。

  • Jinja

    17questions