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

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

ただいまの
回答率

90.33%

  • Python

    9185questions

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

  • Python 3.x

    7390questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

  • LINE Messaging API

    170questions

【Python】LINE Messaging APIにおける多重ループの抜け方

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 532

poker

score 6

LINE Messaging APIによる多重ループから抜ける方法がわかりません。

作りたいものは占いbotのようなものです。
流れは

①「bot:「占い」と入力してください。」
|      ↓   ↑
|      ユーザーが占い以外の言葉を入力。
↓ 
②「bot:性別を入力してください。」
|      ↓   ↑
|      ユーザーが男・女・その他以外の言葉を入力。
↓ 
③「bot:誕生日を入力してください。」
|      ↓   ↑
|      ユーザーが4桁の数字以外を入力。
↓ 
④「bot:血液型を入力してください。」
|      ↓   ↑
|      ユーザーがA,B,AB,O以外の言葉を入力。
↓ 
⑤「bot:あなたはタイプAです。」

上記のようなものになります。
②〜④において、『終了』と入力したら一気に①に戻るプログラムにしたいのですが、うまくいきません。

(途中からです。)
@handler.add(MessageEvent, message=TextMessage)
def handle_text_message(event):
    text = event.message.text
    if text == '占い':
        line_bot_api.reply_message(
            event.reply_token, TextSendMessage(text='占いスタート。\n性別を教えてください。'))
        @handler.add(MessageEvent, message=TextMessage)
        def handle_text_message(event):
            text = event.message.text
            if text in ["男","女"]:
                line_bot_api.reply_message(
                    event.reply_token, TextSendMessage(text='誕生日を教えてください。\n(例:7月7日は「0707」と入力。)'))
                @handler.add(MessageEvent, message=TextMessage)
                def handle_text_message(event):
                    text = event.message.text
                    if 0100 < int(text) < 1232:
                        line_bot_api.reply_message(
                            event.reply_token, TextSendMessage(text='血液型を教えてね。'))
                    elif text == '終了':
                        line_bot_api.reply_message(
                        event.reply_token, TextSendMessage("終了します。"))
                        break #ここがエラーの出ている箇所です。
                    else:
                        line_bot_api.reply_message(
                        event.reply_token, TextSendMessage('誕生日を教えて。'))
            elif text == '終了':
                line_bot_api.reply_message(
                event.reply_token, TextSendMessage("終了します。"))
                break
            else:
                line_bot_api.reply_message(
                event.reply_token, TextSendMessage('性別を教えてください。'))
    else:
        line_bot_api.reply_message(
        event.reply_token, TextSendMessage('占いと入力してね。'))

(省略)


④まで上記のように書きました。

下記のようなエラーメッセージがでます。

 File "〜main.py", line 108
    break
    ^
SyntaxError: 'break' outside loop

初歩的な質問で大変申し訳ありません。
よろしくお願いします。

追記:コード全文記載します。
終了とした関数から抜け出せない状態です。
よろしくお願いします。

# -*- coding: utf-8 -*-

#  Licensed under the Apache License, Version 2.0 (the "License"); you may
#  not use this file except in compliance with the License. You may obtain
#  a copy of the License at
#
#       http://www.apache.org/licenses/LICENSE-2.0
#
#  Unless required by applicable law or agreed to in writing, software
#  distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
#  WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
#  License for the specific language governing permissions and limitations
#  under the License.

from __future__ import unicode_literals

import errno
import os
import sys
import tempfile
import requests
import random
import datetime
from argparse import ArgumentParser

from flask import Flask, request, abort

from linebot import (
    LineBotApi, WebhookHandler
)
from linebot.exceptions import (
    InvalidSignatureError
)
from linebot.models import (
    MessageEvent, TextMessage, TextSendMessage,
    SourceUser, SourceGroup, SourceRoom,
    TemplateSendMessage, ConfirmTemplate, MessageTemplateAction,
    ButtonsTemplate, ImageCarouselTemplate, ImageCarouselColumn, URITemplateAction,
    PostbackTemplateAction, DatetimePickerTemplateAction,
    CarouselTemplate, CarouselColumn, PostbackEvent,
    StickerMessage, StickerSendMessage, LocationMessage, LocationSendMessage,
    ImageMessage, VideoMessage, AudioMessage, FileMessage,
    UnfollowEvent, FollowEvent, JoinEvent, LeaveEvent, BeaconEvent,
    ImageSendMessage
)

app = Flask(__name__)

# get channel_secret and channel_access_token from your environment variable
channel_secret = os.getenv('LINE_CHANNEL_SECRET', '省略')
channel_access_token = os.getenv('LINE_CHANNEL_ACCESS_TOKEN', '省略')

