Pierwszy skrypt :), Odczyt/ zapis do pliku Opcje

[chou]

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.

Ten post edytował chou 13.02.2006, 23:00:09

[TomASS]

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

[Neojawor]

3. Ponoć nieco szybciej:

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.

[dooshek]

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.

[chou]

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

Ten post edytował chou 14.02.2006, 14:08:09