[PHP] pobierz, plaintext 
<?php
$ip = $REMOTE_ADDR;
$data = date('Y-m-d H:i:s');
$zapytanie = "INSERT INTO cms_gbook SET data='$data',nick='$nick',email='$email',www='$www',wpis='$wpis',ip='$ip'";
 
if ($nick==''||$email==''||$wpis=='') {
echo ("Nie wypełniles wszystkich pól");
echo ('<form action="dodaj.php" method="post">');
if ($nick=='') {echo ('Imie/Nick:<br>
<input type="text" size="20" name="nick"><br>');} else echo ('Imie/Nick:<br>
<input type="text" size="20" name="nick" value="'.$nick.'"><br>');
if ($email=='') {echo ('Adres e-mail:<br>
<input type="text" size="20" name="email"><br>');} else echo ('Adres e-mail:<br>
<input type="text" size="20" name="email" value="'.$email.'"><br>');
if ($wpis=='') {echo ('Twój wpis:<br><textarea cols="40" rows="10" name="wpis" ENCTYPE="text/plain" wrap="virtual"></textarea><br>');} else echo ('Twój wpis:<br><textarea cols="40" rows="10" name="wpis" ENCTYPE="text/plain" wrap="virtual">'.$wpis.'</textarea><br>');
echo ('<input type="submit" name="send" value="Wyslij"><input type="reset" value="Wyczysc">
</form>');
} else if (mysql_query ($zapytanie)) {echo ("Dane zostały dodane");} else echo mysql_error();
?>
[PHP] pobierz, plaintext 

Bazy danych nie bede dodawał i formularza jest to plik wykonawczy dodaj.php wyswietlania tez nie daje bo moim zdaniem on jest nie wazny to jest moj pierwszy skrypcik wiec prosze szczere oceny php ucze sie od 2 dni wiec chyba niezle jak na poczatek acha na poczatku niema mysql_connect bo to mam w innym pliku includowane na poczatku ale mysle ze i to wam nie jest potrzbne .

PS. W kodzie na forum sa slashe a ja ich nei dawałem wiec nie zwracac na nie uwagi

Jak na dwa dni nauki, to nieźle, że zabrałeś się za MySQL. Ja go ruszyłem chyba po miesiącu Zacznij formatować kod, jest trochę nieczytelny, mimo, że tak go mało.

tzn na jakiej zasadzie mam go formatowac bo nie czaje ?

1.
Co to jest:

[PHP] pobierz, plaintext 
<?php
$ip = $REMOTE_ADDR;
?>
[PHP] pobierz, plaintext 

lepiej dać:

[PHP] pobierz, plaintext 
<?php
$ip = $_SERVER['REMOTE_ADDR'];
?>
[PHP] pobierz, plaintext 

2.
Przerobiłem troszkę zapytanie:

[PHP] pobierz, plaintext 
<?php
$zapytanie = "INSERT INTO cms_gbook SET data='$data', nick='$nick', email='$email', www='$www',wpis='$wpis',ip='$ip'";
?>
[PHP] pobierz, plaintext 

na:

[PHP] pobierz, plaintext 
<?php
$zapytanie = "INSERT INTO cms_gbook (data, nick, email, www, wpis, ip) VALUES ('$data', '$nick', '$email', '$www', '$wpis', '$ip')";
?>
[PHP] pobierz, plaintext 

INSERT INTO syntax
zamiast date pobierać z php możesz użyć MySQL NOW();
czyli:

[PHP] pobierz, plaintext 
<?php
$zapytanie = "INSERT INTO cms_gbook (data, nick, email, www, wpis, ip) VALUES (NOW(), '$nick', '$email', '$www', '$wpis', '$ip')";
?>
[PHP] pobierz, plaintext 

pod warunkiem, że pole data jest typu date lub datetime.
3.
zamiast sprawdzać $email=='' lepiej użyć empty();

[PHP] pobierz, plaintext 
<?php
$ip = $_SERVER['REMOTE_ADDR'];
$data = date('Y-m-d H:i:s');
$zapytanie = "INSERT INTO cms_gbook (data, nick, email, www, wpis, ip) VALUES (NOW(), '$nick', '$email', '$www', '$wpis', '$ip')";
 
if (empty($nick) OR empty($email) OR empty($wpis)){
	echo 'Nie wypełniles wszystkich pól';
	echo '<form action="dodaj.php" method="post">';
	if (empty($nick)) echo 'Imie/Nick:<br><input type="text" size="20" name="nick"><br>';
	else echo 'Imie/Nick:<br><input type="text" size="20" name="nick" value="'.$nick.'"><br>';
 
	if (empty($email)) echo 'Adres e-mail:<br><input type="text" size="20" name="email"><br>';
	else echo 'Adres e-mail:<br><input type="text" size="20" name="email" value="'.$email.'"><br>';
 
	if (empty($wpis)) echo 'Twój wpis:<br><textarea cols="40" rows="10" name="wpis" ENCTYPE="text/plain" wrap="virtual"></textarea><br>';
	else echo 'Twój wpis:<br><textarea cols="40" rows="10" name="wpis" ENCTYPE="text/plain" wrap="virtual">'.$wpis.'</textarea><br><input type="submit" name="send" value="Wyslij"><input type="reset" value="Wyczysc"></form>';
} 
else if (mysql_query ($zapytanie)) echo ("Dane zostały dodane");
else echo $zapytanie.'<br/>'.mysql_error();
?>
[PHP] pobierz, plaintext 

Jak na pierwszy raz jest dobrze Tak rzymaj

http://wortal.php.pl/wortal/artykuly/pomys...dardy_kodowania

TomASS dzieki za poprawe i rade będe pamietał przy nastpnych skryptach i poprawie obecna ksiege .

A tobie Null dzieki juz kapuje o co chodzi z tym formatowaniem

Przenoszę z Przedszkola na Oceny

[PHP] pobierz, plaintext 
<?php
$zapytanie = "INSERT INTO cms_gbook (data, nick, email, www, wpis, ip) VALUES (NOW(), '$nick', '$email', '$www', '$wpis', '$ip')";
?>
[PHP] pobierz, plaintext 

TomASS : używamy INTO, a nie INOT.
dawhol : nieźle jak na pierwszy skrypt, ja po dwóch dniach nauki. Głowiłem się dlaczego te skrypty na plikach mi nie działają. Jednakże dzięki pomocy manuala udało mi się to naprawić. I co do sprawdzania możesz też użyć:

[PHP] pobierz, plaintext 
<?php
if($nick == null or $email == null or $wpis == null)
?>
[PHP] pobierz, plaintext 

I pisz też nieco czytelniej.
Np. zamiast tak :

[PHP] pobierz, plaintext 
<?php
else if (mysql_query ($zapytanie)) {echo ("Dane zostały dodane");} else echo mysql_error();
?>
[PHP] pobierz, plaintext 

Napisz to tak :

[PHP] pobierz, plaintext 
<?php
else
{
if(mysql_query($zapytanie))
{
echo 'Dane zostały dodane';
}
else
{
echo mysql_error();
}
}
?>
[PHP] pobierz, plaintext 

Wygląda to teraz dużo czytelniej i łatwiej nam się w tym połapać.
Jeszcze po za tym łącz ciągi, gdyż php nie przetworzy tekstu o długości np. 500 linijek. Zdażyło mi się taki ogromny formularz wpisywać w php. Wogóle też używaj apostrofów(' ') zamiast znaków cytatu(" "), jeśli nie masz parsowania zmiennych, a to parsowanie sprawdza się tylko na bardzo małą skalę.

DZIEKI ZA OCENY powprowadzałem porawki wykorzystałem funkcje empty() =) no i poprawiłem estetyke kodu
Jescze raz dzieki za oceny, rady i nauki

Poczytaj jeszcze to:
Temat: Bezpieczenstwo skryptow php
Temat: SQL Injection Insertion

Dzieki Set =) juz co nieco poczytałem choc wiele tetgo jest ale bede pamiętał

A no i najważniejsze:

nie $nick a $_POST['nick'] jeżeli wysyłasz formularzem - metoda post

u siebie mam juz to poprawione ale dzieki za troske

gdy register_globals = on może w taki sposób uzyskać bezpośredni dostęp do $_POST['nick'], ale naturalnie najlepiej robić zawsze tak: $nick = $_POST['nick']