[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 

[Rafal Filipek]

1)

Kod

<?php
function spr(){
    setcookie('test',1);
    return $_COOKIE['test'];
}
var_dump(spr());
?>

gwarantuje ci ze jak odpalisz ten skrypt pierwszy raz to bedziesz mial blad w return $_COOKIE['test']; bo bez przeladowania nie bedzie takiego ciastka

2) twoja funkcja w zaleznosci od warunku zwraca albo $_COOKIE albo caly komunikat, dziwne.
3) w php nie ma funkcji leży_jerzy_na_wieży() wieć staraj się też pisać swoje skrypty stosując angielskie nazwy zmiennych, funkcji itp.
4) nierozumiem czemu sobie utrudniasz tak z tymi datami i czasami. czemu nie przechowujesz timestamp cookies ? zobacz, to co twój skrypt robi od 9 do 46 linijki

Kod

<?php
$now = time();
$last = time() - rand(5000, 10000);
$time = getdate($now-$last);
if($time['hours']-1>0){
    echo 'Ostatnio byłeś tutaj '.($time['hours']-1).' godzin, '.$time['minutes'].' minut, '.$time['seconds'].' sekund temu';
}else{
    echo 'Ostatnio byłeś tutaj '.$time['minuts'].' minut, '.$time['seconds'].' sekund temu';
}
?>

tylko trzeba -1 od godzin bo liczy od 0.

Ten post edytował Rafal Filipek 8.01.2009, 20:47:58