Como compactar codigo

[jandrogdz]

Buen día foro

Necesito de su ayuda para poder compactar un codigo, la situación es que es para un proyecto pero me solicitaron no pasar de cierto numero de lineas de codigo,
me podrian apoyar a compactar este codigo.

Código PHP:

if($cantidad>0 && $cantidad<=100):
      $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad=100";
      $precio=$this->uniqueValue($query) : return $precio->precio; ? return null;      
endif;
 

Este codigo pero se repite como 12 veces con distintos rangos de cantidades y en la consulta el precio cambia.

De antemano muchas gracias

[Malenko]

Yo probaría a usar una función donde la cantidad y el precio sean parámetros.

[pablo_exeg]

buenas

Primero que nada ese codigo lo podes hacer en un script php

y requerirlo la veces que quieras (Primero que nada eso es muy util y solo con una linea incluis todo un script php).

Despues segun lo que dices con distintos rangos y cantidades tendriamos que verlo para saber

[informacionsys]

hola

mm para podrias usar una funcion por ejemplo:

Código PHP:

function Comprobar($cantidad,$maximo,$codigo)
{
 
    if($cantidad>0 && $cantidad<=$maximo):
       $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad=$maximo";
      $precio=$this->uniqueValue($query) ? $precio->precio : null;  
      return $precio; 
   endif;
}
 
// simplemente llamas la funcion pasandole los parametros ,
 
Comprobar(5,100,20);
 

[jandrogdz]

Graicas por responder.

Es correcto lo que dicen, yo lo tengo en una funcion pero no se como hacer para que me arroje los rangoas dinamicos.

La situacion es que de la tabla de precios se manda el codigo y se manda la cantidad ingresada, esta cantidad no siempre sera cerrada es decir no siempre sera de 100 o 200 puede ser de 23 o 187 por dar ejemplos.

Entonces no se como hacer para poder crear dependiendo el rango haga un where distinto.

por ejemplo yo lo tengo de esta forma:

Código PHP:

public function retornaPrecio($codigo,$cantidad){
        if($cantidad>0 && $cantidad<=100):
            $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad=100";
            if($precio=$this->uniqueValue($sql)): return $precio->precio;  else: return null;
            endif;      
        endif;        
    }
 

Espero haberme explicado bien

[jandrogdz]

Malenko No puedo pasar el precio como parametro porque es el dato que necesito obtener de la base de datos

[informacionsys]

no entiendo muy tu caso, como asi dependiendo el rango haga un where distinto , explicate un poco mas desglosado

[jandrogdz]

Si por ejemplo mira el codigo mas o menos va quedando asi:

Código PHP:

