O documento discute princípios de design de código e qualidade de código. Ele apresenta Guilherme Silveira como líder técnico, instrutor e palestrante sobre tópicos como escrever código limpo e evitar código ruim no futuro. O documento também fornece contatos de Guilherme Silveira.

1. Design de Código
qualidade a longo prazo
Guilherme Silveira
@guilhermecaelum

2. Como parar de escrever
o lixo do ano que vem
Guilherme Silveira
@guilhermecaelum

3. aka
“chega de código lixo”
Guilherme Silveira
@guilhermecaelum

5. - líder técnico

6. - líder técnico
- treinamentos

7. - líder técnico
- treinamentos
- online

8. guilherme.silveira@caelum.com.br
- líder técnico
- treinamentos
- online

9. guilherme.silveira@caelum.com.br
@guilhermecaelum
- líder técnico
- treinamentos
- online

10. treinamento?

11. online.caelum.com.br
trial hoje
(descontão no beta)

12. > 2.3 reviews por
resposta

13. online.caelum.com.br
trial hoje
(descontão no beta)

14. start!

15. o mais importante
em um projeto é
escrever código bonito

16. o mais importante
em um projeto é
escrever código bom

17. quem tem
um método zoado aí?

18. Implementation
Design
Architecture

19. architecture >=

20. architecture >= design >=

21. architecture >= design >= implementation

22. Browser
Controller
Hibernate

23. Browser
Controller
JPA

24. Browser
Controller
JPA file upload

25. Browser
Controller
JPA memory

26. Browser
Controller
### cloud ###
JPA memory

27. 14 days and

28. 14 days and

29. Cloud Computing

30. Implementation
Design
Architecture

31. IOC

32. IOC
DESIGN

33. IOC
Usando uma boa prática de design
Paulo Silveira
DESIGN

34. IOC
Usando uma boa prática de design
Paulo Silveira
DESIGN
ARQUITETURA

35. IOC
Usando uma boa prática de design
Paulo Silveira
DESIGN Mudamos a arquitetura e
economizamos $$$.
ARQUITETURA
Sérgio Lopes

36. Architecture
Design
Implementation

37. architecture <=

38. architecture <= design <=

39. architecture <= design <= implementation

40. architecture <= design <= implementation
architecture >=

41. architecture <= design <= implementation
architecture >= design >=

42. architecture <= design <= implementation
architecture >= design >= implementation

43. architecture <= design <= implementation
architecture >= design >= implementation
architecture =

44. architecture <= design <= implementation
architecture >= design >= implementation
architecture = design =

45. architecture <= design <= implementation
architecture >= design >= implementation
architecture = design = implementation

46. design arquitetura
java, ruby, scala, objective-c, c#
servidores, firewalls etc
implementação

47. existe

48. existe
nenhuma arquitetura

49. existe
nenhuma arquitetura
nenhum design

50. existe
nenhuma arquitetura
nenhum design
sem implementação

51. existe implementação

52. a única “coisa” que existe
é a implementação.

53. os mais importantes = os devs

54. o mais importante
em um projeto é
escrever código bom
bonito?

55. arquitetura é o mínimo
se a arquitetura não é boa, problemão

57. o que sobra? design...

58. o que sobra? design...
interface ===> difícil de manter

59. o que sobra? design...
interface ===> difícil de manter
implementation ===> difícil de manter

60. DESIGN
IMPLEMENTAÇÃO

61. DESIGN
Como seu código se comunica?
(design interface de comunicação)
IMPLEMENTAÇÃO

62. DESIGN
Como seu código se comunica?
(design interface de comunicação)
Como seu código é executado?
(design da implementação)
IMPLEMENTAÇÃO

63. veremos code

64. def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end
fonte pequena?

65. def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end
agora sim? fonte 16

