Forma optima para crear funciones en php

[genaro_belen]

Saludos,
Sucede, que estoy desarrollando un website y por mas que he investigado y creado indices y he tratado de modificar mi codigo, sigo teniendo problemas de rendimiento, ya que estoy cargando mucho el servidor y los administradores me estan llamando a la atencion, y no quiero que me tumben la pagina.

En mis investigaciones he encontrado mucha informacion que me dice que puede ser problemas de la conexion, yo utilizo mucho este tipo de funciones.

Es optima esta forma?

Código PHP:

//funcion que muestra la modelo
        function mostrar_modelo($id){
                $db = conectate();//me conecto al servidor y la bd
 
                    //saco el modelo
                    $sql_modelo_vehiculo = $db->prepare("SELECT * FROM tmodelos WHERE id_modelo = ? ");
                    $sql_modelo_vehiculo->execute(array($id));
                    
                    $rs_modelo_vehiculo = $sql_modelo_vehiculo->fetchAll();     
                    
                    foreach($rs_modelo_vehiculo as $modelo){
                            return $modelo['descripcion_modelo'];
                        }
                
            }
 

Creen que ese codigo se puede mejorar para que funcione mejor?
Creen que ahi es que puede estar mi problema?

Gracias

[Nemutagk]

Las funciones no son el problema, el problema de consumo de recursos es la pésima optimización del código, sea o no función, por ejemplo, para que llamas todas las celdas de la tabla tmodelos si solo vas a retornar "descripcion_modelo", es tonto hacer eso, y mas aun hacer un loop para recorrer las demás celdas, mejor solo llama a la celda que necesitas y con fetchRow obtienes el resultado que quieres, aun así, la función que muestras no debería darte problemas, aun como la tienes ahorita, el consumo seria infimo, a menos que tengas miles de usuarios llamando a la función al mismo tiempo, lo mas seguro es que tu problema sea en general la forma en como estas programando y no solo una función...

[genaro_belen]

Eso que dices esta muy bien, me gusta..
Lo unico que solo puse esa funcion, asi como esa hay mas.

Ahh otra cosa, esa funcion de modelo, solo tiene 2 campos.

verificare eso que me dices...

Si tienes alguna aclaracion, no dudes en plantearmelas, please...

Thanks

[Nemutagk]

Con los ejemplos que das no hay forma como ayudarte, no dices el numero de usuarios que acceden al sitio al mismo tiempo ni muestras código completo, así es imposible, es estar adivinando que es lo que esta mal...

[genaro_belen]

Bueno,
Cantidad de usuarios no se decirte, pero esperemos que sean muchos usuarios, mas de 100 simultaneos.

Otra cosa, el codigo no es que no quiera pasarlo, es que es casi imposible, son muuchas paginas, aqui te voy a pasar varias funciones, para que tengas un estimado de como lo estoy haciendo.

Código PHP:

//funcion que consulta todos los datos de un usuario
    function consultar_datos_del_usuario($id){
            global $db;// = conectate();//me conecto al servidor y la bd
                
                $sql_us = $db->prepare("SELECT * FROM tusuarios WHERE usuario_id = ?");
                $sql_us->execute(array($id));
                $rs_us = $sql_us->fetchAll(PDO::FETCH_ASSOC);
                
                foreach($rs_us as $us){
                    return $us;
                }
 
            
        }
 

Código PHP:

//funcion que selecciona los autos vendidos por un determinado usuario
    function consultar_categorias_de_usuario(){
            global $db;// = conectate();//me conecto al servidor y la bd
                
                $sql_cate = $db->prepare("SELECT tipo_usuario_id, tipo_usuario_descripcion FROM tcategoria_usuario");
                $sql_cate->execute();
                $rs_cate = $sql_cate->fetchAll(PDO::FETCH_ASSOC);
                
                return $rs_cate;
 
            
        }
 

Realmente me gustaria saber, cuales faltas de optimizacion le ves a esas funciones
y otra cosa, si tienes algun tips para mejorar el rendimiento, es bienvenido.

Gracias por todo

[Momonito]

Tal vez no sea el más ducho del foro, pero si yo revisase tu trabajo, tal vez te diría que intentases mantener el contacto con la BBDD durante menos tiempo, intento explicarme:

Por lo que me parece entender, estas constantemente conectado con la BBDD, haciendo consultas constamente. Tal vez, sería más optimo hacer una "gran consulta" y trabajar con los datos en la maquina cliente y no mantener esa variable ($db global) corriendo constantemente para cada usuario.

Otra opción, en caso de que tu proyecto necesite trabajar así, es plantearse cambiar se hosting y buscar uno con más potencia que no te ponga esos problemas, al fín y al cabo, la elección de server es tan importante como la programación.

Espero que te sirva de algo este parrafazo :P

[Nemutagk]

Iniciado por Momonito

Tal vez no sea el más ducho del foro, pero si yo revisase tu trabajo, tal vez te diría que intentases mantener el contacto con la BBDD durante menos tiempo, intento explicarme:

Por lo que me parece entender, estas constantemente conectado con la BBDD, haciendo consultas constamente. Tal vez, sería más optimo hacer una "gran consulta" y trabajar con los datos en la maquina cliente y no mantener esa variable ($db global) corriendo constantemente para cada usuario.

Otra opción, en caso de que tu proyecto necesite trabajar así, es plantearse cambiar se hosting y buscar uno con más potencia que no te ponga esos problemas, al fín y al cabo, la elección de server es tan importante como la programación.

Espero que te sirva de algo este parrafazo :P

El realizar varias consultas a la DB en un solo archivo no debería ser el problema, obviamente teniendo en cuenta que las consultas SQL esten bien optimizadas y no este intentando retornar/trabajar con millones de registros...

Iniciado por genaro_belen

....
Otra cosa, el codigo no es que no quiera pasarlo, es que es casi imposible, son muuchas paginas...

No te estoy pidiendo que subas todo el código de tu sitio, si no del script que mas hace trabajar al servidor, los del hosting deben de a verte dicho "tal script consume muchos recursos en servidor" o algo por el estilo, aparte, mas o menos uno entiende que scripts son los mas "pesados" para que el servidor trabaje con ellos, en todo caso, muestra una sola página de tu sitio y si en el estas usando funciones definidas en otro archivo también publícalo, te vuelvo a repetir, el mostrar 1 o 2 funciones no va a ayudar a los que visiten tu post a saber cual es el problema, porque no estamos viendo como programas en general, es estar adivinando, como ultimo consejo, por lo que veo usas funciones comunes, lo mejor en tu caso es usar clases, investiga al respecto, es mejor a tener funciones desperdigadas...

[genaro_belen]

Gracias a Todos, No le habia pasado el codigo porque estaba terminando otras cosas.

Aqui hay una pagina que tiene un gran codigo y creo que esta afectando en el proces.

http://pastebin.com/xvqstAwM

Verificalo a ver que tal lo ves..

Gracias

[shaggyz]

Si tu sitio tiene tráfico y tienes problemas de rendimiento, te recomiendo abandonar el código espagueti http://es.wikipedia.org/wiki/C%C3%B3digo_spaghetti

Te recomiendo buscar información sobre estos temas:

- Programación orientada a objetos en php
- Refactorización
- Patrón de diseño MVC
- Y algún framework que te facilite las tareas (zend, cake, code igniter, etc)

Saludos.