Centralizar o código geralmente é sempre uma boa ideia. Devemos reutilizar o código o máximo possível.
No entanto, é importante observar como fazer isso. Por exemplo, quando você tem um código que compute_prime_number () ou check_if_packet_is_bad (), ele é bom. As chances são de que o próprio algoritmo de funcionalidade evolua e será beneficiado.
No entanto, qualquer pedaço de código que se repita como uma prosa não se qualifica para ser centralizado imediatamente. Isto é mau. Você pode ocultar linhas de código arbitrárias dentro de uma função apenas para ocultar um código, ao longo do tempo, quando várias partes do aplicativo começarem a ser usadas, todas elas deverão permanecer compatíveis com as necessidades de todos os chamados da função.
Aqui estão algumas perguntas que você deve fazer antes de fazer
Tem a função que você está criando seu próprio significado inerente ou é apenas um monte de linhas?
Que outro contexto exigirá o uso das mesmas funções? É provável que você precise generalizar um pouco a API antes de usá-la?
Qual será a expectativa de (diferentes partes dos) aplicativos quando você lançar exceções?
Quais são os cenários para ver que as funções evoluirão?
Você também deve verificar se já existe algo assim. Vi tantas pessoas sempre tendendo a redefinir suas macros MIN, MAX, em vez de procurar o que já existe.
Essencialmente, a pergunta é: "Essa nova função é realmente digna de reutilização ou é apenas uma cópia e colagem ?" Se for o primeiro, é bom ir.