66. for_att
ribute(
attr_na
agora sim? mil fonte

67. 23...
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

68. 23...
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

69. mas e se... 23...
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

70. mas eese...
mas se... 23...
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

71. mas eese...
mas ese... 23...
mas se...
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

72. mas eese...
mas ese... 23...
mas ese...
mas se...
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

73. mas eese...
mas ese... 23...
mas ese...
mas ese...
mas se...
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

74. mas eese...
mas ese... 23...
mas ese...
mas ese...
mas se...
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

75. mas eese...
mas ese... 23...
mas ese...
mas ese...
mas ese...
mas se...
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

76. conciso?

77. def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

78. o fluxo é COMPLEXO
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

79. o fluxo éSEMANTICA
não há COMPLEXO
def _read_attribute(attr_name)
attr_name = attr_name.to_s
attr_name = self.class.primary_key if attr_name == 'id'
value = @attributes[attr_name]
unless value.nil?
if column = column_for_attribute(attr_name)
if unserializable_attribute?(attr_name, column)
unserialize_attribute(attr_name)
else
column.type_cast(value)
end
else
value
end
end
end

80. muitos
ifs?

81. Java?
muitos
ifs?

82. Java? C++?
muitos
ifs?

83. Java? C++?
C?
muitos
ifs?

84. Java? C++?
C?
muitos
ifs?
Basic?

85. Java? C++?
C?
muitos
ifs?
Ruby?
Basic?

86. Clojure?
Java? C++?
C?
muitos
ifs?
Ruby?
Basic?

87. Clojure?
Java? C++?
C?
muitos
ifs?
Scala? Ruby?
Basic?

88. Clojure?
Java? C++?
C?
muitos This
month’s
ifs? new
language?
Scala? Ruby?
Basic?

89. i like repeating myself

90. variáveis
zoadas?

91. Java?
variáveis
zoadas?

92. Java? C++?
variáveis
zoadas?

93. Java? C++?
C?
variáveis
zoadas?

94. Java? C++?
C?
variáveis
zoadas?
Basic?

95. Java? C++?
C?
variáveis
zoadas?
Ruby?
Basic?

96. Clojure?
Java? C++?
C?
variáveis
zoadas?
Ruby?
Basic?

97. Clojure?
Java? C++?
C?
variáveis
zoadas?
Scala? Ruby?
Basic?

98. Clojure?
Java? C++?
C?
variáveis This
month’s
zoadas? new
language?
Scala? Ruby?
Basic?

99. nova

100. nova linguagem

101. nova linguagem
novo

102. nova linguagem
novo mindset

103. nova linguagem
novo mindset
novo

104. nova linguagem
novo mindset
novo uso idiomático

105. nova linguagem
novo mindset
novo uso idiomático
mesmos

106. nova linguagem
novo mindset
novo uso idiomático
mesmos erros

107. complexidade invisivel
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end

108. complexidade invisivel
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
Mateus, programador do cão

109. complexidade invisivel
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
Ahn?
Mateus, programador do cão

110. entendeu?
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end

111. uma linha = conciso?

112. uma linha = bonito?

113. complexidade,

114. complexidade, é natural

115. complexidade, é natural
intenção invisível

116. complexidade, é natural
intenção invisível é do mal

117. pegô?
def cached_attributes
@cached_attributes ||= columns.select { |c|
cacheable_column?(c) }.map { |col| col.name }.to_set
end

118. conciso?
def terere
@cached_attributes ||= columns.select { |c|
cacheable_column?(c) }.map { |col| col.name }.to_set
end

119. def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end

120. def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
how frequently does flow control appear?

121. def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
how many tests do I need?
how frequently does flow control appear?

122. 1 teste
cobertura = 100%

123. sua cobertura
mentiu

124. pq?

125. def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
Text
how frequently does flow control appear?

126. how many ifs?
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
Text
how frequently does flow control appear?

127. how many ifs?
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
Text
how frequently does flow control appear?

128. how many ifs?
how many fors?
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
Text
how frequently does flow control appear?

129. how many ifs?
how many fors?
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
Text
how frequently does flow control appear?

130. how many ifs?
how many fors?
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end
Text
how many tests do I need?
how frequently does flow control appear?

131. refatorar
agora?

132. and now?
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }
.map { |col| col.name }
.to_set
end

133. and now?
how frequently does flow control appear?
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }
.map { |col| col.name }
.to_set
end

134. ,

135. dê enter
tarefa do bem

136. pela “concisão”
menos toques no teclado zoaram meu código
def cached_attributes
@cached_attributes ||= columns.select { |c| cacheable_column?(c) }.map { |col| col.name }.to_set
end

137. pela “concisão”
menos toques no teclado zoaram meu código
def cached_attributes
@cached_attributes ||= 컬륨.걸라 { |c| 캐시_컬륨?(c) }.맵 { |컬| 컬.이름 }.투샛
end

138. pela “concisão”
menos teclas ainda!
def cached_attributes
@ ||= . { | | ?( ) }. { | | . }.
end

139. pela “concisão”
menos teclas ainda!
def cached_attributes
@ ||= . { | | ?( ) }. { | | . }.
end

140. pela “concisão”
menos teclas ainda!
def cached_attributes
@ ||= . { | | ?( ) }. { | | . }.
end

141. não é fazer código bonito

142. não é fazer código bonito
é não fazer deixar cocozinho

143. concisão = com o mínimo de
palavras, deixar claro
tarefa do bem

144. o que acontece?
Client client = clients.lookup(15L);
client.getName();

145. o que acontece?
requisição remota?
Client client = clients.lookup(15L);
client.getName();

146. o que acontece?
requisição remota?
EJB 2 hell!
Client client = clients.lookup(15L);
client.getName();

