Witam mam problem z tym kodem. Po odświeżeniu strony dodają się do bazy puste rekordy, przycisk działa ale coś się wykonuje samoczynnie i nie wiem gdzie jest bład.

Musisz jakoś (if isset POST) sprawdzić, czy został kliknięty przycisk i dopiero wtedy wewnątrz tej zależności wysyłać zapytanie MySQL.

Też tak myślałem ale to nie działa...

Przecież ty nadal wykonujesz zapytanie niezależnie od tego czy POST poszedł czy nie.... Przeczytaj jeszcze raz UWAŻNIE post Damonsson

A do kodu PHP używaj BBOCODE PHP a nie CODE.
I na przyszłość tytułuj tematy jak człowiek a nie jak dwuletnie dziecko
Temat: Jak poprawnie zatytulowac watek

Prosty if i cieszy

[PHP] pobierz, plaintext 
<?php
 
if($_POST['add'])
{
  $add = $_POST['add'];  
  $con = mysql_connect("localhost", "user", "pass");
  if (!$con)
  {
     die('Nie mozna połączyć:' . mysql_error());
  }
  mysql_select_db("baza",$con);
  $sql = nl2br("INSERT INTO tabela SET id='null', arty='$add'");
  if(!mysql_query($sql,$con))
  {
     die('Bład: ' . mysql_error());
  }
     echo "Dodany";
  $_POST['add'] = null;
}
mysql_close($con);
?>
<form action="admin.php" method="POST">
   <textarea rows="10" cols="30" name="add"></textarea><br>
   <input type="submit" value="Dodaj">
</form>
[PHP] pobierz, plaintext 

Dzięki prz3kus teraz działa.

To że działa nie znaczy że jest dobry. Gdzie sprawdzasz czy POST istnieje, czy został wysłany? Po co chcesz zakończyć połączenie w wypadku jego "braku"? Już nie wspominając że w ogóle tu niczego nie filtrujesz.

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['add'])){   
$add = addslashes(htmlspecialchars($_POST['add']));
}
if($_POST['add']){
$add = $_POST['add'];
$con = mysql_connect("localhost", "user", "root");
if (!$con)
{
die('Nie mozna połączyć:' . mysql_error());
}
mysql_select_db("baza",$con);
$sql = nl2br("INSERT INTO tabela SET id='null', arty='$add'");
if(!mysql_query($sql,$con))
{
die('Bład: ' . mysql_error());
}
echo "Dodany";
$_POST['add'] = null;
}
mysql_close($con);
?>
[PHP] pobierz, plaintext 

To fajnie, że sprawdzasz czy istnieje indeks add w tablict $_POST za pomocą isset($_POST['add']), ale pare linijek dalej nadal korzystasz z zapisu if($_POST['add']), co w Twoim przypadku nie jest prawidłowe.

Poprawione.

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['add']))
{   
$add = addslashes(htmlspecialchars($_POST['add']));
$con = mysql_connect("localhost", "user", "pass");
if (!$con)
{
    die('Nie mozna połączyć:' . mysql_error());
}
mysql_select_db("baza",$con);
$sql = nl2br("INSERT INTO tabela SET id='null', arty='$add'");
if(!mysql_query($sql,$con))
{
    die('Bład: ' . mysql_error());
}
    echo "Dodany";
$_POST['add'] = null;
}
mysql_close($con);
?>
[PHP] pobierz, plaintext 

1. nl2br - wiesz co robi ta funkcja ?
2. po co robisz to : $_POST['add'] = null; ?
3. zauważ, że gdy warunek

[PHP] pobierz, plaintext 
if(isset($_POST['add']))
[PHP] pobierz, plaintext 

nie jest spełniony, Twój kod wykonuje tą instrukcję :

[PHP] pobierz, plaintext 
mysql_close($con);
[PHP] pobierz, plaintext 

ale nie wykonuje tej instrukcji :

[PHP] pobierz, plaintext 
$con = mysql_connect("localhost", "user", "pass");
[PHP] pobierz, plaintext 

Dodaje mi <br /> po naciśnięciu enter.

Zapomniałem to wywalić późno jest ;D.

Hmm nie rozumiem dla czego to jest źle...Jeżeli nic nie dodaje chyba niema potrzeby łączyć sie z bazą

