Page tree

 

Ya soy un experto y estoy aquí para ayudar: ¡contesta preguntas pendientes y gana puntos!

Skip to end of metadata
Go to start of metadata

Los strings no deben estar duplicados

 

Clave

S1192

Severidad Menor
RepositorioSonarQube PHP

 

Los strings duplicados hacen que el proceso de refactorización sea propenso a errores, ya que se debe asegurar que todas las apariciones del string se actualicen.

Por otro lado, las constantes pueden ser referenciadas desde muchos lugares, pero necesitan ser actualizadas en un solo lugar

Ejemplo

function run() {
  prepare('action1');          // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Solución

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Excepciones

Para evitar la aparición de algunos falsos positivos, se excluyen aquellos literales de menos de 5 caracteres

  • No labels