štatistika aktivita hodnotitelov – PHP – Fórum – Programujte.com

Ahojte

Chcel by som Váš požiadať o názor na tento kód.

Vytvoril som stránku, kde sa bodujú výrobky... na stránke, kde sa zobrazujú štatistické hodnoty je tabuľka, kde je zobrazená aktivita hodnotiteľov za posledných 12 mesiacov. Teda koľkokrát hodnotili za daný mesiac. Funguje to, ale ten kód sa mi nezdá byť ani trochu dobrý. Príde mi veľmi dlhý a neohrabaný. Vedeli by ste mi povedať, naviesť ma, ako ho zmenšiť? Poprípade zacykliť? 

Výsledok vyzerá takto:

Připojen obrázek.

No a toto je ten kód:

<!-- 
	----------------------------------------------------------------------------
	Aktivita hodnotiteľov za posledných 12 mesiacov
	----------------------------------------------------------------------------
-->

<h2 class="popis_tabuliek">Aktivita enológov (počet bodovaní) za posledných 12 mesiacov</h2>

<table class="statistika_aktivita">


<?php

function mesiac($datum) {
if     ($datum == '01') {echo 'jan';}
elseif ($datum == '02') {echo 'feb';}
elseif ($datum == '03') {echo 'mar';}
elseif ($datum == '04') {echo 'apr';}
elseif ($datum == '05') {echo 'máj';}
elseif ($datum == '06') {echo 'jún';}
elseif ($datum == '07') {echo 'júl';}
elseif ($datum == '08') {echo 'aug';}
elseif ($datum == '09') {echo 'sep';}
elseif ($datum == '10') {echo 'okt';}
elseif ($datum == '11') {echo 'nov';}
elseif ($datum == '12') {echo 'dec';}	
elseif ($datum == '0') {echo 'dec';}
elseif ($datum == '-1') {echo 'jan';}
elseif ($datum == '-2') {echo 'feb';}
elseif ($datum == '-3') {echo 'mar';}
elseif ($datum == '-4') {echo 'apr';}
elseif ($datum == '-5') {echo 'máj';}
elseif ($datum == '-6') {echo 'jún';}
elseif ($datum == '-7') {echo 'júl';}
elseif ($datum == '-8') {echo 'aug';}
elseif ($datum == '-9') {echo 'sep';}
elseif ($datum == '-10') {echo 'okt';}
elseif ($datum == '-11') {echo 'nov';}												
}
?>

<tr>	
	<th>Meno</th>
	<th><?php mesiac(date("m")); ?></th>
	<th><?php mesiac(date("m")-1); ?></th>
	<th><?php mesiac(date("m")-2); ?></th>
	<th><?php mesiac(date("m")-3); ?></th>
	<th><?php mesiac(date("m")-4); ?></th>
	<th><?php mesiac(date("m")-5); ?></th>
	<th><?php mesiac(date("m")-6); ?></th>
	<th><?php mesiac(date("m")-7); ?></th>
	<th><?php mesiac(date("m")-8); ?></th>
	<th><?php mesiac(date("m")-9); ?></th>
	<th><?php mesiac(date("m")-10); ?></th>
	<th><?php mesiac(date("m")-11); ?></th>
	<th>Priemer</th>
</tr>