1.
OK, rozumiem już po co dodałeś tą funkcję. Ale lepiej by było jakbyś użył jej na zmiennej add, a nie całym zapytaniu.
3. Jeżeli nic nie dodajesz, to nie ma potrzeby łączyć się z bazą... a jeżeli się nie połączyłeś z bazą to nie ma potrzeby się z nią rozłączać. Błąd polega na tym, że próbujesz zamknąć drzwi których nie otworzyłeś.

Chyba zaczynam rozumieć oczywiście moja pomyłka wziąłem to

[PHP] pobierz, plaintext 
mysql_close($con);
[PHP] pobierz, plaintext 

za to

[PHP] pobierz, plaintext 
or die(mysql_error());
[PHP] pobierz, plaintext 

Zastanawiam się jeszcze czy takie filtrowanie jest wystarczające...

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['add'])){   
$add = addslashes(htmlspecialchars($_POST['add']));
$con = mysql_connect("localhost", "user", "pass");
if (!$con)
{
    die('Nie mozna połączyć:' . mysql_error());
}
mysql_select_db("baza",$con);
$sql = "INSERT INTO tabela SET id='null', arty='$add'";
if(!mysql_query($sql,$con))
{
    die('Bład:' . mysql_error());
}
    echo "Dodany";
}
?>
[PHP] pobierz, plaintext 

Powiedzmy. Sprawdź jeszcze na tej samej zasadzie czy formularz został wysłany, input/button musi mieć name. Zainteresuj się PDO zamiast mysql_* bo za jakiś czas wylecą z PHP. Aha no i teraz nie kończysz połączenia. dodaj też trim()

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['submit'])&& $_POST['add'] or mysql_close($con)){  
$add = trim(addslashes(htmlspecialchars($_POST['add'])));
$con = mysql_connect("localhost", "user", "pass");
if (!$con)
{
    die('Nie mozna połączyć:' . mysql_error());
}
mysql_select_db("baza",$con);
$sql = nl2br("INSERT INTO tabela SET id='null', arty='$add'");
if(!mysql_query($sql,$con))
{
    die('Bład:' . mysql_error());
}
    echo "Dodany";
}
?>
<form action="admin.php" method="POST">
<textarea rows="10" cols="30" name="add"></textarea><br>
<input type="submit" value="Dodaj" name="">
<input type='hidden' name='submit' />
</form>
[PHP] pobierz, plaintext 

Juz mieli takie pomysły w perl6 żeby ułatwić prace i ten język jest martwy ludzie nie lubią zmian więc nigdy nie wiadomo

W ogóle nie znasz podstaw PHP i zasad działania funkcji.

W PHP bawie sie 3 dzień wiec sie zgodze, natomiast ty masz 6 lat doświadczenia na forum i wciąż masz problem z czytaniem ze zrozumieniem. Pytanie
Twoja odpowiedź niepotrzebny tekst który otwiera nowy temat o którym nie mam pojęcia po 3 dniach nauki później wstawiam kod zrobiony w 5min na kolanie i twoja odpowiedź nabijaj posty dalej dobiesz do 3k przed końcem swiata pozdro .

A ten przykład, zaczerpnąłeś z książki z tutoriala jakiegoś? Jeśli tak to przestań to czytać i sięgnij do innej lektury. W ogóle wiele if'ów jest z będnych w twoim kodzie np.
zamiast:

[PHP] pobierz, plaintext 
$con = mysql_connect("localhost", "user", "pass");
if (!$con)
{
die('Nie mozna połączyć:' . mysql_error());
}
[PHP] pobierz, plaintext 

można użyć:

[PHP] pobierz, plaintext 
$con = mysql_connect("localhost", "user", "pass") or die ('Nie mozna połączyć:' . mysql_error());
[PHP] pobierz, plaintext 

W ogóle te nl2br do niczego nie jest tam potrzebne-wywal to wraz z nawiasami-to ma być zwykły string.

Dodaje mi <br /> po naciśnięciu enter.

To jest polecenie do bazy danych, przecież tego nie wyświetlasz.Jak już chcesz mieć znaki podziału z textarea to powinno być:

[PHP] pobierz, plaintext 
$sql = "INSERT INTO tabela SET id='null', arty=".nl2br($_POST['add']);
[PHP] pobierz, plaintext 

