Det er en noget risikabel måde at anvende data du har modtaget fra brugeren på, da dit site bliver helt åben for SQL Injections.
Overvej at anvende parameteriserede forspørgsler (hvis ellers din ODBC-driver understøtter det), for at sikre dig bedre mod SQL Injections.
Jeg har skrevet en artikel om dette for noget tid siden, som jeg vil mene stadig er anvendelig i dit tilfælde. Jeg vil anbefale, at du tager en time ud af kalenderen og læser den igennem og forstår hvordan du bør gøre, når din kode skal snakke sammen med databasen. Artiklen ligger her:
www.eksperten.dk/guide/1250Når det så er sagt, så ser jeg ikke lige, hvor det er at teksten bliver htmlencoded i dit eksempel, men min tidligere anbefaling vedr. dette står stadig ved magt. Dog bør du overveje et eller andet tjek for om de data du gemmer i databasen indeholder noget der ligner ulovlig indhold (f.eks. script enten som scriptblokke eller som eventhandlers). Det er dog overordentlig svært at lave disse tjek korrekt og grundigt nok selv, så hvis du kan finde en eller anden komponent der kan hjælpe dig med dette, ville det nok være ønskværdigt for dig.