Refactoring
7 views
Skip to first unread message
Kiquenet
Apr 9, 2026, 10:42:54 AM (5 days ago) Apr 9
to AltNet-Hispano
dos opciones, cuàl es mejor ?
jsOnClick = profileId == 5 ? "goToTripsDetail('" + operatorId + "');return false;" : "goToShiftDetail('" + operatorId + "');return false;";
jsOnClick = profileId == 5 ? "goToTripsDetail('" + operatorId + "');return false;" : "goToShiftDetail('" + operatorId + "');return false;";
var jsFunction = profileId == 5 ? "goToTripsDetail" : "goToShiftDetail";
jsOnClick = $"{jsFunction}('{operatorId}');return false;";
La mejor aquí es la segunda (separar jsFunction y luego interpolar), por 3 motivos:
1. Evita duplicación: el string ";return false;" y el patrón de llamada solo están en un sitio.
2. Más mantenible: si cambias la firma JS (p. ej. añadir otro parámetro), lo haces una vez.
3. Más legible: primero decides qué función y luego construyes el onclick.
jsOnClick = $"{jsFunction}('{operatorId}');return false;";
La mejor aquí es la segunda (separar jsFunction y luego interpolar), por 3 motivos:
1. Evita duplicación: el string ";return false;" y el patrón de llamada solo están en un sitio.
2. Más mantenible: si cambias la firma JS (p. ej. añadir otro parámetro), lo haces una vez.
3. Más legible: primero decides qué función y luego construyes el onclick.
Reply all
Reply to author
Forward
0 new messages