python 戻り値が定義されてない

Question

###前提・実現したいこと
list_1から数を１つ取り出す。その数だけランダムに数字を発生させて、それらの和を取る。この動作でかかった時間も測る。
↑の流れを各list_1の要素ごとに和、かかった時間を出力する。
###発生している問題・エラーメッセージ

      for n in list_1:
          create(n)
--->      list_2.append(s)
          list_3.append(end)
      

NameError: name 's' is not defined

###該当のソースコード

import random
import time

list_1=[1,2,4]
list_2=[]
list_3=[]

def create(n):
    start= time.time()
    s=sum(random.randint(0,1000000)for x in range(n))
    end=time.time()-start
    return (s,end)

for n in list_1:
    create(n)
    list_2.append(s)
    list_3.append(end)

for (w1,w2,w3) in zip (list_1,list_2,list_3):
    print "要素" +str(w1) +"\n要素の和" + str


###補足情報(言語/FW/ツール等のバージョンなど)
mac10.12 python2.7.12

Answer 1

sはcreate()のローカル変数なので、関数の外から参照することはできません。
エラーメッセージの通り、そのスコープでsが定義されていません。
現状のコードだとcreate()の戻り値が捨てられているようなので、それを利用してあげれば上手く動くコードになると思います。

Comments

[peanut]

ローカル変数やスコープの意味を調べてみましたが、よくわかりません。
すみませんが参考URLなどあれば載せていただけるとありがたいです。

[D.I]

Carimaticsさんが書かれているのに補足すると
for分内の create(n) を以下のようにするといいと思います
s, end = create(n)

[D.I]

ローカル変数とは、そのメソッド内のみ使われるものです。
def create(n):内で使われている s,end は そのメソッド内でのみ使われ、
その後、破棄されると考えてもらえばいいと思います

[carimatics]

D.Iさんのコメントの通りですが、少し補足します。
「スコープ」というのは、「変数や関数などの名前を利用できる範囲」のことです。

例えば、関数内で定義された変数は、その関数内のみで利用可能な変数です。
つまり、変数を利用できる範囲が限定されています。
（繰り返しの説明になりますが）このような「変数が利用可能な範囲」をスコープと呼びます。
特に、関数内でのみ利用可能な変数は「ローカル変数」と呼ばれます。

スコープの取り扱いについては言語仕様によって異なります。
Pythonのスコープについては以下の公式ドキュメントで解説されていますので、参考にしてください。
http://docs.python.jp/2.7/tutorial/classes.html

公式ドキュメントはある程度の慣れがなければ用語が難しく感じるかもしれないので、以下のURLも参考に挙げておきます。
http://bacspot.dip.jp/virtual_link/www/si.musashi-tech.ac.jp/www/Python_IntroProgramming/02/index-1a.html
http://uxmilk.jp/41353

[peanut]

わかりやすい解説ありがとうございます！
上手く動きました。URLも読んで見ます