A presentation at Code Europe Wroclaw in in Wrocław, Poland by Paweł Lewtak
O b j e c t
C a l i s t h e n i c s 9
s t e p s
t o
b e t t e r
O O
c o d e
A b o u t
m e P a w e ł
L e w t a k S e n i o r
D e v e l o p e r
a t
X s t r e a m
@ p a w e l _ l e w t a k
A g e n d a L e a r n
h o w
t o
m a k e
o u r
c o d e
m o r e : r e a d a b l e r e u s a b l e t e s t a b l e m a i n t a i n a b l e
T h i n g s
w o r t h
k n o w i n g
D R Y K I S S S O L I D Y A G N I G R A S P
C a l i s t h e n i c s
C a l
•
i s
•
t h e n
•
i c s / ˌ k a l ə s ˈ T H e n i k s /
" C a l i s t h e n i c s
a r e
e x e r c i s e s
c o n s i s t i n g
o f
a v a r i e t y
o f
g r o s s
m o t o r
m o v e m e n t s ;
o f t e n r h y t h m i c a l
a n d
g e n e r a l l y
w i t h o u t
e q u i p m e n t o r
a p p a r a t u s . " W i k i p e d i a
O b j e c t
C a l i s t h e n i c s J e f f
B a y
W r i t t e n
f o r
J a v a
W h y
b o t h e r ?
C o d e
i s
r e a d
m o r e t h a n
i t
i s
w r i t t e n A u t h o r
u n k n o w n
Y o u
n e e d
t o
w r i t e
c o d e
t h a t m i n i m i s e s
t h e
t i m e
i t
w o u l d t a k e
s o m e o n e
e l s e
t o u n d e r s t a n d
i t
e v e n
i f
t h a t s o m e o n e
e l s e
i s
y o u A r t
o f
R e a d a b l e
C o d e
b y
D u s t i n
B o s w e l l , T r e v o r
F o u c h e r
R u l e
1 O n l y
o n e
l e v e l
o f i n d e n t a t i o n
p e r
m e t h o d
c l a s s
B o a r d ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ,
d a t a ) :
L e v e l
0
s e l f . b u f
=
" "
f o r
i
i n
r a n g e ( 1 0 ) :
L e v e l
1
f o r
j
i n
r a n g e ( 1 0 ) :
L e v e l
2
s e l f . b u f
=
d a t a [ i ] [ j ]
c l a s s
B o a r d ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ,
d a t a ) :
s e l f . b u f
=
" "
s e l f . c o l l e c t _ r o w s ( d a t a )
d e f
c o l l e c t _ r o w s ( s e l f ,
d a t a ) :
f o r
i
i n
r a n g e ( 1 0 ) :
s e l f . c o l l e c t _ r o w ( d a t a [ i ] )
d e f
c o l l e c t _ r o w ( s e l f ,
r o w ) :
f o r
j
i n
r a n g e ( 1 0 ) :
s e l f . b u f
=
r o w [ j ]
c l a s s
U s e r S e r v i c e ( o b j e c t ) :
d e f
r e g i s t e r ( s e l f ,
u s e r n a m e ,
e m a i l ,
p r o m o _ c o d e
=
F a l s e ) :
u s e r
=
s e l f . c r e a t e _ u s e r ( u s e r n a m e )
i f
e m a i l :
s e n d _ e m a i l ( u s e r ,
e m a i l )
i f
p r o m o _ c o d e :
s e n d _ p r o m o _ c o d e ( u s e r ,
p r o m o _ c o d e )
p r o d u c t s
=
s e l f . g e t _ p r o d u c t s _ b y _ u s e r ( . . . ) i f
p r o d u c t s
i s
N o n e :
p r o d u c t s
=
s e l f . g e t _ p r o d u c t s _ b y _ m e d i a ( . . . )
i f
p r o d u c t s
i s
N o n e :
p r o d u c t s
=
s e l f . g e t _ p r o d u c t s _ b y _ d o m a i n ( . . . )
i f
p r o d u c t s
i s
N o n e :
p r o d u c t s
=
s e l f . g e t _ a n y _ p r o d u c t s ( . . . ) :
i f
p r o d u c t s
i s
N o n e :
r a i s e
E x c e p t i o n ( ' A c c e s s
d e n i e d ' )
e l s e :
. . .
e l s e :
. . .
e l s e :
. . . e l s e :
. . .
p r o d u c t s
=
s e l f . g e t _ p r o d u c t s _ b y _ u s e r ( . . . ) i f
p r o d u c t s
i s
N o n e :
p r o d u c t s
=
s e l f . g e t _ p r o d u c t s _ b y _ m e d i a ( . . . ) i f
p r o d u c t s
i s
N o n e :
p r o d u c t s
=
s e l f . g e t _ p r o d u c t s _ b y _ d o m a i n ( . . . ) i f
p r o d u c t s
i s
N o n e :
p r o d u c t s
=
s e l f . g e t _ a n y _ p r o d u c t s ( . . . ) : i f
p r o d u c t s
i s
N o n e :
r a i s e
E x c e p t i o n ( ' A c c e s s
d e n i e d ' )
e l s e . . .
C h a i n
o f
c o m m a n d
c l a s s
C o m m a n d ( o b j e c t ) :
n e x t _ c o m m a n d
=
N o n e
d e f
a d d ( s e l f ,
n e x t _ c o m m a n d ) :
i f
s e l f . n e x t _ c o m m a n d
i s
N o n e :
s e l f . n e x t _ c o m m a n d
=
n e x t _ c o m m a n d
e l s e :
s e l f . n e x t _ c o m m a n d . a d d ( n e x t _ c o m m a n d )
d e f
_ p r o c e s s ( s e l f ,
a r g s ,
k w a r g s ) :
" " "
c o m p u t a t i o n s
" " "
p a s s
d e f
p r o c e s s ( s e l f ,
a r g s ,
k w a r g s ) :
r e s u l t
=
s e l f . _ p r o c e s s ( * a r g s ,
k w a r g s )
i f
r e s u l t
i s
N o n e :
i f
s e l f . n e x t _ c o m m a n d
i s
n o t
N o n e :
r e t u r n
s e l f . n e x t _ c o m m a n d . p r o c e s s ( * a r g s ,
k w a r g s )
e l s e :
r e t u r n
r e s u l t
r e t u r n
N o n e
c l a s s
G e t P r o d u c t s F o r U s e r ( C o m m a n d ) :
p a s s c l a s s
G e t P r o d u c t s B y M e d i a ( C o m m a n d ) :
p a s s c l a s s
G e t P r o d u c t s B y D o m a i n ( C o m m a n d ) :
p a s s c l a s s
G e t A n y P r o d u c t s ( C o m m a n d ) :
p a s s c o m m a n d s
=
G e t P r o d u c t s F o r U s e r ( ) c o m m a n d s . a d d ( G e t P r o d u c t s B y M e d i a ) c o m m a n d s . a d d ( G e t P r o d u c t s B y D o m a i n ) c o m m a n d s . a d d ( G e t A n y P r o d u c t s ) p r o d u c t s
=
c o m m a n d s . p r o c e s s ( . . . )
B e n e f i t s S i n g l e
r e s p o n s i b i l i t y B e t t e r
n a m i n g S h o r t e r
m e t h o d s R e u s a b l e
m e t h o d s
R u l e
2 D o
n o t
u s e
e l s e
k e y w o r d
d e f
l o g i n
( s e l f ,
r e q u e s t ) :
i f
r e q u e s t . u s e r . i s _ a u t h e n t i c a t e d ( ) :
r e t u r n
r e d i r e c t ( " h o m e p a g e " )
e l s e :
m e s s a g e s . a d d _ m e s s a g e ( r e q u e s t ,
m e s s a g e s . I N F O ,
' B a d
c r e d e n t i a l s ' )
r e t u r n
r e d i r e c t ( " l o g i n " )
d e f
l o g i n
( s e l f ,
r e q u e s t ) :
i f
r e q u e s t . u s e r . i s _ a u t h e n t i c a t e d ( ) :
r e t u r n
r e d i r e c t ( " h o m e p a g e " )
m e s s a g e s . a d d _ m e s s a g e ( r e q u e s t ,
m e s s a g e s . I N F O ,
' B a d
c r e d e n t i a l s ' )
r e t u r n
r e d i r e c t ( " l o g i n " )
d e f
f u n c t i o n ( p a r a m ) :
i f
p a r a m
i s
n o t
N o n e :
v a l u e
=
p a r a m
e l s e :
v a l u e
=
" d e f a u l t "
r e t u r n
v a l u e
d e f
f u n c t i o n ( p a r a m ) :
v a l u e
=
" d e f a u l t "
i f
p a r a m
i s
n o t
N o n e :
v a l u e
=
p a r a m
r e t u r n
v a l u e
d e f
f u n c t i o n ( v a r _ a ,
v a r _ b ,
v a r _ c ,
v a r _ d ) :
i f
v a r _ a :
i f
v a r _ b :
s o m e
c o d e
e l s e :
s o m e
c o d e
e l i f
v a r _ b
a n d
v a r _ c :
i f
n o t
v a r _ d :
s o m e
c o d e
e l s e :
s o m e
c o d e
e l i f
v a r _ b
a n d
n o t
v a r _ c :
s o m e
c o d e
e l s e :
s o m e
c o d e
E x t r a c t
c o d e
D e f a u l t
v a l u e
P o l y m o r p h i s m
S t r a t e g y
p a t t e r n
S t a t e
p a t t e r n
a m a t e u r
B e n e f i t s A v o i d s
c o d e
d u p l i c a t i o n L o w e r
c o m p l e x i t y R e a d a b i l i t y
R u l e
3 W r a p
p r i m i t i v e
t y p e s
i f
i t h a s
b e h a v i o u r
V a l u e
O b j e c t
i n
D D D
c l a s s
V a l i d a t o r ( o b j e c t ) :
d e f
c h e c k _ d a t e ( s e l f ,
y e a r ,
m o n t h ,
d a y ) :
p a s s
1 0 t h
o f
D e c e m b e r
o r
1 2 t h
o f
O c t o b e r ? v a l i d a t o r
=
V a l i d a t o r ( ) v a l i d a t o r . c h e c k _ d a t e ( 2 0 1 6 ,
1 0 ,
1 2 )
c l a s s
V a l i d a t o r ( o b j e c t ) :
d e f
c h e c k _ d a t e ( s e l f ,
y e a r :
Y e a r ,
m o n t h :
M o n t h ,
d a y :
D a y )
b o o l :
p a s s
F u n c t i o n
c a l l
l e a v e s
n o
d o u b t . v a l i d a t o r . c h e c k _ d a t e ( Y e a r ( 2 0 1 6 ) ,
M o n t h ( 1 0 ) ,
D a y ( 1 2 ) )
d e f
c a l c u l a t e _ d i s t a n c e ( s o u r c e _ x ,
s o u r c e _ y ,
t a r g e t _ x ,
t a r g e t _ y ) :
p a s s c a l c u l a t e _ d i s t a n c e ( 1 ,
2 ,
3 ,
4 )
f r o m
c o l l e c t i o n s
i m p o r t
n a m e d t u p l e c l a s s
P o i n t 2 D ( n a m e d t u p l e ( " P o i n t 2 D " ,
" x
y " ) ) :
p a s s d e f
c a l c u l a t e _ d i s t a n c e ( s o u r c e _ p o i n t ,
t a r g e t _ p o i n t ) :
p a s s c a l c u l a t e _ d i s t a n c e ( P o i n t 2 D ( 1 ,
2 ) ,
P o i n t 2 D ( 3 ,
4 ) )
B e n e f i t s E n c a p s u l a t i o n T y p e
h i n t i n g A t t r a c t s
s i m i l a r
b e h a v i o u r
R u l e
4 O n l y
o n e
d o t
p e r
l i n e
O K :
F l u e n t
i n t e r f a c e
c l a s s
P o e m ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ,
c o n t e n t ) :
s e l f . c o n t e n t
=
c o n t e n t
d e f
i n d e n t ( s e l f ,
s p a c e s ) :
s e l f . c o n t e n t
=
"
"
s p a c e s
s e l f . c o n t e n t
r e t u r n
s e l f
d e f
s u f f i x ( s e l f ,
c o n t e n t ) :
s e l f . c o n t e n t
=
s e l f . c o n t e n t
"
"
c o n t e n t
r e t u r n
s e l f P o e m ( " R o a d
N o t
T r a v e l l e d " ) . i n d e n t ( 4 ) \
. s u f f i x ( " R o b e r t
F r o s t " ) \
. c o n t e n t
N o t
O K :
g e t t e r
c h a i n
c l a s s
C a r t S e r v i c e ( o b j e c t ) :
d e f
g e t _ t o k e n ( s e l f ) :
t o k e n
=
s e l f . g e t _ s e r v i c e ( ' a u t h ' ) \
. a u t h _ u s e r ( ' u s e r ' ,
' p a s s w o r d ' ) \
. g e t _ r e s u l t ( ) \
. g e t _ t o k e n ( )
r e t u r n
t o k e n
1 .
W h a t
i f
N o n e
i s
r e t u r n e d
i n s t e a d
o f
o b j e c t ?
2 .
H o w
a b o u t
e x c e p t i o n s
h a n d l i n g ?
c l a s s
F i e l d ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ) :
s e l f . c u r r e n t
=
P i e c e ( ) c l a s s
P i e c e ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ) :
s e l f . r e p r e s e n t a t i o n
=
"
" c l a s s
B o a r d ( o b j e c t ) :
d e f
b o a r d _ r e p r e s e n t a t i o n ( s e l f ,
b o a r d ) :
b u f
=
' '
f o r
f i e l d
i n
b o a r d :
b u f
=
f i e l d . c u r r e n t . r e p r e s e n t a t i o n
r e t u r n
b u f
c l a s s
F i e l d ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ) :
s e l f . c u r r e n t
=
P i e c e ( )
d e f
a d d _ t o ( s e l f ,
b u f f e r ) :
r e t u r n
s e l f . c u r r e n t . a d d _ t o ( b u f f e r ) c l a s s
P i e c e ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ) :
s e l f . r e p r e s e n t a t i o n
=
"
"
d e f
a d d _ t o ( s e l f ,
b u f f e r ) :
r e t u r n
b u f f e r
s e l f . r e p r e s e n t a t i o n c l a s s
B o a r d ( o b j e c t ) :
d e f
b o a r d _ r e p r e s e n t a t i o n ( s e l f ,
b o a r d ) :
b u f
=
' '
f o r
f i e l d
i n
b o a r d :
b u f
=
f i e l d . a d d _ t o ( b u f )
r e t u r n
b u f
B e n e f i t s E n c a p s u l a t i o n D e m e t e r ' s
l a w O p e n / C l o s e d
P r i n c i p l e
R u l e
5 D o
n o t
a b b r e v i a t e
W h y
a b b r e v i a t e ?
T o o
m a n y
r e s p o n s i b i l i t i e s
N a m e
t o o
l o n g
d e f
r e g i s t e r _ u s e r _ s e n d _ w e l c o m e _ e m a i l _ a n d _ a d d _ t o _ d e f a u l t _ g r o u p s ( ) :
p a s s
v s d e f
h a n d l e _ u s e r _ r e g i s t r a t i o n ( ) :
u s e r
=
c r e a t e _ u s e r ( )
s e n d _ w e l c o m e _ e m a i l ( u s e r )
a d d _ t o _ d e f a u l t _ g r o u p s ( )
A v o i d
c o n f u s i o n
a c c
=
0 / /
a c c u m u l a t o r ?
a c c u r a c y ? p o s
=
1 0 0 / /
p o s i t i o n ?
p o i n t
o f
s a l e ?
p o s i t i v e ? a u t h
=
N o n e / /
a u t h e n t i c a t i o n ?
a u t h o r i z a t i o n ?
b o t h ?
D u p l i c a t e d
c o d e
c l a s s
O r d e r ( o b j e c t ) :
d e f
s h i p _ o r d e r ( s e l f ) :
p a s s o r d e r
=
O r d e r ( ) o r d e r . s h i p _ o r d e r ( ) / /
v s c l a s s
O r d e r ( o b j e c t ) :
d e f
s h i p ( s e l f ) :
p a s s o r d e r
=
O r d e r ( ) o r d e r . s h i p ( )
S p l i t
&
e x t r a c t
R e f a c t o r !
T h i n k
a b o u t
p r o p e r
n a m i n g
B e n e f i t s C l e a r
i n t e n t i o n s I n d i c a t e
u n d e r l y i n g
p r o b l e m s
R u l e
6 K e e p
y o u r
c l a s s e s
s m a l l
W h a t
i s
s m a l l
2 0
l i n e s
p e r
m e t h o d 5 0
l i n e s
p e r
c l a s s 1 0
c l a s s e s
p e r
m o d u l e
B e n e f i t s S i n g l e
R e s p o n s i b i l i t y S m a l l e r
m o d u l e s C o h e r e n t
c o d e
R u l e
7 N o
m o r e
t h a n
2
i n s t a n c e
v a r i a b l e
p e r
c l a s s
C l a s s
s h o u l d
h a n d l e
s i n g l e v a r i a b l e
s t a t e
I n
s o m e
c a s e s
i t
m i g h t
b e t w o
v a r i a b l e s
c l a s s
C a r t S e r v i c e ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ) :
s e l f . l o g g e r
=
L o g g e r ( )
s e l f . c a r t
=
C a r t C o l l e c t i o n ( )
s e l f . t r a n s l a t i o n S e r v i c e
=
T r a n s l a t i o n S e r v i c e ( )
s e l f . a u t h _ s e r v i c e
=
A u t h S e r v i c e ( )
s e l f . u s e r _ s e r v i c e
=
U s e r S e r v i c e ( )
c l a s s
C a r t S e r v i c e ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ) :
s e l f . l o g g e r
=
L o g g e r ( )
s e l f . c a r t
=
C a r t C o l l e c t i o n ( )
B e n e f i t s H i g h
c o h e s i o n E n c a p s u l a t i o n F e w e r
d e p e n d e n c i e s
R u l e
8 F i r s t
c l a s s
c o l l e c t i o n s
P y t h o n ' s
c o l l e c t i o n s
m o d u l e
D o c t r i n e ' s
A r r a y C o l l e c t i o n
B e n e f i t s S i n g l e
R e s p o n s i b i l i t y
R u l e
9 D o
n o t
u s e
s e t t e r s / g e t t e r s
A c c e s s o r s
a r e
f i n e
D o n ' t
m a k e
d e c i s i o n s o u t s i d e
o f
c l a s s
L e t
c l a s s
d o
i t ' s
j o b
T e l l ,
d o n ' t
a s k
c l a s s
G a m e ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ) :
s e l f . s c o r e
=
0
d e f
s e t _ s c o r e ( s e l f ,
s c o r e ) :
s e l f . s c o r e
=
s c o r e
d e f
g e t _ s c o r e ( s e l f ) :
r e t u r n
s e l f . s c o r e
U s a g e E N E M Y _ D E S T R O Y E D _ S C O R E
=
1 0 g a m e
=
G a m e ( ) g a m e . s e t _ s c o r e ( g a m e . g e t _ s c o r e ( )
E N E M Y _ D E S T R O Y E D _ S C O R E )
c l a s s
G a m e ( o b j e c t ) :
d e f
_ _ i n i t _ _ ( s e l f ) :
s e l f . s c o r e
=
0
d e f
a d d _ s c o r e ( s e l f ,
s c o r e ) :
s e l f . s c o r e
=
s c o r e
U s a g e E N E M Y _ D E S T R O Y E D _ S C O R E
=
1 0 g a m e
=
G a m e ( ) g a m e . a d d _ s c o r e ( E N E M Y _ D E S T R O Y E D _ S C O R E )
B e n e f i t s O p e n / C l o s e d
P r i n c i p l e
R e c a p 1 .
O n l y
o n e
l e v e l
o f
i n d e n t a t i o n
p e r
m e t h o d , 2 .
D o
n o t
u s e
e l s e
k e y w o r d , 3 .
W r a p
p r i m i t i v e
t y p e s
i f
i t
h a s
b e h a v i o r , 4 .
O n l y
o n e
d o t
p e r
l i n e , 5 .
D o n ’ t
a b b r e v i a t e , 6 .
K e e p
y o u r
e n t i t i e s
s m a l l , 7 .
N o
m o r e
t h a n
t w o
i n s t a n c e
v a r i a b l e
p e r
c l a s s , 8 .
F i r s t
C l a s s
C o l l e c t i o n s , 9 .
D o
n o t
u s e
a c c e s s o r s
H o m e w o r k
C r e a t e
n e w
p r o j e c t
u p
t o 1 0 0 0
l i n e s
l o n g
A p p l y
p r e s e n t e d
r u l e s
a s s t r i c t l y
a s
p o s s i b l e
D r a w
y o u r
o w n
c o n c u l s i o n s
C u s t o m i z e
t h e s e
r u l e s
M a k e
t h e m
y o u r
o w n
F i n a l
t h o u g h t s
T h e s e
a r e
n o t
b e s t
p r a c t i c e s
T h e s e
a r e
j u s t
g u i d e l i n e s
U s e
w i t h
c a r e !
Q u e s t i o n s ?
a m a t e u r
T h a n k
y o u !
@ p a w e l _ l e w t a k
Talk about basic exercises everybody can do to improve code quality. I'll show how you can make your code cleaner, more readable and testable by applying few simple rules. This is something every beginner should know but it's also a good idea for experienced developers to refresh it once in a while.
for free. You
can too.