UnboundLocalErrorの解決方法

Question

前提

AWSで自動電話通知システムを作っています。(Lambda Python3.9)

https://infrastructure-engineer.com/aws-amzaonconnect-008/#index_id3

こちらのサイトを拝見しコードを入力していたのですが下記のようなエラーがでてしまいました。

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

[ERROR] UnboundLocalError: local variable 'phone_number' referenced before assignment
Traceback (most recent call last):
  File "/var/task/lambda_function.py", line 112, in lambda_handler
    phone_number = get_phone_number(contactinfo, priority)
  File "/var/task/lambda_function.py", line 70, in get_phone_number
    return phone_number

該当のソースコード

import json
import boto3
from boto3.dynamodb.conditions import Key, Attr

# boto3からDynamoDBへアクセスするためのオブジェクトを取得
dynamodb = boto3.resource('dynamodb')

# "amazonconnect-contact-list"へアクセスするためのオブジェクトを取得
contactlist = dynamodb.Table("amazonconnect-contact-list")

# "amazonconnect-response-status"へアクセスするためのオブジェクトを取得
responsestatus = dynamodb.Table("amazonconnect-response-status")

# 残り周回数を取得する関数
def get_cycle(sqs_event):

    record = sqs_event['Records'][0]
    json_record = json.loads(record["body"])
    cycle = json_record.get("cycle",-1)

    return cycle


# "amazonconnect-response-status"の内容を返す関数
def status_scan():
    
    scanData = responsestatus.scan()
    items=scanData['Items']

    return scanData["Items"]


# "amazonconnect-contact-list"の内容を返す関数
def operation_scan():

    scanData = contactlist.scan()
    items=scanData['Items']

    return scanData["Items"]


# 電話番号リストの行数を返す関数
def get_contactinfo_item_number(json_contactinfo):

    count = 0
    
    for line in json_contactinfo:
        count += 1

    return count


# 前回のプライオリティを返す関数
def get_before_priority(sqs_event):

    record = sqs_event['Records'][0]
    json_record = json.loads(record["body"])
    priority = json_record["priority"]

    return priority


# 指定されたプライオリティの電話番号を返す関数
def get_phone_number(json_contactinfo, now_priority):
    
    for line in json_contactinfo:
        if line['Priority']==now_priority:
            phone_number = line['Phone']

    return phone_number


def lambda_handler(event, context):

    # 残り周回数を取得
    now_cycle = get_cycle(event)

    # 応答結果を確認
    statustinfo = status_scan()

    # 応答結果がNGの場合
    if statustinfo[0]['CallStatus'] == "NG":
        
        # 電話番号リストを取得
        contactinfo = operation_scan()

        # 電話番号リストの行数を取得
        contactinfo_item_number = get_contactinfo_item_number(contactinfo)

        # 前回のプライオリティを取得
        before_priority = get_before_priority(event)

        # 前回のプラオリティが電話番号リストの行数と同じ場合(リストの最後まで通知した場合)
        if before_priority == contactinfo_item_number:
            
            # プライオリティを1に設定(最初に戻る)
            priority = 1

            # 残り周回数を1減らす
            now_cycle -= 1

        # 前回のプラオリティが電話番号リストの行数と異なる場合(リストの最後まで通知していない場合)
        else:

            # プライオリティに1を足す(次のプライオリティを設定)
            priority = before_priority + 1

        # 残り周回数が0以外の場合
        if now_cycle != 0:

            # 指定したプライオリティの電話番号を取得
            phone_number = get_phone_number(contactinfo, priority)
    
            # boto3からAmazonConnectへアクセスするためのオブジェクトを取得
            connect = boto3.client('connect')
    
            # AmzaonConnectの問い合わせフローを呼び出し電話発信
            connect.start_outbound_voice_contact(
                DestinationPhoneNumber=phone_number,
                ContactFlowId='***',
                InstanceId='***',
                SourcePhoneNumber='***',
            )
    
            # boto3からSQSへアクセスするためのオブジェクトを取得    
            sqs = boto3.resource('sqs')
    
            # "amazonconnect-queue-confirm"へアクセスするためのオブジェクトを取得
            name = 'amazonconnect-queue-confirm'    
            queue = sqs.get_queue_by_name(QueueName=name)
    
            # 現在のプライオリティと周回数をセットしてステータス確認用のキューにメッセージ送信
            response = queue.send_message(MessageBody=json.dumps({"priority": priority, "cycle": now_cycle}))
    

UnboundLocalErrorというエラーについて、ローカル変数とグローバル変数でエラーになったりならなかったりするみたいで、コードを書く順番などが関連しているということでしょうか？

初心者で申し訳ありませんが、回答してくださる方がいればお願いしたいです。

Answer 1

問題を言い換えると，必ずしもnow_priorityに一致するデータがが存在したわけではなかった．ということになります．

def get_phone_number(json_contactinfo, now_priority):
    
    for line in json_contactinfo:
        if line['Priority'] == now_priority: # 条件に一致せず
            phone_number = line['Phone']     # 変数phone_numberが定義されなかったことで

    return phone_number                      # エラーが起きる

ということです．関数の頭にデフォルト値を設定しておくなどして

def get_phone_number(json_contactinfo, now_priority):
    phone_number = "XXX-YYYY-ZZZ"            # デフォルト値を設定する
    for line in json_contactinfo:
        if line['Priority'] == now_priority: # 条件に一致せず
            phone_number = line['Phone']     # 変数phone_numberが定義されなくても

    return phone_number                      # デフォルト値を返すことができる

このように対処することが可能です．もしくは，データjson_contactinfoに必ずnow_priorityを持つデータを入れるよう，対処してください．前回もそうですが，「無い値」を抽出しようとしてエラーが出ています．

Comments

[ps_aux_grep]

実装記事を読んだ身としては，Python初心者に本システムが扱えるとは到底思えません．もう少しPythonやAWSの勉強をなさってから取り組まれる方がスムーズに開発ができ，異常対応もできるようになると思います．

[konan513]

ps_aux_grepさん
自分としましても勉強不足を痛感しています。
その上で2つの質問に丁寧に回答して頂いたことに対してとても感謝しています。
今後とも勉強させていただきます。
今回はありがとうございました。