Dicas de codificação

Use taglibs ou HTL o máximo possível

A inclusão de scriptlets em JSPs dificulta a depuração de problemas no código. Além disso, ao incluir scriptlets em JSPs, é difícil separar a lógica comercial da camada de visualização, o que é uma violação do Princípio de Responsabilidade Única e do padrão de design MVC.

Gravar código legível

O código é escrito uma vez, mas lido muitas vezes. Gastar algum tempo antes para limpar o código que escrevemos vai pagar dividendos pela estrada, já que nós e outros desenvolvedores precisamos lê-lo mais tarde.

Escolha nomes reveladores de intenção

O ideal é que outro programador não tenha que abrir um módulo para entender o que ele faz. Do mesmo modo, eles deveriam ser capazes de dizer o que um método faz sem lê-lo. Quanto melhor pudermos assinar essas ideias, mais fácil será a leitura de nosso código e mais rápido seremos capazes de escrever e mudar nosso código.

Na base de códigos AEM, são usadas as seguintes convenções:

• Uma única implementação de uma interface é chamada de <Interface>Impl, ou seja, ReaderImpl.
• Várias implementações de uma interface são chamadas de <Variant><Interface>, ou seja, JcrReader e FileSystemReader.
• As classes base abstratas são nomeadas como Abstract<Interface> ou Abstract<Variant><Interface>.
• Os pacotes são nomeados como com.adobe.product.module. Cada artefato Maven ou pacote OSGi deve ter seu próprio pacote.
• As implementações Java são colocadas em um pacote impl abaixo de sua API.

Observe que essas convenções não precisam necessariamente ser aplicadas às implementações do cliente, mas é importante que as convenções sejam definidas e aplicadas para que o código possa permanecer preservável.

O ideal é que os nomes revelem a sua intenção. Um teste de código comum para quando os nomes não são tão claros quanto deveriam ser é a presença de comentários explicando para que serve a variável ou o método:

Não se repita

DRY indica que o mesmo conjunto de códigos nunca deve ser duplicado. Isso também se aplica a coisas como literais de cordas. A duplicação de código abre a porta para defeitos sempre que algo tem que mudar e deve ser procurado e eliminado.

Evitar regras CSS nu

As regras de CSS devem ser específicas ao seu elemento de público alvo no contexto do aplicativo. Por exemplo, uma regra CSS aplicada a .content.center seria excessivamente ampla e poderia potencialmente causar impacto em muitos conteúdos em todo o sistema, exigindo que outras pessoas substituíssem esse estilo no futuro. .myapp- centertextseria uma regra mais específica, pois está especificando ** texto centralizado no contexto do aplicativo.

Elimine o uso de APIs obsoletas

Quando uma API está obsoleta, é sempre melhor encontrar a nova abordagem recomendada em vez de depender da API obsoleta. Isso garantirá upgrades mais suaves no futuro.

Gravar código localizável

Quaisquer strings que não estejam sendo fornecidas por um autor devem ser vinculadas em uma chamada para o dicionário i18n do AEM por meio de I18n.get() em JSP/Java e CQ.I18n.get() no JavaScript. Essa implementação retornará a sequência de caracteres transmitida a ela se nenhuma implementação for encontrada, portanto, isso oferta a flexibilidade de implementar a localização após implementar os recursos no idioma principal.

Escapar caminhos de recursos para segurança

Embora os caminhos no JCR não devam conter espaços, a presença deles não deve fazer com que o código quebre. Jackrabbit fornece uma classe de utilitário de Texto com os métodos escape() e escapePath(). Para JSPs, a interface do usuário do Granite expõe uma função granite:encodeURIPath() EL.

Use a API XSS e/ou HTL para proteger contra ataques de script entre sites

AEM fornece uma API XSS para limpar facilmente os parâmetros e garantir a segurança contra ataques de script entre sites. Além disso, o HTL tem essas proteções incorporadas diretamente à linguagem de modelo. Uma planilha de API está disponível para download em Desenvolvimento - Diretrizes e Práticas Recomendadas.

Implemente o registro apropriado

Para código Java, AEM suporta slf4j como a API padrão para mensagens de registro e deve ser usada em conjunto com as configurações disponibilizadas pelo console do OSGi para fins de consistência na administração. O Slf4j expõe cinco níveis diferentes de log. Recomendamos usar as seguintes diretrizes ao escolher o nível em que registrar uma mensagem:

• ERRO: Quando algo está quebrado no código e o processamento não pode continuar. Isso geralmente ocorre como resultado de uma exceção inesperada. Geralmente, é útil incluir rastreamentos de pilha nesses cenários.
• AVISO: Quando algo não funcionou corretamente, mas o processamento pode continuar. Isso geralmente resultará de uma exceção que esperávamos, como PathNotFoundException.
• INFORMAÇÕES: Informações úteis ao monitorar um sistema. Lembre-se de que esse é o padrão e que a maioria dos clientes deixará isso em vigor em seus ambientes. Portanto, não o utilize excessivamente.
• DEPURAÇÃO: Informações de nível mais baixo sobre o processamento. Útil ao depurar um problema com o suporte.
• TRACE: As informações de nível mais baixo, como métodos de entrada/saída. Normalmente, isso só será usado por desenvolvedores.

No caso do JavaScript, console.log deve ser usado somente durante o desenvolvimento e todas as instruções de log devem ser removidas antes da versão.

Evitar programação de culto de carga

Evite copiar o código sem entender o que ele faz. Na dúvida, é sempre melhor perguntar a alguém que tenha mais experiência com o módulo ou a API que você não está claro.