Errore Variabile

[lollo10!]

Ciao, sono alle prime armi col php e non riesco a saltar fuori da un semplicissimo codice :S

<?php
header("Content-type: image/gif");
$user = $_GET['user'];
$country = $_GET['country'];
$sedia = $_GET['sedia'];
    $im = imagecreatefromgif("throne$sedia_1.gif");
    $home = imagecreatefromgif("http://www.sito.$country/imaging/avatar?user=$user&action=sit&img_format=gif");
    imagecopy($im, $home, 20, -15, 0, 0, 64, 110);
    $throne = imagecreatefromgif("throne$sedia_2.gif");
    imagecopy($im, $throne, 0, 0, 0, 0, 70, 110);
//Create Image
imagegif($im);
imagedestroy($im);
?>

Il codice così non funziona e non visualizzo nessuna immagine, mentre se lo scrivo così funziona:

<?php
header("Content-type: image/gif");
$user = $_GET['user'];
$country = $_GET['country'];
    $im = imagecreatefromgif("throne_1.gif");
    $home = imagecreatefromgif("http://www.sito.$country/imaging/avatar?user=$user&action=sit&img_format=gif");
    imagecopy($im, $home, 20, -15, 0, 0, 64, 110);
    $throne = imagecreatefromgif("throne_2.gif");
    imagecopy($im, $throne, 0, 0, 0, 0, 70, 110);
//Create Image
imagegif($im);
imagedestroy($im);
?>

Lo script prende in pratica l'avatar dell'utente di un sito e lo mette su una sedia.
Appena aggiungo la variabile per scegliere quale sedia utilizzare, non visualizzo più nessuna imagine.

Qualcuno sa cos'ho sbagliato?

[lollo10!]

up

[clasku]

<?php 
header("Content-type: image/gif"); 
$user = $_GET['user']; 
$country = $_GET['country']; 
$sedia = $_GET['sedia']; 
$case1 = "throne".$sedia."_1.gif";
$case2 = "throne".$sedia."_2.gif";
    $im = imagecreatefromgif($case1); 
    $home = imagecreatefromgif("http://www.sito.$country/imaging/avatar?user=$user&action=sit&img_format=gif"); 
    imagecopy($im, $home, 20, -15, 0, 0, 64, 110); 
    $throne = imagecreatefromgif($case2); 
    imagecopy($im, $throne, 0, 0, 0, 0, 70, 110); 
//Create Image 
imagegif($im); 
imagedestroy($im); 
?>

[lollo10!]

la differenza qual'è?

[clasku]

variabili con il nome definite prima invece che nella funzione di creazione della gif

$case1 = "throne".$sedia."_1.gif";

prova se va

[lollo10!]

Wow, funziona!

Grazie mille!

[clasku]

prego, però studia bene le concatenazioni di variabili per evitare gli stessi problemi in futuro

ad esempio, potrebbe funzionare anche così

$im = imagecreatefromgif("throne".$sedia."_1.gif"); 

ma non mi piacciono molto queste concatenazioni dentro i parametri delle funzioni, preferisco definire una variabile esternamente e usarla nelle funzioni

ah, per evitare problemi di sicurezza, verifica bene cosa ti arriva in GET...

[lollo10!]

Cosa intendi con l'ultima frase?

[clasku]

che quando ricevi qualche cosa via GET devi essere certo che quel qualcosa sia quello che ti aspetti, per non esporti a problemi di sicurezza