Tema: Ayuda con "proyecto cms"

Hola señores, nuevamente les pido ayuda ya que pretendo hacer un tipo de portal o CMS basico, creado desde cero menos la aplicacion CKEditor que usare como editor.

Tengo algunas dudas sobre el orden y el modelo a usar al momento de escribir el codigo.
Una de estas dudas seria, cuando uno hace una consulta a la base de datos, es correcto ir generando el codigo html dentro del bucle de la consulta ???.

pongo un trozo de la aberracion de codigo que estoy usando para que me digan sus falencias, creo que corresponde al modelo de programacion lineal aunque pretendo llevarlo a estructurado y luego orientado a objetos segun adquiera nuevos conocimientos...

este codigo hace la consulta

Código PHP:

<?php
define('THIS_PAGE','showthemes');
require('var_collected.php');
require('html_head.php');
$line = '<br /><br />__________<br /> Editado Por ';
//RECUPERAR DATOS DEL POST PARA VER A QUE TEMA PERTENECE Y SI ESTE EXISTE_____________________________________________
if(isset($p)){
    
    $sql = "SELECT id_theme,date_creation FROM post WHERE id_post='$p'";
    $cons_post = mysql_query($sql, $conexion) or die ('problema con query');
    $posting = mysql_fetch_array($cons_post);
    $t = $posting['id_theme'];
    
    //Calcular la pagina a la que corresponde el post
    $sql = "SELECT COUNT(id_post) FROM post ";
    $sql.= "WHERE id_theme='$t' AND date_creation <='".$posting['date_creation']."' ORDER BY date_creation ";
    $cons_post = mysql_query($sql, $conexion) or die ('problema con query');
    $num_post = mysql_result($cons_post, 0);            
    $page = (int)(($num_post)/$post_per_page)+1;// $page = ceil($num_post / $post_per_page)+1;
    //Script para posecionar el scroll en el div correspondiente al post
    echo '<script type="text/javascript">';    
     echo 'window.onload = function() { document.getElementById("box_post_'.$p.'").scrollIntoView(true); }';
    echo '</script>';        
    
}

$rows_page = ($page - 1) * $post_per_page;//valor inicial del numero que se vera en la esquina superior derecha #1
//______________________________________________________________________________________________________________________________

//RECUPERAR DATOS DEL TEMA____________________________________________________________
$sql = "SELECT * FROM themes WHERE id_theme='$t' ";
$cons_themes = mysql_query($sql, $conexion) or die ('problema con query');
if($theme = mysql_fetch_array($cons_themes)){
    
    $sql = "SELECT * FROM users,firm,personal WHERE ";
    $sql.= "users.id_user=".$theme['id_user']." AND firm.id_user=".$theme['id_user']." AND personal.id_user=".$theme['id_user'];
    $cons_users = mysql_query($sql, $conexion) or die ('problema con query');
    $users_theme = mysql_fetch_array($cons_users);    
    
    require('bar_new_post.php');
    
}else{    
    require('notice_invalid_link.php');
    require('html_footer.php');
    exit;    
}    


//Si existe una firma y esta esta activada entonces se agrega al contenido
    if($users_theme['state_firm'] == 'y'){
        $theme['content'].= chr(13).'________'.chr(13).chr(13).$users_theme['firm_user'];
    }

