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

如何解决如何在python代码上使用重构方法和函数？

我正处于编写 Python 代码的学习阶段。我已经创建了以下代码并成功获得了结果，但是我被要求重构代码，我不太确定如何继续。我确实参考了与重构相关的多篇文章，但变得更加困惑并且不清楚它是如何完成的。任何帮助将不胜感激。谢谢。

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)```

解决方法

好的代码重构可以由许多不同的步骤组成，具体取决于您的教育者/客户/经理/等。预计，可能需要花费大量的精力和时间。最好问问这个人他们对这个特定项目有什么期望，然后从那里开始。

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

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

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

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

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

# imports go here!

# specific functions
def some_function():
    return

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

为了刚开始的目的，这一切都非常简化。还有很多其他资源可以更深入地了解组织、良好做法和优化。

祝你好运！