[PHP][skrypt] Prosze o sprawdzenie logiczności i błędów, oraz o propozycje Opcje

[thomson89]

Poniżej prosty skrypt, pokazujący kiedy ostatnio byleś na stronie. Proszę o sprawdzenie:
- poprawności ze świata relnego
- poprawności logicznej
- poprawności językowej (php)
Oraz o ewentualne propozycje rozbudowy.

[PHP] pobierz, plaintext 
<?php
function ostatnio()
{
   if(!isset($_COOKIE['kiedy'])){
      $teraz = date("H:i");
      setcookie("kiedy", $teraz);
      return $_COOKIE['kiedy'];
   }
   else {
      //ustawianie nowego czasu
      $teraz = date("H:i");
      $ostatnio = $_COOKIE['kiedy'];
      //zmiana godziny na sekundy
      $minelo = explode(':', $ostatnio);
      $godz = $minelo[0];
      $min = $minelo[1];
      $godz*60*60;
      $min*60;
      $sek1 = $godz+$min;
      //to samo z czasem teraz
      $teraz_cz = explode(':', $teraz);
      $godz = $teraz_cz[0];
      $min = $teraz_cz[1];
      $godz*60*60;
      $min*60;
      $sek2 = $godz+$min;
      //obliczanie ile czasu temu byles
      $sek_temu = $sek2-$sek1;
      if($sek_temu>60){
         $sek_temu/60;
         $min_temu = explode(',', $sek_temu);
         $tresc = "Ostatnio tu byleś".$min_temu[0]." minut temu i ".$min_temu[1]." sekund.";
        
      }
      if($min_temu[0]>60){
         $min_temu[0]/60;
         $godz_temu = explode(',', $min_temu[0]);
         $tresc = "Ostatnio tu byleś".$godz_temu[0]." godzin temu, ".$godz_temu[1]."minut i".$min_temu[0]." sekund.";
      }
      setcookie("kiedy", $teraz);
      return $tresc;
      
   }
}
?>
[PHP] pobierz, plaintext 

I wywołujemy, dołączając plik z funckjami i wywołując:

[PHP] pobierz, plaintext 
<?php
$cos = ostatnio();
echo $cos;
?>
[PHP] pobierz, plaintext 

[SzamanGN]

Nie jestem ekspertem od języka polskiego, ale ja bym napisał w linii 39 pierwszego kodu:

[PHP] pobierz, plaintext 
<?php
$tresc = 'Ostatnie odwiedziny były ';
if ($godz_temu[0]>4) $tresc.=$godz_temu[0].' godzin, ';
if ($godz_temu[0]>1 && $godz_temu[0]<5) $tresc.=$godz_temu[0].' godziny, ';
if ($godz_temu[0]==1) $tresc.=$godz_temu[0].' godzinę, ';
if ($godz_temu[1]>4) $tresc.=$godz_temu[1].' minut, ';
if ($godz_temu[1]>1 && $godz_temu[1]<5) $tresc.=$godz_temu[1].' minuty, ';
if ($godz_temu[1]==1) $tresc.=$godz_temu[1].' minutę, ';
if ($min_temu[0]>4) $tresc.=$min_temu[0].' sekund ';
if ($min_temu[0]>1 && $min_temu[0]<5) $tresc.=$godz_temu[0].' sekundy ';
if ($min_temu[0]==1) $tresc.=$min_temu[0].' sekundę ';
$tresc.='temu.';
?>
[PHP] pobierz, plaintext 

A tak dalej, to:
Jak rozwiążesz problem wspomniany wyżej (jeżeli to będzie dłużej niż 1 doba)?
Dlaczego nie używasz time (łatwiej)?
Czy to jest aż tak ważne aby pokazać dokładną różnicę czasu?
Czy nie lepiej napisać: Ostatnio odwiedziłeś tą stronę DD-MM-YYYY o HH:MM (formatu daty i czasu chyba nie trzeba tłumaczyć)?

Ten post edytował SzamanGN 8.01.2009, 21:51:06