line_bot_api = LineBotApi(channel_access_token)
handler = WebhookHandler(channel_secret)

static_tmp_path = os.path.join(os.path.dirname(__file__), 'static', 'tmp')

# function for create tmp dir for download content
def make_static_tmp_dir():
    try:
        os.makedirs(static_tmp_path)
    except OSError as exc:
        if exc.errno == errno.EEXIST and os.path.isdir(static_tmp_path):
            pass
        else:
            raise


@app.route("/callback", methods=['POST'])
def callback():
    # get X-Line-Signature header value
    signature = request.headers['X-Line-Signature']

    # get request body as text
    body = request.get_data(as_text=True)
    app.logger.info("Request body: " + body)

    # handle webhook body
    try:
        handler.handle(body, signature)
    except InvalidSignatureError:
        abort(400)

    return 'OK'

@handler.add(MessageEvent, message=TextMessage)
def handle_text_message(event):
    text = event.message.text
    if text == '占い':
        line_bot_api.reply_message(
            event.reply_token, TextSendMessage(text='占いスタート。\n性別を教えてください。'))
        @handler.add(MessageEvent, message=TextMessage)
        def handle_text_message(event):
            text = event.message.text
            if text in ["男","女"]:
                line_bot_api.reply_message(
                    event.reply_token, TextSendMessage(text='誕生日を教えてください。\n(例:7月7日は「0707」と入力。)'))
                @handler.add(MessageEvent, message=TextMessage)
                def handle_text_message(event):
                    text = event.message.text
                    if 100 < int(text) < 1232:
                        line_bot_api.reply_message(
                            event.reply_token, TextSendMessage(text='血液型を教えてね。'))
                        @handler.add(MessageEvent, message=TextMessage)
                        def handle_text_message(event):
                            text = event.message.text
                            if text in ["A","B","AB","O"]:
                                line_bot_api.reply_message(
                                    event.reply_token, TextSendMessage(text='あなたはタイプAだよ。'))
                            elif text == '終了':
                                line_bot_api.reply_message(
                                event.reply_token, TextSendMessage("終了します。"))
                                return
                            else:
                                line_bot_api.reply_message(
                                event.reply_token, TextSendMessage('血液型を教えて。'))
                    elif text == '終了':
                        line_bot_api.reply_message(
                        event.reply_token, TextSendMessage("終了します。"))
                        return
                    else:
                        line_bot_api.reply_message(
                        event.reply_token, TextSendMessage('誕生日を教えて。'))
            elif text == '終了':
                line_bot_api.reply_message(
                event.reply_token, TextSendMessage("終了します。"))
                return
            else:
                line_bot_api.reply_message(
                event.reply_token, TextSendMessage('性別を教えてください。'))
    else:
        line_bot_api.reply_message(
        event.reply_token, TextSendMessage('占いと入力してね。'))


if __name__ == "__main__":
    arg_parser = ArgumentParser(
        usage='Usage: python ' + __file__ + ' [--port <port>] [--help]'
    )
    arg_parser.add_argument('-p', '--port', type=int, default=5000, help='port')
    arg_parser.add_argument('-d', '--debug', default=False, help='debug')
    options = arg_parser.parse_args()

    # create tmp dir for download content
    make_static_tmp_dir()

    app.run(debug=options.debug, port=options.port)
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • poker

    2018/01/15 12:28 編集

    試行錯誤して、なんとか動くものができましたが載せたコードで大丈夫でしょうか?また、新たに問題が発生しましたのでご教授いただければと思います。よろしくお願いします。

    キャンセル

  • Udomomo

    2018/01/15 16:12

    なんとか動くというのは、今は「終了」と入力しなければ①〜⑤まで全て正しく動くという状態でしょうか?

    キャンセル

  • poker

    2018/01/15 16:17

    はい。ただループから抜けられず、⑤まで行き着いた後に④に戻る状態です。(⑤で結果が出た後で何か入力すると⑤の結果が出る。⑤の後に④に戻ってしまいます。)⑤まで行った後や途中で終了を入力した時にループを抜ける方法または①に戻る方法を知りたいです。

    キャンセル

回答 1

checkベストアンサー

0

breakforwhileなどループするブロックでのみ使えます。ifの中では使えません。
関数から抜けるにはreturnが使えます。returnとだけ書くと、何も返さずに関数から抜けることができます。
returnするとその時点で関数から抜けるので、例えば

def func(a):
   if a == 0:
       return

   return a + 1


