Forum PHP.pl

Witajcie drodzy koledzy, jako że raczkuje w temacie php mam problem ,z którym nie wiem jak sobie poradzić.

Aktualnie moje linki wyglądają następująco www.jakasstrona.pl/oferta/490.html ,a chciałbym aby wyglądały www.jakasstrona.pl/oferta/490/tytul-strony.html

Stworzyłem w bazie danych odpowiedni wiersz o nazwie SLUG w tabeli offer ,który przetwarza mi tytuł z polskimi znakami i spacjami na przyjazny link czyli np: kula z zębem zamienia na :kula-z-zebem.

Mój plik index wygląda tak:

[PHP] pobierz, plaintext 
<?php 
 
                if(!empty($_GET['oferta']) && is_numeric($_GET['oferta']) ) { include('include/podstrona_oferta.php'); } else {
                    if(!empty($_GET['page']) )  { include('include/podstrona_text.php'); } else {
 
                        //Strona główna
                          include('include/podstrona_main_new.php');
}}
?>            
[PHP] pobierz, plaintext 

Plik odpowiedzialny za wyświetlanie oferty czyli podstrona_oferta.php wygląda tak:

[PHP] pobierz, plaintext 
 
<?php include('include/bannerpage.php'); ?>
<?php include_once ('link.php');?>
<?php  $oferta=mysql_fetch_array(mysql_query("SELECT * FROM offer WHERE AKTYWNE = 1    and  ID ='".$_GET['oferta']."'"));?>
    	<div id="content">
					<section class="breadcrum-sec">
					<div class="container">
						<ol class="breadcrumb">
							<li>
								<a href="/index.php">Strona główna</a>
							</li>
							<li class="active">
								<?php echo ($oferta['TITLE']);?>
							</li>
						</ol>
					</div>
				</section>
				<section class="who_wrap">
					<div class="container">
						<div class="row">
							<div class="col-xs-12 col-sm-12">
							<ul id="lang">
 
                <?php  
                if(stristr($oferta['TEXT'], '<!--more-->'))
                  $oferta['TEXT'] = preg_replace('|^(.*)<!--more-->.*$|Usmi','$1',$oferta['TEXT']); 
//GALERIA 
					if($oferta['START3']!=0)					
					{
					$offer=mysql_fetch_array(mysql_query("SELECT * FROM offer WHERE AKTYWNE = 1   and ID='".$_GET['oferta']."'"));
					echo('');
						 echo(' <p>'.$oferta['TEXT'].'<br/><br></p>');
					 $fotosy=mysql_query("SELECT * FROM offer_foto WHERE OFERTA='".$_GET['oferta']."' ORDER BY POS ASC");
                while($foto=mysql_fetch_array($fotosy)){
                    echo('<a href="'.$wwwlink.'/galerie/'.$foto['SRC'].'" rel="imageOferta" ><img src="'.$wwwlink.'/galerie/miniaturki/'.$foto['SRC'].'"  alt="galeria" style="margin:5px;width:150px;" ></a>');
                }	}
//OFERTA PROMOCYJNA
					if($oferta['START3']!=1 ){
					$offer=mysql_fetch_array(mysql_query("SELECT * FROM offer WHERE AKTYWNE = 1 and ID='".$_GET['oferta']."'"));
 
                echo ('<p>'.$oferta['TEXT'].'</p>');
                echo ('<br><br>Galeria produktu:<br><hr>');
                $fotosy=mysql_query("SELECT * FROM offer_foto WHERE OFERTA='".$_GET['oferta']."' ORDER BY POS ASC");
                while($foto=mysql_fetch_array($fotosy)){
                    echo('<a href="'.$wwwlink.'/galerie/'.$foto['SRC'].'" rel="imageOferta" title="galeria obiektu"><img src="'.$wwwlink.'/galerie/miniaturki/'.$foto['SRC'].'" alt="galeria oferty-'.$foto['ID'].'" title="galeria oferty-'.$foto['ID'].'" style="margin:5px;width:75px;" ></a>');
                }
 
 
}
 ?>
<br>								</div>
							</div>
						</div>
					</div>
[PHP] pobierz, plaintext 

I teraz jeśli zmienie ID na SLUG czyli:

[PHP] pobierz, plaintext 
$oferta=mysql_fetch_array(mysql_query("SELECT * FROM offer WHERE AKTYWNE = 1    and  ID ='".$_GET['oferta']."'"));
 
na 
 
$oferta=mysql_fetch_array(mysql_query("SELECT * FROM offer WHERE AKTYWNE = 1    and  SLUG ='".$_GET['oferta']."'"));
[PHP] pobierz, plaintext 

