Ver Mensaje Individual
  #1 (permalink)  
Antiguo 23/09/2008, 11:01
manguiti
 
Fecha de Ingreso: junio-2007
Ubicación: Cádiz
Mensajes: 369
Antigüedad: 16 años, 10 meses
Puntos: 7
que diriais de esta manera de "programar"?

Me han pasado un codigo que debo revisar y mejorar en algunos aspectos, pero además de que llevo poco tiempo programando en php, creo que todo esto es un caos a pesar, de que hay ciertas cosas que llevan un orden logico...os paso lo que sería parte del codigo de un archivo php de la aplicacion....

Así comienza...

Código PHP:
function alta_ordenes ($Id,$Referencia,$Cliente,$Fecha_Entrada,$Fecha_Prueba,$Fecha_Salida,$Fecha_Conformidad,$Tintas,$Cantidad,$Atendido_Por,$Entrega,$Hora,$Cont_Adjunto,$Soporte,$Tipo_Trabajo,$Prueba,$Descripcion,$Tam_Ancho,$Tam_Alto,$Tam_UM,$Tam_Otros,$Pantone1,$Pantone2,$Tipo_Papel,$Tipo_Interior,$Tipo_Cubierta,$Id_Acabado,$Fecha_Disc,$Num_Archivo,$Fecha_Entrega,$Descripcion_Disconformidad,$Ges_Comercial,$Tec_Impresion,$Tec_Diseno,$Tec_Acabado,$Facturacion,$Fecha,$Papel,$Num_Pliegos,$Otros_Materiales,$Id_Formato,$Id_Maquinas,$Id_Maquina1,$Id_Maquina2,$Incidencia,$Estado,$Trabajo_Envio,$Direccion,$Telefono1,$Telefono2,$Contacto,$Extra1,$Extra2,$Extra3,$Extra4,$Extra5,$Esquema,$Extra6,$diseprueba,$Extra7,$Extra8,$Extra9,$Extra10,$Extra11,$Extra12)

{
    global 
$nombredeladjunto;
    global 
$nombredelesquema;
    global 
$nombredelfilmar;
    global 
$finprueba;
    global 
$findiseno;
    global 
$finprod;

    
$Fecha_Entrada=cambiaf_a_mysql($Fecha_Entrada);
    
$Fecha_Prueba=cambiaf_a_mysql($Fecha_Prueba);
    
$Fecha_Salida=cambiaf_a_mysql($Fecha_Salida);
    
$Fecha_Conformidad=cambiaf_a_mysql($Fecha_Conformidad);
    
$Fecha_Disc=cambiaf_a_mysql($Fecha_Disc);
    
$Fecha_Entrega=cambiaf_a_mysql($Fecha_Entrega);
    
$Fecha=cambiaf_a_mysql($Fecha);

    
////////// NUMERACION INDEPENDIENTE DE DISCONFORMIDADES ////////////

    
if (!$Num_Archivo){
        if (((
$Fecha_Disc)&&($Fecha_Disc<>'0000-00-00'))||(($Fecha_Entrega)&&($Fecha_Entrega<>'0000-00-00'))||(trim($Descripcion_Disconformidad))){
            
$si_maxdis=mysql_query("SELECT max(Num_Archivo) FROM ordenes");
            list(
$maxdis)=mysql_fetch_row($si_maxdis);
            
$Num_Archivo=$maxdis+1;
        }
    }

    
////////////////////////////////////////////////////////////////////

    
$hoy=date("Y-m-d");
//     if ($Estado=='1'){//pasar a diseño si ha pasado fecha conformidad
//         if (($Fecha_Conformidad!='')&&($hoy>=$Fecha_Conformidad))
//             $Estado='2';
//     }
    
if (($Estado=='2')&&($finprueba=='fin')){
            
$Estado='7';
    }
    if (((
$Estado=='2')||($Estado=='7'))&&($findiseno=='fin')){//de diseño pasa a digital o a offset
            
if ($Extra5=='3')
                
$Estado='3';
            elseif (
$Extra5=='4')
                
$Estado='4';
            else
                
$Estado='3';//Si no se sabe si va a digital u offset lo mando a digital
    
}

    if (((
$Estado=='3')||($Estado=='4'))&&($finprod=='fin')) $Estado='5';//si esta en digital o en offset y esta marcado finprod pasa a finalizado

    
if ($nombredeladjunto){
        
$extension explode(".",$nombredeladjunto);
        
$ext=$extension[count($extension)-1];
        
$nombrefinal=microtime().".$ext";
        
move_uploaded_file($Cont_Adjunto,"./adjuntos_NAS/$nombrefinal");
        
$nombreoriginal=$nombredeladjunto;
    }else{
        
$nombrefinal='';
        
$nombreoriginal=$nombredeladjunto;
    }

    if (
$nombredelesquema){
        
$extension explode(".",$nombredelesquema);
        
$ext=$extension[count($extension)-1];
        
$nombrefinalqma=microtime().".$ext";
        
move_uploaded_file($Esquema,"./adjuntos_NAS/$nombrefinalqma");
    }else{
        
$nombrefinalqma='';
    }

    if (
$nombredelfilmar){
        
$extension explode(".",$nombredelfilmar);
        
$ext=$extension[count($extension)-1];
        
$nombrefinalfilmar=microtime().".$ext";
        
move_uploaded_file($Extra9,"./adjuntos_NAS/PARAFILMAR/$nombrefinalfilmar");
//        $nombreoriginal=$nombredeladjunto;
    
}else{
        
$nombrefinal='';
//        $nombreoriginal=$nombredeladjunto;
    
}

    if (
$Estado<='2'$Extra7=$Estado;
    if (
$Estado>='3'$Extra7=$Estado+1;
    if (
$Estado=='7'$Extra7='3';
    
$query "INSERT INTO ordenes VALUES ('$Id','$Referencia','$Cliente','$Fecha_Entrada','$Fecha_Prueba','$Fecha_Salida','$Fecha_Conformidad','$Tintas','$Cantidad','$Atendido_Por','$Entrega','$Hora','$nombrefinal','$Soporte','$Tipo_Trabajo','$Prueba','$Descripcion','$Tam_Ancho','$Tam_Alto','$Tam_UM','$Tam_Otros','$Pantone1','$Pantone2','$Tipo_Papel','$Tipo_Interior','$Tipo_Cubierta','$Id_Acabado','$Fecha_Disc','$Num_Archivo','$Fecha_Entrega','$Descripcion_Disconformidad','$Ges_Comercial','$Tec_Impresion','$Tec_Diseno','$Tec_Acabado','$Facturacion','$Fecha','$Papel','$Num_Pliegos','$Otros_Materiales','$Id_Formato','$Id_Maquinas','$Id_Maquina1','$Id_Maquina2','$Incidencia','$Estado','$Trabajo_Envio','$Direccion','$Telefono1','$Telefono2','$Contacto','$Extra1','$Extra2','$Extra3','$Extra4','$Extra5','$nombrefinalqma','$nombreoriginal','$diseprueba','$Extra7','$Extra8','$nombrefinalfilmar','$Extra10','$Extra11','$Extra12')";

    
mysql_query($query);

}; 
esto es una funcion....el archivo sigue así....