Witam!

Zaczełem dzisiaj pracę z php, *już wcześniej próbowałem, ale mi nie wychodziło *

No cóż pierwszy srypt gotowy, do czego może się przydać nie wiem
A więc:
Skrypt sprawdza czy plik istnieje, jeżeli istnieje to sprawdza czy istnieją w nim jakieś informacje,..
Jeżeli nie istnieją zadne informacje w pliku, to tworzy '@dzisiejszą/aktualną' datę.

Jeżeli plik nie istniał to tworzy nowy plik z nową datą

Jestem ciekawy jak to wygłąda w oku znającego się php`owca
Czy coś robię źle, itp.

Dzięki za zainteresowanie

może się komuś przyda

[PHP] pobierz, plaintext 
<?php
 
$plik = "C:\\......\\time.txt"; #http:\\www.myhomepage.pl\costam.php
 
 
if(file_exists($plik))	{
print "plik istnieje<br>";
 
$plik = fopen("time.txt", "r");
while ($txtinfo = fscanf($plik, "%s\t%s\t%s\n")) {
   list ($data) = $txtinfo;
			}
if (empty($data))	{
print "Data Nie istnieje zostala utworzona nowa data<br>";
$pliksave = fopen('time.txt', 'a+', 1);
$text = date("d-m-y");
fwrite($pliksave, $text);
} else	{
print "istnieje data<br>";
}
fclose($plik);
} else	 {
print "plik nie istnieje<br>";
print "zostl utworzony nowy plik z nowa data<br>";
$pliksave = fopen('time.txt', 'a+', 1);
$text = date('d.m.y')."/".date('H:i:s');
#albo:
# $text="\n Używał=$HTTP_USER_AGENT "."\n IP=$REMOTE_ADDR"."\n Przybył ze strony =$HTTP_REFERER\n".date('d')."-".date('M')."-".date('Y')."/".date('h').":".date('i')."\n\n";
fwrite($pliksave, $text);
}
fclose($plik);
print "to sa informacje z pliku: $data<br>";
 
?>
[PHP] pobierz, plaintext 

p.s.: wydaje mi się, że nie wszystko jest tak jak powinno.

można na końcu dodać

[PHP] pobierz, plaintext 
<?php
 
$plik = fopen("time.txt", "r");
while ($txtinfo = fscanf($plik, "%s\t%s\t%s\n")) {
   list ($data) = $txtinfo;
print "to sa informacje z pliku: $data<br>"; # to jest print z lini 32
			}
fclose($plik);
 
?>
[PHP] pobierz, plaintext 

wtedy będzie zawsze na końcu odczytywał nowe / po zmianie informacje z pliku.

Może "prawdziwym pehapowcem" nie jestem ale coś napisze

To tak: ogólnie nie jest źle, trzeba troszkę nawyków nabrać:
1. Jeśli masz jedną instrukcję po elese, if, while itp.. to możesz napisać:

[PHP] pobierz, plaintext 
<?php
 
else	{
print "istnieje data<br>";
}
 
?>
[PHP] pobierz, plaintext 

Ale możemy prościej napisać:

[PHP] pobierz, plaintext 
<?php
 
else  print "istnieje data<br>";
 
?>
[PHP] pobierz, plaintext 

2. Możesz zamiast prinf używać echo:

[PHP] pobierz, plaintext 
<?php
 
print "Data Nie istnieje zostala utworzona nowa data<br>";
echo "Data Nie istnieje zostala utworzona nowa data<br>";
 
?>
[PHP] pobierz, plaintext 

Nie wiem dlaczego, ale tak jakoś lepiej to wygląda

3. Ponoć nieco szybciej:

[PHP] pobierz, plaintext 
<?php
 
print "to sa informacje z pliku: $data<br>";
 
?>
[PHP] pobierz, plaintext 

Jeste w ten sposób:

[PHP] pobierz, plaintext 
<?php
 
echo "to sa informacje z pliku: ".$data."<br>";
 
?>
[PHP] pobierz, plaintext 

Powodzenia

najszybciej to jest tak:

[PHP] pobierz, plaintext 
<?php
echo 'to sa informacje z pliku: '.$data.'<br>';
?>
[PHP] pobierz, plaintext 

dlatego, że jeśli masz cudzysłów, to php szuka w nim wartości zmiennej, a jeśli masz apostrof, to nie.

Po pierwsze: wciecia, ladne formatowanie kodu powoduje, ze pozniej ktos oprocz ciebie (a i tobie tez pewnie to ulatwi) bedzie w stanie to przeczytac i... zrozumiec.

Staraj sie oddzielac bloki funkcjonalne kodu pojedynczymi odstepami (enterami), dzieki temu kod stanie sie bardziej przejrzysty i blokowy.

Ktos tu radzil, ze jest lepiej nie uzywac klamr {} - radze naprawde ich uzywac zawsze - czesto sie okazuje, ze pozniej dopisujac cos do kodu zapominasz o nich i masz blad gotowy...

Ogolnie proponuje zapoznac sie z ktorymis zasadami kodowania (np. z PEARa) i sie do nich stosowac (odpowiednie wciecia, odpowiednie formatowanie kodu, odpowiednie nazywanie zmiennych/funkcji/metod). Chodzi o to, ze jesli zaczynasz to masz duuuze szanse sie dobrze nauczyc i ladnie pisac kod. Jesli od poczatku to zlejesz to pozniej bedziesz mial problemy z przystosowaniem...

Pisz komentarze! Zapoznaj sie z PHPDoc i dokumentuj swoj kod. Na poczatku jest to bardzo trudne ale znowu, jak sie teraz nauczysz to pozniej bedziesz stosowal. Pojedyncze komentarze zaczynajace sie od // tez sa mile widziane szczegolnie przed kawalkami kodu w ktorych robisz jakies czary i z ktorych bezposrednio nie wynika co tam robisz.

To moja interpretacja Twojego kodu

[PHP] pobierz, plaintext 
<?php
 
$plik = "C:\\......\\time.txt"; #http:\\www.myhomepage.pl\costam.php
 
if (file_exists($plik)) {
  print "plik istnieje<br>";
 
  $plik = fopen("time.txt", "r");
 
  while ($txtinfo = fscanf($plik, "%s\t%s\t%s\n")) {
	list ($data) = $txtinfo;
  }
 
  if (empty($data))  {
	print "Data Nie istnieje zostala utworzona nowa data<br>";
 
	$pliksave = fopen('time.txt', 'a+', 1);
	$text = date("d-m-y");
	fwrite($pliksave, $text);
  } 
  else  {
	print "istnieje data<br>";
  }
 
  fclose($plik);
} 
else {
  print "plik nie istnieje<br>";
  print "zostl utworzony nowy plik z nowa data<br>";
 
  $pliksave = fopen('time.txt', 'a+', 1);
  $text = date('d.m.y')."/".date('H:i:s');
 
  fwrite($pliksave, $text);
}
 
fclose($plik);
 
print "to sa informacje z pliku: $data<br>";
 
?>
[PHP] pobierz, plaintext 

Zauwaz, ze dzieki odpowiednim wcieciom dokladnie widac co jest w czym zagniezdzone - to bardzo pomaga. Nawet wiecej niz bardzo.

Aha, ja bym uzyl funkcji file_put_contents() i file_get_contents() i moze zrobil to troche inaczej (prosciej?) ale jestem juz troche zmeczony. To jeszcze jedna rada - upraszczaj swoj kod! Czesty blad poczatkujacych programistow to znaczne utrudnianie sobie zycia (oczywiscie wynika to tez z nieznajomosci wszystkich mozliwosci jezyka) ale staraj sie upraszczac. Pomysl teraz nad tym co napisales, przejrzyj manuala od php i sprobuj to napisac jeszcze prosciej. Powodzenia.

Dzięki

wszystko się przyda, nie wiem czemu alen stworzenie tego skryptu sprawiło mi wielką przyjemność, już wiem dlaczego większość programistów chodzi wesołych

poprawiłem troszeczkę ten skrypt i wyszło:

[PHP] pobierz, plaintext 
<?php
 
$plikb = 'C:\\.....\\time.txt';
 
if (file_exists($plikb)) {
  print 'plik istnieje<br>';
 
	$cojestwpliku = substr(file_get_contents('time.txt'),0,-1);
 
// sprawdza czy plik jest pusty, jezeli tak tworzy nowa date,. jezeli jest jusz da
ta nic nie robi.
  if (empty($cojestwpliku))  {
	print 'Data Nie istnieje zostala utworzona nowa data<br>';
 
	$text = date('d-m-y');
 
	file_put_contents($plikb, $text);
   }
  else  {
	print 'istnieje data<br>';
  }
 
}
//jezeli plik nie istnial, tworzy nowy z data
else {
  print 'plik nie istnieje<br>';
  print 'zostl utworzony nowy plik z nowa data<br>';
 
  $text = date('d.m.y').'/'.date('H:i:s');
 
  file_put_contents($plikb, $text);
 
}
 
 
print 'to sa informacje z pliku: '.$cojestwpliku.'<br>';
 
?>
[PHP] pobierz, plaintext 

najwięcej w tworzeniu tego skryptu problemów sprawiło mi sprawdzanie czy w pliku są jakieś informacje, dopiero po długotrwałych poszukiwaniach, testeach itd. wpadłem na "empty".
Ale teraz jeżeli mam 2 wartości w pliku:
w pierwszej lini jest wartość:
a=2 | 13:43:01
w drugim
jakis tam tekst


to jak zrobić żeby plik sprawdzał czy istnieje wartość a = 2 i data ?
substr (0,14) ?....


teraz mogę zrobić ten skrypt jako funkcję:

[PHP] pobierz, plaintext 
<?php
 
function datecheck($plikb)	 {
//.. tu wkleić skrypt
}
 
// mogę teraz wywołać ten skrypt tak:
$plikb = 'C:\\.....\\time.txt';
 
datecheck($plikb);
 
?>
[PHP] pobierz, plaintext 

p.s.: jak mam ustawić php.ini żeby był zgodny z ustawieniami na serwerach www, i żeby był bezpieczny, tzn. co jeszcze oprócz registry_globals off
na forum niczego nie znalazłem :/ dzięki jeszcze raz za wszelką pomoc