Terminarz PHP - problem z przewijaniem miesięcy Opcje

[kozak1987]

Z pomocą forum oraz wujka google "napisałem" terminarz w PHP. Wszystko pięknie śmiga - miał zczytywać z bazy dni które mają się podświetlać innym kolorem. Podświetlać aktualny dzień itp.
Napotkałem jednak problem przy tworzeniu przycisków do przełączania miesięcy. Z pomocą znajomego coś tam napisałem, ale ni jak to sie ma do działania.

Tak oto wygląda kod, przyciski od 31 do 44 linijki:

[PHP] pobierz, plaintext 
<?php 
require("kal/connection.php");
connection();
// Funkcja rysująca kalendarz
function rysujKalendarz($miesiac, $rok)
{
global $link;
 
// skonstruowanie zapytania do bazy danych w celu wyciągnięcia dni które mają być zaznaczone i wpisanie tych dni do tablicy $dni_zaznaczone
$sql = "SELECT dzien FROM formularz WHERE miesiac = ".(int)$miesiac." AND rok = $rok ORDER BY dzien ASC";
$res = mysql_query($sql);
$dni_zaznaczone = array();
 
 
while($row = mysql_fetch_array($res))
{
$dni_zaznaczone[] = $row['dzien'];
}
 
// Ustawienie głównych zmiennych
$znacznik = mktime(0, 0, 0, $miesiac, 1, $rok);
$iledni = date("t", $znacznik);
$start = date("w", $znacznik);
$miesiace = array(
"Styczeń", "Luty", "Marzec", "Kwiecień", "Maj", "Czerwiec",
"Lipiec", "Sierpień", "Wrzesień", "Październik", "Listopad", "Grudzień"
);
 
// Rysowanie tabeli
 
$poprzedni = $miesiac - 1;
$nastepny = $miesiac + 1;
 
if($poprzedni < 1) 
{$poprzedni = 12 ; $rok-- ;}
 
if($nastepny > 12)
{$nastepny = 1; $rok++ ;} 
 
 
echo '<table width="400px" border="1" bordercolor="#ffb4ee" cellspacing="0" cellpadding="6"><tr>';
echo '<td colspan="20" style="text-align:center; background:#252525;">';
echo '<a href="terminarz.php?rok='.$poprzedni_rok.'&amp;miesiac='.$poprzedni_miesiac.'">poprzedni </a>' ;
echo '<a href="terminarz.php?rok='.$nastepny_rok.'&amp;miesiac='.$nastepny_miesiac.'"> następny</a>';
echo $miesiace[$miesiac-1].' '.$rok.'</td></tr>';
echo '<tr style="background:#252525;"><td>Nd</td><td>Pn</td>
<td>Wt</td><td>Śr</td><td>Cz</td><td>Pt</td><td>So</td></tr>';
echo '<tr align="right">';
if ($start>0)
echo '<td colspan="'.$start.'" style="background:#252525;"> </td>';
else $pierwszy=true;
 
// Wypisanie wszystkich dni
for ($i=1;$i<=$iledni;$i++) 
{
$dzien = date("w", mktime(0, 0, 0, $miesiac, $i, $rok));
if ($dzien==0 and !$pierwszy) echo '</tr><tr align="right">';
 
// Jeśli jest to dzień zaznaczony, to ustawienie koloru tła na podany kolor
if (in_array($i, $dni_zaznaczone)) $bg = "background:#de00ac;";
// W przeciwnym razie, gdy dzień jest niedzielą, ustawienie specjalnego koloru
else if ($dzien==0) $bg = "background:#1d1d1d;";
// W przeciwnym razie, wiemy że to zwykły dzień i ustawiamy inne tło
else $bg = "background:#252525;";
 
 
$pierwszy = false;
if (mktime(0, 0, 0, date("m"), date("d"), date("Y"))==
mktime(0, 0, 0, $miesiac, $i, $rok)) $bg .= " font-weight:bold;";
echo '<td style="'.$bg.'">'.$i.'</td>';
}
$koniec=7-($start+$iledni)%7;
if ($koniec<>7) echo '<td colspan="'.$koniec.'"
style="background:#252525;"> </td>';
echo '</tr></table>';
 
}
 
