Forum PHP.pl > Niedziałający system newsów

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: Niedziałający system newsów

Forum PHP.pl > Forum > Gotowe rozwiązania

Fipaj

17.03.2005, 13:28:30

Cóż...

Zrobiłem system newsów. Dodawanie i pokazywanie newsów oparte na bazie danych. Działa

Kiedy jednak spróbowałem dodać obsługę komentarzy skrypt się rozwalił...

Mogę prosić szanowną publikę o poprawienie błędów?? Będę dozgonnie wdzięczny...

Błędy te siedzą zapewne w plikach index/komentuj. Index nie potrafi dobrze przekazać do zmiennej w URLu wartości, natomiast w pliku komentuj nie działa nic... Wsyzstkie pliki można oglądać tu: http://www.members.lycos.co.uk/fipaj/news/

Oto pliki:

index.php

[PHP] pobierz, plaintext 
<?php
$bd = @mysql_connect ('localhost', 'fipaj', '');
@mysql_select_db ('fipaj_uk_db');
$zapytanie = @mysql_query ('SELECT * FROM nowosci ORDER BY identyfikator DESC LIMIT 6');
while ($wiersz = @mysql_fetch_array ($zapytanie)) {
    print ('<p>');
    print ('<h3>');
    print $wiersz['tytul'];
    print ('</h3>');
    print ('<small>');
    print ('Data:');
    print $wiersz['data'];
    print ('; Autor:');
    print $wiersz['autor'];
    print (';</small><br>');
    print $wiersz['tresc'];
    $wiersz['identyfikator'] = $id;
    print ('<br><a href=\"komentuj.php?k=' . $id . '\">Komentuj!</a></p>');
}
@mysql_close ($bd);
?>
[PHP] pobierz, plaintext

panel.php