<?php  


	$db = DB();
	$dvanastmesiace = $db->query("
SELECT user_id, name, 
IFNULL(tento.pocet,0) as tento, 
IFNULL(minusjeden.pocet,0) as minusjeden,
IFNULL(minusdva.pocet,0) as minusdva,
IFNULL(minustri.pocet,0) as minustri,
IFNULL(minusstyri.pocet,0) as minusstyri,
IFNULL(minuspat.pocet,0) as minuspat,
IFNULL(minussest.pocet,0) as minussest,
IFNULL(minussedem.pocet,0) as minussedem,
IFNULL(minusosem.pocet,0) as minusosem,
IFNULL(minusdevat.pocet,0) as minusdevat,
IFNULL(minusdesat.pocet,0) as minusdesat,
IFNULL(minusjedenast.pocet,0) as minusjedenast,
IFNULL(minusdvanast.pocet,0) as minusdvanast,

(
IFNULL(tento.pocet,0) + 
IFNULL(minusjeden.pocet,0) +
IFNULL(minusdva.pocet,0) +
IFNULL(minustri.pocet,0) +
IFNULL(minusstyri.pocet,0) +
IFNULL(minuspat.pocet,0) +
IFNULL(minussest.pocet,0) +
IFNULL(minussedem.pocet,0) +
IFNULL(minusosem.pocet,0) +
IFNULL(minusdevat.pocet,0) +
IFNULL(minusdesat.pocet,0) +
IFNULL(minusjedenast.pocet,0) +
IFNULL(minusdvanast.pocet,0)/13
) as priemer 
FROM users


LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())
GROUP BY id_uzivatela ) tento on tento.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-1
GROUP BY id_uzivatela ) minusjeden on minusjeden.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-2
GROUP BY id_uzivatela ) minusdva on minusdva.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-3
GROUP BY id_uzivatela ) minustri on minustri.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-4
GROUP BY id_uzivatela ) minusstyri on minusstyri.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-5
GROUP BY id_uzivatela ) minuspat on minuspat.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-6
GROUP BY id_uzivatela ) minussest on minussest.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-7
GROUP BY id_uzivatela ) minussedem on minussedem.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-8
GROUP BY id_uzivatela ) minusosem on minusosem.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-9
GROUP BY id_uzivatela ) minusdevat on minusdevat.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-10
GROUP BY id_uzivatela ) minusdesat on minusdesat.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-11
GROUP BY id_uzivatela ) minusjedenast on minusjedenast.id_uzivatela = user_id

LEFT JOIN (
SELECT id_uzivatela, COUNT(body) as pocet
FROM bodovanie 
JOIN vina ON id_vino = id_vino_b
WHERE MONTH(date) = MONTH(CURRENT_DATE())-12
GROUP BY id_uzivatela ) minusdvanast on minusdvanast.id_uzivatela = user_id

WHERE
(
IFNULL(tento.pocet,0) + 
IFNULL(minusjeden.pocet,0) +
IFNULL(minusdva.pocet,0) +
IFNULL(minustri.pocet,0) +
IFNULL(minusstyri.pocet,0) +
IFNULL(minuspat.pocet,0) +
IFNULL(minussest.pocet,0) +
IFNULL(minussedem.pocet,0) +
IFNULL(minusosem.pocet,0) +
IFNULL(minusdevat.pocet,0) +
IFNULL(minusdesat.pocet,0) +
IFNULL(minusjedenast.pocet,0) +
IFNULL(minusdvanast.pocet,0)
) != 0


");

foreach ($dvanastmesiace as $mesiace) {
		echo '<tr>
				  
				  <td>'.$mesiace[1].'</td>'.
				 '<td>'.$mesiace[2].'</td>'.
				 '<td>'.$mesiace[3].'</td>'.
				 '<td>'.$mesiace[4].'</td>'.
				 '<td>'.$mesiace[5].'</td>'.
				 '<td>'.$mesiace[6].'</td>'.
				 '<td>'.$mesiace[7].'</td>'.
				 '<td>'.$mesiace[8].'</td>'.
				 '<td>'.$mesiace[9].'</td>'.
				 '<td>'.$mesiace[10].'</td>'.
				 '<td>'.$mesiace[11].'</td>'.
				 '<td>'.$mesiace[12].'</td>'.
				 '<td>'.$mesiace[13].'</td>'.
				 '<td>'.$mesiace[13].'</td>
			 </tr>';
		
		
	}
	
	?>

	
</table>

vidím tam jeden problémek, až to bude déle než rok, budou se ti sčítat měsíce více let. začátek máš v únoru, za rok v únoru ti to do měsíce únor započte jak letošní tak i aktuální únor.

narazil jsem na další věc máš tam blbě tu tabulku měsíců

..Feb-Jan-Dec-Jan-Feb..

ta funkce by šla zjednodušit 

