对于Selenium元素定位的总结

先来看一段项目中出现过的selenium代码

public void setAvailability(){
        selenium.click("//td[2]/div/input");
        selenium.type("//td[3]/div/input", "23");
        selenium.click("//td[4]/div/input");
        selenium.type("//td[5]/div/input", "23");
        selenium.click("//td[6]/div/input");
        selenium.type("//td[7]/div/input", "43");
        selenium.click("//tr[2]/td[2]/div/input");
        selenium.type("//tr[2]/td[3]/div/input", "12");
        selenium.click("//tr[2]/td[4]/div/input");
        selenium.type("//tr[2]/td[5]/div/input", "4");
        selenium.click("//tr[2]/td[6]/div/input");
        selenium.type("//tr[2]/td[7]/div/input", "3");
}

这个代码是出自某developer之手,由于赶进度,匆忙写出的.
这样的代码至少会有两个很严重的问题
1.代码可读性,非常差,即使看到了测试的页面,也很难想象出这些元素到底跟哪个页面相对应.维护起来非常困难.
2.这个代码如果用IE运行的,将非常非常得慢....
第一次看到这个代码,我一度很困惑程序怎么能正确得找得到页面中的元素.

对于这个问题,理想中的解决方案其实很简单,就是让项目中的developer养成良好的编程习惯，把每个页面元素,添加一个唯一的有意义的ID或者Name,这样就能解决如上两个问题,这个时候就体现出team work的重要性了.

再针对上面这个例子,如果我们不能让developer去增加ID的时候，代码依然可以写得更加高效,看了下页面的源代码,发现每个INPUT的父节点DIV都是有ID的，所以代码可以这样改进：

public void setAvailability() {

	    selenium.click("//div[@id='row1']/input");
	    selenium.type("//div[@id='row2']/input", "23");
        selenium.click("//div[@id='row3']/input");
        selenium.type("//div[@id='row4']/input", "23");
        selenium.click("//div[@id='row5']/input");
        selenium.type("//div[@id='row6']/input", "43");
 
}

上面的代码由于在Xpath中增加的ID，程序解析Xpath的速度必然会加快，可读性也会相对提高。

总结：
1.尽量使用ID或者name去定位元素，如果这个元素没有ID或者Name，那么就是用它最近的父节点的ID或者Name去定位。
2.写自动化脚本不是一个人的事情，是一个团队的事情，合作能更好，更轻松得完成任务
3.写出能运行的脚本很容易，但是我们的目标应该是要写出容易维护的脚本。