if($page == 1){
    
    $rows_page++;
    
    //Ultimos cambios reemplazar tags BBCODE x formato HTML 
    $content = $theme['content'];
    require('bbcode_html.php');    
    
    //Preparar variables
    //Rutina para mostrar los botones CITAR - EDITAR - BORRAR    
    $show['options'] = '<a  class="boton" href="editor.php?do=quote_theme&amp;t='.$t.'">Citar</a>';
    if(constant('LOGIN_STATE') == true){
        if($theme['id_user'] == constant('ID_USER') || constant('TYPE_USER') == 'admin'){
            $show['options'].= '<a  class="boton" href="editor.php?do=edit_theme&amp;t='.$t.'">Editar</a>';
            $show['options'].= '<a  class="boton" href="administration/themes.php?do=edit&amp;t='.$t.'">Administrar</a>';
        }
    }
    //Si se ha definido un pais
    if($users_theme['country']){
        $show['country'] = '<tr><td>Pais</td><td><img src="flags/'.$users_theme['country'].'.gif" alt="" /></td></tr>';
    }
    //Conexion online - offline
    if(isset($users_conect_state[$users_theme['id_user']])){
        $show['conect_status']     = '<img src="images/user_online.gif" alt="" />'." Online";
    }else{
        $show['conect_status']     = '<img src="images/user_offline.gif" alt="" />'." Offline";
    }
    //ultima edicion
    if($theme['edit_iduser']){
        $sql = "SELECT name_user FROM users WHERE id_user=".$theme['edit_iduser'];
        $cons_edit = mysql_query($sql, $conexion) or die ('problema con query');
        $name_edit = mysql_fetch_array($cons_edit);    
        $show['edit']        = $line.$name_edit['name_user'].' el '.$fecha->formatDate2($theme['date_edit']);    
    }
    
    $show['id_box']         = 'theme_'.$theme['id_theme'];
    $show['permalink']         = '<a style="color:#fff;" href="showthemes.php?t='.$theme['id_theme'].'">#'.$rows_page.'</a>';
    $show['date_creation']     = $fecha->formatDate2($theme['date_creation']);    
    $show['link_name_user'] = '<a href="member.php?u='.$users_theme['id_user'].'">'.$users_theme['name_user'].'</a>';
    $show['avatar']         = '<img src="'.$users_theme['avatar_user'].'" alt=""/>';//.'?img='.md5(time()).'"/>';
    $show['type_user']         = $users_theme['type_user'];
    $show['date_reg']         = $fecha->formatDate4($users_theme['date_reg']);    
    $show['total_themes']     = $users_theme['total_themes'];
    $show['total_post']     = $users_theme['total_post'];    
    $show['title']             = $theme['title'];
    $show['content']         = $contenido_formateado;
    
    require('html_showthemes.php');//Llamada al box que mustra los datos en pantalla...
    $show = array();
}
//###############################################################################################################

//MOSTRAR LOS POST DEL TEMA______________________________________________________________________________________
//rutina para mostrar registros correspondientes a la pagina
$show     = array();

$ini     = (($page - 1) * $post_per_page);
$cant    = $post_per_page;
if($page == 1){    $cant = $post_per_page - 1;}else{$ini = $ini - 1;}

