2 f5 v0 D' A+ e. t( Cpackage com.thoughtworks.selenium; 6 O2 {" z! S2 m9 @9 jimport org.testng.ITestContext;, P% g* p4 \2 h- z/ x3 }, {
import org.testng.ITestResult; , l6 ~/ w9 U: Aimport org.testng.Reporter;( @. @9 `: I; c& @" Z. i# P
import org.testng.internal.IResultListener;) S( |+ T& w o) q6 Z/ l
" \5 @& I9 P+ R: }" Q6 Jimport java.io.File; 0 ]8 t. e4 l7 Z* x) R ) G/ g9 b. V) P; ]5 w7 K3 zpublic class ScreenshotListener implements IResultListener {* Q; M% i; H y) ^3 u
) Y5 u. F3 n: F2 n
File outputDirectory;1 D% i4 F' j2 {+ a" k; d
Selenium selenium;# J7 F& s4 X# g' q) ~
7 U! p4 Z4 L; W
public ScreenshotListener(File outputDirectory, Selenium selenium) {8 Q8 ^/ Z. h3 ^7 n
this.outputDirectory = outputDirectory; 0 T2 i1 |* B7 ?2 P7 x9 D this.selenium = selenium; 0 c* Q% k4 d( l8 J& ]7 o4 b1 S+ I } ( A' b) ~7 w# n; w) K" I( ` - h# R( G3 r7 o public void onTestFailure(ITestResult result) {) t# |, U5 n" H
Reporter.setCurrentTestResult(result); : c Z6 w9 c c5 g7 A. G# A8 G) c8 r- P1 Q9 U
try { - I" P7 Z% X8 X5 U outputDirectory.mkdirs(); 0 v/ j( p3 v5 z+ y! ]6 F File outFile = File.createTempFile("TEST-" + result.getName(), ".png", outputDirectory);8 Z' t% u& M2 p1 }* p
outFile.delete(); : Y7 h E( t/ p0 q- i# V/ y selenium.captureScreenshot(outFile.getAbsolutePath());* U6 R. x0 l8 Q ?$ z7 L0 j
Reporter.log("<a href='" + + |! K+ a" N4 r5 a3 ^7 z' j( d/ r* \ outFile.getName() +4 @8 w- b1 h9 l, L+ S5 H! {/ z. ?
"'>screenshot</a>"); / M3 A+ ^) |( f# x" p6 | y0 _2 ] } catch (Exception e) { 5 P8 [. p2 O9 t e.printStackTrace(); ( v8 r3 f2 H( E4 v" v O4 ~* p; k Reporter.log("Couldn't create screenshot");* u# [: [ ~ d' c7 m1 f8 H
Reporter.log(e.getMessage());' _/ @" l# Q' G3 x* U
}: B. n! z8 K8 `" _: y
7 x# K, y: @: h- p
Reporter.setCurrentTestResult(null); s" ]& ^$ n! h! R. M) q- P } 1 `7 b u/ V5 f$ n" Q} 9 x( a6 D8 A3 O1 ?. X, `9 C. D" M/ z- U- j" s' e/ |& }
package com.thoughtworks.selenium;& Q0 c/ Q( i6 m8 U- K6 ?) L9 O/ q2 t
import org.testng.ITestContext;( g# K& j2 N* G
import org.testng.ITestResult;4 i4 d& Y% X# M
import org.testng.Reporter;6 C5 T1 d, N! q
import org.testng.internal.IResultListener;& o) m; C% I3 \% W4 d4 g
import java.io.File;4 l U; o" Q/ W, C" B- b# \1 Y
public class ScreenshotListener implements IResultListener {& N# [0 i' i0 n+ Q. e g8 b: s& Y
File outputDirectory; - W: H% D- D8 Z- [$ L( J# u) e Selenium selenium;4 v( \' t( n/ h9 w m" r
public ScreenshotListener(File outputDirectory, Selenium selenium) {9 a# j# o1 R& R
this.outputDirectory = outputDirectory;' r. ?' J" Z; W6 d' Y
this.selenium = selenium; 4 U* m5 `) d8 J* i2 g }3 P7 R, _" ~. j
public void onTestFailure(ITestResult result) { ( `5 H! F' @2 p) | Reporter.setCurrentTestResult(result); # C6 A" V/ x9 Q$ {" c p try { 3 b( `9 Z p: v8 F, m9 j outputDirectory.mkdirs();. p9 E4 F# ^8 N; A
File outFile = File.createTempFile("TEST-" + result.getName(), ".png", outputDirectory); 8 ?. S( {% o1 v9 a( `" t; u outFile.delete();! R) U* V4 l# ?
selenium.captureScreenshot(outFile.getAbsolutePath()); 3 o" O; ?$ a0 u) S Reporter.log("<a href='" ++ B2 T* e8 O1 A, i5 ~3 h
outFile.getName() +! S. K6 \. S- \+ n' q
"'>screenshot</a>");7 u! Z5 f; e2 B3 e4 `* z7 G! j( O/ [
} catch (Exception e) { 9 W9 ?, {# S8 s4 Z3 n; O e.printStackTrace();6 |* g7 y% L! \, A. s8 G5 o
Reporter.log("Couldn't create screenshot");8 t" D0 N7 C5 x3 R
Reporter.log(e.getMessage());$ q1 W6 T$ |4 S8 D
}2 C4 ^7 A, D/ y. N8 Z
Reporter.setCurrentTestResult(null);: S" B: V& y$ a/ l, k
} ' L( [! b2 ~1 y}6 g4 E$ R% ^' z* p0 Q$ z" o
是不是适当增加空白行就提高了代码的可读性呢,这里说的是适当增加空白行,并不代表随意增加。事实上,如果增加一些无意义的空白行反而会使代码的可读性变差。" n- I3 z' u f/ O4 N
* s( \& ?, Z5 b* ^$ h' w- I$ c# T7 O
我们习惯于使有关联的代码彼此靠近,无明显关联的代码相互分隔。因此我们不但需要使用空白行间隔代码,还要调整代码位置,把有关联的代码放在一起,通常我们把被调用的函数放在调用函数的下面。这样别人在读我们的代码的时候再也不用经历来回“跳跃”的痛苦了。 / ^3 x' ]6 u& i, G 5 L- X+ s/ V) {& i* e3 B最后,我们通常把实体变量定义在类的顶部,这个只是我们Java程序员的习惯操作,如果突然在类的中间位置出现一个变量声明会让人觉得很奇怪。如果你想说,定义在中间是不想让定义位置和调用位置离太远,那么只能说明你的类定义出现了问题。建议你看一下旧文代码洁癖系列(三):整洁的类和函数。 ( o5 R; w; i6 b$ z# E8 _# @ 9 Z3 h% s* T3 k+ J- _+ v- x+ w横向格式 3 e6 }5 L' L6 M0 Z* m9 I) R; V & p/ R- ]% D5 v3 x, \介绍完垂直格式,紧接着自然要来介绍一下它的兄弟,横向格式。或许你会问为什么不叫水平格式,我的回答是:也可以叫水平格式,只要你喜欢。横向格式也是需要间隔和靠近的,这里的间隔主要是为了起到强调的效果。举个例子。 0 ]1 t" K- G; W. a& k 5 h3 t2 }# S* B# K+ K3 m" Y& D$ @int lineSize = line.length();! y4 J Q* k: o# n7 R
19 A$ E8 V4 H& n7 p5 n) K v
这里等号两边的空格分别是为了强调左右两边的元素。横向格式另一个比较重要的元素就是缩进,Java程序对缩进没有强制性要求,而Python程序对缩进的要求非常严格,稍有不慎,执行的结果就会不同。但是Java程序员也要注意缩进,因为我们的代码是层级关系,而缩进可以帮我们快速理清层级关系。4 E* z$ n* G" W( |
3 o% y3 o4 |2 d8 r/ o' Q最后,横向代码格式对每行代码长度是有要求的,如果代码过长,那么在阅读的时候就需要左右滑动,而这个操作其实是不受人喜欢的。虽然现在代码编辑器可显示的长度变大了,但我们还是习惯每行代码最多100个字符。 1 A/ d( G# W- S. Q ?9 B$ |/ f: U: }$ e# P6 R! J1 J, ^9 a `
团队的规则 + U7 Z( _# q. W , ^2 w; T" R' X' V! J) i: k5 l每一个优秀的团队都已一套属于自己的代码格式要求,有些是特定的,有些是使用公共的。我们team所用的代码规范就是Google的代码规范,阿里的代码规范也是比较被大家认可的。这里给大家一个小福利,在我的公众号后台回复【代码规范】,就可以获得一份阿里的Java开发手册。 : J7 u; V+ |6 I2 `: i8 l--------------------- * Z- X z+ z" r# t3 J! _5 R; \# x 9 d, k1 v; u. O2 g9 b+ e3 [ ' y r: t# r+ k0 j& ?& n0 X- {! i! e6 M) d: [- W( d( m
1 X) Y& x1 P- A4 G2 V' e9 N! s, v$ m
* m- s$ F( E6 d1 Z; N3 T " }/ n# B/ O! h t/ H 4 L. p( d7 J. }. E & ~1 O" F: L$ ~" w( P ' O' j7 f. t% `8 @+ c7 ?/ p( H h) T3 N. Y6 R
; e1 }9 m% X2 d' i: u5 _3 }+ S
: o: G5 H4 V5 }0 Q& D' b & e$ O: Z& d5 @ A : \: B9 l6 O0 c - @! K5 }$ a S, o % @5 q5 c9 {5 a% w6 P 5 C" O4 _! \' `1 {) n7 B3 X ! A7 A& \( P9 A2 [2 \8 T5 a 9 S& S/ }* Z2 C1 P8 M0 R # d4 \# e# c: }5 y4 a* f }8 S7 H: c! D' v' P1 m8 U
5 u5 u" r1 y1 S( c/ _: n( t
! Y4 g0 j. h8 p1 z: Z: | 2 i8 p8 v3 f9 M* X: G: F * |4 U0 f; E3 _: ~! e 6 x# u$ ]6 s' k2 y) q% K3 g