とすると、a=0だった場合、何も返されずに関数から抜けます。a+1は実行されません。

⑤から抜けられない件は、⑤の結果を出す部分のソースコードがないのでわかりません。

-- 追記 --
handlerがいくつもあり、テキストを入力するとどのhandlerが呼ばれるかわからないのが、変な動作の原因になっている気がします。
以下のように直すと、handlerが1つになります。

@handler.add(MessageEvent, message=TextMessage)
def handle_text_message(event):
    text = event.message.text
    if text == '占い':
        # メッセージを返す処理
    elif text in ['男', '女']:
        # メッセージを返す処理
    ...
    elif text in ['A', 'B', 'O', 'AB']:
        # 結果を返す処理
    else:
        # その他の入力があったときのエラーメッセージを返す処理


このとき、最初にいきなり「男」と入力しても、占いのフローを無視して「誕生日を教えてください」と返されますが、
フローに依存せずに適当に結果を返してもよければ問題ないでしょう。
botは本来ステートレス(以前の会話を覚えていないこと)であるという想定で作られているので、botに複雑な処理を求めるとコードも複雑になってしまいます。

もし、ステートフル(今までの会話のやり取りを覚えており、それをふまえて会話すること)なサービスにしたいのであれば、bot単体だけでは厳しいです。

少しコーディングが難しくなってもよければ、結果を出す用のスクリプトとデータベースを用意する方がよいと思います。
例えば、以下のものを用意します。
・受付コード:ユーザと占い師を仲介し、それぞれと情報をやり取りするだけ
・占い師コード:受付からユーザの入力をもらい、占いをするだけ
・データベース:ユーザの入力を記憶しておくだけ

例として、③と④の流れは以下のようにできます。

③「0101」と入力
・受付は占い師に「0101」を伝え、「血液型を教えてください」と返す
・占い師はデータベースに「0101」を入れておく。

④「A」と入力
・受付は占い師に「A」を伝え、結果を判定するよう伝える
・占い師はデータベースに「A」を入れる
・占い師コードの結果判定用の関数が動く。データベースからそのユーザの情報を全て取り出し、必要なものが全てあれば結果を判定し、受付に伝える。
・受付はユーザに結果を伝える

こうすると、受付コードはユーザとの会話の流れを覚えておく必要がありません。実際に占っているのは背後の占い師コードなので、botがステートレスでも、ユーザからはbotが占ってるように見えます。
また、このように役割を分担すると、将来機能を追加したりメンテナンスしたりするときに、どこをいじればよいかわかりやすくなり、コードが複雑になりにくいです。

もしこういう実装を自力でやるのは難しすぎると思うのであれば、コーディングなしでノンプログラマーでもステートフルなLine botを作れるツールが世の中にあるので、それを使うと良いと思います。(「チャットボットツール」で検索するといろいろ出てきます。ものによっては個人向け無料プランもあります)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/15 17:12

    return
    をコードの
    break #ここがエラーの出ている箇所です。(③の動作中のコード)
    に入れて実行したところ、エラーもなく動いたのですが③で「終了」と入力すると「終了します。」と出力されますが、適当な文字を入力すると'誕生日を教えて。' と出力されるため、③の関数から抜けられていないようです。

    キャンセル

  • 2018/01/15 17:30

    コード全文記載しました。よろしくお願いします!

    キャンセル

  • 2018/01/15 17:47

    コードの構成全体を直した方がいいかもしれません。回答に追記しました。

    キャンセル

  • 2018/01/15 17:56

    回答ありがとうございます!
    そうなんですね…。elifで分岐させるやり方で一度試してみようかと思います!
    ありがとうございました!

    キャンセル

  • 2018/01/15 18:07

    elifに直して構築したところ、 if 0100 < int(text) < 1232:の部分でエラーがでてしまいました^^;
    今回のbotの作成はこのやり方では難しそうなので、結果を出す用のスクリプトとデータベースを用意する方法を勉強してみたいと思います。

    キャンセル

  • 2018/01/15 18:15

    おっとすみません。textが占いでも男・女でもない場合にそこの条件分岐に行くのですが、そのときtextが4ケタ数字でない場合にint型に変換できず、エラーになっているんだと思います。
    ご自身でコーディングを続けるなら、例えばtext.isdigit()などの条件を追加する必要があります。これは文字列中に数値しかない場合にTrueになります。

    キャンセル

  • 2018/01/15 19:02

    ありがとうございます!条件式にtext.isdigit()==Trueを入れて動きました!

    キャンセル

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

  • Python

    9185questions

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

  • Python 3.x

    7390questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

  • LINE Messaging API

    170questions