requestをimportして動作するプログラムが動かない（仮想環境下）

Question

初めましてpython初心者のgoliraです。
今、pythonの仮想環境vent上でrequestを下記プログラムで動かそうとしたのですが
プログラム自体は立ち上がるのですが、"天気"と打っても反応が無く、他のif文である
"干支"、"選ぶ"等のキーワードにも何故か反応しなくなりました。
どうすれば正常に動作するのか、ご教授お願いします。

１：プログラムコード

from import_eto import eto_command
from my_random import choice_command, dice_command
from import_hiduke import(kyou_command, ima_command,
youbi_command)
from forecast import weather_command

def len_command(command):
＿ cmd, text = command.split()
＿ length = len(text)
＿ response = "文字列の長さは{}です".format(length)
＿ return response

def heisei_command(command):
＿ heisei, year_str = command.split()
＿ if year_str.isdigit():
＿ year = int(year_str)

＿ if year >= 1989:
＿ heisei_year = year - 1988
＿ response = "西暦{}年は、平成{}年です".format(year, heisei_year)
＿ else:
＿ response = "西暦{}年は、平成ではありません".format(year)

＿ else:
＿ response = "数値を指定して下さい"
＿ return response

aisatu_file = open("aisatu.txt", encoding="utf-8")
raw_data = aisatu_file.read()
aisatu_file.close
lines = raw_data.splitlines()

bot_dict = {}
for line in lines:
＿ word_list = line.split(",")
＿ key = word_list[0]
＿ response = word_list[1]
＿ bot_dict[key] = response

while True:
＿ command = input("なんか言え ")
＿ response = ""
＿ try:
＿ for key in bot_dict:
＿ if key in command:
＿ response = bot_dict[key]
＿ break

＿ if "平成" in command:
＿ response = heisei_command(command)

＿ if "長さ" in command:
＿ response = len_command(command)

＿ if "干支" in command:
＿ response = eto_command(command)

＿ if "選ぶ" in command:
＿ response = choice_command(command)

＿ if "サイコロ" in command:
＿ response = dice_command(command)

＿ if "今日" in command:
＿ response = kyou_command()

＿ if "現在" in command:
＿ response = ima_command()

＿ if "曜日" in command:
＿ response = youbi_command(command)

＿ if "天気" in command:
＿ response = weather_command()
＿ if not response:
＿ response = "日本語喋れや豚野郎"

＿ print(response)

＿ if "さようなら" in command:
＿ print("二度と開くなよまじで")
＿ break

except Exception as e:
    print("予期せぬエラーが発生しました")
    print("種類：", type(e))
    print("内容:", e)

”import_eto.py”のコード

#coding:utf-8

def eto_command(command):
＿ eto, year = command.split()
＿ eto_list = ("子", "丑", "寅", "卯", "辰", "巳", "午",
＿ "未", "申", "酉", "戌", "亥")
＿ eto_number = (int(year) + 8) % 12
＿ eto_name = eto_list[eto_number]

＿ response = "{}年生まれの干支は「{}」です".format(year, eto_name)
＿ return response

"my_random.py"のコード

import random

def choice_command(command):
＿ data = command.split()
＿ choiced = random.choice(data[1:])
＿ response = "「{}」が選ばれました".format(choiced)
＿ return response

def dice_command(command):
＿ num = random.randrange(1, 100)
＿ response = "「{}」の目が出ました".format(num)
＿ return response

"import_hiduke.py"のコード

rom datetime import date, datetime

def kyou_command():
＿ today = date.today()
＿ response = "今日の日付は{}です".format(today)
＿ return response

def ima_command():
＿ now = datetime.now()
＿ response = "現在日時は{}です".format(now)
＿ return response

def youbi_command(command):
＿ try:
＿ data = command.split()
＿ year = int(data[1])
＿ month = int(data[2])
＿ day = int(data[3])
＿ one_day = date(year, month, day)

＿ weekday_str = "月火水木金土日"
＿ weekday = weekday_str[one_day.weekday()]

＿ response = "{}は、{}曜日です".format(one_day, weekday)

＿ except IndexError:
＿ response = "３つの値（年月日）を空白で区切って入力して下さい"
＿ except ValueError:
＿ response = "正しい日付を入力して下さい"

＿ return response

"forecast.py"のコード

import requests

def weather_command():
＿ base_url = "http://weather.livedoor.com/forecast/webservice/json/v1"
＿ city_code = "230010"
＿ url = "{}?city={}".format(base_url, city_code)
＿ r = requests.get(url)
＿ weather_data = r.json()
＿ city = weather_data["location"]["city"]
＿ label = weather_data["forecasts"][0]["dateLabel"]
＿ telop = weather_data["forecasts"][0]["telop"]

＿ response = "{}の{}の天気は「{}」です".format(city, label, telop)
＿ return response

２：エラーコード

なんか言え 干支
予期せぬエラーが発生しました
種類： <class 'ValueError'>
内容: not enough values to unpack (expected 2, got 1)
なんか言え 選ぶ
予期せぬエラーが発生しました
種類： <class 'IndexError'>
内容: Cannot choose from an empty sequence
なんか言え 天気
なんか言え 天気
なんか言え 天気

