---
title: "Snygga upp koden"
type: "forum-thread"
url: "https://www.webforum.nu/amne/asp/54759-snygga-upp-koden"
topic: "ASP"
topic_url: "https://www.webforum.nu/amne/asp"
author: "medialabs"
published: "2002-10-01T12:14:48.000Z"
updated: "2002-10-01T21:19:49.000Z"
replies: 7
views: 281
page: 1
pages: 1
language: "sv-SE"
site: "webForum — webforum.nu"
rights: "Upphovsrätten till varje inlägg tillhör dess författare."
attribution: "Citera som: webForum, https://www.webforum.nu/amne/asp/54759-snygga-upp-koden"
---

# Snygga upp koden

## #1 — medialabs, 2002-10-01T12:14Z

hej!

Skulle nån vilja ta sig en titt på denna kod och se om man kan snygga upp i asp koden.

```

<!-- #include file = "../inc/adovbs.inc" -->
<!-- #include file = "artikel_losenord.asp" -->

<%
IF Request.QueryString("alt") = "login" THEN
	IF Request.Form("losen") = losenord AND Request.Form("anv") = anvandare THEN
		Session("access") = "YaHoO"
	END IF
END IF
%>

<body bgcolor="#FFFFFF">
<p align="center"><font face="Verdana" size="6">Artiklar<b><font face="Verdana" size="1">
<script language="JavaScript" type="text/JavaScript" src="../js/funktioner.js"></script>
  </font></b><br>
  <font size="1"> <font size="2">Administration</font></font></font></p>
<%IF Session("access") = "YaHoO" THEN%>

<div align="center"></div>
<div align="center"> 

  <%
Set Connect = Server.CreateObject("ADODB.Connection")
Connect.Open "Driver={Microsoft Access Driver (*.mdb)}; DBQ=" & server.mappath("kontakter.mdb") 

IF Request.Querystring("alt") ="del" Then 
SQL = "DELETE * FROM kunder WHERE ID = " & Request.Querystring("id")
Connect.execute(SQL)
Response.Redirect "artikel_admin.asp"
ELSE

SQL = "SELECT * From kunder ORDER BY Kontaktperson"
Set RecSet = Connect.Execute(SQL)

DO UNTIL RecSet.EOF %>
  <tr> 
    <table width="300" border="0" style="border: #330099 1px solid" align="center">
      <td> 
          <table border="0" align="center">
            <tr> 
              <td> 
                <div align="center"><b><font size="1" face="Verdana">Artikel</font></b><font size="1" face="Verdana"><br>
                  <a href="javascript:popup('las.asp?id=<%=RecSet("id")%>',500,500);"><%=RecSet("Foretagsnamn")%></a></font></div>
              </td>
            </tr>
          </table>
      </td>
      </tr>
      <tr> 
        <td height="12"> 
          <div align="center"><font size="1" face="Verdana"><b><a href="artikel_admin.asp?alt=del&id=2" onClick="return confirm('säkert?');">Radera</a>
            / <a href="artikel_andra.asp?id=<%=RecSet("id")%>">&Auml;ndra</a></b></font></div>
        </td>
      </tr>
    </table>
  <font size="1" face="Verdana">&nbsp;&nbsp;&nbsp; </font><br>
  <tr>
    <%
RecSet.MoveNext
Loop

RecSet.Close
Set RecSet = nothing
Connect.Close
Set Connect = nothing
END IF
%>
</div>
<%ELSE%>
<form name="form2" method="post" action="artikel_admin.asp?alt=login">
  <font size="1"> <font face="Verdana, Arial, Helvetica, sans-serif"> </font></font> 
  <table border="0" align="center">
    <tr> 
      <td colspan="2"> 
        <div align="center"><font size="1"></font> <font size="1"><font face="Verdana"><b>LOGGA 
          IN</b></font></font></div>
      </td>
    </tr>
    <tr> 
      <td><font size="1"><font face="Verdana, Arial, Helvetica, sans-serif">Anv. 
        namn:</font></font></td>
      <td><font size="1"><font size="1"><font face="Verdana, Arial, Helvetica, sans-serif"> 
        <input type="text" name="anv" style="border: #000000 1px solid; font-family: verdana; font-size: 10px" size="15" maxlength="20">
        </font></font></font></td>
    </tr>
    <tr> 
      <td><font size="1"><font face="Verdana, Arial, Helvetica, sans-serif">L&ouml;senord:</font></font></td>
      <td><font size="1"><font face="Verdana, Arial, Helvetica, sans-serif"> </font><font size="1"><font face="Verdana, Arial, Helvetica, sans-serif"> 
        <input type="text" name="losen" style="border: #000000 1px solid; font-family: verdana; font-size: 10px" size="15" maxlength="20">
        </font></font></font></td>
    </tr>
    <tr> 
      <td>&nbsp;</td>
      <td><font size="1"><font face="Verdana, Arial, Helvetica, sans-serif"> 
        <input type="submit" style="border: #000000 1px solid; font-family: verdana; font-size: 10px" name="Submit2" value="Logga in">
        </font></font></td>
    </tr>
    <tr> 
      <td colspan="2">
        <div align="center"><font face="Verdana" size="1">(<b>admin</b> p&aring; 
          b&aring;da &auml;r f&ouml;rvalt)</font></div>
      </td>
    </tr>
  </table>
</form>
<% END IF %>
```

Permalänk: https://www.webforum.nu/p/54759

## #2 — m_soderlund, 2002-10-01T13:10Z

Du kan ju till att börja med skriva \<%Option Explicit%\> i början av dokumentet så att du måste deklarera alla variabler. Det ger en liten prestandaökning..
(Dim Connect, SQL....)

Permalänk: https://www.webforum.nu/p/804502

## #3 — fundrusse, 2002-10-01T15:21Z

Det ger ingen prestandaökning alls..
Det ända syftet med att dimma grejorna är att man skall kunna felsöka lättare..

Dvs programmera med Explicit

Permalänk: https://www.webforum.nu/p/804649

## #4 — voigtann1, 2002-10-01T15:22Z

du använder ju inget ifrån
\<!-- #include file = "../inc/adovbs.inc" --\>

Permalänk: https://www.webforum.nu/p/804653

## #5 — Josef, 2002-10-01T15:23Z

Men jag har för mig att asp.dll hanterar variablerna snabbare om de finns deklarerade... Men jag kanske kommer ihåg fel.

Permalänk: https://www.webforum.nu/p/804654

## #6 — Denniz PoP, 2002-10-01T15:32Z

> **Josef skrev:**
>
> Men jag har för mig att asp.dll hanterar variablerna snabbare om de finns deklarerade... Men jag kanske kommer ihåg fel.

Det har jag också för mig :)

Typ att den hela tiden måste kontrollera om varibeln existerar eller inte om man inte använder Option Explicit. Om man använder det så vet den hela tiden om varibeln i fråga existerar eller inte, därav en liten prestandaökning.

Permalänk: https://www.webforum.nu/p/804672

## #7 — crisse6, 2002-10-01T21:15Z

snygga upp din kod med stylesheets ;) jakla massa font face.... och en del i styles i dina tabeller...

Permalänk: https://www.webforum.nu/p/805071

## #8 — Josef, 2002-10-01T21:19Z

...och så blir det mycket lättare att ändra senare. Byta färger, fonter osv...

Permalänk: https://www.webforum.nu/p/805076

---

Tråden på webben: https://www.webforum.nu/amne/asp/54759-snygga-upp-koden
