StrongholdNet - forum graczy Stronghold :: Zobacz temat

Tay

Mam "mały" problem przy pisaniu systemu newsów. Otóż mam podstronę z rozszerzeniem .html z takim oto formularzem:

Kod:

<html>
<body>
<form action="news.php" method="post">
<table border="0">
<tr>
<td>Tytuł newsa:</td><td><input type="text" name="tytul" size="15"></td>
</tr>
<tr>
<td>Treść newsa:</td><td> <textarea name="tresc" rows="5" cols="20"></textarea></td>
</tr>
</table>
<input type="submit" value="Wyslij"><br>

</body>

Służy ona jak widać do dodawania newsów.

Następnie jest plik news.php do którego przesyłane są dane z formularza (te div'y tak na razie tylko wstawiłem, kombinuję tam z nimi, ale nie o to chodzi):

Kod:

<html>
<head>
<link rel="stylesheet" type="text/css" href="style.css"/>
</head>
<body>
<?php
require "connection.php";
connection();
$id = $_POST['id'];
$tytul = $_POST['tytul'];
$tresc = $_POST['tresc'];
$data = $_POST['data'];
$zapytanie = "INSERT INTO newsy VALUES ('$id')";
mysql_query($zapytanie);
$zapytanie = "INSERT INTO newsy VALUES ('$tytul')";
mysql_query($zapytanie);
?>
<div id="NEWS">
<div id="TYTUL_NEWSA"><h3>$tytul</h3></div>
<div id="TRESC_NEWSA"><p>Treść newsa</p></div>
<div class="pasek4"></div>
</div>
<div id="NEWS">
<div id="TYTUL_NEWSA"><h3>Tytuł newsa</h3></div>
<div id="TRESC_NEWSA"><p>Treść newsa</p></div>
<div class="pasek4"></div>
</div>

</body>
</html>

I tutaj mam problem:

Kod:

$zapytanie = "INSERT INTO newsy VALUES ('$id')";
mysql_query($zapytanie);
$zapytanie = "INSERT INTO newsy VALUES ('$tytul')";
mysql_query($zapytanie);

Problem polega na tym, iż gdy dodaję sobie newsa w formularzu, następnie wchodzę w phpmyadmin żadnych rekordów w bazie nie ma. Zmieniałem już te wpisy na różne sposoby. Jestem baardzo zielony w php, więc proszę o pomoc. Tak samo zastanawia mnie jak "wyciągnąć" potem te newsy z bazy, aby były one widoczne na np. stronie głównej. Z góry dzięki za pomoc, mam nadzieję, że rozumiecie o co mi chodzi

Siwy

Podstawy SQL'a się kłaniają

. Po pierwsze - pokaż strukturę tabeli (eksport z PHPMyAdmina). W każdym razie do dodawania rekordów używa się jednego zapytania, a nie jak Ty robisz 2, czy więcej

.

$id zbędny - od tego sa pola id w tabeli z nadanym auto_increment (autouzupełnianiem kolejną liczbą). $data to powinien być IMO albo czas od 1 stycznia 170 (time()), albo data w formie SQLowej, czyli odpowiednio sformatowena funkcja date do postaci YYYY-MM-DDD (oczywiście dla pola w tabeli musisz nadać odpowiedni rodzaj - DATE). Inna sprawa, że w formularzu nie są tworzone zmienne $_POST['id'], $_POST['data']

.

Jeśli zaś chodzi o finalne dodanie - najlepiej sprawdzać, czy funkcja mysql_query() zwróciła true (co oznacza, że wszystko poszło ok) i też else, aby wyświetlić jaki jest problem (funkcja mysql_error()). Samo zapytanie, zależnie od tego, jak wygląda struktura tabeli, powinno mieć mniej więcej taki kształt:

Kod:

$zapytanie = "INSERT INTO newsy VALUES(NULL [pole auto_increment samo doda id], '$tytul', '$tresc', $$data'";

if(mysql_query($zapytanie))
{
echo "Poprawnie dodano newsa!";
}
else
{
echo "Wystąpił błąd".mysql_error();
}

Co się zaś tyczy samego HTMLa - do tworzenia formularzy nie używaj tabel

. Najlepiej skonstruować formularz na label'ach.

Jeśli chodzi o 'wyciąganie' newsów z bazy - mówię, to są podstawy SQL'a. Z czego się uczysz btw.?

Kod powinien mieć mniej więcej taki kształt:

Kod:

$zapytanie = "SELECT * FROM newsy ORDER BY id DESC"; // * oznacza wszystko, w innym wypadku wymieniaj pola jakie chcesz wyciagnac po przecinku

$wynik = mysql_query($wynik);
$ile = mysql_num_rows($wynik); // przeliczenie zwroconych wynikow

for($i = 0; $i < $ile; $i++)
{
$news = mysql_fetch_assoc($wynik); // ta funkcja zwraca aktualny rekord w postaci tabeli z indeksami jako nazwy pol

echo "<h1>".$news['tytul']."</h2>";
echo "<p class=\"data\">".$news['data']."</p>";
echo "<p>".$news['tresc']."</p>";
}

Tay

Siwy

Też uczę się z tej książki (tzn. teraz już często do manuala także zaglądam

). Które wydanie?

Tay

Wydanie czwarte

Odyn · Administrator Wiek: 35 Posty: 857 Medale: Brak

Tay

Zrobiłem, to co poradziliście i podmieniłem kod, jednakże po otwarciu pliku news.php jest taki oto napis:

Wystšpił błšdYou have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

Natomiast plik news.php wygląda następująco:

Kod:

<html>
<head>
<link rel="stylesheet" type="text/css" href="style.css"/>
</head>
<body>
<?php
require "connection.php";
connection();
$tytul = $_POST['tytul'];
$tresc = $_POST['tresc'];
$zapytanie = "INSERT INTO newsy VALUES (NULL, '$tytul', '$tresc'";

if(mysql_query($zapytanie))
{
echo "Poprawnie dodano newsa!";
}
else
{
echo "Wystąpił błąd".mysql_error();
}

$zapytanie = "SELECT * FROM newsy ORDER BY id DESC"; // * oznacza wszystko, w innym wypadku wymieniaj pola jakie chcesz wyciagnac po przecinku

$wynik = mysql_query($zapytanie);

while($news = mysql_fetch_assoc($wynik))
{
echo "<h1>".$news['tytul']."</h2>";
echo "<p class=\"data\">".$news['data']."</p>";
echo "<p>".$news['tresc']."</p>";
}
?>
</body>
</html>

Proszę o dalszą pomoc

Siwy

Require to funkcja. Choć błąd nie tego dotyczy. Nie zamknąłeś nawiasu za values - IMO tego dotyczy błąd. Jednak dalej nie przedstawiłeś nam struktury tabeli, a co za tym idzie wydaje mi się, że następny błąd (po poprawieniu tego) będzie w insercie - pola mogą być w innej kolejności/jest ich więcej etc.

[ Dodano: 12 Wrzesień 11, 19:19 ]
Cwaniaczek, ja mam 3 ;P. Btw. Masz tam wszystko co trzeba opisane, jeśli chodzi o to, o co teraz pytasz

. Przewertuj ją dokładnie. Co nie oznacza, że nie chcę Ci pomóc

.

Tay

Siwy

Cytat:

(...)wydaje mi się, że następny błąd (po poprawieniu tego) będzie w insercie - pola mogą być w innej kolejności/jest ich więcej etc.

Cytat:

Wystšpił błšdColumn count doesn't match value count at row 1

Miałem rację

.

Są dwa sposoby dodawania danych do bazy, pierwszy, gdy dodajemy dane do każdego pola, czyli taki jaki stosujesz teraz:

Kod:

INSERT INTO tabela VALUES(wszystkie dane po przecinku w kolejności w jakiej są pola);

Można dodawać dane tylko do określonych pół, np.

Kod:

INSERT INTO `id`, `tytul` VALUES(NULL, 'Tytuł');

Czyli po prostu dodawaj datę, jako ostatnią wartość, np.

Kod:

INSERT INTO newsy VALUES(NULL, 'Tytuł', 'Treść', 'Data');

Btw.

Cytat:

latin2_swedish_ci

wtf? xd

Cytat:

Czyli mam zastąpic cudzysłów nawiasem okrągłym?

Jeeej xd. Wartość jest przekazywana funkcji, jak słusznie mówisz, w nawiasie, jednak wewnątrz nawiasu także musi być cudzysłów

. Czyli...

Kod:

require('wzgledna_sciezka_do_pliku.php');

Tay

Dzięki, jesteś wielki, poprawiłem wszystko i działa

Jednakże... tak, ja to zawsze mam jakiś problem ;D Newsy pokazują się, ale w każdym z nich nie ma wartości z tabeli newsy, tylko wszędzie jest tak:
tytul
tresc

Dzieje się tak zapewne z powodu tego:

Kod:

echo "<h1>".$news['tytul']."</h2>";
echo "<p class=\"data\">".$news['data']."</p>";
echo "<p>".$news['tresc']."</p>";

Tylko na co mam zamienić słowa tytul i tresc, aby wyświetlał mi się tytuł i treść newsa z bazy danych?

Poniżej załączam kod z tych dwóch plików (na wszelki wypadek) oraz linki do stron:

www.tay.cba.pl/news.php

Kod:

<html>
<head>
<link rel="stylesheet" type="text/css" href="style.css"/>
</head>
<body>
<?php
require ('connection.php');
connection();
$tytul = $_POST['tytul'];
$tresc = $_POST['tresc'];
$data = $_POST['data'];
$zapytanie = "INSERT INTO newsy VALUES(NULL, '$tytul', '$tresc', '$data')";

if(mysql_query($zapytanie))
{
echo "Poprawnie dodano newsa!";
}
else
{
echo "Wystąpił błąd".mysql_error();
}

$zapytanie = "SELECT 'tytul', 'tresc' FROM newsy ORDER BY id DESC"; // * oznacza wszystko, w innym wypadku wymieniaj pola jakie chcesz wyciagnac po przecinku

$wynik = mysql_query($zapytanie);

while($news = mysql_fetch_assoc($wynik))
{
echo "<h1>".$news['tytul']."</h2>";
echo "<p class=\"data\">".$news['data']."</p>";
echo "<p>".$news['tresc']."</p>";
}
?>
</body>
</html>

www.tay.cba.pl/addnews.html

Kod:

<html>
<body>
<form action="news.php" method="post">
<table border="0">
<tr>
<td>Tytuł newsa:</td><td><input type="text" name="tytul" size="15"></td>
</tr>
<tr>
<td>Treść newsa:</td><td> <textarea name="tresc" rows="5" cols="20"></textarea></td>
</tr>
<tr>
<td>Data:</td><td> <input type="text" name="data" size="15"></td>
</tr>
</table>
<input type="submit" value="Wyslij"><br>

</body>
</html>

Siwy

Hmm po pierwsze oddziel dodawanie newsa od ich wyświetlania

. Bo co każde wyświetlenie dodają się chyba puste rekordy mi się wydaje. Albo poprzez sprawdzenie czy coś przyszło z formularza, albo przez osobny plik

.

Aby sprawdzić czy przyszły jakieś dane z formularza użyj

Kod:

// moze byc tez np. GET
if($_SERVER['REQUEST_METHOD'] == 'POST')
{
kod dodawania newsa
}
else
{
wyswietlanie nmewsow
}

Co do Twojego problemu wydaje mi się, że rozwiązaniem będzie usunięcie cudzysłowów z nazw pól

. Czyli

Kod:

SELECT tytul, tresc FROM newsy ORDER BY id DESC;

Zawsze nazwy pól podaje się albo bez niczego, albo w tych tajemniczych znakach koło jedynki - `

.

Tay

Siwy

Zostało Ci jeszcze uwierzytelnianie admina

.

Cytat:

Jeszcze raz dzięki, choć znając mnie jeszcze nieraz będę musiał szukać u Ciebie pomocy

Nie ma sprawy, to przyjemność

. Dodaje kopa, aby dalej się uczyć etc., bo widać tego efekty nie tylko w postaci własnych skryptów, ale także pomocy innym

.

Tay

Niestety napotkałm kolejny problem. Ale skoro to dla Ciebie przyjemność, to możesz mi pomóc, prawda?:D

Otóż dodałem do tych newsów div'y, aby to jakoś wyglądało:

Kod:

while($news = mysql_fetch_assoc($wynik))
{
<div id="NEWS">
<div id="TYTUL_NEWSA">
echo "<h3>".$news['tytul']."</h3>";
</div>
<div id="TRESC_NEWSA">
echo "<p>".$news['tresc']."</p>";
</div>
<div class="pasek4">
</div>
</div>
}
?>

Problem z tym, że mam błąd. Nie wiem jak osadzić html w php. Próbowałem naprawdę wielu sposobów - dodawałem cudzysłowy, echa itd. Jednak jestem na to za głupi i muszę kolejna raz prosić Cię o przysługę ;D Pomożesz?

PS: Chciałbym też aby w jednym takim divie z treścią wyświetlała się treść jednego newsa, bo teraz to chyba będą się wyświetlać wszystkie wrzucone do tabeli treści. Podobnie z tytułami. No i aby najnowsze newsy były u góry