Как я могу оптимизировать эту глючную логику? - PullRequest
Новая
       0

Как я могу оптимизировать эту глючную логику?

0 голосов
/ 08 июля 2011

Я имею дело с данными формы примерно с 30 экземплярами этих наборов из 4 полей (каждое из разных имен и функций).Есть ли способ сделать это более управляемым?

<!--- Lifeguard Instructor --->
<!--- Is the date defined? --->
<cfif len(Trim(form.lifeguardInstrcutorExp)) EQ 0>
    <cfinvokeargument name="lifeguardInstrcutorExp" 
        value="#defaultDate#">
<cfelse>
    <cfinvokeargument name="lifeguardInstrcutorExp" 
        value="#CreateODBCDate(Form.lifeguardInstrcutorExp)#">
</cfif>
<!--- Is a Company defined? --->
<cfif len(Trim(form.lifeguardInstrcutorCompany)) EQ 0>
    <cfinvokeargument name="lifeguardInstrcutorCompany" value="">
<cfelse>
    <cfinvokeargument name="lifeguardInstrcutorCompany" 
        value="#Trim(Form.lifeguardInstrcutorCompany)#">
</cfif>
<!--- Has a file been specificed? --->
<cfif not len(Trim(form.lifeguardInstrcutorImage)) EQ 0>
    <cffile action="upload" accept="#defaultFileAccepted#" 
        filefield="lifeguardInstrcutorImage" 
        destination="#destination#" 
        nameConflict="makeUnique">
    <cfinvokeargument name="lifeguardInstrcutorImage" 
        value="#pathOfFile##cffile.serverFile#">
<cfelse>
</cfif>
<!--- Do We have a hard copy? --->
<cfinvokeargument name="lifeguardInstrcutorOnFile" 
    value="#Trim(form.lifeguardInstrcutorOnFile)#">

1 Ответ

0 голосов
/ 18 июля 2011

Я хотел бы, чтобы вызываемая вами функция имела аргументы с параметрами по умолчанию, а затем передавал бы форму в качестве набора аргументов.

Функция: 

<cffunction name="myFunctionUsingForm">
        <cfargument name"lifeguardInstructorExp" type="string" default="#defaultDate#" />
        <cfargument name"lifeguardInstrcutorCompany" type="string" default="" />
        <cfargument name"lifeguardInstrcutorImage" type="any" default="" />
        <cfargument name"lifeguardInstrcutorOnFile" type="boolean" default="false" />

Призвание: 

<cfset myFunctionUsingForm(argumentCollection=form) />
...