Optimizar función

[Reenokazar]

Que tal, he hecho una función en PHP para insertar registros con un número de argumentos indefinidos. Me gustaría que algún maestro me diga si es posible optimizarla (seguro que si) y me explique como, si es posible. Aquí el código:

Código PHP:

  function insertarRegistro(){ 
     
    // Primer argumento es la tabla 
    $tabla = func_get_arg(0); 
    // Seguno argumento son los campos 
    $campos = func_get_arg(1); 
     
    // El resto son los datos a insertar 
    for ($i = 2; $i < func_num_args(); $i++){ 
         
        // Que al final no agregue la coma para que no de error de sintaxis MySQL 
        if($i+1 < func_num_args()){ $coma = ","; }else{ $coma = ""; } 
         
      $datos = $datos."'".func_get_arg($i)."'$coma"; 
    } 
     
    // Inserto registro 
    print $sql = "INSERT INTO $tabla ($campos) VALUES ($datos)";  
} 
 
insertarRegistro("datos_catalogo","nombre,codigo,catid,foto,descripcion,archivo,destacado,publicar",$nombre,$codigo,$catid,$foto,$descripcion,$archivo,$destacado,$publico);  
   

[pateketrueke]

este otro ejemplo seria mas "óptimo"

Código PHP:

  function insertar($tabla, $campos = array())
{
  $sql = 'INSERT INTO';

  $vals = $cols = array();
  foreach ($campos as $key => $value)
  {
    $value = is_numeric($value)? $value
               : "'" . addslashes(htmlspecialchars($value)) . "'";

    $vals []= $value;
    $cols []= $key;
  }

  $cols = join(', ', $cols);
  $vals = join(",\n", $vals);

  $sql .= "\n$tabla($cols)\n";
  $sql .= "VALUES($vals)";
  return $sql;
}

$sql = insertar("datos_catalogo", array(
  "nombre" => $nombre,
  "codigo" => $codigo,
  "catid" => $catid,
  "foto" => $foto,
  "nombre" => $nombre,
  "codigo" => $codigo,
  "catid" => $catid,
  "foto" => $foto,
  "descripcion" => $descripcion,
  "archivo" => $archivo,
  "destacado" => $destacado,
  "publico" => $publico
));

echo $sql;  
   

[jonatanc]

Hola , yo te presento otra solución que a mi parecer es mas viable y menos php dependiente jaja.

Código php:

<?php
 
/**
 * Script Author: Jonatan Lucio Casagrande <[email protected]>
 * Script Function:
 * Script Info: -
 */
 
function insertaSQL($tabla,$parametros){
 
if(isset($tabla) && is_array($parametros) && (count($parametros)>=0)){
 
$campos='';
 
$valores='';
 
foreach($parametros as $campo=>$valor){
 
$campos.="{$campo},";
 
$valores.="'{$valor}',";
 
}
 
$campos=substr($campos,0,strlen($campos)-1);
 
$valores=substr($valores,0,strlen($valores)-1);
 
$sql="INSERT INTO {$tabla} ({$campos}) VALUES({$valores})";
 
echo $sql;
 
unset($campos,$valores,$tabla,$parametros);
 
}
 
}
 
//-- Invocación
 
$parametros=array("id"=>"12","nombre"=>"elias");
insertaSQL('usuarios',$parametros);
 
?>
 

Saludos!

[pochos]

bueno jugando un poco se me ocurrio esto ..

Código:

function insertarRegistro($tabla, $campo_valor){
    
	foreach ($campo_valor as $key => $value){
		$campos .= ','.$key;
		$datos .= ",'".$value . "'";
	}
	$campos = substr($campos, 1, strlen($campos));//remueve primer caracter -> la coma
	$datos = substr($datos, 1, strlen($datos));//remueve primer caracter -> la coma
    // Inserto registro
    print $sql = "INSERT INTO $tabla ($campos) VALUES ($datos)"; 
}
$campo_valor['nombre'] = 'juan';
$campo_valor['apellido'] = 'perez'; // osea seria $campo_valor['nombredelcampo'] = 'valor';
insertarRegistro('nombre_tabla',$campo_valor);

pero bueno tambien seguramente se puede optimizar

[rusinho]

Una variacion de tu funcion:

Código PHP:

  
function insertarRegistro(){
    // Primer argumento es la tabla
    $tabla = func_get_arg(0);
    // Seguno argumento son los campos
    $campos = func_get_arg(1);
    
    // El resto son los datos a insertar
    for ($i = 2; $i < func_num_args(); $i++){        
      $datos = $datos."'".func_get_arg($i)."'";
    }
    $datos=substr($datos,0,strlen($datos)-1); //Quitar coma al final
     // Inserto registro
    print $sql = "INSERT INTO $tabla ($campos) VALUES ($datos)"; 
}
insertarRegistro("datos_catalogo","nombre,codigo,catid,foto,descripcion,archivo,destacado,publicar",$nombre,$codigo,$catid,$foto,$descripcion,$archivo,$destacado,$publico);  
   

Solo elimino el if interno en el for, para acelerar un poco la construccion de la variable $datos, y al salir del for le quito la ultima coma.

Espero sea de tu utilidad.

[Reenokazar]

Excelente, gracias a todos!