[F1] optimizando el codigo, jugando con AS3

[salvadoresc]

resulta que me puse a jugar un rato con flash  y estoy travesenadolo la cosa es, que ahorita me enfrento a un dilema, ya que tengo dos variables, las cuales cada una puede tener valores desde cero hasta 4. y segun los valores de estas dos variables puedo tener diferentes combinaciones, que segun la combinacion me deberian asignar 8 diferentes resultados.

la cosa que he hecho una serie de decisiones para poder determinar cual combinacion es la que tengo y lo trate de hacer asi:

Código: [Seleccionar]

if (erv == 0 && env == 0) {
puntosPares=2;
}
if (erv == 0 && env == 1 || env == 2 || env == 3 || env == 4  ) {
puntosPares=3;
}
if (erv == 1 && env == 0 ) {
puntosPares=4;
}
if (erv == 2 && env == 0 ) {
puntosPares=5;
}
if (erv == 1 ||  erv == 2 && env == 3 || env == 4 ) {
puntosPares=6;
}
if (erv == 3 ||  erv == 4 && env == 0 || env == 1 || env == 2 ) {
puntosPares=7;
}
if (erv == 1 ||  erv == 2 && env == 1|| env == 2 ) {
puntosPares=8;
}
if (erv == 3 ||  erv == 4 && env == 3 || env == 4 ) {
puntosPares=9;
}
asi deberian de ser las asignaciones de valores

var1              var2                 resultado
0                    0                     1
0                 1,2,3,4               2
1                       0                  3
2                        0                4
1,2                   3,4                5
3,4                 0,1,2               6
1,2                  1,2                 7
3,4                  3,4                 8

pero de la forma en que lo tengo si selecciono la combinacion 1,0  puntosPares me devuelve 8 
estaba pensando modificarlo y evaluar en if separados por ejemplo asi:

Código: [Seleccionar]

if (erv == 1 ||  erv == 2 ) {
if (env == 1 || env == 2 ) {
puntosPares=8;
}
}
que opinan? cual podría ser la mejor manera de evaluar estas condiciones?

[salvadoresc]

me respondo a mi mismo 

bueno no exactamente, como mencione antes combinar los IF me funciono, siento que quizas podria haber sido mas eficiente pero al menos me funciono 

Código: [Seleccionar]

if (erv == 0 && env == 0) {
puntosPares=2;
}
if (erv == 0  ) {
if(env == 1 || env == 2 || env == 3 || env == 4){
puntosPares=3;
}
}
if (erv == 1 && env == 0 ) {
puntosPares=4;
}
if (erv == 2 && env == 0 ) {
puntosPares=5;
}
if (erv == 1 ||  erv == 2  ) {
if(env == 3 || env == 4){
puntosPares=6;
}
}
if (erv == 3 ||  erv == 4 ) {
if(env == 0 || env == 1 || env == 2){
puntosPares=7;
}
}

if (erv == 1 ||  erv == 2  ) {
if( env == 1|| env == 2){
puntosPares=8;
}
}
if (erv == 3 ||  erv == 4  ) {
if(env == 3 || env == 4){
puntosPares=9;
}
}
gracias a los que leyeron  igual si consideran alternativas, ponganlas. 

[Ideas]

Código: [Seleccionar]

combinacion(4,3);

function combinacion(ev:Number, en:Number)
{
switch (ev)
{
case 0:
if (en == 0)
{
trace("Puntos Pares 1");
}
if (en != 0)
{
trace("Puntos Pares 2");
}
break;

case 1:
if(en== 0)
{
trace("Puntos Pares 3");
}
if(en==1 || en==2)
{
trace("Puntos 7");
}
if(en>2)
{
trace("Puntos 5");
}
break;

case 2:
if(en==0)
{
trace("Puntos 4");
}
if(en==1 || en==2)
{
trace("Puntos 7");
}
if(en>2)
{
trace("Puntos 5");
}
break;

case 3:
case 4:
if(en <=2)
{
trace("Puntos 6");
}
if(en >=3)
{
trace("Puntos 8");
}
break;

}
}

Esta anterior en un bucle de 100000 repeticiones me genera 387ms vs 420ms de los if.

ó

Código: [Seleccionar]

function combinacion(ev:Number, en:Number)
{
switch (ev)
{
case 0:
switch(en)
{
case 0:
trace("Puntos 1");
break;
case 1: case 2: case 3: case 4:
trace("Puntos 2");
break;
}
break;

case 1:
switch(en)
{
case 0:
trace("Puntos 3");
break;
case 1: case 2:
trace("Puntos 7");
break;
case 3: case 4:
trace("Puntos 5");
break;
}
break;

case 2:
switch(en)
{
case 0:
trace("Puntos 4");
break;
case 1: case 2:
trace("Puntos 7");
break;
case 3: case 4:
trace("Puntos 5");
break;
}
break;

case 3: case 4:
switch(en)
{
case 0: case 1: case 2:
trace("Puntos 6");
break;
case 3: case 4:
trace("Puntos 8");
break;
}
break;
}
}

