Krijg sql query niet aan de praat

Door stefan op 11-01-2010 10:37

1.117 views

<?php
$host = "localhost";
$gebruikersnaam = "deb***_root";
$wachtwoord = "***";
$databasenaam = "deb***_cms";

mysql_connect($host,$gebruikersnaam,$wachtwoord);
mysql_select_db($databasenaam);

  $habbo=$_GET['habbonaam'];
  $username=$_GET['username'];
  $text=$_GET['text'];

		if(empty($text)) {
	echo ubb('[fullalert]Je moet wel een tekst invoeren voordat je een bericht in het gastenboek kunt zetten!<br>
			   <input type="button" value="Sluiten" onclick="javascript:melding_close();" />[/fullalert]');
		}else{
	mysql_query('INSERT INTO guestbook (to_user,from_user,tekst,date,ip) VALUES ("'.$habbo.'","'.$username.'","'.$text.'",NOW(),"'.$_SERVER['REMOTE_ADDR'].'")"');
	
	echo'<table width="100%">
    <tr><td width="100%"><table width="100%" style="border: dashed 1px; border-color: #EFEFEF;" cellspacing="0" cellpadding="0">
      <tr>
        <td style="background: #EFEFEF;"><div style="float: left; margin-top: 5px;"><img src="http://www.habbo.nl/habbo-imaging/avatarimage?user='.$username.'&amp;action=sta&amp;frame=3&amp;direction=2&amp;head_direction=2&amp;gesture=sml&amp;size=s&amp;img_format=png" border="0" onerror="this.src=(\'http://www.habbowereld.nl/assets/img/site/error_habboSmall.png\')" /></div>
          <div style="float: left; margin-left: -25px;"><strong><a href="http://www.habbowereld.nl/leden/profiel/'.$username.'">'.$username.'</a></strong></div><br>'.$text.'
          <div align="right" style="margin-bottom: 5px; margin-right: 5px;"><font style="color: gray; font-size: 10px;">'.date("d-m-Y").'</font></div>
         </td>
      </tr>
    </table>
    </td></tr></table>';
		}
		
	?>

wat is er verkeerd?

Onbekende gebruiker

11-01-2010 12:33

Dat is dus mijn punt.
Of mysqli of pdo of weetikveelwat nou beter is dat boeit me niet.
Als ze verdwijnen, dan denk ik dat je dat op zo'n manier als [php]eregi[/php] kunt vinden.
Verder verwacht ik ook wel dat als je host update naar een nieuwe php versie (wat goed is, want sommige zitten nog met php 4 en andere nog met de versie voor php 5.3) dat ze dat melden en ook melden wat wellicht de gevolgen kunnen zijn.

Joren de Wit

11-01-2010 12:38

Maar dat neemt niet weg dat je beter voor pdo of mysqli kunt kiezen bij het maken van nieuwe scripts. Ten eerste voorkom je dat je in de toekomst deze scripts opnieuw moet scripten als mysql_* wel daadwerkelijk verdwijnt en daarnaast ontwikkel je dan met een functionaliteit die actief verder ontwikkeld wordt en niet een die enkel onderhouden wordt.

Bovendien is de mysqli extensie er juist voor gemaakt om met MySQL versies > 4.1.3 te werken. Aangezien de meeste host nu toch wel op MySQL 5 draaien zou dat een logische keuze zijn (of uiteraard de keuze voor het OO PDO).

Richard van Velzen

11-01-2010 13:08

mysql_* zal niet verdwijnen, dat is pure flauwekul. Dankzij de nieuwe library (mysqlnd) zal de support ook weer sterk verbeteren, indien je met mysqlnd compiled uiteraard.

Het grootste minpunt aan mysql_* is dat het geen support voor nieuwere features (denk aan prepared statements) biedt. Maar dat is vaak ook helemaal niet nodig, dat kun je prima zelf via queries doen, zoals het hoort.

Reageren

Inloggen om te reageren