Wyświetla mi się tekst tej strony , ale nie mam zdjęć, i tutaj mam problem. Linijka odpowiedzialna za zdjęcia to:

[PHP] pobierz, plaintext 
 $fotosy=mysql_query("SELECT * FROM offer_foto WHERE OFERTA='".$_GET['oferta']."' ORDER BY POS ASC");
                while($foto=mysql_fetch_array($fotosy)){
                    echo('<a href="'.$wwwlink.'/galerie/'.$foto['SRC'].'" rel="imageOferta" ><img src="'.$wwwlink.'/galerie/miniaturki/'.$foto['SRC'].'"  alt="galeria l" style="margin:5px;width:150px;" ></a>');
[PHP] pobierz, plaintext 

Ogólnie tabela offer_foto składa się z: ze swojego ID, numeru oferty ( czyli OFERTA) i SRC czyli scieżki . - sprawdza które oferty maja taką samą cyfrę i wyświetlają ją w ofercie.

Nie wiem co zmienić w tym fragmencie kodu odpowiedzialnym za zdjęcia aby się pokazywały, ewentualnie co dodać

[PHP] pobierz, plaintext 
<?php  $oferta=mysql_fetch_array(mysql_query("SELECT * FROM offer WHERE AKTYWNE = 1    and  ID ='".$_GET['oferta']."'"));?>
[PHP] pobierz, plaintext 

.

Proszę o pomoc

Źle do tego podchodzisz
1. Robisz zapytania w pętli
2. Zawsze o ile to możliwe przeszukuj bazę po ID nie żadnym SLUGu
3. Staraj się wrzucać sformatowany kod, bo ten słabo się czyta

Pokaż struktury tabel, które wykorzystujesz na tej stronie, to ogarniemy to jednym zapytaniem

Tak to wygląda


DESCRIBE offer_foto;

Field Type Null Key Default Extra
ID int(3) NO PRI NULL auto_increment
OFERTA nt(3) NO NULL
SRC varchar(255) NO NULL
POS int(3) NO 999



DESCRIBE offer:


Field Type Null Key Deafult Extra
ID int(255) NO PRI NULL auto_increment
MAIN_FOTO int(4) NO NULL
PROMUJ tinyint(1) NO NULL
START tinyint(1) NO NULL
START1 tinyint(1) NO NULL
START2 tinyint(1) NO NULL
START3 tinyint(1) NO NULL
START4 tinyint(1) NO NULL
TITLE varchar(50) NO NULL
SLUG varchar(255) NO NULL
TITLESEO varchar(300) NO NULL
DESCRIPTION varchar(300) NO NULL
TAGS varchar(300) NO NULL
TEXT text NO NULL
KAT varchar(50) NO NULL

Jeszcze kilka uwag
1. Staraj się unikać takiego kodu mysql_fetch_array(mysql_query()), bo jeżeli mysql_query() nic nie zwróci (zwróci FALSE), php rzuci błędem
2. mysql_* jest już przestarzałe pomyśl nad przejściem na mysqli_* lub PDO
3. Każdą zmienną pochodzącą od użytkownika escape'uj np. za pomocą mysql_real_escape_string()

[PHP] pobierz, plaintext 
<?php
$query = mysql_query("SELECT * FROM offer o LEFT JOIN offer_foto of ON (of.oferta = o.id) WHERE o.id = $id");
 
$tmp = [];
    while($row = mysql_fetch_assoc($query)) {
        if (!array_key_exists($row['ID'], $tmp)) {
            $tmp[$row['ID']] = [
                'TITLE' => $row['TITLE'],
                'TEXT' => $row['TEXT'],
                'START3' => $row['START3'],
                'IMAGES' => []
            ];
        }
 
        $tmp[$row['ID']]['IMAGES'][] = [
            'ID' => $row['ID'],
            'SRC' => $row['SRC']
        ];
    }
 
?>
///....
 
<?php foreach ($tmp as $id => $data) : ?>
    <?php if ($data['START3'] != 1) : ?>
        <p><?= $data['TEXT'];?><br/><br></p></div>
    <?php endif; ?>
 
    <?php foreach ($data['IMAGES'] as $image) : ?>
        <a href="<?= $image['SRC']; ?>"></a>
    <?php endforeach; ?>
<?php endforeach; ?>
[PHP] pobierz, plaintext 

W zapytaniu podmień * na nazwy kolumn , ponieważ kolumnę ID masz w dwóch tabelach, i mogą pojawić się problemy - sprawdź na googlu jak tworzyć aliasy nazw kolumn

Ten post edytował CuteOne 16.05.2016, 18:25:49

Dzięki zaraz zabieram się za lekturę i poprawki