[php]Losowy błąd generowania tabelki fwrite Opcje

[Moris299]

Sprawa wygląda tak:
mam kod który pobiera od użytkownika ip, host i referera a następnie umieszcza w tabelce
kod tabelki jednak lubi się w losowym momencie wykrzyaczyć, o tak:
http://i.imgur.com/0hzefw9.png

tzn kolejne tabele zaczynają wchodzić w tabelę wcześniej
http://i.imgur.com/RbDPkLN.png

i tak po kilkuset wejściach potrafi się umieścić kilka tabel w jednej tabeli i wszystko się rozlatuje
za pierwszym razem popsuło się po 42 wpisach a za drugim po 83

skąd to się bierze? czemu tak się dzieje?

[PHP] pobierz, plaintext 
<?php
//pozyskanie informacji z $_SERVER i załadowanie ich do zmiennych
date_default_timezone_set('Europe/Warsaw');
$agent = $_SERVER['HTTP_USER_AGENT'] . "\n\n";
$ip = $_SERVER['REMOTE_ADDR'] . "\n\n";
$host = gethostbyaddr($_SERVER['REMOTE_ADDR']);
$data = date('j-n-Y H:i:s');
$ref = $_SERVER['HTTP_REFERER'];
$ref_licz = strlen($ref);
if($ref_licz == 0) { 
	$ref = 'brak informacji o HTTP referer';
}
$br = '<br />';
 
 
 
// wpis
// przypisanie zmniennej $file nazwy pliku 
$file = "adresy_ip.html"; 
// uchwyt pliku, otwarcie do dopisania 
$fp = fopen($file, "a+"); 
// blokada pliku do zapisu 
flock($fp, 2); 
 
// zapisanie danych do pliku
 
fwrite($fp, '<hr><table style="border: 0px solid black;"><tr><td class="ab"><b>Data i godzina:</b></td><td> ');
fwrite($fp, "$data ");
fwrite($fp, '</td></tr><tr><td class="ab"><b>User Agent:</b></td><td>');
fwrite($fp, "$agent"); 
fwrite($fp, '</tr><tr><td class="ab"><b>IP:</b></td><td>');
fwrite($fp, "$ip"); 
fwrite($fp, '</td></tr><tr><td class="ab"><b>Host:</b></td><td>');
fwrite($fp, "$host"); 
fwrite($fp, '</td></tr><tr><td class="ab"><b>Referer:</b></td><td>');
fwrite($fp, "$ref"); 
fwrite($fp, '</td></tr></table><hr>'); 
 
 
// odblokowanie pliku 
flock($fp, 3); 
 
// zamknięcie pliku 
fclose($fp); 
 
 
?>
[PHP] pobierz, plaintext 

[trueblue]

Z tego powodu, że niepoprawnie otwierasz/zamykasz znaczniki. Choćby w linii 29.
Gdybyś prześledził źródło wygenerowanej strony, zauważyłbyś.

[Moris299]

faktycznie

ale w 29 linijce nie mogę namierzyć błędu? poprawiłem za to 31

[PHP] pobierz, plaintext 
fwrite($fp, '<hr><table style="border: 0px solid black;"><tr><td class="ab"><b>Data i godzina:</b></td><td> ');
fwrite($fp, "$data ");
fwrite($fp, '</td></tr><tr><td class="ab"><b>User Agent:</b></td><td>');
fwrite($fp, "$agent"); 
fwrite($fp, '</td></tr><tr><td class="ab"><b>IP:</b></td><td>');
fwrite($fp, "$ip"); 
fwrite($fp, '</td></tr><tr><td class="ab"><b>Host:</b></td><td>');
fwrite($fp, "$host"); 
fwrite($fp, '</td></tr><tr><td class="ab"><b>Referer:</b></td><td>');
fwrite($fp, "$ref"); 
fwrite($fp, '</td></tr></table><hr>'); 
[PHP] pobierz, plaintext 

[trueblue]

Tak. Otwierałeś w linii 29 <td> bez zamknięcia w 31.

[Moris299]

Ok, dzięki

zresetowałem
adresy_ip.html

ruch na mojej stronce jest na tyle duży, że za kilka minut już będę w stanie powiedzieć czy pomogło



