Pythonで別の関数内のある関数の変数を使用する

Question

私はビデオファイルをダウンロードするプログラムを持っています。ここではフルですが、短いプログラムを心配しないでください。

import pafy 

def download(): 
    url = raw_input('Please enter the path to the video\n') 
    video = pafy.new(url) 
    vid_title = video.title 
    best = video.getbest() 
    streams = video.streams 
    print(vid_title) 
    for stream in streams: 
     print(stream) 
    print'Resolution: ',best.resolution,'\nExtension : ', best.extension 
    user_choice = raw_input("Do you want to download this video\ny or n\n") 
    if user_choice == 'y': 
     print'Your video will downloaded soon' 
     filename = best.download(filepath = '/home/mark/new_projects') 
     another_download() 
    elif user_choice =='n': 
     print'You have chosen not to download a video' 
     another_download() 


def another_download(): 
    another_choice = raw_input('Would you like to download another video\ny or n\n') 
    if another_choice == 'y': 
     download() 
    else: 
     print'Thank for using my program' 

download() 

私はより小さな機能に分解したいと思います。私はこれをやろうとしました：

しかし、これを試してみると、私は最高のものが宣言されていないと言っているエラーが出ます。私はグローバル変数としてベストを宣言したくありません。ある関数の変数を別の関数の中で使用する方法はありますか？

Answer 1

は、ここにあなたのためのカップルのオプションがあります。

ちなみにリターンはあなたの問題を解決します。私はできるだけベストを尽くして寝そべります。

オプション1：あなたが最初download()を呼び出していると仮定すると、あなたはあなたが必要なものを返すurl()持ち、download()方法で変数にそれを保存することができます

def url(): 
    url = raw_input('Please enter the path to the video\n') 
    video = pafy.new(url) 
    vid_title = video.title 
    best = video.getbest() 
    streams = video.streams 
    print(vid_title) 
    for stream in streams: 
     print(stream) 
    print'Resolution: ',best.resolution,'\nExtension : ', best.extension 
    return best 

def download(): 
    user_choice = raw_input("Do you want to download this video\ny or n\n") 
    if user_choice == 'y': 
     best = url() 
     print'Your video will downloaded soon' 
     filename = best.download(filepath = '/home/mark/new_projects') 
     another_download() 
    elif user_choice =='n': 
     print'You have chosen not to download a video' 
     another_download() 

オプション2：あなたは、グローバル変数を使用することができ、しかし私は、この場合には、それらを使用しての波及効果を知らない：

best = None 
def url(): 
    global best 
    url = raw_input('Please enter the path to the video\n') 
    video = pafy.new(url) 
    vid_title = video.title 
    best = video.getbest() 
    streams = video.streams 
    print(vid_title) 
    for stream in streams: 
     print(stream) 
    print'Resolution: ',best.resolution,'\nExtension : ', best.extension 

def download(): 
    global best 
    user_choice = raw_input("Do you want to download this video\ny or n\n") 
    if user_choice == 'y': 
     print'Your video will downloaded soon' 
     filename = best.download(filepath = '/home/mark/new_projects') 
     another_download() 
    elif user_choice =='n': 
     print'You have chosen not to download a video' 
     another_download() 

私はこれらのソリューションのいずれかが何をしたいあなたを与えるだろうと思うが、それはそれ自体ないとして、私は、この特定の場合には最初のをお勧めします複雑なプログラムのようです。

Answer 2

大きな関数を小さな関数に分割するのは良い習慣ですが、より緊急な問題は、メインループを使用して、そのように連鎖する代わりに関数を返すようにすることです。

今すぐダウンロード（） - > another_download（） - > download（） - > another_download（） - > download（） - > ...ので、n個のvidsをダウンロードしたい場合、最後の1つのフィニッシュまで2〜1の機能がぶら下がっています。

def url(): 
    ... 
    return best 

def download(): 
    best = url() 
    ...