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

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

ただいまの
回答率

89.99%

SyntaxError: 'return' outside function エラー原因

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 13K+

horisuke3232

score 109

raspberry Pi を使って人感センサーに反応したらiPhoneにお知らせする物を作りたいと思い、調べたところ下記のサイトを発見しました。 
https://www.sho-yamane.me/entry/2015/06/07/195407 
こちらを参考にgmailのアドレス、パスワードを変更して実行してみたのですが、

line 52
 return msg
 SyntaxError: 'return' outside function

とエラーが表示されます。

#!/usr/bin/env python
# -*- coding: utf-8 -*-
from __future__ import print_function
import os.path
import datetime
import smtplib
import socket
import time
import RPi.GPIO as GPIO
GPIO.setmode(GPIO.BCM)
GPIO.setup(25, GPIO.IN)
from contextlib import closing
from email import Encoders
from email.Utils import formatdate
from email.MIMEBase import MIMEBase
from email.MIMEMultipart import MIMEMultipart
from email.MIMEText import MIMEText

#Gmailアカウント
ADDRESS = "アドレス"
PASSWARD = "パスワード"

#SMTPサーバの設定(Gmail用)
SMTP = "smtp.gmail.com"
PORT = 587
def create_message(from_addr, to_addr, subject, body, mime=None, attach_file=None):
    """
    メッセージを作成する
    @:param from_addr 差出人
    @:param to_addr 宛先
    @:param subject 件名
    @:param body 本文
    @:param mime MIME
    @:param attach_file 添付ファイル
    @:return メッセージ
    """
    msg = MIMEMultipart()
    msg["From"] = from_addr
    msg["To"] = to_addr
    msg["Date"] = formatdate()
    msg["Subject"] = subject
    body = MIMEText(body)
    msg.attach(body)

# 添付ファイル
    if mime != None and attach_file != None:
        attachment = MIMEBase(mime['type'],mime['subtype'])
        file = open(attach_file['path'])
        attachment.set_payload(file.read())
        file.close()
        Encoders.encode_base64(attachment)
        msg.attach(attachment)
        attachment.add_header("Content-Disposition","attachment", filename=attach_file['name'])

    return msg


    if mime != None and attach_file != None:
        attachment = MIMEBase(mime['type'],mime['subtype'])
        file = open(attach_file['path'])
        attachment.set_payload(file.read())
        file.close()
        Encoders.encode_base64(attachment)
        msg.attach(attachment)
        attachment.add_header("Content-Disposition","attachment", filename=attach_file['name'])

    return msg

def send(from_addr, to_addrs, msg):
    """
    メールを送信する
    @:param from_addr 差出人
    @:param to_addr 宛先(list)
    @:param msg メッセージ
    """
    smtpobj = smtplib.SMTP(SMTP, PORT)
    smtpobj.ehlo()
    smtpobj.starttls()
    smtpobj.ehlo()
    smtpobj.login(ADDRESS, PASSWARD)
    smtpobj.sendmail(from_addr, to_addrs, msg.as_string())
    smtpobj.close()

if __name__ == '__main__':
#宛先アドレス
    to_addr = "アドレス"
#件名と本文
    subject = "侵入者あり!"
    body = "たった今侵入者がありました!警戒してください!"
#添付ファイル設定(text.txtファイルを添付)
    mime={'type':'text', 'subtype':'comma-separated-values'}
    attach_file={'name':'test.txt', 'path':'./text.txt'}
#メッセージの作成(添付ファイルあり)
#msg = create_message(ADDRESS, to_addr, subject, body, mime, attach_file)
#メッセージ作成(添付ファイルなし)
while True:
    inputValue = GPIO.input(25)
if (inputValue == True):
    msg = create_message(ADDRESS, to_addr, subject, body)
    send(ADDRESS, [to_addr], msg)
    time.sleep(1)

何が原因なのでしょうか?

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

Pythonはインデントを正しく書かないと動作しないのはご存知でしょうか?

以下の記事の

Pythonを始めよう (1/5):CodeZine(コードジン)
https://codezine.jp/article/detail/1617

ブロックは全てインデントで記述する

の箇所を読んでみて下さい。

codeのマークアップがされているようなので、タブ文字でもないですよね。
正しいインデントを適用して、それを実行してみて下さい。

それでも上手く行かないようでしたら、この質問のコードを修正したものに置き換えてから、もういちど聞いてください。


(追記)

後半は、たぶんこうするのが正しいと思います。

if __name__ == '__main__':
    #宛先アドレス
    to_addr = "アドレス"
    #件名と本文
    subject = "侵入者あり!"
    body = "たった今侵入者がありました!警戒してください!"
    #添付ファイル設定(text.txtファイルを添付)
    mime={'type':'text', 'subtype':'comma-separated-values'}
    attach_file={'name':'test.txt', 'path':'./text.txt'}
    #メッセージの作成(添付ファイルあり)
    #msg = create_message(ADDRESS, to_addr, subject, body, mime, attach_file)
    #メッセージ作成(添付ファイルなし)
    while True:
        inputValue = GPIO.input(25)
        if (inputValue == True):
            msg = create_message(ADDRESS, to_addr, subject, body)
            send(ADDRESS, [to_addr], msg)
            time.sleep(1)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/23 07:51

    回答ありがとうございました。
    わかりやすいサイトの紹介もありがとうございました。
    無事原因が特定できました。
    コード修正し表記しました。

    別な質問となってしまうのですが、上記のコードで実行したところ、空白のまま何も動作がありません。
    人感センサーを動かしてみましたが、メールが送信されませんでした。
    なぜなんでしょうか?

    キャンセル

  • 2016/08/23 08:00

    修正例を記載しました。

    これは元のコード例が良くないですね。
    わざとなのかどうかわかりませんが。

    Pythonは、コードをサイトに掲載する際にこの手の問題がつきまといますので、ご注意ください。
    teratailでも質問にインデントの無いコードが貼られることが良くあります。

    キャンセル

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

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