// Wywołanie funkcji z aktualną datą
rysujKalendarz(date("m"),date("Y"));
 
?>
[PHP] pobierz, plaintext 

Tak więc jak już wcześniej wspominałem jestem zielony w tym temacie i robię to raz bo muszę dwa, że już zostało tak mało do skończenia że nie chcę komuś tego zlecać. Domyślam się, że brakuje jakiejś zmiennej?
Ze strony estetycznej natomiast chciałbym, żeby linki "poprzedni miesiąc" i "następny miesiąc" były po bokach tej komórki w której się znajdują natomiast nazwa aktualnego miesiąca i rok żeby były pomiędzy linkami.

[camikazee]

[PHP] pobierz, plaintext 
// Rysowanie tabeli
    $poprzedni_rok = $rok;
    $nastepny_rok = $rok;
 
    if($miesiac==1){
    	$poprzedni_miesiac = 12;
    	$poprzedni_rok--;
    }
    else{
    	$poprzedni_miesiac = $miesiac - 1;
    }
 
    if($miesiac==12){
    	$nastepny_miesiac = 1;
    	$nastepny_rok++;
    }     
    else{
    	$nastepny_miesiac = $miesiac + 1;
    }
 
    echo '<table width="400px" border="1" bordercolor="#ffb4ee" cellspacing="0" cellpadding="6"><tr>';
    echo '<td colspan="20" style="text-align:center; background:#252525;">';
    echo '<a href="terminarz.php?rok='.$poprzedni_rok.'&amp;miesiac='.$poprzedni_miesiac.'">poprzedni </a>' ;
    echo '<a href="terminarz.php?rok='.$nastepny_rok.'&amp;miesiac='.$nastepny_miesiac.'"> następny</a>';
[PHP] pobierz, plaintext 

[Pilsener]

Po co tak kombinować? Wystarczy użyć: strtotime

[PHP] pobierz, plaintext 
echo strtotime('+1 month');
[PHP] pobierz, plaintext 

- jak rok przeskoczy to też uwzględni, nie trzeba dodatkowych warunków, że jak ostatni miesiąc to miesiąc 1 a rok plus jeden i odwrotnie.

[camikazee]

Ja się dostosowałem do jego przykładu. Poza tym jeżeli użyjesz strtotime, to najpierw będziesz musiał złączyć datę, sprawdzić strtotime a potem znów ją rozbić na rok i miesiąc by dalej w skrypcie się wszystko zgadzało. Ok, ja też twierdzę, że można cały kalendarz zrobić lepiej, prościej i elastyczniej wykorzystując funkcje PHP do obsługi dat.

[Pilsener]

Po co rozbijać? Masz funkcje do tego:

[PHP] pobierz, plaintext 
echo strftime('%Y-%m',strtotime('+1 month'));
[PHP] pobierz, plaintext 

[camikazee]

Ale skoro tak zrobi, to będzie musiał rozbić datę na rok i miesiąc, bo ona te parametry przenosi osobno, świadomie bądź nie.

[PHP] pobierz, plaintext 
echo date('Y-m', strtotime($rok.'-'.$miesiac.' -1 month'));
echo date('Y-m', strtotime($rok.'-'.$miesiac.' +1 month'));
[PHP] pobierz, plaintext 

Lub mozolnie tak

[PHP] pobierz, plaintext 
$poprzedni_rok = date('Y', strtotime($rok.'-'.$miesiac.' -1 month'));
$nastepny_rok = date('Y', strtotime($rok.'-'.$miesiac.' +1 month'));
$poprzedni_miesiac = date('m', strtotime($rok.'-'.$miesiac.' -1 month'));
$nastepny_miesiac = date('m', strtotime($rok.'-'.$miesiac.' +1 month'));
[PHP] pobierz, plaintext 

