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

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

ただいまの
回答率

90.00%

python-bottleでエラー500がでる

解決済

回答 1

投稿

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

Erbin

score 4

スマートフォンでLEDを光らすor消すがしたい

ラズベリーパイゼロWを用いてiotしたいので、簡単にできるツールを探した結果bottleにたどりつき、HelloWorldはできました。
しかし、JavaScriptの実装やボタンの配置などをしようとコードを書いて表示させようとしたときに以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

n
127.0.0.1 - - [29/Aug/2019 15:46:08] "GET / HTTP/1.1" 200 291
favicon.ico
Traceback (most recenr call last):
  File "/usr/lib/python2.7/dist-packages/bottle.py", line 862, in _handle return route.call(**args)
  File "/usr/lib/python2.7/dist-packages/bottle.py", line 1740, in wrapper rv = callback(*a, **ka)
  File "/home/pi/Documents/controlLED_on_web.py", line 37, in index led_num = int(led)
ValueError: invalid literal for int() with base 19: 'favicon.ico'
127.0.0.1 - - [29/Aug/2019 15:46:12] "GET /favicon.ico HTTP/1.1" 500 745

該当のソースコード

from bottle import route, run
import RPi.GPIO as GPIO

host = '192.168.1.8'

GPIO.setmode(GPIO.BCM)
led_pins = [2, 4, 5]
led_states = [0, 0, 0]
switch_pin = 17

GPIO.setup(led_pins[0], GPIO.OUT)
GPIO.setup(led_pins[1], GPIO.OUT)
GPIO.setup(led_pins[2], GPIO.OUT)
GPIO.setup(switch_pin, GPIO.IN, pull_up_down=GPIO.PUD_UP)

def switch_status():
    state = GPIO.input(switch_pin)
    if state:
        return 'Up'
    else:
        return 'Down'

def html_for_led(led):
    l = str(led)
    result = " <input type='button' onClick='changed(" + l + ")' value='LED " + l + "'/>"
    return result

def update_leds():
    for i, value in enumerate(led_states):
        GPIO.output(led_pins[i], value)

@route('/')
@route('/<led>')
def index(led="n"):
    print(led)
    if led != "n":
        led_num = int(led)
        led_states[led_num] = not led_states[led_num]
        update_leds()
    response = "<script>"
    response += "function changed(led)"
    response += "{"
    response += "  window.location.href='/' + led"
    response += "}"
    response += "</script>"

    response += '<h1>GPIO Control</h1>'
    response += '<h2>Button=' + switch_status() + '</h2>'
    response += '<h2>LEDs</h2>'
    response += html_for_led(0) 
    response += html_for_led(1) 
    response += html_for_led(2) 
    return response

run(host='0.0.0.0', port=80)

試したこと

favicon.icoと書いてあったため https://www.favicon.cc/?action=icon&file_id=14586 のアイコンをダウンロードし上記の@route('/')の下に下記のコードを書いてみたりしました(変数の型が違うみたいなことを言われたためやめた)。どうにかしてfavicon.icoを送れればいいと思ってますがあってますか?

@route('/favicon.ico')
def favcon():
  fav=None
  return fav('favicon.ico', root='./static')


またこのサイトを見て.icoが送れそうなところを抜粋して書いたりしました。

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

使っているラズベリーパイはRasPizeroWでアプリはPython3(IDLE)です。
なお、python-bottleのバージョンが古いのかと思いコマンドでインストールするやつをやったところ最新のやつでした。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

現在 index()関数の実装は

@route('/')
@route('/<led>')
def index(led="n"):
    print(led)
    if led != "n":
        led_num = int(led)
        # (以下略)

となっておりますので、'/favicon.ico' が呼ばれた場合 led='favicon.ico' の引数で index関数が呼ばれてしまいます。

その場合は、led != 'n' の条件文は Trueとなりますので、その後の処理が行われ、最終的に

led_num = int(led)

にて、文字列 'favicon.ico' が int型にキャストできずにエラーが発生してしまいます。

とりあえずの修正としては上記のコードを

@route('/')
@route('/<led:int>')
def index(led=None):
    print(led)
    if (led is not None) and (0 <= led < len(led_status)):
        led_num = int(led)
        # (以下略)

のようにすることで

  • @route('/<led:int>') とすることで、int型以外の引数で index()関数が呼ばれないようにする
  • 引数がない事を明確にするために、デフォルト引数を led=None に変更する(これは好み)
  • ledの数の範囲外の値で呼ばれた場合の条件文のなかに入らないようにする

できるのではないでしょうか。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/08/29 19:17

    すみません。読み間違えてました。404になって結果画面が表示されないので、どうにかできませんか?たびたびすみません。。

    キャンセル

  • 2019/08/29 19:22 編集

    ん?
    「404 Not found」になるのは、あくまでも "http://****/favicon.ico " を呼んだときかと思うのですが、違います??
    通常通り、 "http://*****/ " を呼んだ場合や、"http://*****/1 " などと数値の場合は問題なく index関数が呼ばれるので従来どおりの動作をすると思いますよ。

    キャンセル

  • 2019/08/29 23:10

    できてました、、、
    http://***/Noneでできるのかなぁ~って思ってできなくて勘違いしてました、、申し訳ないです。
    この度はありがとうございました!

    キャンセル

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

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