function mesiac($datum) {
$datum=($datum + 12) % 12
if     ($datum == 1) {echo 'jan';}
elseif ($datum == 2) {echo 'feb';}
elseif ($datum == 3) {echo 'mar';}
elseif ($datum == 4) {echo 'apr';}
elseif ($datum == 5) {echo 'máj';}
elseif ($datum == 6) {echo 'jún';}
elseif ($datum == 7) {echo 'júl';}
elseif ($datum == 8) {echo 'aug';}
elseif ($datum == 9) {echo 'sep';}
elseif ($datum == 10) {echo 'okt';}
elseif ($datum == 11) {echo 'nov';}
elseif ($datum == 0) {echo 'dec';}											
}

A JEŠTĚ 1 MOŽNOST PŘES funkce

function mesiac($datum) {
echo substr(date("F", mktime(null, null, null, $datum)),0,3); 
}
-- date vrátí celý název měsíce, fungují správně i mínusové měsíce  
-- substr zkrátí na 3 znaky 

na to SQL se ještě podívám, myslím že by mělo jít zjednodušit máš tam příliš těch podmínek a subquery

teď jsem ještě něco hledal a zjistil jsem, že není třeba ten substr, pokud se do formátu date místo 'F' dá 'M' takže stačí

echo date("M", mktime(null, null, null, $datum));

Ďakujem Milan, že si si našiel znova na mňa čas. Pozriem sa na to.

Než tu hromadu ifů, to bych raději použil pole. Teď nejde o to, jak zjistit název měsíce, ale obecně o podobné mapování. 

function mesiac($num) {
    $nazvy = ['jan','feb','mar','apr','máj','jún',
              'júl','aug','sep','okt','nov','dec'];
    $idx = (11 + $num) % 12;
    return $nazvy[$idx];
}

Opakující se kód jde samozřejmě hodit do cyklu.

$curmonth = date('n');
for ($i = 0; $i < 12; $i++)
	echo '<th>' . mesiac($curmonth - $i) . '</th>';

Nad tím SQL se mi nechce přemýšlet (hlavně nad náročností toho, co jsem vyplodil). Nějak nevidím potřebu těch joinů (rozhodně tam nepotřebuješ ještě tabulku vina). Když už to chceš namastit do jednoho řádku, tak bych asi zkusil nějak takhle: 

SELECT name,
       m1, m2, m3 ... m12,
       (m1 + m2 + m3 ... m12) / 12 avg
FROM   users
       LEFT JOIN (SELECT user_id,
                         Sum(Month(datum) = 1) m1,
                         Sum(Month(datum) = 2) m2,
                         Sum(Month(datum) = 3) m3,
                         ...
                  FROM   bodovanie
                  GROUP  BY uid) pocty
              ON pocty.user_id = user_id;

#5 gna
ahoj, je vidět že v php děláš víc než já, to tvoje použití pole na měsíce vypadá vizuálně lépe, o tom cyklu na výpis měsíců jsem také uvažoval, já to radil z práce furt někdo rušil takže jsem nahodil jen takové první nápady.

hm u toho tvého SQL mi chvilku trvalo než jsem pochopil, že sčítáš výsledek vyhodnocení True=1, jen si to pak musí správně spárovat do té tabuky, tam by měl stačit posun indexu, na celkový počet bych do subquery přidal Sum(1) celkem

ještě by tam měl mít limit na datum, buď poslední den aktuálního měsíce - rok, nebo Aktuální rok or ((Akt.rok-1) and měsíc>aktuální)

průměr by měl být jen z měsíců kdy bylo hodnoceno, to bych udělal až v php ve foreach 

foreach ($dvanastmesiace as $mesiace) {
		$count=0;
		for ($i = 2; $i < 14; $i++)
			if ($mesiace[i] > 0) $count++;
		echo '<tr>				  
			<td>'.$mesiace[1].'</td>';
			$curmonth = date('n');
			for ($i = 0; $i < 12; $i++)
				$month = ($curmonth +11 - $i) % 12;
				 echo '<td>'.$mesiace[$month+2].'</td>';
			echo '<td>'.$mesiace[14]/$count.'</td>
			 </tr>';
		
	}

Ahojte, 

Ďakujem Vám za reakcie.

Dostal som sa k tomu až teraz a už nerozumiem ani tomu svojmu kódu :D

Pozriem sa na Vaše rady a aplikujem ich.