Comments

[LouiS0616]

こちら https://teratail.com/help/question-tips#questionTips3-5-1 を参照し、インデントが崩れないように質問を編集してください。

[LouiS0616]

あと、手元で実行できるようにimport_eto.pyなども可能なら下さいな。

[goliragolira]

いつも迅速な回答、ありがとうございます。直ぐに対処してみます。

[yoorwm]

「初めまして」なのに「いつも迅速な回答」なのですか？

[goliragolira]

初めて見て頂く人も居るかと、思いまして、すいません、、、

[退会済みユーザー]

コードブロックの前後を```で囲んでください。もしくは、コード部分をマウスで選択して上側のタブ</>でも同じことができます。

Answer 1

とりあえずエラーの原因だけ指摘します。

なんか言え 干支

予期せぬエラーが発生しました
種類： <class 'ValueError'>
内容: not enough values to unpack (expected 2, got 1)

import_eto.eto_command を見る限り、西暦も併せて入力すべきです。

なんか言え 選ぶ

予期せぬエラーが発生しました
種類： <class 'IndexError'>
内容: Cannot choose from an empty sequence

やはり my_random.choice_command を見る限り、選択肢を渡すべきです。

---

基本的には書いたとおりにしか動いていないので、関数の呼び出し方にご留意ください。
また、例外の安易な握り潰しは絶対にしないようにしてください。

Wandboxにコードを起こして再現させたので、一応共有しておきます。Wandbox

コードブロックの適用の仕方

次のようにコードブロックを利用出来ます。

def main():
    print('Hello World!')

if __name__ == __main__:
    main()

方法は二つあります。
0. コードをハイライトした状態で<code>ボタンを押す
0. マークダウンを手動で入力する Qiita - Markdown記法 チートシート - コードの挿入

書いてみた

部分的ですが、参考までに。

main.py

from command import eto_command, choice_command

command_dict = {
    '干支': eto_command,
    '選ぶ': choice_command,
}

def interpret_command(input_str):
    command, *args = input_str.split()

    if command in command_dict:
        return command_dict[command](*args)

    return f'「{command}」の意味がわかりません。'


def main():
    while True:
        input_str = input('なんか入力してね: ')
        if not input_str:
            continue
        
        if 'さようなら' == input_str:
            print('了解。システムを終了します。')
            break

        print(
            interpret_command(input_str), end='\n\n'
        )


if __name__ == '__main__':
    main()

command.py

import random


def eto_command(*year_strs):
    if len(year_strs) != 1:
        return 'コマンドと一緒に西暦を一つだけ入力してね。'

    year_str = year_strs[0]
    try:
        year = int(year_str)
    except ValueError:
        return f'{year_str}は西暦だと解釈できないよ。'

    eto = [
        "子", "丑", "寅", "卯", "辰", "巳", "午", "未", "申", "酉", "戌", "亥"
    ][(year + 8) % 12]

    return f'{year}年生まれの干支は「{eto}」です。'


def choice_command(*candidate_list):
    if not candidate_list:
        return 'そもそも選択肢がないよ。'

    choiced = random.choice(candidate_list)
    return f'「{choiced}」が選ばれました。'

実行例 おまけ：Wandbox

なんか入力してね: 干支
コマンドと一緒に西暦を一つだけ入力してね。

なんか入力してね: 干支 2018
2018年生まれの干支は「戌」です。

なんか入力してね: 干支 2018 2019
コマンドと一緒に西暦を一つだけ入力してね。

なんか入力してね: 干支 3.14
3.14は西暦だと解釈できないよ。

なんか入力してね: 選ぶ
そもそも選択肢がないよ。

なんか入力してね: 選ぶ 1 2 3
「1」が選ばれました。

なんか入力してね: 選ぶ 1 2 3
「1」が選ばれました。

なんか入力してね: 選ぶ 1 2 3
「2」が選ばれました。

なんか入力してね: 面白いこと言って
「面白いこと言って」の意味がわかりません。

なんか入力してね:
なんか入力してね: さようなら
了解。システムを終了します。

Comments

[goliragolira]

ありがとうございます！
試してみます！

[goliragolira]

動きました！
本当にありがとうごさいます！

[LouiS0616]

解決されたようで何よりです。

今回のコードが失敗した原因は主に次の二つですね。
1. 関数の正しい用法を把握できていなかったこと。
2. エラーを適切にハンドリング出来ず、情報が潰れてしまっていたこと。

例外を安易に使うと一気にデバッグが難しくなります。
tryは、ここぞという場面以外では使わない方が良いでしょう。もちろん、その『ここぞ』を判断するためには幾度とない失敗が必要ではあるのですが。

ただし except Exception はほぼ全ての例外を捕捉してしまうので厳禁です。

---
コードブロックの適用も出来るようにしておいてくださいね。

[goliragolira]

エラーの把握をするためにも、try、exceptの使い方には気を付けたいと思います。