Problema con connessione al database

[Colino]

Ho scritto queste righe ma non riesco a capire quali possano essere gli errori per cui non riesce ad accedere:

		$db_host = "localhost";
		$db_user = "root";
		$db_password = "";
		$db_database = "my_database";
		$db_tabella = "tab_news";
		
		$connessione = mysql_connect($db_host,$db_user,$db_password);

	if (isset($_REQUEST['submit'])){
		
		$n_title = $_REQUEST['news_title'];
		$n_data = $_REQUEST['news_data'];
		$n_body = $_REQUEST['news_body'];

		
		
		$query = "INSERT INTO tab_news (news_title,news_data,news_body) VALUES (".$n_title ."," . $n_data .",". $n_body . ");";

		mysql_query($query,$connessione) or die ("\nQuery fallita");

cosa genera il problema??
la pagina la sto testando in locale con il phpmyadmin

[neroux]

Se tutti i valori non siano di tipo integer mancherebbero gli apici nell'INSERT

$query = "INSERT INTO tab_news (news_title,news_data,news_body) VALUES ('".$n_title ."','" . $n_data ."','". $n_body . "')"; 

Però attenzione, così il codice sia probabilmente pronto per un sql injection -> mysql_real_escape_string().

[Colino]

Allora,
ho provato ad inserire il tutto tra apici non va...
a cosa potrebbe essere dovuto??

Come si può rimediare al problema delle sql injection?

[E.d.i.73]

Hai dimenticato nel codice di selezionare il database tramite la funzione mysql_select_db($db),comunque prova così:

<?php
$db_host = "localhost";
$db_user = "root";
$db_password = "";
$db_database = "bla";
$db_tabella = "news";

mysql_connect($db_host,$db_user,$db_password);
mysql_select_db($db_database);
if (isset($_REQUEST['submit'])){

$n_title = $_REQUEST['news_title'];
$n_data = $_REQUEST['news_data'];
$n_body = $_REQUEST['news_body'];

$query = "INSERT INTO tab_news (news_title,news_data,news_body) VALUES ('$n_title ',' $n_data','$n_body')";

mysql_query($query) or die ("\nQuery fallita");
mysql_close();
}
?>

[Colino]

grazie adesso funziona,
esiste un modo per ovviare al problema delle sql injection?

[neroux]

Certo, usando mysql_real_escape_string() per MySQL o prepared statements (il miglior metodo IMHO).

http://it.wikipedia.org/wiki/SQL_injection

[Colino]

Scusate,
vorrei aggiungere un campo per le foto

<input name="news_pic" type="file" id="news_pic" />

ma quando faccio pressione sul tasto di invio mi restituisce:

 Notice: Undefined index: news_pic

perchè?

[E.d.i.73]

Nell'html devi aggiungere l'enctype nel form:
prova in questo modo:

<form action="foto.php"method="post"enctype="multipart/form-data">
<input type="hidden" value="foto"name="action"/>
<input name="news_pic" type="file" id="news_pic" />




<input type="submit" name="submit" value="Invia"/>
</form>

per quando riguarda il codice php non so se le foto le vuoi trasferire su una cartella sul server per poi richiamarle od altro..fammi sapere ciaoo.