---
title: "Funktionsanrop"
type: "forum-thread"
url: "https://www.webforum.nu/amne/asp/167622-funktionsanrop"
topic: "ASP"
topic_url: "https://www.webforum.nu/amne/asp"
author: "devotion"
published: "2007-12-28T13:29:11.000Z"
updated: "2007-12-28T14:04:13.000Z"
replies: 4
views: 372
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/167622-funktionsanrop"
---

# Funktionsanrop

## #1 — devotion, 2007-12-28T13:29Z

Hej!

Jag har en funktion som anropas så här:

```
lExternalTaskId=CreateNewTask(sCustomerNr, sCustomerName, sCustomerOrderer, nPlanDateType, sPriority, tmPlanStartDate, tmPlanStopDate, sTask, sCustomT1, sCustomT2, sCustomT3, sTaskNote)
```

det som händer egentligen, är att en rad "hämtas" från en tabell/databas och "läggs till" en annan...

Allt fungerar finfint, men jag tycker det ser så fult ut.. :) Det blir så många parametrar...

Kan man gör på något annat sätt...

Hela funktionen ser ut så här:

```
'******************************
'Funktion för att skapa nytt uppdrag
'
'******************************
Function CreateNewTask(sCustomerNr, sCustomer, sCustomerOrderer, nPlanDateType, sPriority, tmPlanStartDate, tmPlanStopDate, sTask, sCustomT1, sCustomT2, sCustomT3, sTaskNote)
	dim lNextId
	dim sSQL
	dim sUser, sOwner, sCreatedBy
	
	sUser="(Kö)"
	sOwner="(Kö)"
	sCreatedBy="*********"
	
	lNextId=GetNextExternalId("tblTask")
	
	sSQL="INSERT INTO [Uppdrag NU Pro v3 Testföretag 2].dbo.[tblTask] ([Id], [Modified], [Status], [TaskNr], [Priority], [PlanDateType], [PlanStartDate], [PlanStopDate], [CreatedBy], [CreatedDate], [Owner], [User], [CustomerNr], [CustomerName], [CustomerOrderer], [Task], [CustomT1], [CustomT2], [CustomT3], [TaskNote] ) VALUES (" _
		& lNextId & ", GetDate(), 2, '" & GetNextTaskNr() & "', " & sPriority & ", " & nPlanDateType & ", '" & tmPlanStartDate & "', '" & tmPlanStopDate & "', '" & sCreatedBy & "', GetDate(), '" & sOwner &"', '" & sUser & "', '" & sCustomerNr & "', '" & sCustomer & "', '" & sCustomerOrderer & "', '" & sTask & "', '" & sCustomT1 & "', '" & sCustomT2 & "', '" & sCustomT3 &"', '" & sTaskNote &"')"
	ExecuteSQL(sSQL)
	
	CreateNewTask=lNextId
	
End function
```

Mvh
Henrik

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

## #2 — voigtann1, 2007-12-28T13:31Z

Kan lägga den i en Dictionary.. men egentligen: Om du måste ha alla värden så måste du ju ändå ha alla värden.

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

## #3 — devotion, 2007-12-28T13:34Z

Jo, det är väl så jag har tänkt, att allt måste med... Det blir så klumpigt bara.. Och blandar man i Dictionary, så kanske koden blir ännu mer svåröverskådlig....

  :)  

Mvh
Henrik

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

## #4 — voigtann1, 2007-12-28T13:38Z

> **devotion skrev:**
>
> Jo, det är väl så jag har tänkt, att allt måste med... Det blir så klumpigt bara.. Och blandar man i Dictionary, så kanske koden blir ännu mer svåröverskådlig....
> 
>   :)  
> 
> Mvh
> Henrik

Om allt måste med så går det ju knappast att korta ner in-datan ;) eller? 
Dictionary är inte direkt svår att se... men slutresultatet blir det samma: Måste fylla dictionary innan och sen hämta ut det igen. Tycker du kan behålla din struktur som det är.

Du kan ju även skapa egna klass-objekt för att ha som in data.. lite mer kod för klasser men då gör du "custom"-klass som faktiskt ha alla värden du vill ha... även om dom nu skulle vara tomma. Kanske lite onödigt att skapa en klass just bara för detta men om du hittar andra användningsområden i ditt projekt kan det kanske vara något att rekommendera?

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

## #5 — devotion, 2007-12-28T14:04Z

> **voigtann1 skrev:**
>
> Om allt måste med så går det ju knappast att korta ner in-datan ;) eller?

  :r 

Nä, jag vet... Tänkte bara om man kunde göra på annat sätt... :)   

Mvh
Henrik

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

---

Tråden på webben: https://www.webforum.nu/amne/asp/167622-funktionsanrop