nadal się psuje :/

Skopiowałem kod, skasowałem wszystkie elementy php i zostawiłem sam HTML oraz dodałem wcięcia i wynik jest taki, że kod jest poprawny, Walidator też nie zwraca błędu. Czyli jednak nie w składni HTML jest problem tylko gdzieś z PHP ale nie mam pojęcie gdzie i czemu :/

[HTML] pobierz, plaintext 
<table style="border: 0px solid black;">
	<tr>
		<td class="ab">
			<b>Data i godzina:</b>
		</td>
		<td>
			$data
		</td>
	</tr>
	<tr>
		<td class="ab">
			<b>User Agent:</b>
		</td>
		<td>
			$agent
		</td>
	</tr>
	<tr>
		<td class="ab">
			<b>IP:</b>
		</td>
		<td>
			$ip
		</td>
	</tr>
	<tr>
		<td class="ab">
			<b>Host:</b>
		</td>
		<td>
			$host
		</td>
	</tr>
	<tr>
		<td class="ab">
			<b>Referer:</b>
		</td>
		<td>
			$ref
		</td>
	</tr>
</table> 
[HTML] pobierz, plaintext 

[Damonsson]

Po prostu lock puszcza Ci pewnie gdzieś w połowie. Daj sobie IFa czy jest lock/unlock i wtedy ładuj dane wewnątrz IFa.

[PHP] pobierz, plaintext 
if (flock($fp, LOCK_EX)) {  
    fwrite($fp, '<hr><table style="border: 0px solid black;"><tr><td class="ab"><b>Data i godzina:</b></td><td> ');
    fwrite($fp, "$data ");
    fwrite($fp, '</td></tr><tr><td class="ab"><b>User Agent:</b></td><td>');
    fwrite($fp, "$agent"); 
    fwrite($fp, '</tr><tr><td class="ab"><b>IP:</b></td><td>');
    fwrite($fp, "$ip"); 
    fwrite($fp, '</td></tr><tr><td class="ab"><b>Host:</b></td><td>');
    fwrite($fp, "$host"); 
    fwrite($fp, '</td></tr><tr><td class="ab"><b>Referer:</b></td><td>');
    fwrite($fp, "$ref"); 
    fwrite($fp, '</td></tr></table><hr>'); 
    flock($fp, LOCK_UN);
}
 
[PHP] pobierz, plaintext 

Ten post edytował Damonsson 18.01.2017, 01:19:00

[Moris299]

to może zamiast ifa przed każdym fwrite dać lepiej locka przed każdym fwritem?
jest jakaś zasada, że lock bedzie trzymał tylko przez ileś tam pleceń?

Dodane 19:51 18-01-2017:

dobra najwyżej przerobię to tak by wpisywało do mysqla zamiast do pliku

[Pyton_000]

A możesz mi powiedzieć po co te kombinacje z fwrite?

Można dużo prościej:

[PHP] pobierz, plaintext 
<?php
date_default_timezone_set('Europe/Warsaw');
$agent = $_SERVER['HTTP_USER_AGENT'] . "\n\n";
$ip = $_SERVER['REMOTE_ADDR'] . "\n\n";
$host = gethostbyaddr($_SERVER['REMOTE_ADDR']);
$data = date('j-n-Y H:i:s');
$ref = $_SERVER['HTTP_REFERER'];
$ref_licz = strlen($ref);
if($ref_licz == 0) {
    $ref = 'brak informacji o HTTP referer';
}
$br = '<br />';
 
 
$table = "<hr><table style=\"border: 0px solid black;\"><tr><td class=\"ab\"><b>Data i godzina:</b></td><td> 
$data 
</td></tr><tr><td class=\"ab\"><b>User Agent:</b></td><td>
$agent
</tr><tr><td class=\"ab\"><b>IP:</b></td><td>
$ip
</td></tr><tr><td class=\"ab\"><b>Host:</b></td><td>
$host
</td></tr><tr><td class=\"ab\"><b>Referer:</b></td><td>
$ref
</td></tr></table><hr>";
 
file_put_contents('adresy_ip.html', $table, FILE_APPEND | LOCK_EX);
 
[PHP] pobierz, plaintext 

[Moris299]

pomogło, dzięki