147. o que acontece?
requisição remota?
EJB 2 hell!
Client client = clients.lookup(15L);
client.getName();

148. o que acontece?
requisição remota?
EJB 2 hell!
Client client = clients.lookup(15L);
client.getName();
Quer ver uma mágica?

149. o que acontece?
requisição HTTP?
inferno de “rest” proxy
aka active resource
Client client = clients.lookup(15L);
client.getName();

150. o que acontece se?
client.save
def save
database.save(this)
end

151. o que acontece se?
an HTTP remote request!?
client.save
def save
database.save(this)
end

152. o que acontece se?
an HTTP remote request!?
ClientRule.checkUniqueEnroll is invoked!?
client.save
def save
database.save(this)
end

153. o que acontece se?
an HTTP remote request!?
ClientRule.checkUniqueEnroll is invoked!?
“invisible invocation(s)” pattern
client.save
def save
database.save(this)
end

154. composition

155. composition is good

156. composition is good
if invisible,

157. composition is good
if invisible, is evil

158. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

159. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

160. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

161. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

162. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

163. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

164. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

165. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

166. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

167. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name; 9 quebras de fluxo
}
return identifierNames.get(cls);
5 testes
} 3 negações

168. quem escreveu?

169. meu amigo
sérgio

170. private String getIdentifierName(Class<?> cls) {
if (!identifierNames.containsKey(cls)) {
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name;
}
return identifierNames.get(cls);
}

171. private String getIdentifierName(Class<?> cls) {
if (identifierNames.containsKey(cls)) {
return identifierNames.get(cls);
}
String name = null;
if (cls.isAnnotationPresent(Identifier.class)) {
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier.name() != null && !"".equals(identifier.name().trim())) {
name = identifier.name();
}
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name;
}

172. private String getIdentifierName(Class<?> cls) {
if (identifierNames.containsKey(cls)) {
return identifierNames.get(cls);
}
String name = null;
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier != null && identifier.name() != null && !"".equals(identifier.name().trim()
name = identifier.name();
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
identifierNames.put(cls, name);
return name;
}

173. private String getIdentifierName(Class<?> cls) {
if (identifierNames.containsKey(cls)) {
return identifierNames.get(cls);
}
String name = null;
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier != null && identifier.name() != null && !"".equals(identifier.name().trim()
name = identifier.name();
}
if (name == null) {
name = cls.getName().substring(cls.getName().lastIndexOf('.') + 1);
}
return cache(cls, name);
}
private String cache(Class<?> cls, String value) {
identifierNames.put(cls, name);
return value;
}

174. private String getIdentifierName(Class<?> cls) {
if (identifierNames.containsKey(cls)) {
return identifierNames.get(cls);
}
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier == null || identifier.name() == null || "".equals(identifier.name().trim())
return cache(cls, cls.getName().substring(cls.getName().lastIndexOf('.') + 1));
}
return cache(cls, identifier.name());
}
private String cache(Class<?> cls, String value) {
identifierNames.put(cls, name);
return value;
}

175. private String getIdentifierName(Class<?> cls) {
if (identifierNames.containsKey(cls)) {
return identifierNames.get(cls);
}
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier == null || identifier.name() == null || "".equals(identifier.name().trim())
return cache(cls, nameFor(cls));
}
return cache(cls, identifier.name());
}
private String cache(Class<?> cls, String value) {
identifierNames.put(cls, name);
return value;
}
private String nameFor(Class<?> cls) {
return cls.getName().substring(cls.getName().lastIndexOf('.') + 1);;
}

176. private String getIdentifierName(Class<?> cls) {
if (identifierNames.containsKey(cls)) {
3 quebras de fluxo
}
return identifierNames.get(cls);
0 negações
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier == null || nullOrEmpty(identifier.name())) {
return cache(cls, nameFor(cls));
}
return cache(cls, identifier.name());
}
private static boolean nullOrEmpty(String value) {
2 quebras de fluxo
}
return value == null || “”.equals(value.trim());
0 negações
private String cache(Class<?> cls, String value) {
identifierNames.put(cls, name);
return value;
}
private String nameFor(Class<?> cls) {
return cls.getName().substring(cls.getName().lastIndexOf('.') + 1);;
}

177. private String getIdentifierName(Class<?> cls) {
if (identifierNames.containsKey(cls)) {
3 quebras de fluxo
}
return identifierNames.get(cls);
0 negações
Identifier identifier = (Identifier) cls.getAnnotation(Identifier.class);
if (identifier == null || nullOrEmpty(identifier.name())) {
return cache(cls, nameFor(cls));
}
return cache(cls, identifier.name());
}
private static boolean nullOrEmpty(String value) {
2 quebras de fluxo
}
return value == null || “”.equals(value.trim());
0 negações
private String cache(Class<?> cls, String value) {
identifierNames.put(cls, name);
return value;
}
private String nameFor(Class<?> cls) {
return cls.getName().substring(cls.getName().lastIndexOf('.') + 1);;
}

