När jag skriver ut min sql: Hej Magnus. Jo det ser jag att du tar den i ifsatsen, men var är ifsatsen? innan eller efter själva uppdateringen? Ok sorry, jag missuppfattade dig om sqlen. HAHAH Datorer lever sitt eget liv :) Innan.Uppdateringsproblem.
<code>
UPDATE t_answer set vyes = vyes +1 WHERE id = 4
</code>
Jag försöker så här:
<code>
sql = "UPDATE t_answer set " + sAnsw + " = " + sAnsw + " +1 WHERE id = " + iId;
Response.Write(sql);
OleDbCommand cmd = new OleDbCommand(sql, DBConn);
DBConn.Open();
cmd.ExecuteNonQuery();
DBConn.Close();
</code>
Variablen sAnsw kommer ifrån:
<code>
string sAnsw = "";
if (Voteja.Checked)
{
sAnsw = "vyes";
}
else if (Votenej.Checked)
{
sAnsw = "vno";
}
else if (Votevetinte.Checked)
{
sAnsw = "vdontKnow";
}
'det ser väldigt jobbigt ut med radioknappar, måste man göra så här?
</code>
Jag får inga error, men heller ingenting uppdaterat.
Vad tror ni???Sv: Uppdateringsproblem.
Försök att skriva sqlfrågan så här
<code>
sql = "UPDATE t_answer SET sAnsw =" + sAnsw + 1 + " WHERE id =" + iID;
</code>
Var tar du ut din variabel? Gör du det innan eller efter du använder den? Om du gör det efter kan du ju inte få värdet i den eftersom den inte är global.
Sen skulle du nog kolla SQL-injections också, det är bättre du använder lagrade procedurer.
Vad gäller radioknapparna tror jag att du kan använda radioButtonList och läsa ut värdet av den.Sv:Uppdateringsproblem.
Jag visade ju hur sql:en ser ut utskriven (med response.write.
Inga fel där.
Eftersom det finns tre olika fält i databasen,vyes,vno och vdontKnow, så tycker jag att det verkar
bra att använda sig av den variablem.
radioButtonList kände jag inte till, utan jag skall titta på den.
SQL-injections tror jag inte kan förekomma, då detta är en röstnings-applikation, och det finns enbart
radioknapparna där. Inga texboxar el dyl.
Var jag tar variablen??
La jag in för mycket kod... titta i if-satsen.
Av någon okänd anledning, denna fungerar nu.Sv: Uppdateringsproblem.
Sv:Uppdateringsproblem.
Vad jag har gjort nu, är att jag har brytet ner en stor class till många små.
Jag har fått det överskådligare och lättare att läsa/först.
Just nu så "rinner" det på av bara h-e, det fungerar kanon.
Tack för visat intresse.