-
Elegant Expresion Writing
Hi!
I have an expresion with a lot of conditionals, I’m asking if i can “clean” the code instead of use a lot of “if else”.
Thanks
/* ----------- OSCILACION ----------- */
/////////// EDEN COMENTA: ///////////
/*Esta expresión crea una función de oscilacion automática en función del tiempo.*/
/////////// VARIABLES ///////////
//Eje X
invertir_x=effect("Oscilador 2D")("Invertir Fase Posicion X").value ;
frecuencia_x=effect("Oscilador 2D")("Frecuencia Posicion X");
amplitud_x=effect("Oscilador 2D")("Amplitud Posicion X");
multiplicador_x=effect("Oscilador 2D")("Multiplicador Posicion X");
rozamiento_x=effect("Oscilador 2D")("Rozamiento Posicion X");
desfase_x=effect("Oscilador 2D")("Desfase Posicion X");
//Eje Y
invertir_y=effect("Oscilador 2D")("Invertir Fase Posicion Y").value;
frecuencia_y=effect("Oscilador 2D")("Frecuencia Posicion Y");
amplitud_y=effect("Oscilador 2D")("Amplitud Posicion Y");
multiplicador_y=effect("Oscilador 2D")("Multiplicador Posicion Y");
rozamiento_y=effect("Oscilador 2D")("Rozamiento Posicion Y");
desfase_y=effect("Oscilador 2D")("Desfase Posicion Y");
activar_pulsos_y=effect("Oscilador 2D")("Pulsos Posicion Eje Y").value;
duracion_pulsos_y=effect("Oscilador 2D")("Duracion Pulsos Posicion Y");
periodo_pulsos_y=effect("Oscilador 2D")("Periodo Pulsos Posicion Y")
t_y=time%(periodo_pulsos_y+duracion_pulsos_y);/////////// CODIGO ///////////
//Eje X
switch (invertir_x) {
case 0:
if (rozamiento_x==0){
oscilacion_x=multiplicador_x*Math.sin(desfase_x+time*(frecuencia_x))*(amplitud_x);
}else{
oscilacion_x=multiplicador_x*Math.sin(desfase_x+time*(frecuencia_x))*(amplitud_x)/Math.exp(rozamiento_x*time);
}
break;
case 1:
if (rozamiento_x==0){
oscilacion_x=multiplicador_x*Math.cos(desfase_x+(Math.PI/2)+time*(frecuencia_x))*(amplitud_x);
}else{
oscilacion_x=multiplicador_x*Math.cos(desfase_x+(Math.PI/2)+time*(frecuencia_x))*(amplitud_x)/Math.exp(rozamiento_x*time);
}
break;
}
//Eje Y
switch (invertir_y) {
case 0:
if (activar_pulsos_y==1){
if (rozamiento_y==0){
oscilacion_y= (t_y < duracion_pulsos_y) ? multiplicador_y*((amplitud_y)*Math.sin(desfase_y+t_y*(frecuencia_y)*Math.PI*2)) : 0;
}else{
oscilacion_y= ease(t_y,0,duracion_pulsos_y,(multiplicador_y*((amplitud_y)*Math.sin(desfase_y+t_y*(frecuencia_y)*Math.PI*2))),0);
}
}else{
if (rozamiento_y==0){
oscilacion_y= multiplicador_y*Math.sin(desfase_y+time*(frecuencia_y))*(amplitud_y);
}else{
oscilacion_y= multiplicador_y*Math.sin(desfase_y+time*(frecuencia_y))*(amplitud_y)/Math.exp(rozamiento_y*time);
}
}
break;
case 1:
if (activar_pulsos_y==0){
if (rozamiento_y==0){
oscilacion_y= (t_y < duracion_pulsos_y) ? multiplicador_y*((amplitud_y)*Math.cos(desfase_y+(Math.PI/2)+t_y*(frecuencia_y)*Math.PI*2)) : 0;
}else{
oscilacion_y= ease(t_y,0,duracion_pulsos_y,(multiplicador_y*((amplitud_y)*Math.cos(desfase_y+(Math.PI/2)+t_y*(frecuencia_y)*Math.PI*2))),0);
}
}else{
if (rozamiento_y==0){
oscilacion_y= multiplicador_y*Math.cos(desfase_y+(Math.PI/2)+time*(frecuencia_y))*(amplitud_y);
}else{
oscilacion_y= multiplicador_y*Math.cos(desfase_y+(Math.PI/2)+time*(frecuencia_y))*(amplitud_y)/Math.exp(rozamiento_y*time);
}
}
break;
}
dimension=3;
try
{value[2]}
catch(array)
{dimension=2;}
try
{value[1]}
catch(array)
{dimension=1;}
x=effect("Oscilador 2D")("Activar Posicion Eje X").value;
y=effect("Oscilador 2D")("Activar Posicion Eje Y").value;
if (dimension>=2){
if (x==1) {
[value[0]+oscilacion_x,value[1]];
}
if (y==1){
[value[0],oscilacion_y+value[1]];
}
if (x==1 && y==1){
[oscilacion_x+value[0],oscilacion_y+value[1]];
}
if (x==0 && y==0){
[value[0],value[1]];
}
}
/* ===================================== */ -
2 Replies
-
1- To start with, you dont need to use “if else” to use or not use decay since the formula that takes decay into account is valid for no decay as well (Math.exp(0) = 1).
Similarly you dont need to do sin(x) —> cos(x+Pi/2) to get the – sign. Multiply by (-1) directly.
Also, i dont understand why you need both an amplitude and a multiplicador since they have the same affect…2- To shorten the length of the expression and make it easier to read, you could define functions.
For instance, at the beginning of the expression you define a functionfunction oscFunction1(t, multiplicador,desfase, frecuencia, amplitud, rozamiento, invertir)
{
var sign = (invertir) ? -1 : +1;
return(sign * multiplicador * amplitud * Math.sin(desfase+ t *(frecuencia))/Math.exp(rozamiento*t));
}then you can call that function anytime with the appropriate parameters, for instance the first block of your code (Eje X) reads:
oscilacion_x= oscFunction1(time, multiplicador_x,desfase_x, frecuencia_x, amplitud_x, rozamiento_x,invertir_x);
You can do similarly for the y oscillations. I didnt look in details but basically you can probably write a single function that works for all cases.
3- You can also replace “dimension = 3; try… try…” by
try{dimension = value.length;}
catch(e){dimension = 1;}
4- the last part of your code is a bit strange: you ask the expression to calculate oscilacion_x and oscilation_y and at the end you check if you actually need these calculations. For instance if your variables x and y are both 0, then you have calculated lots of things to return… [value[0], value[1]].
You could do:a) estimate the dimension
b) osc_x = (x==0) ? 0 : oscilacion_x (calculate osc_x only in the second case);
c) if (dimension >= 2) osc_y = (y==0) ? 0 : oscilacion_y (calculate osc_y only in the second case);
d) return the result:
switch (dimension)
{
case 1 : value + osc_x; break;
case 2 : [value[0]+osc_x,value[1]+osc_y]; break;
default : [value[0]+osc_x,value[1]+osc_y, value[2]];
}Xavier
-
Thanks for your words you had help me a lot!
Reply to this Discussion! Login or Sign Up