Php, table, tr et td (w3c)

[vdom]

Salut,

Je ne m'en sort pas avec le w3c, il me calle sans cesse des erreurs, si vous pouvez m'aider :

Code: Tout sélectionner

function get_tip_top() {
      global $settings, $prefix, $root_path, $skin_path;

      $cres = mysql_query("SELECT * FROM {$prefix}category ORDER by cat_id") or report();
      
      if(mysql_num_rows($cres) == 0) $data = "Aucune catégorie";
      else {
      
         $data = "<table cellpadding='0' cellspacing='0' align='left'>";
         $counter = 0;
         $td = 0;
         
         //while($crow = mysql_fetch_assoc($cres)) {
         for($i=0;$i<15;$i++) {//while($crow = mysql_fetch_assoc($cres)) {
         
            $crow = mysql_fetch_assoc($cres);
                        
            $cat_url = "$root_path/".plaintext($crow['cat_art'])."-$crow[cat_id]-1.html";

            $gdata = "";
            
            if($td == 3) {
               $data .= "</tr><tr><td><div class=\"index-art\">";
               $td = 0;
            }
            else
               $data .= "<td><div class=\"index-art\">";
            
            
            $res = mysql_query("SELECT * FROM {$prefix}art gm, {$prefix}category ca WHERE ca.cat_id=gm.gam_cat_id AND gm.gam_cat_id=$crow[cat_id] GROUP BY gm.gam_id ORDER BY gm.gam_id DESC LIMIT 3") or report();
            if(mysql_num_rows($res) == 0) $clixint = "Clixint Technologies 2008"; //$gdata .= "";
         
            else {

               $num = 1;
               while($row = mysql_fetch_assoc($res)) {         
            
                  $gam_url = "$root_path/art/".plaintext($row['ar_title'])."-$row[ar_id].html";
         
                  if(strstr($row['ar_thumb_url'], "http"))         
                     $ar_thumb = $row['ar_thumb_url'];               
                  else
                     $ar_thumb = "$root_path/gam_previews/$row[ar_thumb_url]";
      
                  if(strlen($row["ar_desc"])>68) {
                     $row["ar_desc"] = str_cut($row["ar_desc"],68);
                     $row["ar_desc"] = $row["ar_desc"].'... ';
                  }
                  
                  if($num == 1)
                  $gdata .= "<div class=\"index-cat\"><h2><a href=\"$cat_url\" title=\"$row[cat_name]\">$row[cat_name]</a></h2></div>
                  <div class=\"myart\">
                         <div class=\"arttitle\"><h3><a href=\"$ar_url\" title=\"$row[ar_title]\">$row[ar_title]</a></h3></div>
                        <div class=\"cadre-art\">
                           <a href=\"$ar_url\" title=\"$row[ar_title]\">
                           <img title=\"$row[ar_title]\" src=\"$ar_thumb\" alt=\"\" />
                           </a>
                           <div class=\"artdesc\">$row[ar_desc]</div>
                        </div>
                  </div>";

                  else $gdata .= "<div class=\"myart2\">
                     <div class=\"artTitle2\"><a href=\"$ar_url\" title=\"$row[ar_title]\">$row[ar_title]</a></div>
                  </div>";
                  
                  $num++;
               }
               $data .= $gdata."</td>";               
            }   
         
            $td++;
         }
                  
         $data .= "</div></td></tr></table>";
      }
      
      return $data;
   }

[WEBCodeur]

Pour moi il est impensable de faire la chose que tu cites comme un code:

Code: Tout sélectionner

$data = "<table cellpadding='0' cellspacing='0' align='left'>";

Pour faire ca, y'a pas pas besoin de faire si complexe...
Tu fais ton template xHTML et tu appel tes variables dans tes cellules, et tu les repetes.
Excuses moi si je ne suis pas claire. Mais je suis surpris de voir ca ^^

[rudddy]

c'est vrai que en général, il faut essayer de coder le html en dehors du php, c'est beaucoup plus clair même si ca consomme un chouya plus de ressources serveurs !

[vdom]

Oui je sais mais bon comme je suis parti sur ça, c'est plus simple pour moi de continuer sur ce trend...
Même si là je ne vois pas comment m'y prendre ^^

Si quelqu'un veut essayer, je peux lui envoyer le fichier en mp

Merci,

[bproductiv]

si tu donnais l'url de la page en question ca aiderait pour voir les erreurs w3C

[vdom]

Ok,

Voici : http://www.addictojeux.com

Merci pour votre aide,

[spout]

Il faudrait faire ça sans table, via un <div> en float:left; avec une largeur définie qui remplace les <td>, sans oublier un clear:both après tous les <div> flottants. C'est beaucoup plus simple qu'avec des tableaux.

[bproductiv]

il n'y a pas de TR au debut du tableau.

[Protée]

Est-ce normal que mon Kaspersky détecte un cheval de Troie sur ton site (-http://www.addictojeux.com/skins/default/css/clix_functions.js) ?

[vdom]

w3c ok

Merci à tous,

Nb: Protée, je regarde ça c'est étrange...