Czy można prościej?

[Pilsener]

Można prościej:

[PHP] pobierz, plaintext 
$date = $_GET['date'].'-01'; //data w postaci YYYY-MM (plus dzień, bo taką datę łatwo obsłużyć)
$date_month_next = strftime('%Y-%m',strtotime('+1 month',strtotime($date)));
$date_month_prev = strftime('%Y-%m',strtotime('-1 month',strtotime($date)));
 
//wystarczy wstawić do url:
echo '<a href="'.$date_month_prev.'">month prev</a>';
[PHP] pobierz, plaintext 

Teoretycznie wystarczą dwie linijki kodu

Ten post edytował Pilsener 3.04.2012, 10:35:14

[kozak1987]

Camikazee niestety twój sposób nie działa - ojawia sie tak samo jak to co ja napisałem, są linki ale zero akcji. Zauważyłem że po najechaniu na link to dobrze liczy bo jako następny miesiąc wyrzuca 5, a poprzedni 3 - tak więc to samo w sobie myślę że działa natomiast nie rysuje nowego miesiąca. Nie zmienia nazwy miesiąca itp - po prostu zero akcji.

Pilsner - twoje rozwiązanie w ogóle nie działa, po kliknięciu w link przenosi do strony której nie ma.

Pozostali - nie ogarniam o czym mówicie ani nie wiem gdzie to powklejać.

[camikazee]

Bo my nie pisaliśmy całego kodu tylko modyfikacje, które miałeś zrobić.
Idąc Twoją metodą, masz kod:

[PHP] pobierz, plaintext 
    <?php
 
    require("kal/connection.php");
	connection();
	// Funkcja rysująca kalendarz
	function rysujKalendarz($miesiac, $rok)
	{
	global $link;
 
	// skonstruowanie zapytania do bazy danych w celu wyciągnięcia dni które mają być zaznaczone i wpisanie tych dni do tablicy $dni_zaznaczone
	$sql = "SELECT dzien FROM formularz WHERE miesiac = ".(int)$miesiac." AND rok = $rok ORDER BY dzien ASC";
	$res = mysql_query($sql);
	$dni_zaznaczone = array();
 
 
	while($row = mysql_fetch_array($res))
	{
	$dni_zaznaczone[] = $row['dzien'];
	}
 
    // Ustawienie głównych zmiennych
    $znacznik = mktime(0, 0, 0, $miesiac, 1, $rok);
    $iledni = date("t", $znacznik);
    $start = date("w", $znacznik);
    $miesiace = array(
    "Styczeń", "Luty", "Marzec", "Kwiecień", "Maj", "Czerwiec",
    "Lipiec", "Sierpień", "Wrzesień", "Październik", "Listopad", "Grudzień"
    );
 
    // Rysowanie tabeli
    $poprzedni_rok = $rok;
    $nastepny_rok = $rok;
 
    if($miesiac==1){
    	$poprzedni_miesiac = 12;
    	$poprzedni_rok--;
    }
    else{
    	$poprzedni_miesiac = $miesiac - 1;
    }
 
    if($miesiac==12){
    	$nastepny_miesiac = 1;
    	$nastepny_rok++;
    }     
    else{
    	$nastepny_miesiac = $miesiac + 1;
    }
 
    echo '<table width="400px" border="1" bordercolor="#ffb4ee" cellspacing="0" cellpadding="6"><tr>';
    echo '<td colspan="20" style="text-align:center; background:#252525;">';
    echo '<a href="terminarz.php?rok='.$poprzedni_rok.'&amp;miesiac='.$poprzedni_miesiac.'">poprzedni </a>' ;
    echo '<a href="terminarz.php?rok='.$nastepny_rok.'&amp;miesiac='.$nastepny_miesiac.'"> następny</a>';
    echo $miesiace[$miesiac-1].' '.$rok.'</td></tr>';
    echo '<tr style="background:#252525;"><td>Nd</td><td>Pn</td>
    <td>Wt</td><td>Śr</td><td>Cz</td><td>Pt</td><td>So</td></tr>';
    echo '<tr align="right">';
    if ($start>0)
    echo '<td colspan="'.$start.'" style="background:#252525;"> </td>';
    else $pierwszy=true;
 
    // Wypisanie wszystkich dni
    for ($i=1;$i<=$iledni;$i++)
    {
    $dzien = date("w", mktime(0, 0, 0, $miesiac, $i, $rok));
    if ($dzien==0 and !$pierwszy) echo '</tr><tr align="right">';
 
    // Jeśli jest to dzień zaznaczony, to ustawienie koloru tła na podany kolor
    if (in_array($i, $dni_zaznaczone)) $bg = "background:#de00ac;";
    // W przeciwnym razie, gdy dzień jest niedzielą, ustawienie specjalnego koloru
    else if ($dzien==0) $bg = "background:#1d1d1d;";
    // W przeciwnym razie, wiemy że to zwykły dzień i ustawiamy inne tło
    else $bg = "background:#252525;";
 
 
    $pierwszy = false;
    if (mktime(0, 0, 0, date("m"), date("d"), date("Y"))==
    mktime(0, 0, 0, $miesiac, $i, $rok)) $bg .= " font-weight:bold;";
    echo '<td style="'.$bg.'">'.$i.'</td>';
    }
    $koniec=7-($start+$iledni)%7;
    if ($koniec<>7) echo '<td colspan="'.$koniec.'"
    style="background:#252525;"> </td>';
    echo '</tr></table>';
 
    }
 
    $rok = ($_GET['rok'] == '') ? date('Y') : $_GET['rok'];
    $miesiac = ($_GET['miesiac'] == '') ? date('m') : $_GET['miesiac'];
 
    // Wywołanie funkcji z aktualną datą
    rysujKalendarz($miesiac, $rok);
 
    ?>
