Forum PHP.pl > Problem z wyświetlaniem danych z MySQL

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: Problem z wyświetlaniem danych z MySQL

alkesz1718

16.08.2017, 17:36:57

Witam,
Mam pewien problem z modułem a mianowicie wyświetla on z bazy danych daną ilość artykułów z ilością komentarzy, oceny i wyświetleń od najwyższej do najniższej...

Wszystko wyświetla się prawidłowo w ilości komentarzy i ocenach jeśli jednak przejdę do wyświetlanie wg odwiedzin to pokazuje mi w wszystkich artykułach najwyższą liczbę odwiedzin z 1 art gdzie realna liczba jest całkiem inna...

Niżej podaję kod odpowiedzialny za wyświetlanie art wg wyświetleń/odwiedzin:

[PHP] pobierz, plaintext 
if ($_GET['mode']=='views'){
$cache = $dle_api->load_from_cache ( "top100views",  3600, $tpl->result['content'] ); 
if( $cache === false ) {
if ($top100_config['category'] != 0){
$category_true = "AND category IN (".$top100_config['category'].")";
} else {
$category_true = "";
}
$sql_result = $db->query( "SELECT a.id, a.title, a.category, a.alt_name, b.news_read FROM " . PREFIX . "_post a INNER JOIN " . PREFIX . "_post_extras b WHERE approve=1 ".$category_true." ORDER BY news_read DESC LIMIT ".$top100_config['news_num'] );
 
$top100 = <<<HTML
<thead>
<tr>
<td align="center">$top100_config[title_num]</td>
<td align="center">$top100_config[title_name]</td>
<td align="center">$top100_config[title_views_head]</td>
</tr>
</thead>
HTML;
$top100 .= <<<HTML
<tbody>
HTML;
$i=1;
while ( $row = $db->get_row($sql_result) ) {
$row['category'] = intval( $row['category'] );
 
if( $config['allow_alt_url'] == "yes" ) {
 
			if( $row['flag'] and $config['seo_type'] ) {
 
				if( $row['category'] and $config['seo_type'] == 2 ) {
 
					$full_link = $config['http_home_url'] . get_url( $row['category'] ) . "/" . $row['id'] . "-" . $row['alt_name'] . ".html";
 
				} else {
 
					$full_link = $config['http_home_url'] . $row['id'] . "-" . $row['alt_name'] . ".html";
 
				}
 
			} else {
 
				$full_link = $config['http_home_url'] . date( 'Y/m/d/', $row['date'] ) . $row['alt_name'] . ".html";
			}
 
		} else {
 
			$full_link = $config['http_home_url'] . "index.php?newsid=" . $row['id'];
 
		}
$row['title']=strip_tags( stripslashes( $row['title'] ) );
$row['news_read']=strip_tags( stripslashes( $row['news_read'] ) );
$full_link=strip_tags( stripslashes( $full_link ) );
 
$top100 .='';
 
if ($row['news_read']!=0){		
$top100 .= <<<HTML
<tr>
<td align="center">&nbsp;&nbsp;$i&nbsp;&nbsp;</td>
<td><a href="$full_link" title=""><strong>$row[title]</strong></a></td>
<td align="center">$row[news_read]</td>
</tr>
HTML;
 
$i++;	
}
}
 
$top100 .= <<<HTML
</tbody>
HTML;
$tpl->load_template( 'top100.tpl' );
$tpl->set( '{title}', $top100_config['title_views'] );
$tpl->set( '{top100}', $top100 );
$tpl->compile( 'content' );
$dle_api->save_to_cache ( "top100views", $tpl->result['content'] ); 
} 
else { 
    var_dump($cache); 
}
$tpl->clear();	
$db->free();
}
[PHP] pobierz, plaintext

Kod łączy ze sobą 2 tabele z których wyciąga dane:
- nazwa artykułu
- ilość wyświetleń
- id
- (rzekomo kategorię ale wycofałem tę funkcję)

Prawdopodobnie gdzieś w tym kodzie jest błąd ale już nie mam pomysłu gdzie.

Dla przykładu wyglądu podaję stronę testową:
http://djalexn.hol.es/?do=top100&mode=views

jak będzie potrzebny cały kod to chętnie podeślę i prosiłbym o jak najszybszą odpowiedź i najlepiej z podanym gotowym rozwiązaniem...

Z góry dziękuję i pozdrawiam

b4rt3kk

21.08.2017, 16:11:45

Zwróć uwagę na to zapytanie:

[PHP] pobierz, plaintext 
$sql_result = $db->query( "SELECT a.id, a.title, a.category, a.alt_name, b.news_read FROM " . PREFIX . "_post a INNER JOIN " . PREFIX . "_post_extras b WHERE approve=1 ".$category_true." ORDER BY news_read DESC LIMIT ".$top100_config['news_num'] );
[PHP] pobierz, plaintext

Brakuje nam tutaj warunku połączenia tabeli post z tabelą post_extras.

Prawidłowe zapytanie powinno wyglądać mniej więcej tak (mniej więcej, bo nie wiem na podstawie jakiego klucza jest stworzone dowiązanie):

[PHP] pobierz, plaintext 
$sql_result = $db->query( "SELECT a.id, a.title, a.category, a.alt_name, b.news_read FROM " . PREFIX . "_post a INNER JOIN " . PREFIX . "_post_extras b ON a.id = b.id_post WHERE approve=1 ".$category_true." ORDER BY news_read DESC LIMIT ".$top100_config['news_num'] );
[PHP] pobierz, plaintext

alkesz1718

21.08.2017, 16:47:55

Cytat

Prawidłowe zapytanie powinno wyglądać mniej więcej tak (mniej więcej, bo nie wiem na podstawie jakiego klucza jest stworzone dowiązanie):

[PHP] pobierz, plaintext 
$sql_result = $db->query( "SELECT a.id, a.title, a.category, a.alt_name, b.news_read FROM " . PREFIX . "_post a INNER JOIN " . PREFIX . "_post_extras b ON a.id = b.id_post WHERE approve=1 ".$category_true." ORDER BY news_read DESC LIMIT ".$top100_config['news_num'] );
[PHP] pobierz, plaintext

Ogólnie to to rozwiązanie pomogło ale musiałem zmienić:
ON a.id = b=id_post
na
ON a.id = b.eid
Tak jak mam w swojej bazie danych i problem się rozwiązał

Co do "braku warunku połączenia tabeli post z tabelą post_extras" to powiem ci że to nie do końca jest prawda (a przynajmniej tak mnie się wydaje) gdyż podobny kod mam na wyświetlanie innych danych... np:

[PHP] pobierz, plaintext 
$sql_result = $db->query( "SELECT a.id, a.title, a.category, a.alt_name, b.rating FROM " . PREFIX . "_post a INNER JOIN " . PREFIX . "_post_extras b WHERE approve=1 ".$category_true." ORDER BY rating DESC LIMIT ".$top100_config['news_num'] );
[PHP] pobierz, plaintext

no i tutaj o dziwo działa jak należy

b4rt3kk

21.08.2017, 17:18:40

Być może jest ok, albo myślisz, że jest ok, bo czegoś nie zauważasz, ale jako wynik takiego zapytania bez klauzuli ON otrzymujesz połączenie kartezjańskie, czyli każdy wpis z tabeli A z każdym wpisem z tabeli B.

Dla pewno zdebuguj to sobie.

alkesz1718

21.08.2017, 20:28:03

Hmmm rzeczywiście, po usunięciu ciasteczek itd. przestała działać inna funkcja... gdy użyłem klauzulę ON naprawiło się... chyba muszę bardziej się podszkolić w PHP i MySQL

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.