Mình đã viết code kết nối tới csdl, còn đoạn code cho class hiển thị dữ liệu nhưng còn dài và có nhiều đoạn trùng nhau.

Mình cần nhiều dữ liệu trong trường channel_name, xếp theo thứ tự order từ bảng channels, rồi echo ra các tag html để chèn vào các trang html khác nhau. Mình chưa viết cho ngắn gọn lại được, vẫn còn đoạn code trùng nhau ở các hàm, vì các hàm chỉ khác nhau ở chỗ echo thôi.

<?php
class Channel{
    public function getData(){
        try{
          return mysql_query("SELECT channel_name from channels ORDER BY channels.order ASC LIMIT 10");
          }

        catch (exception $error){
            echo "Error:" .$error;
        }
    }

    public function showHomeChannel(){
        $result = $this->getData();
        if(mysql_num_rows($result)>0){
            while($row = mysql_fetch_array($result)){
                echo '<div><a href="#">'.$row["channel_name"].'</a></div>';
            }
        }
    }

    public function showSelectedChannel(){
      $result = $this->getData();
        if(mysql_num_rows($result)>0){
            while($row = mysql_fetch_array($result)){
                echo '<option value='.$channel.'>'.$row["channel_name"].'</option>';
            }
        }
    }
}
?>

Mong đc giúp đỡ.

Code của bạn cũng đã ngắn gọn lắm rồi còn gì :-) Có muốn ngắn hơn nữa thì chỉ có bỏ cái câu lệnh if(mysql_num_rows($result)>0) đi vì thực sự không cần thiết lắm.

Các hàm vẫn trùng đoạn code này:
$result = $this->getData();
        if(mysql_num_rows($result)>0){
            while($row = mysql_fetch_array($result)){...

vì chỉ khác khi echo thôi.

Các hàm vẫn trùng đoạn code này:
$result = $this->getData();
        if(mysql_num_rows($result)>0){
            while($row = mysql_fetch_array($result)){...

vì chỉ khác khi echo thôi.

Trùng có 2-3 dòng đâu có đáng gì :-) Còn nếu bạn muốn gọn hơn thì có thể làm thế này:

<?php
...
public function showSelectedChannel($mysqlResult) {
    while ( $row = mysql_fetch_array($mysqlResult) ) {
        echo '<option value='.$channel.'>'.$row["channel_name"].'</option>';
    }
}
Rồi ở bên ngoài bạn có thể query rồi truyền cái $mysqlResult vào cho 2 hàm con.