MSPM5 - niah, bulwersik?

Jeśli już coś robisz, to rób to dobrze, albo wcale. 5 minut na kolanie nie wystarczy. Moja odpowiedź była adekwatna do tego o co pytałeś. Nie poprawiłeś błędów o które już Ci ktoś wyżej wspomniał, zbudowałeś warunki które w ogóle nie mają sensu.
Przeczytaj pierw jakiś kurs (wikibooks), poznaj zasady działania PHP, a następnie przemyśl to jak Twój skrypt powinien działać.

Do czego Ci te pole ukryte i pusta wartość name? Sprwdź też typ danych, utwórz połączenie wtedy gdy warunki zostną spełnione, tak samo z zakończeniem.


Perl jest martwy? Skoro wiesz lepiej.

Coś tam jeszcze żyje.Dalej mnie zastanawia czy to język webowy czy okienkowy.Próbowałem się pobawić przy wdrożeniu jednej aplikacji- dla mnie to magiel i czarna magia więc sobie darowałem- taki offtop z mojej strony.

Heh, fajnie. Jakieś 10 lat temu też próbowałem się pobawić pewnym językiem. I dałem sobie spokój, bo to była dla mnie czarna magia. Tym językiem był Javascript. O którym dziś nie powiedziałbym, że "coś tam jeszcze żyje". Tego samego nie powiem ani o C ani (na przykład) o fortranie. Mnogość pakietów (i istniejących rozwiązań) w tych językach jest naprawdę poważnym argumentem za tym, by nie mówić "coś jeszcze żyje".

Mowilem o Perl6 ,ze jest martwy moim zdaniem nic sie nie dzieje wszyscy dalej siedza w 5.10 ogolnie perl zostal zepchniety z web przez PHP a do aplikacji okienkowych sa lepsze narzedzia. Teraz jest niezastapiony w pracy administratora generowanie statystyk, automatyzacja itd Perl nie jest martwy ale nie jest modny bo to jak napisal Niktoś magiel i czarna magia. Nie zbudowalem blad jest w tym ,ze przerobilem jakis przyklad wziety z jakiegos kursu zeby dzialal pozniej wy cos zaczeliscie podpowiadac i zaczalem niektore linijki wymieniac i zrobil sie magiel ;D gdybym pisal to od zera to by tego nie bylo a ten kod ktos napisal zmienilem pare rzeczy czegos niezrozumialem i dla tego tak to wyglada. Pusta wartosc name to przypadek napisales ,ze input/button musi mieć name mialem dopisac name a robilem cos innego jeszcze i pomieszaly mi sie pliki.

Język nie ma być modny, tylko opłacalny, przejrzysty, skalowalny, spełniający założenia programisty PHP6 też jest w powijakach, ale nie powiedziałbym że jest martwy. A magiel? czyli znaczy się burdel? PHP ma tak od zawsze.

Ja próbowałem uruchomić perla poprzez fastCGI na IIS zajęło mi to dwa dni aby pojawił się napis is work na białej stronie.Dwa dni mi to zajęło, gdyż błędy wyświetlenia tylko napisów było spowodowane tym, że brakowało mi szeregu modułów, które musiałem ręcznie doinstalowywać w CPAN'ie -wybacz ale to tylko napisy, więc takie coś chyba powinno być obsługiwane standardowo przy instalacji perla. No właśnie skusiłem się na perla bo o statystyki mi chodziło, a wyszukałem sobie programik godny uwagi -cholewcia wie ile modułów musiałbym zainstalować aby go uruchomić,więc sobie darowałem.
Godnym zainteresowania w tym języku programowania jest studio programistyczne ActivePearl -niestety o dziwo płatne, zważywszy ,że tacy potentaci jak Microsoft, Java Microsystem, PHP i jego frameworki udostępniają swoje platformy za darmo.
Dlatego ten język(perl) jak i flash pójdą w zapomnienie gdyż są płatne.

MSP.... Autorze tematu. Pisałem CI, żebyś nl2br użył tylko na zmiennej add, wtedy nikt się do tego nie przyczepi

Co do filtrowania zmiennych - jest to wystarczające filtrowanie.