public function retornaPrecio($codigo,$cantidad){
        if($cantidad>0 && $cantidad<=100):
            $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad=100";
            if($precio=$this->uniqueValue($sql)): return $precio->precio;  else: return null;
            endif;      
        endif;        
        if($cantidad>100 && $cantidad<=200):
            $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad=200";
            if($precio=$this->uniqueValue($sql)): return $precio->precio;  else: return null;
            endif; 
        endif;
        if($cantidad>200 && $cantidad<=300):
            $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad=300";
            if($precio=$this->uniqueValue($sql)): return $precio->precio;  else: return null;
            endif; 
        endif;
        if($cantidad>300 && $cantidad<=500):
            $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad=500";
            if($precio=$this->uniqueValue($sql)): return $precio->precio;  else: return null;
            endif; 
        endif;
 

Lo que necesito es que los rangos sean dinamicos y que el valor de la cantidad del where tmbn sea dinamico peor no tengo idea de como hacerlo.
La cantiad ya esta guardada en la base de datos:
La tabla de precio tiene precioID,cantidad,precio,productoID

Espero haberme dado a entender

[GatorV]

Puedes usar un switch:

Código PHP:

switch(true) {
      case $cantidad <= 100:
              $qty = 100;
      case $cantidad <= 200:
              $qty = 200;
              break;
       case $cantidad <= 300;
              $qty = 300;
              break;
}
$sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad=$qty";
if($precio=$this->uniqueValue($sql)): return $precio->precio;  else: return null;
 

Saludos.

[jandrogdz]

Por pura duda GatorV no habria problema por ejemplo al poner 25 y que agarre otro precio ya que 25 es menor a 200.

Simple duda

[informacionsys]

hummm si viste el primer ejemplo que te di de la funcion ,y si le pasas el maximo como parametro , es decir el 200 o el 100 , la cuestion es que ese valor de donde lo obtienes siempre va hacer asi , 100 200 o 300?

[jandrogdz]

Ese valor viene de la base de datos y si siempre seran esos, pero lo que tengo que hacer es la cantidad que ingresa el cliente compararla con la cantidad de la base de datos para poder retornar el precio.

[GatorV]

A mi código al primero le falta un break; jeje, pero no, los switches se evaluan de arriba a abajo (como todo el código) por lo que si pones primero tus opciones más cortas ahí se quedan, en el caso de 25, al evaluar el primer case, va a dar true y ahí va a quedar.

Saludos.

[jandrogdz]

Ok muchas gracias a todos por su ayuda de verdad aprendi algo nuevo en los switches que puedes hacer validaciones dobles. y gracias a todos por sus consejos me hicieron volar mas ideas ya he conseguido compactarloe me ahorre hasta un 60% de lineas de codigo a la forma en la que yo lo tenia.

Quedo de la siguiente forma:

Código PHP:

private function comprobar($cantidad){
        switch($cantidad):
            case $cantidad>0 && $cantidad<=100:  return 100; break;
            case $cantidad>100 && $cantidad<=200:  return 200; break;
            case $cantidad>200 && $cantidad<=300:  return 300; break;
            case $cantidad>300 && $cantidad<=500:  return 500; break;
            case $cantidad>500 && $cantidad<=1000:  return 1000; break;
            case $cantidad>1000 && $cantidad<=2000:  return 2000; break;
            case $cantidad>2000 && $cantidad<=3000:  return 3000; break;
            case $cantidad>3000 && $cantidad<=4000:  return 4000; break;
            case $cantidad>4000 && $cantidad<=5000:  return 5000; break;
            case $cantidad>5000 && $cantidad<=6000:  return 6000; break;
            case $cantidad>6000 && $cantidad<=8000:  return 8000; break;
            case $cantidad>8000 && $cantidad<=10000:  return 10000; break;
        endswitch;
    }
    
    public function retornaPrecio($codigo,$cantidad){
        $nuevaCantidad=$this->comprobar($cantidad);
        $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad='$nuevaCantidad'";
        $precio=$this->uniqueValue($sql);
        $precio ? $precio->precio : null;   
        return $precio;  
    }
 

Otra pregunta como puedo hacer para que en el metodo retornPrecio hacer un if mas compacto.
Intente asi:

Pero me dice que la variable $precio no existe

Código PHP:

$nuevaCantidad=$this->comprobar($cantidad);
        $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad='$nuevaCantidad'";
        $precio=$this->uniqueValue($sql) ? $precio->precio : null;   
        return $precio;
 

y en el where trate de hacer esto pero no funciono

Código PHP:

$sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad='$this->comprobar($cantidad)'";
 

[GatorV]

No es necesario que pongas la condición doble, si lo haces como te indiqué primero solo es necesario que uses la condición mayor y con eso basta .

Respecto a tu if, puedes probar algo así:

Código PHP:

$precio = ($rowPrecio = $this->uniqueValue($sql) !== null) ? $rowPrecio->precio: null;
 

Saludos.

[jandrogdz]

GatorV. me sigue mandando el mismo error. COn tu ejemplo

[informacionsys]

hola

prueba asi:

Código PHP:

$rowPrecio = $this->uniqueValue($sql);
$precio = ($rowPrecio !== null && empty($rowPrecio)) ? $rowPrecio->precio : null;
 

[GatorV]

¿Que tipo de dato devuelve $this->uniqueValue?

[dashtrash]

Ese switch duele a los ojos...

Código PHP:

private function comprobar($cantidad){
        $temp=(1+(intval($cantidad/100)))*100;
         return $temp<8001?$temp:10000;
      }
 

[dashtrash]

Y si lo quieres hacer aún más compacto:

Código PHP:

public function retornaPrecio($codigo,$cantidad){      
        $sql="SELECT precio FROM precio WHERE productoID='$codigo' AND cantidad= IF(FLOOR($cantidad/100)<8,(1+FLOOR($cantidad/100))*100,10000)";
        $precio=$this->uniqueValue($sql);
        $precio ? $precio->precio : null;   
        return $precio;  
    }
 

[jandrogdz]

dashtrash

Me podrias explicar el proceso de esa funcion pork aun no logro captar como es el funcionamiento

[dashtrash]

Tu switch lo único que hace es calcular, dado un número, en qué rango de los múltiplos de 100 está.
La parte entera de la división del número entre 100, va a ser 0 para numeros entre 0 y 99.1 para numeros entre 100 y 199, etc.
Asi que, si a esa parte entera le sumas 1 y lo multiplicas por 100, tienes el límite superior que buscas.
Como, por algún motivo, si es superior a 8000, devuelves 10000 (si es superior, la función con los switch no devuelve nada, lo cual generaría un error), le añado esa condición.

[jandrogdz]

entonces esa funcion hace la tarea de todo el switch?

[dashtrash]

Si.Y en el siguiente post que he puesto, no necesitas ni la función con el switch.Se calcula dentro de la query.El parser mysql lo considerará una constante, por lo que no afectará al rendimiento.

[Triby]

Funcionará crear la cadena de la consulta e inmediatamente retornar el valor?... debería!

Código PHP:

return ($precio = $this->uniqueValue($sql)) ? $precio->precio : null;
 

Me extraña que tengas límite en la cantidad de líneas, tal vez el proyecto es escolar.