Błąd w kodzie Opcje

[tommy4]

Witam, mam pewien problem, mianowicie jestem w trakcie robienia systemu newsów i gdy dodawałem edycję, coś się zepsuło, a patrzę na to od 5h i nic nie umiem znaleźć, to pomyślałem, że może doświadczeni użytkownicy coś będą o tym wiedzieć.

[PHP] pobierz, plaintext 
<?php
require_once "../main.php";
require_once "../naglowek.php";
 
if(isset($_POST['nid']))
{
	$nid = $_POST['nid'];
	echo '<tr><td class="border">';
	if($nid == '') 
	{
		echo '<div class="main">Musisz wybrać newsa!</div>';
	}
	$nedit = $nid;
	$edycja = 1;
	echo "</td></tr>";
}
 
 
 
if (isset($_POST['tresc']) && isset($_POST['temat']))
{
	echo '<tr><td class="border" style="border-top:none;">';
	$trescnewsa = $_POST['tresc'];
	$tematnewsa = $_POST['temat'];
	$tematnewsa = htmlspecialchars($tematnewsa);
	$czas = time();
	if ($edycja == 1)
	{
		$sql = "UPDATE newsy SET temat=$tematnewsa, tresc=$trescnewsa WHERE id=$nedit";
		if(@mysql_query($sql))
		{
			echo '<div class="main">Twój news został zmieniony.</div>';
		}
		else 
		{
			echo '<div class="main">Błąd podczas edycji newsa.</div>';
		}
 
	} 
	else
	{
		if ($edycja == 0)
{
		$sql2 = "INSERT INTO newsy (temat,tresc,data) VALUES ('$tematnewsa','$trescnewsa', '$czas')";
		if(@mysql_query($sql2))
		{
			echo '<div class="main">Twój news został dodany.</div>';
		}
		else
		{
			echo '<div class="main">Błąd podczas dodawania newsa.</div>';
		}
}
	}
	echo '</td></tr>';
}
 
 
 
 
echo'<tr><td class="border" style="border-bottom:none">';
$sql = @mysql_query("SELECT temat,id FROM newsy ORDER BY id DESC");
if (!sql)
{
	exit('<div class="main">Problem podczas pobierania newsów z bazy danych!</div>');
}
echo '</td></tr><tr><td class="border" style="border-top:none;border-bottom:none;">';
?>
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post"><div class="main">Newsy: <select name="nid" size="1">
<option selected value="">Wybierz newsa</option>
<option value="">------------------</option>
<?php
$sql = @mysql_query('SELECT id, temat, data FROM newsy ORDER BY data DESC');
if (!$sql)
{
	exit ('<div class="main">Nie można pobrać listy newsów!</div>');
}
 
while($newsl = mysql_fetch_array($sql))
{
	$nid = $newsl['id'];
	$ntemat = $newsl['temat'];
	echo "<option value='$nid'>$ntemat</option>";
}  
?>
</select><input type="submit" value="Edytuj"></div></form>
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post"><table border="0"><tr><td><div class="main">Temat:</div></td><td>
<input type="text" name="temat" 
<?php 
if ($edycja == 1)
{ 
	echo 'value="';
	$sql = @mysql_query('SELECT temat FROM newsy WHERE id='.$nedit);
	$row = @mysql_fetch_array($sql);
	echo $row['temat'];
}
?>" size="30"></td></tr><tr>
<td valign="top"><div class="main">Treść:</div></td><td>
<textarea name="tresc" rows="15" cols="40">
<?php 
if ($edycja == 1)
{
	$sql = @mysql_query('SELECT tresc FROM newsy WHERE id='.$nedit);
	$row = @mysql_fetch_array($sql);
	echo $row['tresc'];
} 
?>
</textarea></td></tr><tr><td colspan=2>
<div style="text-align:center;"><input type="submit" name="zapisz" value="Zapisz"></div></td></tr></table>
</form>
<?php
echo'</td></tr>';
require_once "../stopka.php";
 
?>
[PHP] pobierz, plaintext 

Jak to wygląda i jak działa możecie zobaczyć tutaj. Z góry dziękuję.

[-r0m4n-]

Jak byś mógł pokazać jakiegoś loga, było by może lepiej znaleźć usterkę...

Pozdrawiam

[nospor]

nie wiem czy wiesz, ale lista wyboru oprócz znacznikow <option> musi tez miec <select. u ciebie jest tylko znacznik zamykajacy </select> nie ma otweirajacego. nadaj mu tez wlasciwa nazwe

[siutek]

qrcze no... ja nie widze zadnego bledu w dzialaniu, opisz dokladniej co i jak :/

[-Guest-]

nospor:

[HTML] pobierz, plaintext 
<select name="nid" size="1">
[HTML] pobierz, plaintext 

Jest, tylko musisz przesunąć suwak w prawo.

siutek:

Jak chcę edytować, to po zapisie dodaje mi nowy news, a nie edytuje istniejący.

r0m4n:

O co chodzi z logiem?

[tommy4]

Przepraszam, że napisałem jako guest, ale nie zauważyłem, że nie jestem zalogowany. Jeszcze raz proszę o pomoc, ponieważ nie umiem nadal znaleźć nigdzie błędu.

[Hacker]

"Wyechuj" może przed if-em edytuj i zobacz co i jak albo po każdym if-ie jakiś komunikat

A błąd jest taki gdzie dodajesz nid-a w tym drugim formularzu??

Ten post edytował Hacker 18.12.2005, 18:01:32

[tommy4]

[PHP] pobierz, plaintext 
<?php
require_once "../main.php";
require_once "../naglowek.php";
 