ActivePERL, do jasnej. "PERL" od "Practical Extraction and Report Language". Nie od Perły, a tym bardziej nie od piwa "Perła". Przepraszam, że tak ostro, ale irytuje mnie zniekształcanie nazw własnych.


Tak. A PHPStorm też jest płatny - i czego wg Ciebie to dowodzi? że nie mogę pisać w PHP? Proszę Cię, trzymajmy jakiś poziom dyskusji.
Zresztą, od kiedy istnieje ActivePerl? Bo sam Perl to już ponad 20 lat... Popatrz, jakie to dziwne: bez ActivePerla tyle czasu przetrwał? No niemożliwe, prawda?

Perl taki jest moze nie idealnie przejrzysty ale spelnia swoje zadanie od lat. Jest elastyczny robi to co sie chce a po drugiej stronie jest python ktory jest latwy, przejrzysty i ma dobra reklame a ze dzisiaj sie leci na latwizne to wiekszosc wybierze pythona bo jest teraz na topie i o to mi chodzilo chociaz jak niektorzy mowia python to nie tylko zabawka ale tez potezne narzedzie. Uzylem slowa magiel w porownaniu do tego co stworzylem i jakims cudem nie posypalo bledami ale prawdziwy magiel zaczal sie jak sprobowalem zrobic to. nie rozumiem dla czego chcesz sprawdzac typ danych w moich pseudo artykolach to nie kalkulator chyba ,ze masz cos innego na mysli.

hahah widzialem ,ze pisales i wiedzialem ze sie ktos przyczepi bylem pewien ale nie wiedzialem jak to zrobic pozatym dalej cos nie dziala jakis blad jest w tym kodzie zaraz cos z tym zrobie pewnie cudzyslow zgubiony

[PHP] pobierz, plaintext 
$sql = "INSERT INTO tabela SET id='null', arty=".nl2br($_POST['add']); 
[PHP] pobierz, plaintext 

JA tam błędu nie widzę... ale na kij filtrujesz zmienną $add, ja potem dodajesz do bazy i tak $_POST['add']

Blad jest napewno bo sie strona nie wczytuje i sypie bladami. Kod juz jest poprawiony (chyba) wkleje jak poprawie WSZYSTKO tylko nie bijcie wiem ze to wyglada strasznie Hmm w ktorym miejscu mam to filtrowac wydaje mi sie ,ze jest dobrze odfiltrowalo mi <hr> wczesniej sie robily krechy na stronie. Blad jest gdzie indziej za szybko wszystko robie jakis nawias.

To zapodaj te błędy

Jak robisz takie coś :

[PHP] pobierz, plaintext 
$add  = $_POST['add'];
$add - filtruj($add);
 
$query = "INSERT INTO ...." . $_POST['add'];
 
[PHP] pobierz, plaintext 

to dodajesz nieprzefiltrowane dane, bo przefiltrowana jest $add, a nie $_POST['add'] .

Wrzcam caly kod bo zrobil sie balagan

Tresc bledu: Parse error: syntax error, unexpected ')' in /virtual/****/admin.php on line 10

[PHP] pobierz, plaintext 
<!DOCTYPE html><html lang="pl">
<head><title>****</title><meta charset="ISO-8859-2"></head>
     <body background="black" bgproperties="fixed">
      <style type="text/css">
      body {background-color:black;color:white;}
      </style>
           <center>
             <h1>
<?php
if(isset($_POST['submit'])&& $_POST['add'])){  
$add = trim(addslashes(htmlspecialchars($_POST['add'])));
$con = mysql_connect("localhost", "user", "pass") or die ('Nie mozna połączyć:' . mysql_error());
mysql_select_db("baza",$con);
$sql = "INSERT INTO tabela SET id='null', arty=".nl2br($_POST['add']);
if(!mysql_query($sql,$con))
{
die('Bład:' . mysql_error());
}
echo "Dodany";
}
?>
<form action="admin.php" method="POST">
<textarea rows="10" cols="30" name="add"></textarea><br>
<input type="submit" value="Dodaj" name="submit">
</form></h1>
[PHP] pobierz, plaintext 

W 10 lini za dużo o jedno zamknięcie nawiasu

tak wiem juz to poprawilem ale ma inne bledy jeszcze

wpisuje jeden wyraz i wyskakuje

