řazení v news podle zhlédnutí – PHP – Fórum – Programujte.com

Ahoj!

Na HomePage zobrazují 11 newsek. Zobrazují se podle datumu vložení a času takto:

$result = dbquery(
"SELECT tn.*, tc.*, tu.user_id, tu.user_name, tu.user_status
FROM ".DB_NEWS." tn
LEFT JOIN ".DB_USERS." tu ON tn.news_name=tu.user_id
LEFT JOIN ".DB_NEWS_CATS." tc ON tn.news_cat=tc.news_cat_id
WHERE ".groupaccess('news_visibility')." AND (news_start='0'||news_start<=".time().")
AND (news_end='0'||news_end>=".time().") AND news_draft='0'
GROUP BY news_id 
ORDER BY news_sticky DESC, news_datestamp DESC LIMIT ".$_GET['rowstart'].",".$items_per_page
);

Chtěl bych, aby se články/newsky řadily takto nadále, ale s tím, že se pro každý den seřadí podle počtu zhlédnutí, ostatní zůstane stejné. Upravil jsem (viz. poslední řádek) kód takto:

//Modification  - Kvetos, 20.06.2014
$result = dbquery(
"SELECT tn.*, tc.*, tu.user_id, tu.user_name, tu.user_status
FROM ".DB_NEWS." tn
LEFT JOIN ".DB_USERS." tu ON tn.news_name=tu.user_id
LEFT JOIN ".DB_NEWS_CATS." tc ON tn.news_cat=tc.news_cat_id
WHERE ".groupaccess('news_visibility')." AND (news_start='0'||news_start<=".time().")
AND (news_end='0'||news_end>=".time().") AND news_draft='0'
GROUP BY news_id
	ORDER BY news_sticky DESC, news_reads DESC LIMIT ".$_GET['rowstart'].",".$items_per_page
		);

Tato změna se projevila tak, že se u akituálního dne řadí články podle zhlédnutí reads od největšího počtu zhlédnutí ki nejmenšímu. Což je OK, tak to chci.

Avšak co je špatně, že se zároveň pod tím zobrazují všechny zbývající články z ostatních dnů (za poslední 2 roky) sestupně POUZE od největšího počtu zobrazení k nejnižšímu, bez sestupného řazení podle dnů.

Jak bych to měl vyřešit. Díky za každopu radu.

1. ak chces pouzivat sql tak sa nauc alebo si aspon pozri co robia dane prikazy ktore pouzivas
2. neprepisuj to druhe zoradenie ... order by funguje tak ze najprv tabulku zoradi podla posledneho potom zachova poradie a zoradi ju podla skorsieho atd az kym nenarazi na prve ... ty chces najprv zoradit podla reads potom podla datestamp a nakoniec dat sticky ako prve

#2 Flowy
Aha, takže jen prohodit. Rozumím.

#2 Flowy
Tak jsem změnil ten poslední řádek na tohle:

 ORDER BY news_sticky DESC, news_datestamp DESC, news_reads DESC LIMIT ".$_GET['rowstart'].",".$items_per_page

1/ FUNGUJE, částečně!!!

2/ Ale ještě chybička, reads mi neřadí sestupně, tedy od nejčtenější newsky do nejméně čtené v každém dni.

3/ Pokud se neplatu, tak řazení se zadává přes groupaccess. Takže bych tam potřeboval propašovat asi:

WHERE ".groupaccess('news_visibility')." AND (news_start='0'||news_start<=".time().")
AND (news_end='0'||news_end>=".time().") AND news_draft='0'

Tedko  jsem zkoušel, nefunguje tato 2. změna; zobrazuje se vše, chybu nevypisuje..., ale neseřazuje...,  ještě to chce vylepšit, ale jak?

D.

Tohle, tato mpje úprava nefunguje:

WHERE ".groupaccess('news_visibility')." AND ORDER BY news_datestamp DESC,AND (news_start='0'||news_start<=".time().")
AND (news_end='0'||news_end>=".time().") AND news_draft='0'

Celé to vypadá takto:

if ($rows) {
		$result = dbquery(
			"SELECT tn.*, tc.*, tu.user_id, tu.user_name, tu.user_status
			FROM ".DB_NEWS." tn
			LEFT JOIN ".DB_USERS." tu ON tn.news_name=tu.user_id
			LEFT JOIN ".DB_NEWS_CATS." tc ON tn.news_cat=tc.news_cat_id
	WHERE ".groupaccess('news_visibility')." AND ORDER BY news_datestamp DESC,AND (news_start='0'||news_start<=".time().")
				AND (news_end='0'||news_end>=".time().") AND news_draft='0'
			GROUP BY news_id
			ORDER BY news_sticky DESC, news_datestamp DESC, news_reads DESC LIMIT ".$_GET['rowstart'].",".$items_per_page
		);
		

Vypisuje chybu:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ORDER BY news_datestamp DESC,AND (news_start='0'||news_start<=1403504812) AN' at line 5
 

Jestli ti to funguje, tak ziram, protoze pouzivas konstrukce, ktere vubec neznam, coz ale mozne je, v sql jsem spis amater.
Ja bych to resil tak, ze si privne vyberu id clanku, ktere chci a dalsim selectem je teprve seradim. Cili neco jako 

SELECT sloupce
FROM (SELECT id FROM ... WHERE ... ORDER BY time DESC LIMIT 11)
LEFT JOIN ostatni tabulky...
ORDER BY pocet_precteni DESC

#6 peter
Však taky ty Kvetosovy konstrukce jsou pěkně zastaralé a nebezpečné.

#5 Kvetos
Končí ti to chybou, protože máš AND před ORDER BY

Panové, dík za připomínky, pěkně se v tom placatím. Ale snad to vyřeším podle Vašich rad. Zatím díky.

#5 Kvetos
zoradovat tabulky pred joinom je zbytocne kedze joinom sa pravdepodobne aj tak neudrzi poradie ... moderny system by taketo stavy mal zakazat

a tiez by ti to v oracle vobec nepreslo pretoze order by MUSI byt vzdy na konci (okrem limit)

Dodatečně, díky. Zatím jsem tento kód nedokončil.