[PHP] Przycisk-odnośnik Opcje

[Adalberd]

Witajcie,

Po dodaniu produktu do bazy, chce żeby wyświetlił się przycisk z odnośnikiem do strony głównej, kombinuje w ten sposób:

[PHP] pobierz, plaintext 
 
 $ins = @mysql_query("INSERT INTO mojabazax SET produkt1='$produkt1', produkt2='$produkt2', '");
 
    if($ins) echo "<center>dodano produkt do bazy<p>";  
	if($ins) echo "<input type="button" value="POWRÓT" onclick="document.location = 'http://wp.pl'>" ;
 
	    else echo "Nie udało się wprowadzić samochodu";
 
    mysql_close($connection);
}
 
?> 
[PHP] pobierz, plaintext 

Gdzie popełniam błąd?

[mlawnik]

Co ci się pojawia?

[PHP] pobierz, plaintext 
 $ins = @mysql_query("INSERT INTO mojabazax SET produkt1='$produkt1', produkt2='$produkt2', '");
 
    if($ins){ 
    echo '<center>dodano produkt do bazy<p>';
    echo '<input type="button" value="POWRÓT" onclick="document.location = 'http://wp.pl'>';
    }  
    else{ 
    echo 'Nie udało się wprowadzić samochodu';
    }
 
    mysql_close($connection);
 
 
?> 
[PHP] pobierz, plaintext 

Edit: Sorki, poprawilem.
Tak jak napisali, Jeden if wystarczy i ' zamiast ".

Ten post edytował mlawnik 3.04.2011, 16:19:40

[kadlub]

[PHP] pobierz, plaintext 
 
 <?php
 $ins = @mysql_query("INSERT INTO mojabazax  (produkt1, produkt2) values ('".$produkt1."','".$produkt2."' ");
 
    if($ins) echo "<center>dodano produkt do bazy<p>";  
	if($ins) echo '<a href="http://wp.pl">Powrót</a>' ;
 
	    else echo "Nie udało się wprowadzić samochodu";
 
    mysql_close($connection);
}
 
?> 
 
 
[PHP] pobierz, plaintext 

Ten post edytował kadlub 3.04.2011, 16:33:17

[kamil4u]

Musisz pozamieniać " na \" wew. echo.
Nie lepiej zastosować zwykły link <a> odpowiednio go modyfikując CSS? Moim zdaniem tak.

Poza tym masz 2 razy ten sam warunek if($ins) - użyj "wyrażeń blokowych":

[PHP] pobierz, plaintext 
if($ins){
//kod
} else {
//kod
}
[PHP] pobierz, plaintext 

I na koniec - lepiej używać ' niż "(w PHP) - poczytaj o tym, było kilka razy na forum.

Ten post edytował kamil4u 3.04.2011, 16:17:59

[Adalberd]

ok, posłuchałem wszystkich rad, zastosowałem ' ' zamiast " ", zostosowałem tylko jedenego if-a, i zrobiłem odnośnik:

[HTML] pobierz, plaintext 
'<a href="www.wp.pl">Powrót</a>'
[HTML] pobierz, plaintext 

Później CSS zrobie przycisk skoro tak można.

Dziękuje,
temat do skasowania.