Ajuda para melhorar código

zking · 03/08/2014

Hola amigos, bueno este es el dilema en mi script se repite mucho un row , hay algo para optimizarlo?
Les dejare la manera que uso y una opcional pero no se si este bien.

PHP:

while ($row = $result->fetch_array()) {

if(1 == $row['Estado']){
echo "Exto";
}elseif (2 == $row['Estado']){
echo "Exito 2";
}else{
echo "XXXX";
}

PHP:

while ($row = $result->fetch_array()) {

$var = $row['Estado'];

if(1 == $var){
echo "Exto";
}elseif (2 == $var){
echo "Exito 2";
}else{
echo "XXXX";
}

}

Cicklow · 03/08/2014

es lo mismo... no te afecta... solo lees la variable... solo que de la forma 2 estas creando una variable mas...

zking · 03/08/2014

cicklow disse:
es lo mismo... no te afecta... solo lees la variable... solo que de la forma 2 estas creando una variable mas...

Gracias, Grande [MENTION=9679]cicklow[/MENTION]

Cicklow · 03/08/2014

zasudark disse:
Gracias, Grande [MENTION=9679]cicklow[/MENTION]

aca tenes tips para codificar en PHP
50+ best practices to optimize PHP code performance | _MD_ ProductionS

:encouragement:

TonchitoZ · 03/08/2014

En vez de usar un IF podrías utilizar un switch

PHP:

while ($row = $result->fetch_array()) {
	switch ($row['Estado']) {
		case 1:
			echo "Exto";
			break;
		case 2:
			echo "Exito 2";
			break;
		default:
			echo "XXXX";
	}
}

Mejor estructura, más rápido y mejor de leer para un programador :encouragement:

Te dejo la documentación por si acaso PHP: switch - Manual

zking · 03/08/2014

TonchitoZ disse:
En vez de usar un IF podrías utilizar un switch

PHP:

while ($row = $result->fetch_array()) { switch ($row['Estado']) { case 1: echo "Exto"; break; case 2: echo "Exito 2"; break; default: echo "XXXX"; } }

Mejor estructura, más rápido y mejor de leer para un programador :encouragement:

Te dejo la documentación por si acaso PHP: switch - Manual

Gracias, lo leeré :encouragement:

Javieer · 04/08/2014

Supongo que los resultados vienen de SQL así que es mas rápido hacer esto

PHP:

while ($row = $result->fetch_array())
	{ 
	if("1" === $row['Estado'])
		{ 
		echo "Exto"; 
		}
	elseif ("2" === $row['Estado'])
		{	
		echo "Exito 2"; 
		}
	else
		{ 
		echo "XXXX"; 
		}  
	}

RafaG · 04/08/2014

He estado leyendo algo sobre esto y no encuentro nada específico sobre PHP.

Según este hilo de Stackoverflow (allí hablan de Java, no de PHP): Java speed access array index versus temp variable - Stack Overflow , la segunda opción, en la que se usa la variable temporal, es más rápida, aunque puede depender de la máquina virtual.

En este otro hilo c - More efficient to use temporary variable or direct from array? - Stack Overflow hablan de lo mismo pero sobre C++. Al parecer el segundo ejemplo sería el más rápido, pero escribas lo que escribas el compilador generará el código ensamblador óptimo.

Por último, en este otro hilo sobre C# ( .net - In C#, Is it slower to reference an array variable? - Stack Overflow ) llegan a una conclusión similar: la segunda opción es más rápida, además de más legible, aunque el compilador podría optimizar el código.

En conclusión, yo elegiría la segunda opción: es más legible, elegante y al parecer es más rápida.

Un saludo.

zking · 04/08/2014

Javieer disse:
Supongo que los resultados vienen de SQL así que es mas rápido hacer esto

PHP:

while ($row = $result->fetch_array()) { if("1" === $row['Estado']) { echo "Exto"; } elseif ("2" === $row['Estado']) { echo "Exito 2"; } else { echo "XXXX"; } }

Gracias por responder 😛

Ajuda para melhorar código

zking

Cicklow

zking

Cicklow

TonchitoZ

zking

Javieer

RafaG

zking

Tópicos similares