$sql = "SELECT * FROM post WHERE id_theme='$t' ORDER BY date_creation LIMIT ".$ini.",".$cant;
$cons_post = mysql_query($sql, $conexion) or die ('problema con query');
while($post = mysql_fetch_array($cons_post)){
    
    $rows_page++;
    
    $sql = "SELECT * FROM users,firm,personal ";
    $sql.= "WHERE users.id_user=".$post['id_user']." AND firm.id_user=".$post['id_user']." AND personal.id_user=".$post['id_user'];
    $cons_users = mysql_query($sql, $conexion) or die ('problema con query');
    $users_post = mysql_fetch_array($cons_users);

    //Si existe una firma y esta esta activada entonces se agrega al contenido
    if($users_post['state_firm'] == 'y'){
        $post['content'].= chr(13).'________'.chr(13).chr(13).$users_post ['firm_user'];
    }
    
    //Ultimos cambios reemplazar tags BBCODE x formato HTML
    $content = $post['content'];
    require('bbcode_html.php');
    
    //Preparar variables
    //Rutina para mostrar los botones CITAR - EDITAR - BORRAR    
    $show['options'] = '<a  class="boton" href="editor.php?do=quote_post&amp;p='.$post['id_post'].'">Citar</a> ';    
    if(constant('LOGIN_STATE') == true){
        if($post['id_user'] == constant('ID_USER') || constant('TYPE_USER') == 'admin'){
            $show['options'].= '<a  class="boton" href="editor.php?do=edit_post&amp;p='.$post['id_post'].'">Editar</a>';    
        }
        if(constant('TYPE_USER') == 'admin'){
            $show['options'].= '<a  class="boton" href="editor.php?do=delete_post&amp;p='.$post['id_post'].'">Borrar</a>';
        }
    }
    //Si se ha definido un pais
    if($users_post['country']){
        $show['country'] = '<tr><td>Pais</td><td><img src="flags/'.$users_post['country'].'.gif" alt="" /></td></tr>';
    }
    //Conexion online - offline
    if(isset($users_conect_state[$users_post['id_user']])){
        $show['conect_status']     = '<img src="images/user_online.gif" alt="" />'." Online";
    }else{
        $show['conect_status']     = '<img src="images/user_offline.gif" alt="" />'." Offline";
    }
    //ultima edicion
    if($post['edit_iduser']){
        $sql = "SELECT name_user FROM users WHERE id_user=".$post['edit_iduser'];
        $cons_edit = mysql_query($sql, $conexion) or die ('problema con query');
        $name_edit = mysql_fetch_array($cons_edit);        
        $show['edit'] = $line.$name_edit['name_user'].' el '.$fecha->formatDate2($theme['date_edit']);    
    }
    $show['id_box']         = 'post_'.$post['id_post'];
    $show['permalink']         = '<a style="color:#fff;" href="showthemes.php?p='.$post['id_post'].'">#'.$rows_page.'</a>';
    $show['date_creation']     = $fecha->formatDate2($post['date_creation']);        
    $show['link_name_user'] = '<a href="member.php?u='.$users_post['id_user'].'">'.$users_post['name_user'].'</a>';
    $show['avatar']         = '<img src="'.$users_post['avatar_user'].'" alt="" />';//.'?img='.md5(time()).'"/>';
    $show['type_user']         = $users_post['type_user'];
    $show['date_reg']         = $fecha->formatDate4($users_post['date_reg']);
    $show['total_themes']     = $users_post['total_themes'];
    $show['total_post']     = $users_post['total_post'];    
    $show['title']             = $post['title'];
    $show['content']         = $contenido_formateado;
    
    require('html_showthemes.php');
    $show = array();
}

//---------------------------
require('bar_new_post.php');
require('html_footer.php');
?>

este lo imprime en pantalla.

Código PHP:

<div class="contenedor" id="box_<?php echo $show['id_box'];?>" style="background:#D8D8D8;"> 
<table width="100%" border="0" cellspacing="0" >
<tr class="bar_title">
<td width="200">
    <div style="padding:2px;"><?php echo $show['date_creation'];?></div>
</td>
<td align="right">
    <div style="padding:2px;"><?php echo $show['permalink'] ;?></div>
</td>
</tr>

<tr>
<td valign="top" style="padding:5px;">  
    <div class="link_name_user">
    <?php echo $show['link_name_user'];?>
    </div>
    <div style="margin-top:5px;">    
    <?php echo $show['avatar'];?>
    </div>
    <div style="margin-top:5px;">       
        <table width="100%" border="0">
        <tr>
            <td width="70">Tipo user </td>
            <td><?php echo $show['type_user'];?></td>
        </tr>
        <tr>
            <td>Registrado</td>
            <td><?php echo $show['date_reg'];?></td>
        </tr>
        <tr>
            <td>Temas</td>
            <td><?php echo $show['total_themes'];?></td>
        </tr>
        <tr>
            <td>Posts</td>
            <td><?php echo $show['total_post'];?></td>
        </tr>
        <?php echo $show['country'];?>
        <tr>
            <td>Estado</td>
            <td><?php echo $show['conect_status'];?></td>
        </tr>
        </table>
     </div>
</td>
<td valign="top" style="padding:5px;background:#FFF;">
    <div style="padding:2px;">
    <?php echo $show['title'];?>
    <hr />
    <?php echo $show['content'].$show['edit'];?>
    </div>
</td>
</tr>

<tr>
<td align="center" style="padding:5px;">
    <div style="padding:2px;">
    &nbsp;
    </div>
</td>
<td valign="top" style="padding:5px;background:#FFF;">
    <div style="padding:2px;">   
    <?php echo $show['options'];?>
    </div>
</td>
</tr>        
</table>
</div>

