[PHP]Dodawanie newsa, Jak nie to, to to. Opcje

[MateuszS]

Witam. Mam takie cuś:

[PHP] pobierz, plaintext 
<? 
            
$_POST['tytulnewsa'] = $tytulnewsa;
$_POST['trescnewsa'] = $trescnewsa;
            
$data = $_SESSION['data'];
$trescnewsa = $_SESSION['trescnewsa'];
$tytulnewsa = $_SESSION['tytulnewsa'];
            
$data = (date("G:i:s, dS F Y"));
            
if(!isSet($tytulnewsa) && !isSet($trescnewsa)) 
{
     if(!empty($tytulnewsa) and !empty($trescnewsa)) 
     {
 
                $zapytanie = mysql_query("INSERT INTO `news` (Data,Nick,Tytul,Tresc)
                                          VALUES ('$data','$_SESSION[login]','$tytulnewsa','$trescnewsa')");
              
                header('location: '.$_SERVER['PHP_SELF']);
                exit;
     }
}                              
 
else
{
      echo '<div class="else">';
      echo 'Błędnie wypełniony formularz!';
      echo '</div>';
}    
 
            
ob_end_flush();
            
?>
[PHP] pobierz, plaintext 

Gdybym to zrobił na szybko to było by tak:
1. Wchodzę na stronę newsa a tam jest już "Błędnie wypełniony formularz"
2. Po odświeżaniu strony dodaje się 10 kolejnych newsów.

Aby temu zapobiec, Zrobiłem tak:
1. Dodałem

[PHP] pobierz, plaintext 
<?php
header('location: '.$_SERVER['PHP_SELF']);
 exit;
?>
[PHP] pobierz, plaintext 

2. Użyłem isSet, żeby nie było tego napisu "Błednie... " zaraz po wejściu na stronę.

Ten kod główny powyżej to ten który teraz mam.

Problem polega na tym, że nie zapisuje mi do bazy żadnych rekordów. Czy da się to naprawić żeby nie było w/w problemów?

Ten post edytował MateuszScirka 13.01.2009, 16:14:04

[Strarus]

Nie wiem czy będzie dobrze ale może się uda (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Ja bym zrobił inaczej zapytanie do MySQL:

[PHP] pobierz, plaintext 
<?php
"INSERT INTO `news` SET `data`='$data', `login`='$_SESSION[login]', `tytulnewsa`='$tytulnewsa', `trescnewsa`='$trescnewsa'"
?>
[PHP] pobierz, plaintext 

[MateuszS]

Ten element jest dobrze bo newsy dodawało. Twoje zapytanie jest chyba trochę nie poprawne. Tak się robi przy SELECT. Ale dzięki, Pozdrawiam i czekam na odpowiedzi.

[help_mee]

a czy łączysz się z bazą ? czy tylko dałeś część kodu?

[Strarus]

Zapytanie mam dobre bo u mnie jest tak:

[PHP] pobierz, plaintext 
<?php
$add = mysql_query("INSERT INTO `news` SET `content`='$content', `data`='$data', `nick`='$nick', `img`='$img'");
?>
[PHP] pobierz, plaintext 

I wszystko działa (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
A dlaczego przypisujesz dacie puste session (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) I potem ją definiujesz?

[MateuszS]

Tak łącze include (jest wcześniej w kodzie).

Połączenie itp jest dobrze, zapytanie też jest dobrze. Bo gdy usuwałem issety, mieszałem instrukcje warunkowe to działało tyle że nie tak jak trzeba.

Wyrażę się inaczej bo to dość zamotane:

Chcę aby po wejściu do panelu nie pokazało się od razu: "błąd przy dodawaniu newsa" tak jak po wykonaniu instrukcji = false. Poza tym chcę aby gdy odświeżam stronę nie dodawało kolejnych pustych newsów. I po ostatnie chce zeby je wogole dodawalo

[piotrooo89]

popatrz jak przypisujesz zmienne...

[PHP] pobierz, plaintext 
<?php
$_POST['tytulnewsa'] = $tytulnewsa;
$_POST['trescnewsa'] = $trescnewsa;
?>
[PHP] pobierz, plaintext 

pierwsza zawsze jest zmienna do której chcesz coś zapisać natomiast druga to zmienna (text, cokolwiek) w której trzymasz co ma się znaleźć w pierwszej.

[PHP] pobierz, plaintext 
<?php
$tytulnewsa=$_POST['tytulnewsa'];
$trescnewsa=$_POST['trescnewsa'];
?>
[PHP] pobierz, plaintext 

[Strarus]

Muszę już iść ale masz mój kod, którego używam:

[PHP] pobierz, plaintext 
<?php
 if(isset($_SESSION['zalogowany'])) { 
include('db_connect.inc.php');
if(isset($_POST['content']) && !empty($_POST['content']) && isset($_POST['nick']) && !empty($_POST['nick']) && isset($_POST['img']) && !empty($_POST['img'])) {
  $content = ($_POST['content']);
  $data = date('d.m.Y H:i:s');
  $nick = strip_tags($_POST['nick']);
  $img = ($_POST['img']);
  $add = mysql_query("INSERT INTO `news` SET `content`='$content', `data`='$data', `nick`='$nick', `img`='$img'");
  if(!$add) {
    echo 'Błąd podczas dodawania danych do bazy';
  } 
} else {
   echo '<form method="post" action="dodaj.php"><table>
<tr>
<td>Nick:</td>
<td><input type="text" name="nick" /></td>
</tr>
<tr>
<td>Treść:</td>
<td><textarea name="content" cools="40" rows="4"></textarea></td>
</tr>
<tr>
<td>Obrazek numer:</td>
<td><select name="img">
        <option>1</option>
        <option>2</option>
        <option>3</option>
    </select></td>
</tr>
</table><br/><input type="submit" value="Dodaj!" /></form>
<br /><img src="news/news1.png" width="50px" height="50px" /> - 1
<br /><img src="news/news2.png" width="50px" height="50px" /> - 2
<br /><img src="news/news3.png" width="50px" height="50px" /> - 3
<br/>
<br/><a href="logout.php">Wyloguj!</a>';
 }
} else {
  echo '<a href="login.php">Zaloguj!</a>';
}
?>
[PHP] pobierz, plaintext 

Może się przyda (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[MateuszS]

Piotrooo, no dobra ale to nie ma nic wspolnego z tym ze nie dodaje mi nic do bazy...

Gdy zmienie instrukcje, wywale isSety itp to dziala dobrze tylko ze po odswiezaniu dodaje mi rekordy a po wejsciu na strone formularza od razu pisze "Blednie wyp. formularz" jeszcze przed wypelnieniem

[erix]

A wiesz, jaka jest różnica między isset" title="Zobacz w manualu PHP" target="_manual a emtpy" title="Zobacz w manualu PHP" target="_manual? Będziesz miał pustą zmienną i isset" title="Zobacz w manualu PHP" target="_manual zwróci true.

[MateuszS]

Wiec jak powinno byc zeby bylo dobrze?

[erix]

Dowiesz się, jak ruszysz swoje cztery litery i przeczytasz manuala.

[MateuszS]

Moje 4 litery ruszały się całą noc i całą Biologię, bo siedziałem nad tym prostym skryptem. Serio próbowałem już wszystkiego co mi tylko do głowy przyszło. Skoro wiecie to czemu nie napiszecie. Na każdy post w tym dziale można odp. "Rusz 4 litery i czytaj manuala". Jednak nie zawsze wystarczy przeczytać manuala po angielsku żeby się dow. co jest źle. Wg mnie wszystko jest dobrze. A nawet taki prosty kod nie działa....

[PHP] pobierz, plaintext 
<?php
$trescnewsa=$_POST['trescnewsa'];
            $tytulnewsa=$_POST['tytulnewsa'];
            
            $data = $_SESSION['data'];
            $trescnewsa=$_SESSION['trescnewsa'];
            $tytulnewsa=$_SESSION['tytulnewsa'];
            
            $data = (date("G:i:s, dS F Y"));
            
            
            if(!empty($tytulnewsa) and !empty($trescnewsa)) 
            {
 
                $zapytanie = mysql_query("INSERT INTO `news` (Data,Nick,Tytul,Tresc)
                            VALUES ('$data','$_SESSION[login]','$tytulnewsa','$trescnewsa')");
 
            }
?>
[PHP] pobierz, plaintext 

[piotrooo89]

a co trzymasz w zmiennych sesyjnych? jest tam coś? bo jak nie masz nic to nic dziwnego że nie działa.

[erix]

Na każdy post w tym dziale można odp. "Rusz 4 litery i czytaj manuala".

Posiedziałbyś dłużej w tym dziale, to byś się zdziwił, jacy ludzie potrafią być leniwi... I gdyby wszyscy ruszali swoje cztery litery jak należy, to byłby tu jeden wątek na godzinę, a nie na minutę. A nie narzekaj na angielski, bo jest i będzie językiem programistów.

Twierdzisz, że jest dobrze, ok:

[PHP] pobierz, plaintext 
<?php
$trescnewsa=$_POST['trescnewsa'];
           $tytulnewsa=$_POST['tytulnewsa'];
           
           $data = $_SESSION['data'];
           $trescnewsa=$_SESSION['trescnewsa'];
           $tytulnewsa=$_SESSION['tytulnewsa'];
?>
[PHP] pobierz, plaintext 

Wyciągasz treść z formularza, ale zaraz ją zapisujesz pustymi danymi, więc jak ma być dobrze? Nie trzeba mieć 40 fakultetów, żeby zrozumieć, że jak do pudełka z muchą wstawisz rosiczkę, to już tej muchy nie będzie.

Poza tym, jest taka funkcja mysql_error" title="Zobacz w manualu PHP" target="_manual.

[MateuszS]

Więc z moich zamyślnych konkluzji, zawżdy powiedziałbym iż nawet ciężki pracy, wypracowałem takie cu:

[PHP] pobierz, plaintext 
<? 
            
            
            $trescnewsa=$_POST['trescnewsa'];
            $tytulnewsa=$_POST['tytulnewsa'];
            
                        
            $data = (date("G:i:s, dS F Y"));
            
 
            if( !empty($tytulnewsa) and !empty($trescnewsa))
            {
                    $zapytanie = mysql_query("INSERT INTO `news` (Data,Nick,Tytul,Tresc)
                                                VALUES ('$data','$_SESSION[login]','$tytulnewsa','$trescnewsa')");
             
                    header('location: '.$_SERVER['PHP_SELF']);
                    exit;
                                                
            }
            
            else 
            {
                
                if (isSet($tytulnewsa) and isSet($tytulnewsa))
                {
                    echo '<div class="else">';
                    echo 'Błąd przy wypełnianiu formularza!';
                    echo '</div>';
                }   
            }
            
    
            ?>
[PHP] pobierz, plaintext 

Wszystko do bazy zapisuje, po odswiezaniu nie dodaje nowych rekordow all ok tylko tego. W tym kodzie po blednym wypelnieniu formularza (zostawienie pustych miejsc) nie wyswietla sie error. ale po usunieciu issetow i troche pozmienianiu wyswietla sie nawet przed wypelnieniem pol. Pomysl na issety wzialem ze starego skryptu komentarza, zalaczam:

[PHP] pobierz, plaintext 
<?php
if(isSet($_POST["haslo"]) && isSet($_POST["login"])) 
{
if( $_POST['login'] != $login && $_POST['haslo'] != $haslo)
    {
        echo '<br /><br />';
        echo '<center>';
        echo 'Podane dane są nieprawidłowe';
    }
}
?>
[PHP] pobierz, plaintext 

Prosze o odpowiedź

[erix]

ale po usunieciu issetow i troche pozmienianiu wyswietla sie nawet przed wypelnieniem pol.

Zamień isset" title="Zobacz w manualu PHP" target="_manual na !empty" title="Zobacz w manualu PHP" target="_manual.

[MateuszS]

yyy jezeli to zamienie na !empty to wywali blad gdy bede wysylal wiad.

[PHP] pobierz, plaintext 
<?php
else
           {
               
               if (!empty($tytulnewsa) and !empty($tytulnewsa)) // ?!
               {
                   echo '<div class="else">';
                   echo 'Błąd przy wypełnianiu formularza!';
                   echo '</div>';
               }  
           }
           
   
           ?>
[PHP] pobierz, plaintext 

Ten post edytował MateuszScirka 14.01.2009, 16:20:14

[magra]

spróbuj tak:

[PHP] pobierz, plaintext 
<?php
if( $_POST && !empty($tytulnewsa) and !empty($trescnewsa))
           {
                   $zapytanie = mysql_query("INSERT INTO `news` (Data,Nick,Tytul,Tresc)
                                               VALUES ('$data','$_SESSION[login]','$tytulnewsa','$trescnewsa')");
            
                   header('location: '.$_SERVER['PHP_SELF']);
                   exit;
                                               
           }
           
           elseif ($_POST)
           {
                   echo '<div class="else">';
                   echo 'Błąd przy wypełnianiu formularza!';
                   echo '</div>';
           }
?>
[PHP] pobierz, plaintext 

[MateuszS]

Działa! Dzieki Ci bardzo! (IMG:http://forum.php.pl/style_emoticons/default/yahoo.gif) (IMG:http://forum.php.pl/style_emoticons/default/thumbsupsmileyanim.gif) (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)