如何在 python 代码上使用带有函数的重构方法？

Question

我正处于编写 python 代码的学习阶段。我创建了以下代码并成功获得了结果，但是，我被要求重构代码，但我不太确定如何进行。我确实提到了与重构相关的多个 post，但变得更加困惑并且不清楚它是如何完成的。任何帮助将不胜感激。谢谢

import pandas as pd
import numpy as np
pd.set_option('display.max_columns',100)
data = pd.read_excel (r'S:\folder\file1.xlsx')
df_mail =pd.DataFrame(data,columns= ['CustomerName','CDAAccount','Transit'])
print(df_mail)
df_maillist =df_mail.rename(columns={'CDAAccount':'ACOUNT_NUM','Transit':'BRANCH_NUM'})
print(df_maillist)

## 1) Read SAS files 
pathcifbas = 'S:\folder\custbas.sas7bdat'
pathcifadr = 'S:\folder\cusadr.sas7bdat'
pathcifacc = 'S:\folder\cusact.sas7bdat'

##custbas.sas7bdat 

columns=['CIFNUM','CUSTOMR_LANGUG_C']
dfcifbas = pd.read_sas(pathcifbas)
print(dfcifbas.head())
df_langprf= dfcifbas[columns]
print(df_langprf.head())
df_lang =df_langprf.rename(columns={'CUSTOMR_LANGUG_C':'Language Preference'})
print(df_lang)

## cusadr.sas7bdat

dfcifadr = pd.read_sas(pathcifadr)
print(dfcifadr.head())
cols=['CIFNUM','ADRES_STREET_NUM','ADRES_STREET_NAME','ADRES_CITY','ADRES_STATE_PROV_C','FULL_POSTAL','ADRES_COUNTRY_C','ADRES_SPECL_ADRES']
df_adr= dfcifadr[cols]
print(df_adr.head)

### Renaming the columns
df_adrress =df_adr.rename(columns={'ADRES_CITY':'City','ADRES_STATE_PROV_C':'Province','FULL_POSTAL':'Postal Code','ADRES_COUNTRY_C':'Country','ADRES_SPECL_ADRES':'Special Address'})
print(df_adrress)

## cusact.sas7bdat

dfcifacc = pd.read_sas(pathcifacc)
print(dfcifacc.head())
colmns=['CIFNUM','ACOUNT_NUM','BRANCH_NUM','APLICTN_ID']
df_acc= dfcifacc[colmns]
print(df_acc)
## Filtering the tables with ['APLICTN_ID']== b'CDA'
df_cda= df_acc['APLICTN_ID']== b'CDA'
print(df_cda.head())
df_acccda = df_acc[df_cda]
print(df_acccda)

## Joining dataframes (df_lang), (df_adrress) and (df_acccda) on CIF_NUM
from functools import reduce
Combine_CIFNUM= [df_acccda,df_lang,df_adrress ]
df_cifnum = reduce(lambda left,right: pd.merge(left,right,on='CIFNUM'), Combine_CIFNUM)
print(df_cifnum)

#convert multiple columns object byte to string

df_cifnumstr= df_cifnum.select_dtypes([np.object])
df_cifnumstr=df_cifnumstr.stack().str.decode('latin1').unstack()
for col in df_cifnumstr:
   df_cifnum[col] = df_cifnumstr[col]
print(df_cifnum) ## Combined Data Frame

# Joining Mail list with df_cifnum(combined dataframe)

Join1_mailcifnum=pd.merge(df_maillist,df_cifnum, on=['ACOUNT_NUM','BRANCH_NUM'],how='left')
print(Join1_mailcifnum)

## dropping unwanted columns

Com_maillist= Join1_mailcifnum.drop(['CIFNUM','APLICTN_ID'], axis =1)
print(Com_maillist)

## concatenating Street Num + Street Name = Street Address

Com_maillist["Street Address"]=(Com_maillist['ADRES_STREET_NUM'].map(str)+ ' ' + Com_maillist['ADRES_STREET_NAME'].map(str))
print (Com_maillist.head())

## Rearranging columns

Final_maillist= Com_maillist[["CustomerName","ACOUNT_NUM","BRANCH_NUM","Street Address","City","Province","Postal Code","Country","Language Preference","Special Address"]]
print(Final_maillist)

## Export to excel

Final_maillist.to_excel(r'S:\Data Analysis\folder\Final_List.xlsx',index= False, sheet_name='Final_Maillist',header=True)```

Answer 1

好的代码重构可以由许多不同的步骤组成，具体取决于您的educator/client/manager/etc。期望，可能需要花费大量不同的精力和时间。最好询问此人他们对这个特定项目的期望并从那里开始。

但是，对于 Python 相对较新的人，我建议您从可读性和组织性入手。确保你所有的变量名都是明确的和可读的（假设你没有使用像匈牙利符号这样的必需模式）。作为起点，Python 命名约定倾向于使用小写字母和下划线，某些对象或 class 名称除外。 Python 实际上有一个非常深入的风格指南，叫做 PEP-8。你可以在这里找到它

https://www.python.org/dev/peps/pep-0008/

我个人最喜欢的是评论。注释应始终包含某事的“原因”，而不一定是“如何”（您的代码应具有足够的可读性以使这部分相对明显）。对于没有大量个人选择的较小脚本或作业来说，这有点困难，但最好记住这一点。

如果您了解了面向对象编程，您肯定应该将任务拆分为函数和 classes。在您的特定情况下，您可以为加载文件、对文件内容执行特定操作和导出等操作创建单独的函数。如果您注意到一堆功能往往具有相似的主题，那么可能是考虑为这些功能创建 class 的好时机！

最后，这再次是个人偏好（无论如何对于基本脚本），但我希望看到可读性和组织性的主要声明。

# imports go here!

# specific functions
def some_function():
    return

if __name__ == "__main__":
    # the start of your program goes here!

为了刚开始的目的，这一切都被大大简化了。还有很多其他资源可以更深入地介绍组织、良好实践和优化。

祝你好运！

如何在 python 代码上使用带有函数的重构方法？

How to use refactoring method with functions on python code?

python

refactoring

pandas