Forum PHP.pl

Witam.
Mam napisany prosty skrypt do pokazywania stron.
Skrypt zakończony jest funkcją updatującą statystyki.

końcówka skryptu wygląda tak:

[PHP] pobierz, plaintext 
	$view = new Core_Template($data);
	echo  $view->getView();
	Core_Statistics::update($router);
[PHP] pobierz, plaintext 

W czym problem:
Odpaliłem phpmyadmin z tabelką statystyk i robię F5 skryptu mojej strony -> wynik statystyki zwiększają się o dwa.
Jeśli zakomentuję 'echo $view->getView();' statystyki zwiększają się o jeden (czyli tak jak powinno być).
Metoda getView() wygląda tak:

[PHP] pobierz, plaintext 
  public function getView() {
	return $this->res;
  }
[PHP] pobierz, plaintext 

Problem: dlaczego zwykłe wyświetlenie $this->res powoduje że skrypt zachowuje się jakby wykonywał się dwa razy (IMG:style_emoticons/default/questionmark.gif) ?
Gdzie można szukać rozwiązania bo spotykam się z tym po raz pierwszy.

pozdrawiam
Grzegorz

Pokaż jeszcze skąd się bierze $router i co się dzieje w Core_Statistics::update

[PHP] pobierz, plaintext 
static function update($router) {
 
		global $db;
		$data = date("Y-m-d");
		$ip = $_SERVER['REMOTE_ADDR'];
 
		$sql = "INSERT INTO statisctics_details VALUES('', :time, :mod, :ip)";
		$q = $db->prepare($sql);
		$q->bindValue(':time', time(), PDO::PARAM_STR);
		$q->bindValue(':mod', $router->mod, PDO::PARAM_STR);
		$q->bindValue(':ip', $ip, PDO::PARAM_STR);
		$q->execute();
 
		$tab = '';
		$sql = "SELECT * FROM statisctics_views WHERE data= :data";
		$q = $db->prepare($sql);
		$q->bindValue(':data', $data, PDO::PARAM_STR);
		$q->execute();
		$tab = $q->fetchAll(PDO::FETCH_ASSOC);
 
		if (count($tab)==0) {
			$sql = "INSERT INTO statisctics_views VALUES('', :data, '1', :ip)";
			$q = $db->prepare($sql);
			$q->bindValue(':data', $data, PDO::PARAM_STR);
			$q->bindValue(':ip', $ip, PDO::PARAM_STR);
			$q->execute();
		} else {
 
			$id = 0;
			$ogladany = 0;
			foreach ($tab as $row) {
				if ($row['ip']==$ip) {
					$id = (int)$row['id'];
					$ogladany = (int)$row['ogladany'];
				}
			}
 
			if ($id!=0) {
				$ogladany++;
				$sql = "UPDATE statisctics_views SET ogladany = :ogladany WHERE id = :id";
				$q = $db->prepare($sql);
				$q->bindValue(':id', $id, PDO::PARAM_INT);
				$q->bindValue(':ogladany', $ogladany, PDO::PARAM_INT);
				$q->execute();
			} else {
				$sql = "INSERT INTO statisctics_views VALUES('', :data, '1', :ip)";
				$q = $db->prepare($sql);
				$q->bindValue(':data', $data, PDO::PARAM_STR);
				$q->bindValue(':ip', $ip, PDO::PARAM_STR);
				$q->execute();
			}
 
		}
  }
[PHP] pobierz, plaintext 

A jesli chodzi o router to poprostu klasa rozkladajaca url na odpowiedni modul i jego parametry :

[PHP] pobierz, plaintext 
public function __construct() {
		$this->url = substr($_SERVER['REQUEST_URI'], 1);
		$this->method =  $_SERVER['REQUEST_METHOD'];
 
		$args = array();
		$routes = array();
 
		$routes['mod_mainpage'] = array('method'=>'GET', 'type'=>'literal', 'action'=>'/^(,[a-z]{2})?$/', 'args'=>array('1'=>'lang'), 'langswitch'=>array('/,[a-z]{2}$/', ',{LANG}'));
		$routes['mod_page'] = array('method'=>'GET', 'type'=>'literal', 'action'=>'/^[a-zA-Z0-9\-\_]+(,[a-z]{2})?.htm$/', 'args'=>array('1'=>'lang'), 'langswitch'=>array('/(,[a-z]{2})?.htm$/', ',{LANG}.htm'));
		$routes['mod_content'] = array('method'=>'GET', 'type'=>'segment', 'action'=>'/^[a-zA-Z0-9\-\_]+\/[a-zA-Z0-9\-\_]+(,[0-9]+)(,[0-9]+)(,[a-z]{2})?.htm$/', 'args'=>array('1'=>'catid', '2'=>'newsid', '3'=>'lang'), 'langswitch'=>array('/(,[a-z]{2})?.htm$/', ',{LANG}.htm'));
		$routes['mod_category'] = array('method'=>'GET', 'type'=>'segment', 'action'=>'/^[a-zA-Z0-9\-\_]+(,[0-9]+)(,[0-9]+)(,[a-z]{2})?\/$/', 'args'=>array('1'=>'catid', '2'=>'page', '3'=>'lang'), 'langswitch'=>array('/(,[a-z]{2})?\/$/', ',{LANG}/'));
		$routes['mod_contact'] = array('method'=>'GET', 'type'=>'literal', 'action'=>'/^contact(,[a-z]{2})?.htm$/', 'args'=>array('1'=>'lang'), 'langswitch'=>array('/(,[a-z]{2})?.htm$/', ',{LANG}.htm'));
		$routes['mod_sitemap'] = array('method'=>'GET', 'type'=>'literal', 'action'=>'/^sitemap(,[a-z]{2})?.htm$/', 'args'=>array('1'=>'lang'), 'langswitch'=>array('/(,[a-z]{2})?.htm$/', ',{LANG}.htm'));
 
		if ($this->url!='') {
			foreach ($routes as $mod => $data) {
				if ($this->method==$data['method']) {
 
					if ($data['action']!='') {
						if (preg_match($data['action'], $this->url, $results)) {
 
							$this->mod = $mod;
 
							if (isset($data['args'])) {
								$args = array();
								foreach ($data['args'] as $key=>$val) {
									if (isset($results[$key])) {
										$args[$val] = substr($results[$key], 1);
									}
 
								}
								if (isset($args['lang'])) {
									$this->lang = $args['lang'];
								}
 
								$this->args = $args;
							}
 
							if ($this->url!='') {
								$this->langpattern = preg_replace($data['langswitch'][0], $data['langswitch'][1], $this->url);
							}
						}
					}
				}
			}
		} else {
			$this->langpattern = ',{LANG}';
		}
	}
[PHP] pobierz, plaintext 

a czy np zła konstrukcja np .htaccessa może powodować takie ponowne wykonanie tego samego skryptu ? (np wystąpił jakiś błąd i następuje przeładowanie ?)

Dzięki nospor - znalazłem.
Faktycznie, jeśli brakuje zmiennej to PHP wywala info natomiast jeśli w trakcie generowania strony dynamicznie liknowane są pliki np js to już trudniej to wyszukać.
Ciekawostką jest to, że jak miałem źródło strony

[HTML] pobierz, plaintext 
<script type="text/javascript" src="/js/mod_page.js"></script>
[HTML] pobierz, plaintext 

a pliku 'mod_page.js' fizycznie nie było na dysku to po kliknięciu w link '/js/mod_page.js' nie dostawałem 404 tylko ponownie tą samą stronę (dlatego statystyki zliczały się x2).
ciekawe...

Ten post edytował elmozaur 5.04.2016, 21:52:15