if(isset($_POST['nid']))
{
    $nid = $_POST['nid'];
    echo '<tr><td class="border">';
    if($nid == '') 
    {
        echo '<div class="main">Musisz wybrać newsa!</div>';
    }
    $nedit = $nid;
    echo 'if(isset($_POST[nid])) - wartość edycji:'.$edycja.'!';
    $edycja = 1;
    echo 'if(isset($_POST[nid])) - wartość edycji(2):'.$edycja.'!';
    echo "</td></tr>";
}
 
if (isset($_POST['tresc']) && isset($_POST['temat']))
{
    echo '<tr><td class="border" style="border-top:none;">';
    $trescnewsa = $_POST['tresc'];
    $tematnewsa = $_POST['temat'];
    $tematnewsa = htmlspecialchars($tematnewsa);
    $czas = time();
    echo 'if($edycja == 1) w isset POST(tresc i temat): '.$edycja.'!';
    if ($edycja == 1)
    {
        $sql = "UPDATE newsy SET temat=$tematnewsa, tresc=$trescnewsa WHERE id=$nedit";
        if(@mysql_query($sql))
        {
            echo '<div class="main">Twój news został zmieniony.</div>';
        }
        else 
        {
            echo '<div class="main">Błąd podczas edycji newsa.</div>';
        }
 
    } 
    else
    {
        echo 'if($edycja == 2) w isset POST(tresc i temat): '.$edycja.'!';
        if ($edycja == 0)
{
        $sql2 = "INSERT INTO newsy (temat,tresc,data) VALUES ('$tematnewsa','$trescnewsa', '$czas')";
        if(@mysql_query($sql2))
        {
            echo '<div class="main">Twój news został dodany.</div>';
        }
        else
        {
            echo '<div class="main">Błąd podczas dodawania newsa.</div>';
        }
}
    }
    echo '</td></tr>';
}
 
 
 
 
echo'<tr><td class="border" style="border-bottom:none">';
$sql = @mysql_query("SELECT temat,id FROM newsy ORDER BY id DESC");
if (!sql)
{
    exit('<div class="main">Problem podczas pobierania newsów z bazy danych!</div>');
}
echo '</td></tr><tr><td class="border" style="border-top:none;border-bottom:none;">';
?>
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post"><div class="main">Newsy: <select name="nid" size="1">
<option selected value="">Wybierz newsa</option>
<option value="">------------------</option>
<?php
$sql = @mysql_query('SELECT id, temat, data FROM newsy ORDER BY data DESC');
if (!$sql)
{
    exit ('<div class="main">Nie można pobrać listy newsów!</div>');
}
 
while($newsl = mysql_fetch_array($sql))
{
    $nid = $newsl['id'];
    $ntemat = $newsl['temat'];
    echo "<option value='$nid'>$ntemat</option>";
}  
?>
</select><input type="submit" value="Edytuj"></div></form>
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post"><table border="0"><tr><td><div class="main">Temat:</div></td><td>
<input type="text" name="temat" 
<?php 
echo 'if($edycja == 1) w formularzu: '.$edycja.'!';
if ($edycja == 1)
{ 
    echo 'value="';
    $sql = @mysql_query('SELECT temat FROM newsy WHERE id='.$nedit);
    $row = @mysql_fetch_array($sql);
    echo $row['temat'];
}
?>" size="30"></td></tr><tr>
<td valign="top"><div class="main">Treść:</div></td><td>
<textarea name="tresc" rows="15" cols="40">
<?php 
echo 'if($edycja == 1) w formularzu (2): '.$edycja.'!';
if ($edycja == 1)
{
    $sql = @mysql_query('SELECT tresc FROM newsy WHERE id='.$nedit);
    $row = @mysql_fetch_array($sql);
    echo $row['tresc'];
} 
?>
</textarea></td></tr><tr><td colspan=2>
<div style="text-align:center;"><input type="submit" name="zapisz" value="Zapisz"></div></td></tr></table>
</form>
<?php
echo'</td></tr>';
require_once "../stopka.php";
 
?>
[PHP] pobierz, plaintext 

TO JEST NOWY KOD... Kliknijcie tutaj żeby sprawdzić jak działa.

EDIT: no i widocznie nie ma tej zmiennej, po załadowaniu strony po raz drugi:/

Ten post edytował tommy4 18.12.2005, 18:23:41

[Ociu]

Proponuje tak:

[PHP] pobierz, plaintext 
<?php
function editForm($id) {
$sql = 'SELECT * FROM news WHERE id='.$id.'';
$r = mysql_fetch_row(mysql_query($sql));
 
print '<form action="newsy.php?edit='.$id.'&formed=true" method="POST"> etc...';
}
 
function edit($id, $temat, $tresc) {
mysql_query('UPDATE news SET temat="'.$temat.'", tresc="'.$tresc.'" WHERE id="'.$id.'"');
}
 
if(isset($_GET['edit']) && isset($_GET['formed'])) {
edit($_GET['edit'], $_POST['temat'], $_POST['tresc']);
} else {
editForm($_GET['edit']);
}
?>
[PHP] pobierz, plaintext 

[tommy4]

OK, jeszcze raz tą edycję spróbuję zrobić od początku.

EDIT: JEEEEEEEEEESSSSSSSSSSTTTTT!!!!!!!!! Ludzie dzięki Wam bardzo, jak będę miał jeszcze jakieś problemy to napiszę. Swoją drogą ciekawe newsy napisaliście. Teraz się zajmę usuwaniem newsów, a Wam jeszcze raz dziękuję!

Ten post edytował tommy4 19.12.2005, 05:35:16