Tengo que reemplazar en una aplicación JEE (no la he creado yo) todas las apariciones de la función eval()
. El motivo es dado tras realizar un análisis del código mediante una solución de medición de la calidad y la seguridad del software (Kiuwan), el cuál indica como una de las soluciones para mejorar las métricas, eliminar las apariciones de dicha función.
Según esto, se me plantean varios casos en los que no se muy bien cómo realizar el reemplazo. Me gustaría saber si los cambios que he hecho son correctos y en el caso en el que no sepa como hacerlo, me podáis orientar (las líneas con eval están comentadas):
CASO 1
edicionGrid = function(formid,nombregrid) {
$("#Act_Buttons td:first").remove();
//var gruposLocal = eval("grupos_" + nombregrid);
var gruposLocal = (new Function('return ' + 'grupos_' + nombregrid))();
...
Error:
VM1089:1 Uncaught ReferenceError: grupos_evaluacion is not defined
...
CASO 2
myelemSelect = function(value, options) {
//Crea el elemento DOM para el tipo de campo MODCAMPO_SELECT
var nombreCampo = options["name"];
var select;
var id = jQuery(idGridParaCombos).getGridParam('selrow');
var valorSeleccionado;
var rowData;
if (id) {
var rowData = jQuery(idGridParaCombos).getRowData(id);
//valorSeleccionado = eval("rowData." + nombreCampo);
valorSeleccionado = (new Function('return rowData.' + nombreCampo))();
} else {
rowData = jQuery(idGridParaCombos).getRowData();
if(rowData != '') {
valorSeleccionado = value;
//valorSeleccionado = eval("rowData[0]." + nombreCampo);
valorSeleccionado = (new Function('return rowData[0].' + nombreCampo))();
}
}
...
Error:
VM1089:1 Uncaught ReferenceError: rowData is not defined
...
CASO 3
for(var i = 0; i < opciones["colModel"].length ; i++) {
if(opciones["colModel"][i].editoptions != null) {
if(opciones["colModel"][i].editoptions.custom_element != undefined) {
var evalFunction = opciones["colModel"][i].editoptions.custom_element = eval(opciones["colModel"][i].editoptions.custom_element);
}
if(opciones["colModel"][i].editoptions.custom_value != undefined) {
opciones["colModel"][i].editoptions.custom_value = eval(opciones["colModel"][i].editoptions.custom_value);
}
}
}
Nota: En este caso no se como realizar el reemplazo.
CASO 4
if (isDefined("gridLoadComplete")) {
gridLoadCompleteSoloLectura();
//eval("gridLoadComplete();"); //Ejecuto loadcomplete propio de la pagina
gridLoadComplete();
}
Nota: Llamo directamente a la función. No se si es correcto.
CASO 5
//eval("$(\"#jqgh_" + nuevoString+ " .ui-icon-" + arOrden[i] + "\").removeClass(\"ui-state-disabled\");");
$("#jqgh_" + nuevoString + " .ui-icon-" + arOrden[i]).removeClass("ui-state-disabled");
Nota: En este caso, parece que funciona correctamente.
He puesto un ejemplo de cada uno de los casos en los que aparece la función eval()
y me gustaría que me ayudarais o me digais si está bien hecho o no.
En otras apariciones he realizado el reemplazo y en principio parece que no da problemas, al igual que en los casos 4 y 5.
Gracias.
new Function
, mientras que no es lo mismo que eval, es muy parecido. Prueba a usarwindow["functionName"](arguments);
en tus llamadas como explican en esta respuesta.window
se refieren al objeto global del documento, por tanto sólo te hace falta el nombre de la función y que esté declarada en este ámbito general. Si no lo estuviera, el enlace explica cómo acceder a namespaces. Es decir,window["grupos_" + nombregrid]();
debería funcionar.