Buenas, tengo un script hecho, que utiliza mucho la funcion eval(), pero no es nada recomendable, en estos casos. Como podría reemplazarlo?
trabajar con arrays/objetos, en este caso añadir:
eval("$HERO." + group + ".push('" + item.id.replace("Check","") + "')")
eliminar objetos
eval("delete $HERO." + item[0]);
operaciones
eval("$HERO.skills." + skill + "" + operator + "=" + countSkill);
eval("$ELEMENTSPOINTS." + type + "." + element);
En este caso quiero mostrar en el objeto $ELEMENTSPOINTS el elemento de su tipo, se que se puede hacerse así:
$ELEMENTSPOINTS[type];
pero es solo para mostrar el subobjecto ELEMENTSPOINTS con todas sus propiedades, pero no el valor de uno en concreto.
Utilizo tanto el eval(), porque es dinamico, todos los eval estan dentro de una funcion que contiene parametros, cuyos parametros los utilizo en el eval(). Como lo hago sin utilizarlo?
Los objetos, son los siguientes:
$HERO={
name:null,
magic:[],
weapons:[],
gems:[],
skills:{
attack:0,
defenser:0,
speed:0
}
}
const $ELEMENTSPOINTS={
magic:{ // +
lightning:{
attack:10
},
ice:{
attack:4,
defenser:3
},
fire:{
attack:8
},
wind:{
defenser:2
}
},
weapons:{
sword:{
attack:5,
speed:1 // the speed substraction in the operation
},
shield:{
defenser:10,
speed:5
},
hatchet:{
attack:10,
speed:4
},
crossbow:{
attack:7,
speed:3
}
},
gems:{ // +
diamond:{
lightning:2,
ice:3,
fire:5,
wind:4
},
esmerald:{
lightning:2,
ice:3,
fire:5,
wind:4
},
ruby:{
lightning:3,
ice:2,
fire:3,
wind:2
},
sapphire:{
lightning:2,
ice:2,
fire:3,
wind:4
}
}
}
Un ejemplo de cuando utilizo eval:
ActionSkills = (type,element,operator)=>{
if(type != "gems"){
/*
The buttons' group aren't gems, so all skills' type got by the specificed element's sub-objects are iterated, and each iteration
gets skill's number and if the skill's type is 'speed' and operator is '+' the operator is become to '-'
(because the speed substraction of speed's total count), else if the operator is '-', it is became in '+' (because the button is inactive).
Finally is add/substraction the value get with the skill's total specificed. Thanks to eval()
Also, if the button's group clicked is 'magic', apart from does previous it, this runs the function ActionGems()
*/
for(skill in eval("$ELEMENTSPOINTS." + type + "." + element)){
countSkill = eval("$ELEMENTSPOINTS." + type + "." + element + "." + skill);
//console.log(skill + ":" + countSkill);
(skill == "speed" && operator == " + ")?(operator = "-"):(skill == "speed" && operator == "-")?(operator = " + "):null;
eval("$HERO.skills." + skill + "" + operator + "=" + countSkill);
}
(type == "magic")?ActionGems(type,element,operator):null;
}else{
/*
As are gems, the operation is different. So this executes the function ActionGems()
*/
ActionGems(type,element,operator);
}
}
Gracias de antemano
eval("$HERO.skills." + skill + "" + operator + "=" + countSkill);
Para qué quieres el eval?
$HERO.skills[skill] + = countSkill;
Si quieres usar distintos operadores usas una función.
function operar(operacion) {
switch(operacion) {
case "sumar":
$HERO.skills[skill] += countSkill;
break;
case "restar":
$HERO.skills[skill] -= countSkill;
break;
}
}
operar("sumar");
También deberías usar template strings por no volverte loco con tanta coma y +
eval(`$HERO.skills.${skill}${operator}=${countSkill}`);
Todos estos con la excepción de uno se pueden remplazar con la notación en corchetes:
eval("$HERO." + group + ".push('" + item.id.replace("Check","") + "')")
$HERO[group].push(item.id.replace("Check", ""));
eval("delete $HERO." + item[0]);
delete $HERO[item[0]];
eval("$ELEMENTSPOINTS." + type + "." + element);
$ELEMENTSSPOINTS[type][element];
El otro también necesita notación en corchetes pero necesitas manejar los casos de operación:
if(operator === '+'){
$HERO.skills[skill] += countSkill;
} else if (operator === '-') {
$HERO.skills[skill] -= countSkill;
}
O el
switch como lo menciona @XSSStringManolo
Muchisimas gracias a los dos! Me a funcionado muy bien.
@XSStringManolo, no cai en que podia hacerlo de esa manera.