Bład:Unknown column 'lol' in 'field list'
wpisuje 2 wyrazy i wywala

Bład:You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '2' at line 1

Żebyś się nie męczył:

[PHP] pobierz, plaintext 
<?php
if((isset($_POST['submit']))&&(!empty($_POST['add']))){
$add = trim(addslashes(htmlspecialchars(nl2br($_POST['add']))));
$con = mysql_connect("localhost", "user", "pass") or die ('Nie mozna połączyć:' . mysql_error());
mysql_select_db("baza",$con);
$sql =mysql_query("INSERT INTO tabela SET id='null', arty='$add'",$con) or die('Bład:' . mysql_error());
 
echo "Dodany";
}
?>
[PHP] pobierz, plaintext 

zmienilem ta linijke

[PHP] pobierz, plaintext 
$sql = "INSERT INTO tabela SET id='null', arty=".nl2br($_POST['add']);
[PHP] pobierz, plaintext 

na to co bylo wczesniej i nie ma bledu

[PHP] pobierz, plaintext 
$sql = nl2br("INSERT INTO tabela SET id='null', arty='$add'");
[PHP] pobierz, plaintext 

dla czego tak sie dzieje

Wydaje mi się że brakuje apostrofów. Spróbuj czegoś takiego:

[PHP] pobierz, plaintext 
$sql = "INSERT INTO tabela SET id='null', arty='".nl2br($_POST['add'])."'";
[PHP] pobierz, plaintext 

Ew. Potestuj zapytanie bezpośrednio w bazie

Wyświetl sobie treść zapytania SQL (u Ciebie w zmiennej $sql). Z nami też możesz się podzielić

Teraz dziala.

[PHP] pobierz, plaintext 
$sql = "INSERT INTO tabela SET id='null', arty='".nl2br($_POST['add'])."'";
[PHP] pobierz, plaintext 

tresc tego zapytania

[PHP] pobierz, plaintext 
 INSERT INTO tabela SET id='null', arty='dupa'Dodany
[PHP] pobierz, plaintext 

i poprzedniego

[PHP] pobierz, plaintext 
INSERT INTO tabela SET id='null', arty=dopa2Bład:Unknown column 'dopa2' in 'field list'
[PHP] pobierz, plaintext 

Niktoś przerobiłem linijke w tym kodzie który dałeś nie zauważylem go wczśniej. Zmieniłem tak jak koxu1996 no i działa

[PHP] pobierz, plaintext 
$sql = mysql_query("INSERT INTO tabela SET id='null', arty='".nl2br($_POST['add'])."'",$con) or die('Bład:' . mysql_error());
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
if((isset($_POST['submit']))&&(!empty($_POST['add']))){
$add = trim(addslashes(htmlspecialchars($_POST['add'])));
$con = mysql_connect("localhost", "user", "pass") or die ('Nie mozna połączyć:' . mysql_error());
mysql_select_db("baza",$con);
$sql = mysql_query("INSERT INTO tabela SET id='null', arty='".nl2br($_POST['add'])."'",$con) or die('Bład:' . mysql_error()); 
echo "Dodany";
}
?>
<form action="admin.php" method="POST">
<textarea rows="10" cols="30" name="add"></textarea><br>
<input type="submit" value="Dodaj" name="submit">
</form></h1> 
[PHP] pobierz, plaintext 

Nadal jest ten błąd o którym pisałem. $add jest przefiltrowana prawidłowo, a $_POST['add'] nie jest przefiltrowana. Powineneś dodać $add do bazy.

chodzi Ci o ta linie
$sql = mysql_query("INSERT INTO tabela SET id='null', arty='".nl2br($_POST['add'])."'",$con) or die('Bład:' . mysql_error());

tak

Po coś to wymyślili. Poczytaj o sql injection.

O to chodzi ? http://php.net/manual/en/function.gettype.php

Wszytko co powinno Cie interesować z is_* w tym przypadku (art) is_string, do tego sprawdzasz długość np. mb_strlen jak spełniają warunek, przesyłasz, jak nie unset i tyle, tak samo możesz dodatkowo sprawdzać czy dane POST, to na pewno POST. http://php.net/manual/pl/book.filter.php ale to już inna bajka, są też inne opcje, ale to już byś musiał określić co chcesz wprowadzać do bazy.