en funcionamiento (servidor de pruebas)
http://dominiodepruebas.net16.net/forum/showthemes/t-1/

espero sus sugerencias para mejorar este "codigo" asi sea necesario empezar todo nuevamente.

Gracias...

Iniciado por WindLaze

Yo creo que si vas a empezar un proyecto el cuál podría ser largo o pretendas extenderlo. Sería que, desde el principio, comenzaras a orientarlo a objetos, aunque sea tedioso vale la pena, para futuras actualizaciones. Ahora, si en el caso de no poder utilizar objetos (PHP5) tendrás que diferenciando creando funciones en forma modular, que vendría siendo prácticamente lo mismo en cuanto a orden.

Por otra parte, el código se ve bastante simple, pero a la vez complejo de entender. Por lo que, separar la lógica, los querys y el html sería una buena forma de ordenarte. Así mismo, si quieres hacer consultas a la base de datos y a la vez generar el HTML es factible, pero para mí sería más cómodo asignar los valores devueltos a una variable en especifico y generar separadamente el HTML del query. Supongo que además de enterder fácilmente lo que se hace, también facilita la parte del diseño de la página.

Tienes razon ,lo que pasa es que empece al reves y ahora recien me estoy metiendo en lo que es la oop entonces tengo que volver sobre mis pasos y reescribir todo el codigo, he tratado de ordenar el codigo por bloques html y php a los que llamo con require lo malo de esto es que no hay encapsulamiento y otras bondades que ofrece la oop...

Iniciado por unreal4u

el motor del foro lo creaste tu ?

...
Lo primero y más importante: tu código es MUY vulnerable a inyecciones SQL, te recomiendo que trabajes con mi class que soluciona eso:


Ya que estás en eso, ocupa mi class que fue nominada por algún sitio X al cual no le seguí la pista y se me perdió
Hace la creación de código HTML4.01 y XHTML1.0 válido un chiste y algo sumamente fácil de ocupar, hasta el flash es HTML válido
HTML Utilities - PHP Classes

Concuerdo totalmente con WindLaze en que ocupando classes te va a alivianar harto la pega y te ayudará a mejorar bastante el modelo MVC de tu aplicación.

Otro consejo muy útil: ten sólo un index.php y a lo más dos a 3 carpetas en el servidor: el index debería poder incluir todas las demás partes.
Investígate sobre las funciones ob_start, ob_get_contents y ob_end_clean para saber cómo NO realizar ninguna salida sino hasta asegurarte que todo está bien y que puedes imprimir.

También investiga sobre set_error_handler y register_shutdown_function, a mi gusto, esas dos están entre las funciones más powa de PHP. Combinando ambas, permite la recuperación hasta de un error fatal (ojo: no de parsing que ocurre ANTES que un error fatal pueda producirse).

Por el momento con eso tendrás harta entretención

Saludos !!

A que te refieres con motor todo el codigo lo escribí yo pero es muy básico por eso quiero mejorarlo.

Sobre las inyecciones Mysql he saneado las variables que confronto en las consultas con (int) para los enteros y con expresiones regulares para los string como mail, name, password, etc. los contenidos los guardo de esta manera htmlentities($content, ENT_QUOTES).Espero que sea suficiente...

modelo MVC este dato es lo que andaba buscando me interesa mucho documentarme sobre este respecto.

Iniciado por tokugawa

Utiliza algun Framework que sea flexible, es decir que no de mucho drama al incorporar librerias externas, en este caso yo uso CodeIgniter, porque de todo lo que he probado es lo mas simple que encontré,pero tambien es muy completo(los otros como cake tienen asistentes y asdf que facilitan tarea, pero no me gusta trabajar con algo que no puedo controlar o no se 100% como está funcionando)

Por el momento esto esta fuera de mi alcance ya que no he usado ningun framework, pero igual persigo lo mismo controlar 100% la aplicacion.

____

Agradesco su valiosa ayuda, tengo bastante en que entretenerme...Saludos!!
____
Por lo que estoy viendo voy a tener que reescribir gran parte del codigo... Plaf
http://jacobo.tarrio.org/es/tech/php_claro