php - Dynamiske paneler - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php mysql

Bruger #16025 @ 19.10.12 17:06

Kære udviklere,

Nu hvor jeg er igang med at bygge mit eget personlige "cms" til eget brug, er jeg stødt på et problem.

Jeg skal have lavet nogle såkaldte "paneler" på siden, som kommer fra en database.

Jeg har bygget det op i min html, således:

PHP kode

 <body>
     <d:header>
         <?php $functions->get_panels('top'); 
         ?>
     </d:header>
 
     <d:content>
         <?php $functions->get_panels('content');
         ?>
     </d:content>
 
     <d:footer>
         <?php $functions->get_panels('footer'); 
         ?>
     </d:footer>
 </body>

I min php ser det således ud

PHP kode

 public function get_panels($type)
 {
     $i = 0;
 
     parent::ExecuteQuery("SELECT panels_title, panels_text FROM dab93_panels WHERE panels_type = '$type'");
     $row = parent::FetchRow();
     $num = parent::NumRows();
 
     if($num > 0)
     {
         $str_from = array("[a", "url=\"", "]", "[/a");
         $str_to = array("<a", "href=\"", ">", "</a");
 
         $row["panels_text"] = str_replace($str_from, $str_to, $row["panels_text"]);
         
         while($i < $num)
         {
             echo( '<section class="panel '.$type.'">' . $row["panels_title"] . " " . $row["panels_text"] ."</section>\n" );
             $i++;
         }
     }
 }

Problemet ligger i at den dublikerer min data, hvis jeg har to "typer" der er ens.

I min tabel har jeg dette: Link

Jeg har med vilje lavet 2 "paneler" som ligger under typen "content". Problemet er at den øverste hvor der står "hello world" under titlen, ved "content", der bliver den udskrevet to gange med samme data istedet for at den tager det nye data og udskriver dette.

Nuværende resultat: Link

Er der nogle der kan finde fejlen? Jeg kan ikke. På forhånd tak.

5 svar postet i denne tråd vises herunder
1 indlæg har modtaget i alt 6 karma

Sorter efter stemmer Sorter efter dato

Bruger #10216 @ 19.10.12 18:33

4.283

Det vil aldrig virke, som du tror det virker! Du henter kun een række, og that's it.

I seneste stump deler du en tekststreng ud i et array.

Du kan måske løse det ved at skrive:

PHP kode

 public function get_panels($type)
  {
      $i = 0;
  
      parent::ExecuteQuery("SELECT panels_title, panels_text FROM dab93_panels WHERE panels_type = '$type'");
      
      $num = parent::NumRows();
  
      if($num > 0)
      {
          while($row = parent::FetchRow())
          {
              $str_from = array("[a", "url=\"", "]", "[/a");
              $str_to = array("<a", "href=\"", ">", "</a");
  
              $row["panels_text"] = str_replace($str_from, $str_to, $row["panels_text"]);
          
              echo( '<section class="panel '.$type.'">' . $row["panels_title"] . " " . $row["panels_text"] ."</section>\n" );
              $i++;
          }
      }
  }

Men uanset hvad du gør, skal du pille din database ud af den klasse. Din klasse (hvad end den hedder) skal ikke extend'e fra din database. Det bliver et helvede at holde styr på. Din (view) klasse skal have - når nu du ikke bruger MVC - din database klasse som input parameter i constructor på den klasse vi ser på. Derfra kan du så f.eks. benytte $this->db->query('....');

Indlæg senest redigeret d. 19.10.2012 18:33 af Bruger #10216

Bruger #14645 @ 19.10.12 17:59

806

Nu sidder jeg ikke med PHP men ser da ud til du kun henter en enkelt række ud, men udskriver den antal-rækker gange:

PHP kode

 $row = parent::FetchRow();
 $num = parent::NumRows();
 //....
 while($i < $num)
 {
    echo( '<section class="panel '.$type.'">' . $row["panels_title"] . " " . $row["panels_text"] ."</section>\n" );
    $i++;
 }

Du skal nok have alt det udenfor-while'n, men inden for if'en ind under whilen også samt, hente en ny række i hver iteration i whilen.

Bruger #16025 @ 19.10.12 18:14

442

Det er lige det jeg tænkte. Normalt bruger jeg foreach i denne situation, men kunne ikke få det til at virke.

Det mærkelige er dog at det virker i en anden situation, hvor jeg skal udskrive alle styles der tilhører det valgte tema:

PHP kode

 <?php $functions->get_styles(); ?>

PHP kode

 public function get_styles()
 {
     $i = 0;
 
     parent::ExecuteQuery("SELECT themes_styles, themes_name FROM dab93_themes INNER JOIN dab93_webinfo ON themes_id = webinfo_themes_id");
     $row = parent::FetchRow();
 
     if(parent::NumRows() > 0)
     {
         $row["themes_styles"] = explode(",", $row["themes_styles"]);
         $num = count($row["themes_styles"]);
 
         while($i < $num)
         {
             echo( '<link rel="stylesheet" type="text/css" href="http://localhost/dab93/application/themes/'.$row["themes_name"].'/css/' . $row["themes_styles"][$i] . '" />
 ' );
             $i++;
         }
     }
 }

Det er netop dét der undrer mig.

Bruger #17081 @ 19.10.12 18:35

1.258

Det vil aldrig virke, som du tror det virker! Du henter kun een række, og that's it.

I seneste stump deler du en tekststreng ud i et array.

Du kan måske løse det ved at skrive:
PHP kode
 public function get_panels($type)
  {
      $i = 0;
  
      parent::ExecuteQuery("SELECT panels_title, panels_text FROM dab93_panels WHERE panels_type = '$type'");
      
      $num = parent::NumRows();
  
      if($num > 0)
      {
          while($row = parent::FetchRow())
          {
              $str_from = array("[a", "url=\"", "]", "[/a");
              $str_to = array("<a", "href=\"", ">", "</a");
  
              $row["panels_text"] = str_replace($str_from, $str_to, $row["panels_text"]);
          
              echo( '<section class="panel '.$type.'">' . $row["panels_title"] . " " . $row["panels_text"] ."</section>\n" );
              $i++;
          }
      }
  }
Men uanset hvad du gør, skal du pille din database ud af den klasse. Din klasse (hvad end den hedder) skal ikke extend'e fra din database. Det bliver et helvede at holde styr på. Din (view) klasse skal have - når nu du ikke bruger MVC - din database klasse som input parameter i constructor på den klasse vi ser på. Derfra kan du så f.eks. benytte $this->db->query('....');

$this->speak->the_truth($this->man);

Indlæg senest redigeret d. 19.10.2012 18:37 af Bruger #17081

Bruger #16025 @ 19.10.12 18:45

442

@Michael
Det virker, tusind tak! Og ja, jeg skal nok kigge på det du skriver

Dynamiske paneler

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler