ASP opgave ( tror jeg nok)

Nu ved jeg ikke hvordan din database ser ud, men jeg kan se et mønstre i systemet:

platformnummer, platformnan:
1, pc
2, dreamcast
3, playstation
4, nintendo 64
5, playstation 2
6, xbox
7, macintosh
8, gameboy

Så jeg vil gå ud fra, at der bare skal tilføjes "9, platformen"..

Og lige en anden ting:
Der er noget galt med validering af platformen.. eksempel

PC: http://gamesupport.dk/show.asp?show=platform&platform=1

hvis du erstatter "1" (platform=1) med "9", vil ikke finde platformen, men siden vises rigtigt.

Men: hvis "1" erstattes af "a", fejler siden.. den crasher.. hvikelt er dårlig validering..

Løsningen findes her: http://www.w3schools.com/vbscript/func_isnumeric.asp

Hej
tak for svarene. men i er altså nød til at tage dansk :-)
Hvor og hvad skal jeg gøre ?

Jeg går ud fra, at du på linie 14 i filen show.asp, har noget i stil med:

sql = "select * from TABELNAVN where en_eller_anden_ID = " & reqeust.querystring("platform")

den vigtige del er: reqeust.querystring("platform")
Det kan være at reqeust.querystring("platform") er smidt over i en variabel, og så står der noget andet i linie 14..

Egentligt er det nok linie 13 eller 12..

smid lige linierne 1-14 herind.. HUSK: stien til databasen, brugernavn, kodeord og andre vitale oplysningern, skal skjules... f.eks. med "***"..

Response.Buffer = True
    DesignStart
   
    ConnectToDatabase   
   
    Server.ScriptTimeOut = 200

    Show = request("show")

    platform = request("platform")

    if platform = 1 then
        Response.write "<img src='grafik/fyld-trans.gif' width='100%' height='10'><br>" & vbcrlf
        Response.write topbar("PC CD-ROM")
        Stats("PC CD-ROM")
        menubar = "pc"
        show_count = "PC CD-ROM"
    elseif platform = 2 then
        Response.write "<img src='grafik/fyld-trans.gif' width='100%' height='10'><br>" & vbcrlf
        Response.write topbar("Dreamcast")
        Stats("Dreamcast")
        menubar = "dreamcast"
        show_count = "Dreamcast"
    elseif platform = 3 then
        Response.write "<img src='grafik/fyld-trans.gif' width='100%' height='10'><br>" & vbcrlf
                Response.write topbar("Playstation")
        Stats("Playstation")
        menubar = "playstation"
        show_count = "Playstation"
    elseif platform = 4 then
        Response.write "<img src='grafik/fyld-trans.gif' width='100%' height='10'><br>" & vbcrlf
        Response.write topbar("Nintendo 64")
        Stats("Nintendo 64")
        menubar = "n64"
        show_count = "Nintendo 64"
    elseif platform = 5 then
        Response.write "<img src='grafik/fyld-trans.gif' width='100%' height='10'><br>" & vbcrlf
        Response.write topbar("PLAYSTATION 2")
        Stats("Playstation 2")
        menubar = "playstation2"
        show_count = "Playstation 2"
    elseif platform = 6 then
        Response.write "<img src='grafik/fyld-trans.gif' width='100%' height='10'><br>" & vbcrlf
        Response.write topbar("XBOX")
        Stats("Xbox")
        menubar = "xbox"
        show_count = "Xbox"

Der er fejl, og plads til optimering..

1) Fejl.
Du henter en værdi, uden at checke typen af værdien.. rettelse:

    platform = request("platform")
    if isNumeric(platform) and platform <> "" then
      ' koden her
    else
      response.write "Fejl i platform type"
    end if

2) "koden" der optimeres
Når man skal checke om noget har en bestemt værdi, ud af flere værdier, er det bedre at bruge Select Case, i stedet for If-sætninger:

Select Case platform
case 1
  ' kode til platform = 1
case 2
  ' kode til platform = 2
case 3
  ' kode til platform = 3
case 4
  ' kode til platform = 4
case 5
  ' kode til platform = 5
case 6
  ' kode til platform = 6
case else
  ' kode til en platform der er ugyldig
end select


Du behøver ikke "Case Else", hvis du bruger denne metode, da platform aldrig bliver ugyldig:

if isNumeric(platform) and platform <> "" then
  select case platform
  ' case sætninger med deres koder
  end select
else
  response.write "Ugyldig platform"
end if


Du bestemmer selv, hvordan det skal se ud.. du kunne også ændre koden, så den vælger "platform = 1", i stedet for at skrive "Ugyldig platform"..

Det er op til dig.