[PHP] pobierz, plaintext 
<html>
<head>
<title>Panel administracyjny</title>
</head>
<body>
<?php
if (isset ($_POST['wyslij'])) {
    if ($_POST['haslo'] == &#092;"TeSt\") {
        $bd = @mysql_connect ('localhost', 'fipaj', '');
        @mysql_select_db ('fipaj_uk_db');
        $_POST['tytul'] = $tytul;
        $_POST['tresc'] = $tresc;
        $_POST['autor'] = $autor;
$zapytanie = @mysql_query (&#092;"INSERT INTO nowosci VALUES ('$tytul', '$tresc', '$autor', NOW(), '')\");
        @mysql_close ($bd);
        print ('News został dodany...');
    }
    else {
        print ('Niepoprawne hasło...');
    }
}
else {
    print ('<form action=\"panel.php\" method=\"post\">');
    print ('Hasło: <input type=\"password\" name=\"haslo\">');
    print ('Treść: <input type=\"text\" name=\"tresc\">');
    print ('Tytuł: <input type=\"text\" name=\"tytul\">');
    print ('Autor: <input type=\"text\" name=\"autor\">');
    print ('<input type=\"submit\" name=\"wyslij\" value=\"wyslij\">');
    print ('</form>');
}
@mysql_close ($bd);
?>
</body>
</html>
[PHP] pobierz, plaintext

komentuj.php

[PHP] pobierz, plaintext 
<html>
<head>
<title>Komentuj newsa</title>
</head>
<body>
<?php
$bd = @mysql_connect ('localhost', 'fipaj', '');
@mysql_select_db ('fipaj_uk_db');
$zapytanie = @mysql_query ('SELECT * FROM nowosci WHERE identyfikator = '$k' LIMIT 1');
while ($wiersz = @mysql_fetch_array ($zapytanie)) {
    print ('<p>');
    print ('<h3>');
    print $wiersz['tytul'];
    print ('</h3>');
    print ('<small>');
    print ('Data:');
    print $wiersz['data'];
    print ('; Autor:');
    print $wiersz['autor'];
    print (';</small><br>');
    print $wiersz['tresc'];
}
print ('Komentarze:');
$zapytanie = @mysql_query ('SELECT * FROM komentarze WHERE post_identyfikator = $k');
while ($wiersz = @mysql_fetch_array ($zapytanie)) {
    print ('<p>');
    print $wiersz['autor'];
    print ('<br>');
    print $wiersz['tresc'];
    print ('</p>');
}
print ('Dodaj komentarz:');
print ('<form method=\"post\" action=\"komentuj.php\">');
print ('Autor: <input type=\"text\" name=\"autor\">');
print ('Treść: <input type=\"text\" name=\"tresc\">');
print ('<input type=\"submit\" name=\"wyslij\" value=\"wyślij\"></form>');
if (isset ($_POST['wyslij'])) {
    $_POST['autor'] = $autor;
    $_POST['tresc'] = $tresc;
    $zapytanie = @mysql_query ('INSERT INTO komentarze VALUES ($autor, $tresc, $k)');
    print ('Komentarz został dodany');
}
@mysql_close ($bd);
?>
</body>
</html>
[PHP] pobierz, plaintext

revyag

17.03.2005, 13:41:18

Plik index.php, wiersz 17. - Czy czasem to przypisanie nie powinno być odwrotnie skonstruowane ?
Plik komentuj.php. linie 38-39
Dlaczego do zmiennych $_POST przypisywane są wartości z pół formularza, które przecież będą zawierać te same wartości co zmienne $_POST po submicie ? To jest bez sensu.

zbig13

17.03.2005, 14:09:38

Jeszcze zwroc uwage na wiersz 9, 25-26 i 42-43 w pliku komentuj.php. Zmienne przekazywane w pasku adresu w przeglądarce powinny być pobierane metodą $_GET[], np. zamiast:

[PHP] pobierz, plaintext 
<?php
$zapytanie = @mysql_query ('SELECT * FROM nowosci WHERE identyfikator = '$k LIMIT 1); 
?>
[PHP] pobierz, plaintext

powinno być:

[PHP] pobierz, plaintext 
<?php
$zapytanie = @mysql_query (&#092;"SELECT * FROM nowosci WHERE identyfikator = {$_GET['k']} LIMIT 1\"); 
?>
[PHP] pobierz, plaintext

Niektóre serwery mogą obslugiwać samą zmienna $k, ale zalecane jest stosowanie $_GET[]. A poza tym w twoim zapytaniu jest błąd, bo tam, gdzie masz WHERE identyfikator='$k' następuje zamknięcie łańcucha. Chyba chodziło ci o WHERE identyfikator=' . $k . '

Fipaj

17.03.2005, 16:23:44

Cały czas nie działa...

Vertical

17.03.2005, 16:40:17

Może chodzi o łączenie stringów:
zamiast

[PHP] pobierz, plaintext 
<?php
$zapytanie = @mysql_query ('SELECT * FROM nowosci WHERE identyfikator = '$k' LIMIT 1');
?>
[PHP] pobierz, plaintext

powinno być:

[PHP] pobierz, plaintext 
<?php
$zapytanie = @mysql_query ('SELECT * FROM nowosci WHERE identyfikator = '. $k .' LIMIT 1');
?>
[PHP] pobierz, plaintext

i tak dalej....
Tak apropos to uprościłbym troszkę wyświetlanie. Zamiast:

[PHP] pobierz, plaintext 
<?php
print ('<p>');
	print ('<h3>');
	print $wiersz['tytul'];
?>
[PHP] pobierz, plaintext

napisałbym:

[PHP] pobierz, plaintext 
<?php
printf ('<p><h3> %sn', $wiersz['tytul']);
?>
[PHP] pobierz, plaintext

Fipaj

17.03.2005, 16:55:29

Nic... Nie idzie... Chyba trzeba będzie plik komentuj.php przepisać od nowa... ale nie mam pomysłu, co tam może być nie tak... zbyt zawiłe to dla mnie

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.