[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 

[Maxik]

Dokładnie się nie przygladałem, ale zamiast date użyłbym unixowego znacznika czasu, prościej obliczać.

[Crozin]

Skrypt ten polegnie, gdy ostatnio byłbym wczoraj (inny dzień). Jak już wspomniano wcześniej - lepiej będzie Ci operować na dacie w formacie UNIXowym (time" title="Zobacz w manualu PHP" target="_manual).

I co to za zadanie: Ostatnio byłeś tu 6 minut temu i 12 sekund.
Chyba: Ostatnio byłeś tutaj 6 minut i 12 sekund temu (chociaż i tak lepiej zapisać 6 minut temu - po co sekundowa dokładność?)

[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

[thomson89]

Dziekuję.

[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