Simplificar mi este codigo

[jandrogdz]

Buen dia foro

Tengo una duda hay alguna forma de simplificar este codigo o alguna opcion que ustedes conoscan.

Código PHP:

if(isset($_REQUEST['Cotizaciones']))
            {
                $cot = $_REQUEST['Cotizaciones']; 
                $sqlCot=$conexion->query("INSERT INTO Acceso_menu VALUES ('$cot', '$usuarioID')");
            } else { }
 
            if(isset($_REQUEST['Diseño']))
            {
                $arte = $_REQUEST['Diseño'];
                $sqlArte=$conexion->query("INSERT INTO Acceso_menu VALUES ('$arte', '$usuarioID')");
            } else { }
 
            if(isset($_REQUEST['Produccion']))
            {
                $prod = $_REQUEST['Produccion'];
                $sqlProd=$conexion->query("INSERT INTO Acceso_menu VALUES ('$prod', '$usuarioID')");
            } else { }
 
            if(isset($_REQUEST['Envio']))
            {
                $env = $_REQUEST['Envio'];
                $sqlEnv=$conexion->query("INSERT INTO Acceso_menu VALUES ('$env', '$usuarioID')");
            } else { }
 
            if(isset($_REQUEST['Facturacion']))
            {
                $fac = $_REQUEST['Facturacion'];
                $sqlFac=$conexion->query("INSERT INTO Acceso_menu VALUES ('$fac', '$usuarioID')");
            } else { }
 
            if(isset($_REQUEST['Usuarios']))
            {
                $usu = $_REQUEST['Usuarios'];
                $sqlUsu=$conexion->query("INSERT INTO Acceso_menu VALUES ('$usu', '$usuarioID')");
            } else { }
 
            if(isset($_REQUEST['Clientes']))
            {
                $cte = $_REQUEST['Clientes'];
                $sqlCte=$conexion->query("INSERT INTO Acceso_menu VALUES ('$cte', '$usuarioID')");
            } else { }
 
            if(isset($_REQUEST['Proveedores']))
            {
                $prov = $_REQUEST['Proveedores'];
                $sqlProv=$conexion->query("INSERT INTO Acceso_menu VALUES ('$prov', '$usuarioID')");
            }
            else { }
 
            if(isset($_REQUEST['Almacen']))
            {
                $alm = $_REQUEST['Almacen'];
                $sqlAlm=$conexion->query("INSERT INTO Acceso_menu VALUES ('$alm', '$usuarioID')");
            }
 

[mogurbon]

pos podrias quitarle los else vacios y esa es la idea? que inserte varias veces un mismo userid ?

[jandrogdz]

Gracias por contestar.

Perdon ese codigo era el anterior los else ia se lo habia kitado pero kisiera saber si hay una forma de simplificar ese codigo. Lo que recibo son checkbox no se si alla alguna manera de hacer un tipo ciclo que vaya viendo si tiene algo o no el valor.

[montealegreluis]

Hola jandrogdz, creo que con un array sería mucho más simple

Código PHP:

$validKeys = array(
    'Cotizaciones',
    'Diseño',
    'Produccion',
    'Envio',
    'Facturacion',
    'Usuarios',
    'Clientes',
    'Proveedores',
    'Almacen',
);
foreach ($validKeys as $key) {
    if(isset($_REQUEST[$key])) {
        $value = $_REQUEST[$key]; 
        $sqlCot=$conexion->query("INSERT INTO Acceso_menu VALUES ('$value', '$usuarioID')");
    }
}
 

Saludos!

[jandrogdz]

Gracias montealegreluis eso era ecxactamente lo que estaba buscando te lo agradesco mucho

[montealegreluis]

De nada, saludos! XD