178. um “if if if if if if” dói
(aka. sanduíche íche íche pattern)

179. ninguém acerta de primeira

180. esse dói?
select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo

181. esse dói?
select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo
Maurício Aniche

182. esse dói?
select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo
Como você testaria isso?
Maurício Aniche

183. esse dói?
select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo
Como você testaria isso?
mock.assert(_.select(“select from ...”))?
Maurício Aniche

184. esse dói?
select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo
Como você testaria isso?
mock.assert(_.select(“select from ...”))?
1 teste?
Maurício Aniche

185. sua cobertura
mente

186. select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo limit 2

187. select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo limit 2

188. select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo limit 2

189. Facinho...
select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo limit 2

190. Facinho...
select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo limit 2
Martin Fowler

191. Facinho...
select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo limit 2
sql também é linguagem...
Martin Fowler

192. Facinho...
select * from Contatos where a=b or (c=d && e=f)
order by primeiro_campo limit 2
sql também é linguagem...
mina gata hein...
Martin Fowler

193. menos de 5 testes não garanto
mas a cobertura diz “100%”...
mentirosa ...

194. coverage 100% would be
128 tests?

195. conscientizar-se da complexidade
de nosso código
Jim Webber

196. conscientizar-se da complexidade
de nosso código
não adianta esconder o spaguetti,
ele continua lá.
Jim Webber

197. conscientizar-se da complexidade
de nosso código
não adianta esconder o spaguetti,
ele continua lá.
tem que deixar explícito!
Jim Webber

198. OO 101

199. se você tem que executar algo sempre
que cria um Cliente?

200. se você tem que executar algosempre
que cria um Cliente?

201. se você tem que executar algo sempre
que cria um Cliente?

202. declare dependência
e código no
CONSTRUTOR

203. declare dependência
e código no
CONSTRUTOR
mas e no Rails, Javabeans, Hibernate, etc?

204. java (the javabeans way) rails (the ruby way)
dumb constructor dumb constructor
getter+setter attr_accessor
the scala way
dumb constructor
val

205. java (the javabeans way) rails (the ruby way)
dumb constructor dumb constructor
public variable public variable
the scala way
dumb constructor
public variable

206. O padrão de OO no mercado é a
“orgia dos objetos” pattern.
Todo mundo pega todo mundo.

207. Duvidou?
Spring: CTRL+F static
Rails: ActiveRecord.methods.size

208. positioning
code

209. a new scala project
vraptor + hibernate

210. modele REGRAS

211. Vamos refatorar?

212. no lugar ADEQUADO

213. post post post
Ah! Quebrou encapsulamento dá nisso!

214. post post post
Ah! Quebrou encapsulamento dá nisso!

215. post post post
Ah! Quebrou encapsulamento dá nisso!

216. post post post
Ah! Quebrou encapsulamento dá nisso!

217. post post post
Ah! Quebrou encapsulamento dá nisso!

218. Posso agulhar?
post post post
Ah! Quebrou encapsulamento dá nisso!

219. no lugar ADEQUADO
se eu mudar, eu quebro... mas ai
tudo bem! encapsulamento++
demeter++

220. no lugar ADEQUADO
se eu mudar, eu quebro... mas ai
tudo bem! encapsulamento++
demeter++

221. arquitetura insuficiente
==>
melhore

222. caso contrário

223. 1. implementação existe,
o resto é
interpretação

224. 2. código é complexo

225. 2. código é complexo
faça bunitu!
não se preocupe
com fofurice

226. 3. esconder intenção
não é simplificar

227. 4. se é difícil de testar,
é difícil de usar

228. 5. deixe seus devs
aprender e melhorar

229. 6. ao mudar linguagem
verifique 3 vezes
os princípios

230. 7. melhore

231. 7. refatore

232. 7. refatore
toda hora

233. 7. refactor
toda hora
para o melhor
não para o mais fofo

234. lembre-se
pair-programming
brown bag refactoring
code review

235. @guilhermecaelum
guilherme.silveira@caelum.com.br
http://online.caelum.com.br
te vejo lá

236. sobrou tempo?
seu código
hibernate

237. presentation code != domain code
controller
model
view
if discussion.author==myself ||
myself.roles[:moderator]

238. presentation code != domain code
controller
model
if discussion.author==myself
view myself.roles[:moderator]
if mysel.canEdit(discussion)

239. esses ifs não ficam repetidos em várias views

240. abra *.jsp, *.erb, *.ssp, *.similares
CTRL+F, if com algo != de boolean
CTRL+F, if com chamada != de boolean