[PHP]klasa gallery Opcje

[devil-war]

Czy ten kod wygląda na obiektowy

Prosze o opinie

[PHP] pobierz, plaintext 
<?php
class gallery{
 
private $baza;//Laczenie z baza 
public  $biblioteka = '1';
 
 
function __construct(sqlcon $baza,$type,functions $fun,$biblioteka) 
{
$this->db = $baza;
$this->id_type = $type;
$this->func = $fun;
$this->libary = $biblioteka;
}
 
public function show_folders($image){
 
$galeria = $this->db->query('SELECT * FROM '.DB_PREFIX.'gallery_folders WHERE `folder_id_type`= '.intval($this->id_type).'');
WHILE($g = $this->db->fetch_assoc($galeria)){
 
$g_t = array('folder_id' => $g['id'],
'folder_nazwa' => $g['folder_nazwa'],
'folder_naz_t' => $this->func->utnij($g['folder_nazwa'],'12'));
 
echo' <div class="zdjecie">
<a href="gallery.php?act=open_folder&folder_id='.$g['id'].'"><img src="'.$image.'"
                border="0" alt="'.$g['folder_nazwa'].'"></a><br />
               <a href="gallery.php?act=open_folder&folder_id='.$g['id'].'">'.$this->func->utnij($g['folder_nazwa'],'12').'</a>
               </div>';
 
 
}
}
public function show_photos($id){
 
$this->id = $id;
 
$query = $this->db->query('SELECT * FROM '.DB_PREFIX.'gallery WHERE `folder_id` = '.intval($this->id).'');
 
$q = $this->db->query('SELECT * FROM '.DB_PREFIX.'gallery_folders WHERE `id` = '.intval($this->id).'');
while($qq=$this->db->fetch_assoc($q)){
echo '<B>ť</B> <b>'.$qq['folder_nazwa'].'<b>'; 
}
echo '<br><br>';
 
while($g=$this->db->fetch_assoc($query)){
 
if($this->libary == '1')
{
$bib = 'lightbox[roadtrip]';
}
elseif($this->libary == '2')
{
$bib = 'lytebox[vacation]';
}
elseif($this->libary == '3')
{
$bib = 'lightbox[atomium]';
}
 
elseif($this->libary == '4')
{
$bib = 'prettyPhoto[gallery]';
}
 
echo '<a href="'.$g['big_image'].'" border="0" rel="'.$bib.'">
<img border="0" src="'.$g['small_image'].'"  height="80" width="80"></a>';
 
 
}
}
public function load_libary_photos(){
 
if($this->libary == '1')
{ 
echo'<link rel="stylesheet" href="./scripts/thickbox/css/lightbox.css" type="text/css" media="screen" />
<script src="./scripts/thickbox/js/prototype.js" type="text/javascript"></script>
<script src="./scripts/thickbox/js/scriptaculous.js?load=effects,builder" type="text/javascript"></script>
<script src="./scripts/thickbox/js/lightbox.js" type="text/javascript"></script>';
}
elseif($this->libary == '2')
{ 
echo'<script type="text/javascript" language="javascript" src="./scripts/lytebox/lytebox.js"></script>
<link rel="stylesheet" href="./scripts/lytebox/lytebox.css" type="text/css" media="screen" />';
}
elseif($this->libary == '3')
{
echo'<script type="text/javascript" src="./scripts/slimbox/js/mootools.js"></script>
<script type="text/javascript" src="./scripts/slimbox/js/slimbox.js"></script>
<link rel="stylesheet" href="./scripts/slimbox/css/slimbox.css" type="text/css" media="screen" />
';
}
 
elseif($this->libary == '4')
{
echo'<script src="./scripts/bib4/js/jquery.js" type="text/javascript" charset="utf-8"></script>
<link rel="stylesheet" href="./scripts/bib4/css/prettyPhoto.css" type="text/css" 
media="screen" title="prettyPhoto main stylesheet" charset="utf-8" />
<script src="./scripts/bib4/js/prettyPhoto.js" type="text/javascript" charset="utf-8"></script>
';
 
}
echo'<style>
#wrapper {
    widt: 100%;
    clear: both;
}
.zdjecie {
    floa: left;
    text-alig: center;
    paddin: 10px 0;
}
.zdjecie img {
    displa: block;
    margi: 0 10px;
}
</style>';
}
 
 
//Koniec klasy
}
?>
[PHP] pobierz, plaintext 

Ten post edytował devil-war 19.09.2008, 13:28:24

[Shili]

Względnie, jak już ktoś się uprze, żeby cokolwiek wyświetlić, to niech to będzie jakaś jedna metoda show.

Class gallery, a nazwy właściwości, metod i zmiennych wszelakiego typu po polsku. Proponuję przenieść się jednak na ten angielski, w najgorszym przypadku konsekwentnie pisać po polsku. Wszystko.

Baza jak rozumiem jest to obiekt bazy danych?
Jeśli tak, to spokojnie możesz użyć konstrukcji: construct(nazwaKlasyBazyDanych $baza, ...) będziesz miał pewność, że obiekt zawsze i wszędzie bez względu na poziom zmęczenia będzie obiektem bazy danych.

[PHP] pobierz, plaintext 
<?php
if($this->biblioteka == '1')
{
$bib = 'lightbox[roadtrip]';
}
if($this->biblioteka == '2')
{
$bib = 'lytebox[vacation]';
}
if($this->biblioteka == '3')
{
$bib = 'lightbox[atomium]';
}
 
if($this->biblioteka == '4')
{
$bib = 'prettyPhoto[gallery]';
}
?>
[PHP] pobierz, plaintext 

Jeśli właściwość $this->biblioteka może być albo 1, albo 2, albo 3, albo 4, to nie stosuj czterech ifów po sobie. Lepiej jest zrobić elseif. W tym przypadku jeśli $this->biblioteka będzie 1 to i tak sprawdzi wszystkie inne warunki, które spokojnie mógłby sobie pominąć.

Poza tym w jakimś tam stopniu wygląda na obiektowy, ale jednocześnie wygląda na straszliwie zakręcony, trudny do modyfikacji i rozbudowy. A to wszystko przez ten spagettowaty kod html w środku.