[mxgxw]

Código: [Seleccionar]

combinacion(4,3);

function combinacion(ev:Number, en:Number)
{
switch (ev)
{
case 0:
if (en == 0)
{
trace("Puntos Pares 1");
}
if (en != 0)
{
trace("Puntos Pares 2");
}
break;

case 1:
if(en== 0)
{
trace("Puntos Pares 3");
}
if(en==1 || en==2)
{
trace("Puntos 7");
}
if(en>2)
{
trace("Puntos 5");
}
break;

case 2:
if(en==0)
{
trace("Puntos 4");
}
if(en==1 || en==2)
{
trace("Puntos 7");
}
if(en>2)
{
trace("Puntos 5");
}
break;

case 3:
case 4:
if(en <=2)
{
trace("Puntos 6");
}
if(en >=3)
{
trace("Puntos 8");
}
break;

}
}

Esta anterior en un bucle de 100000 repeticiones me genera 387ms vs 420ms de los if.

ó

Código: [Seleccionar]

function combinacion(ev:Number, en:Number)
{
switch (ev)
{
case 0:
switch(en)
{
case 0:
trace("Puntos 1");
break;
case 1: case 2: case 3: case 4:
trace("Puntos 2");
break;
}
break;

case 1:
switch(en)
{
case 0:
trace("Puntos 3");
break;
case 1: case 2:
trace("Puntos 7");
break;
case 3: case 4:
trace("Puntos 5");
break;
}
break;

case 2:
switch(en)
{
case 0:
trace("Puntos 4");
break;
case 1: case 2:
trace("Puntos 7");
break;
case 3: case 4:
trace("Puntos 5");
break;
}
break;

case 3: case 4:
switch(en)
{
case 0: case 1: case 2:
trace("Puntos 6");
break;
case 3: case 4:
trace("Puntos 8");
break;
}
break;
}
}

Yo lo haría así:

Código: [Seleccionar]

var resultado:Array = Array();
resultado[0] = [1,2,2,2,2];
resultado[1] = [3,7,7,5,5];
resultado[2] = [4,7,7,5,5];
resultado[3] = [6,6,6,8,8];
resultado[4] = [6,6,6,8,8];

Para obtener las respuestas:

Código: [Seleccionar]

nuevoValor = resultado[var1][var2];

Y te garantizo que aunque el memory footprint del mío es más grande, la ejecución es más rápida jijijijiji

[hkadejo]

Yo lo haría así:

Código: [Seleccionar]

var resultado:Array = Array();
resultado[0] = [1,2,2,2,2];
resultado[1] = [3,7,7,5,5];
resultado[2] = [4,7,7,5,5];
resultado[3] = [6,6,6,8,8];
resultado[4] = [6,6,6,8,8];

Para obtener las respuestas:

Código: [Seleccionar]

nuevoValor = resultado[var1][var2];

Y te garantizo que aunque el memory footprint del mío es más grande, la ejecución es más rápida jijijijiji

Menos codigo = mas legible y mas mantenible...quedate con esta opcion salvadoresc 

[salvadoresc]

excelente  con el detalle que te decia que ahi falta el cuarto valor de la variable uno pero se arregla facil por que es exactamente los mismos resultados de la 3 asi que o se agrega o si hay un cuatro se le resta uno da igual. pero si creo que vamos a modificar esto, igual tengo que hacerle unos ligeros cambios cosmeticos al FLA. de momento estoy esperando que me instalen la CS5 por que mi lenovo cayo en combate =/

[mxgxw]

excelente  con el detalle que te decia que ahi falta el cuarto valor de la variable uno pero se arregla facil por que es exactamente los mismos resultados de la 3 asi que o se agrega o si hay un cuatro se le resta uno da igual. pero si creo que vamos a modificar esto, igual tengo que hacerle unos ligeros cambios cosmeticos al FLA. de momento estoy esperando que me instalen la CS5 por que mi lenovo cayo en combate =/

Sorry no me fije ya lo arreglo fksajdfhkjfsa

[salvadoresc]

ya lo implemente y quedo nice, solo encontre un ligero error:

en as hay que usar el constructor new para construir el arreglo

Código: [Seleccionar]

var resultado:Array = new Array();
encontre otros errorcitos que tenia por otro lado y ya estamos hoy sigo agregando mas tonteras ><