[PHP] pobierz, plaintext 

[kozak1987]

Jesteś wielki!!!
Tydzień szukałem rozwiązaniem a wystarczyło napisać temat, pare godzin i wszystko śmiga.
Jeszcze od strony estetycznej da radę zrobić tak, aby aktualny miesiąc i rok był pomiędzy tymi przyciskami? Zamieniam linijki miejscami i wywala mi cały czas błąd.

[Lorum3]

[PHP] pobierz, plaintext 
 
 
    echo '<table width="400px" border="1" bordercolor="#ffb4ee" cellspacing="0" cellpadding="6"><tr>';
    echo '<td colspan="20" style="text-align:center; background:#252525;">';
    echo '<a href="terminarz.php?rok='.$poprzedni_rok.'&amp;miesiac='.$poprzedni_miesiac.'">poprzedni </a>' ;
    echo $miesiace[$miesiac-1].' '.$rok;
    echo '<a href="terminarz.php?rok='.$nastepny_rok.'&amp;miesiac='.$nastepny_miesiac.'"> następny</a>';
    echo '</td></tr>';
    echo '<tr style="background:#252525;"><td>Nd</td><td>Pn</td>
 
[PHP] pobierz, plaintext 

[camikazee]

[PHP] pobierz, plaintext 
echo '<table width="400px" border="1" bordercolor="#ffb4ee" cellspacing="0" cellpadding="6"><tr>';
echo '<td colspan="20" style="text-align:center; background:#252525;">';
echo '<a href="terminarz.php?rok='.$poprzedni_rok.'&miesiac='.$poprzedni_miesiac.'">poprzedni </a> ';
echo $miesiace[$miesiac-1].' '.$rok;
echo ' <a href="terminarz.php?rok='.$nastepny_rok.'&miesiac='.$nastepny_miesiac.'"> następny</a></td></tr>';
[PHP] pobierz, plaintext 

spóźniony

Ten post edytował camikazee 3.04.2012, 18:26:21

[kozak1987]

Dzięki Panowie bardzo. Problem rozwiązany, można zamknąć