求助：将Python脚本转化为可被调用的模块？

Question

以下是我写的一段代码，目的是从指定目录中批量提示文件中的指定内容（PackageName），以脚本的形式运行可以满足需求，但现在想做成模块以被其它脚本调用，自己改了很久都不能满足，现发出来求助下高手，不胜感激。

# -*- coding: GBK -*-
__author__ = 'Kevin'
import os
import re

def traverse(path):
    '''
    遍历指定路径下的所有文件
    '''
    filesName = []
    for root,dirs,files in os.walk(path):
        for fn in files:
            filesName.append(os.path.join(root,fn))
        return filesName

def reCrash(s):
    '''
    正则匹配提取出crashed package name
    如：从// CRASH: com.wandoujia.phoenix2 (pid 3903)中提取出com.wandoujia.phoenix2
    '''
    m = '//\sCRASH:\s(.*)\s[(]pid\s(.*)[)]'
    n = re.match(m,s)
    if n:
        return n.group(1)
    else:
        return None

filesList = traverse('D:\\TestDoc')

crashCount = []

for i in range(len(filesList)):
    try:
        for line in open("%s" %filesList[i]):
            mCrash = reCrash(line)
            if mCrash:
                crashCount.append(mCrash)
    except:
        print('Faild to open file the %s.' %filesList[i])

print(crashCount)

通过网友的帮助问题已解决，代码如下：

# -*- coding: GBK -*-
__author__ = 'A'
import os
import re

def traverse(path):
    '''
    遍历指定路径下的所有文件
    '''
    filesName = []
    for root,dirs,files in os.walk(path):
        for fn in files:
            filesName.append(os.path.join(root,fn))
    return filesName

def reCrash(s):
    '''
    正则匹配提取出crashed package name
    如：从// CRASH: com.wandoujia.phoenix2 (pid 3903)中提取出com.wandoujia.phoenix2
    '''
    m = '//\sCRASH:\s(.*)\s[(]pid\s(.*)[)]'
    n = re.match(m,s)
    if n:
        return n.group(1)
    else:
        return None

def crashlist(filesList):
    crashCount = []
    for filesName in filesList:
        try:
            for line in open(filesName):
                mCrash = reCrash(line)
                if mCrash:
                    crashCount.append(mCrash)
        except:
            print('Faild to open file the %s.' %filesName)
    return crashCount

if __name__ == '__main__':
    filesList = traverse('D:\\TestDoc')
    print crashlist(filesList)

Answer 1

你这个代码写的不给力,应该属于python新手级别,你要将这些代码作为一个模块调用,很显然你是调用模块中的属性,而模块的属性就是模块顶层缩进的那些names,包括函数名,类名,以及其它模块级的全局标量,所以你这个代码要改,就应该把下面的那些代码通过一个函数封装起来

def func(directory):
    filesList = traverse(directory)

    crashCount = []

    for afile in filesList:
        try:
            for line in open(afile):
                mCrash = reCrash(line)
                if mCrash:
                    crashCount.append(mCrash)
        except:
            print('Faild to open file the %s.' % afile)

    print(crashCount)

这样你在其它的模块中导入该模块后,func这个属性就可见了,你也就可以通过调用这个函数来干你想干的事,我声明代码不一定完全正确,而且你的代码有很多风格上的问题