python+selenium链接页面自动化，求大师指点优化代码？
python+selenium链接页面自动化，求大师指点优化代码？代码是可以跑通的，感觉写的有点乱，求指点
#!/usr/bin/env python
#_*_coding:utf-8_*_
from selenium import webdriver
import time
from selenium.webdriver.common.action_chains import ActionChains
def OpenUrl():
driver=webdriver.Chrome()
driver.maximize_window()
driver.get('http://*********')
return driver
#5个主页面
links=['AiBB','HOT51','Live秀','MV','歌手']
#5个主页面的共同链接页面
link=['大星闻','爱豆评选','关于我们','联系我们','招贤纳士','隐私政策','沪B2-20090096','版权声明','0908243','沪网文［2014］0557-107',
'（沪 ）字第 1158 号','沪市文演（经）00-1060','上海市互联网违法和不良信息举报中心举报电话：021-62650919 客户邮箱：service@ikuaishou.com']
#5个主页面的共同链接页面，xpath定位的
def c():
driver.find_element_by_xpath('//a[@class="aibbLogo"]').click()
time.sleep(2)
driver.find_element_by_xpath('//a[@class="huoxiuLogo"]').click()
driver.find_element_by_xpath('//*[@id="index"]/div/div/div/div/div/a').click()
#5个主页面
def b(j):
driver.find_element_by_link_text(j).click()
#5个主页面的共同链接页面
def a(i):
driver.find_element_by_link_text(i).click()
#执行其中1页面（mv、歌手需要鼠标指上去并点击），加了判断
for j in links:
driver=OpenUrl()
if j==links or j==links:
faxian=driver.find_element_by_link_text('发现')
ActionChains(driver).move_to_element(faxian).perform()
time.sleep(1)
if j==links:
driver.find_element_by_xpath('//*[@id="header_nav"]/li/dl/dd/a').click()
elif j==links:
driver.find_element_by_xpath('//*[@id="header_nav"]/li/dl/dd/a').click()
else:
#执行其他4个主页面
b(j)
#判断了5个主页中不同的链接并执行
if j==links:
driver.find_element_by_link_text('打榜规则').click()
elif j==links:
driver.find_element_by_xpath('//*[@id="mvdetails-zc"]/div/div/div/div/div/div/a/img').click()
elif j==links:
driver.find_element_by_xpath('//*[@id="id-ui-videoitem"]/div/div/div/div/div/div/div/a/div/img').click()
driver.find_element_by_xpath('//*[@id="id-ui-videoitem"]/div/div/div/div/div/div/div/a/div').click()
elif j==links:
driver.find_element_by_xpath('//*[@id="sing-zc"]/div/div/div/div/a').click()
driver.find_element_by_xpath('//*[@id="yi-list-con"]/div/a/img').click()
elif j==links:
driver.find_element_by_xpath('//*[@id="head_image"]/a/img')
#5个主页面中共同的链接并执行，link定位的
for i in link:
a(i)
#5个主页面中共同的链接并执行，xpath定位的
c()
driver.quit()
根据童鞋贴的代码 个人有一下建议：
1：链接公用信息单独拉出来
2：自己封装click操作 调用时只需要传入元素定位的方式和元素标识，如element_click('xpath=/xxx')
3：可以把所有页面元素集中管理 比如用json或者xml
4：封装页面操作 参考PageObject模型 菜鸟@大虾 发表于 2017-2-28 09:46
根据童鞋贴的代码 个人有一下建议：
1：链接公用信息单独拉出来
2：自己封装click操作 调用时只需要传入 ...
好的 感谢建议我再看看
页:
[1]
