faustf

questions about byref

5 posts in this topic

hi guy i have  a   script  

 

with   2  function  

the  first is 

Func _Connection_Open($ConnectionString)
    Local $oErrorHandler = ObjEvent("AutoIt.Error", "_ErrFunc")

     $objConn = ObjCreate("ADODB.Connection")
     $objConn.Open($ConnectionString)
    If @error Then Return SetError($ADO_ERR_CONNECTION, @error, $ADO_RET_FAILURE)

    Return SetError($ADO_ERR_SUCCESS, $ADO_EXT_DEFAULT, $ADO_RET_SUCCESS)

EndFunc   ;==>_Connection_Open

and  second  function is  

 

Func _SQL_Query($sql_query)

    $rsCustomers = $objConn.Execute($sql_query)
    With $rsCustomers
        While Not .EOF
            ConsoleWrite(.Fields("id").Value & " - " & .Fields("code").Value & @LF)
            ;$ps_Country_ID_array[$b] = (.Fields("id").Value)
            ;$b += 1
            .MoveNext
        WEnd
        .Close
    EndWith

    EndFunc

when  a  run  the  script  and  call the  first  function go  all ok ,  when i call a  second  give me error , $objConn undeclare global variable ,  so i think  i must  pass $objConn by the  first to second ,  and  i think  use  byref , but  i dont know  how  

some  one  can  explain me ??  i try  to red  a wiki  ,  and   i try to put  in first    Func _Connection_Open(byref $objConn ,$ConnectionString) ,  but  not  work  good 

 

 

Share this post


Link to post
Share on other sites



The only way to pass the variable byref is to call the second function from the first. But unless you set the function parameter of the second function to the same name as the variable in the calling function (both would have to be called $objConn), you still wouldn't be able to use the variable name in the second function. In your case you probably don't want to transfer it byref because you have the possibility to change the contents of the variable in the second function.

If the first function doesn't directly call the second function, you will have to use a global variable.

Func _Connection_Open($ConnectionString)
    Local $oErrorHandler = ObjEvent("AutoIt.Error", "_ErrFunc")

    $objConn = ObjCreate("ADODB.Connection")
    $objConn.Open($ConnectionString)
    If @error Then Return SetError($ADO_ERR_CONNECTION, @error, $ADO_RET_FAILURE)
    _SQL_Query("query string", $objConn) ; <<<<<<<<<<<<<<<<<<<<<< Something like this.
    Return SetError($ADO_ERR_SUCCESS, $ADO_EXT_DEFAULT, $ADO_RET_SUCCESS)

EndFunc   ;==>_Connection_Open
Func _SQL_Query($sql_query, $objConn)

    $rsCustomers = $objConn.Execute($sql_query)
    With $rsCustomers
        While Not .EOF
            ConsoleWrite(.Fields("id").Value & " - " & .Fields("code").Value & @LF)
            ;$ps_Country_ID_array[$b] = (.Fields("id").Value)
            ;$b += 1
            .MoveNext
        WEnd
        .Close
    EndWith

EndFunc   ;==>_SQL_Query

 


If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.
Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag Gude
How to ask questions the smart way!

I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from.

Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays.  -  ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script.  -  Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label.  -  _FileGetProperty - Retrieve the properties of a file  -  SciTE Toolbar - A toolbar demo for use with the SciTE editor  -  GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI.  -   Latin Square password generator

Share this post


Link to post
Share on other sites

yes  but  i  dont  want  use global  because   this   couple function is inside  the  udf  , and  usally   is much better  if in udf not  use global variable ,  not  exist  a  sy some  word of  command  ???   thankz  at  all 

 

Share this post


Link to post
Share on other sites

Global variables exist for a reason, that reason is that 2 functions can't share local variables, or their contents, unless one function calls the other function and passes the contents of that variable to the second function. If the 2 functions aren't talking directly to each other you MUST use a global variable.

The only other way to make it work is if you have a third function, that calls the first function, that first function returns the contents of the variable you want to use in the second function back to the calling function, and then you call the second function using the returned value(s).


If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.
Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag Gude
How to ask questions the smart way!

I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from.

Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays.  -  ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script.  -  Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label.  -  _FileGetProperty - Retrieve the properties of a file  -  SciTE Toolbar - A toolbar demo for use with the SciTE editor  -  GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI.  -   Latin Square password generator

Share this post


Link to post
Share on other sites

#5 ·  Posted (edited)

I see you're using part of my UDF, but you changed it.
The question is why ? What was your intention ?
 

If you need your own "easier to use" Function then you should make your functions based on UDF function. So you should: create wrapper.


By the way.
I use this UDF for a long time (started from _sql.au3 and last few months I changed almost everything making ADO.au3 ), and for this UDF I did wrapper, created under my specific CRM application.

 

EDIT:
If you make description of what you care then I'll try to make examples of how to adapt ADO.au3 to the specific case.

But first you must simply answer/give some feedback in your previous Threads which you open last days.
I simply need to know if my examples works for you (answer in the desired therad where I post example)

Edited by mLipok

Signature beginning:   Wondering who uses AutoIT and what it can be used for ?
* GHAPI UDF - modest begining - comunication with GitHub REST API *
ADO.au3 UDF     POP3.au3 UDF     XML.au3 UDF    How to use IE.au3  UDF with  AutoIt v3.3.14.x  for other useful stuff click the following button

Spoiler

Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind. 

My contribution (my own projects): * Debenu Quick PDF Library - UDF * Debenu PDF Viewer SDK - UDF * Acrobat Reader - ActiveX Viewer * UDF for PDFCreator v1.x.x * XZip - UDF * AppCompatFlags UDF * CrowdinAPI UDF * _WinMergeCompare2Files() * _JavaExceptionAdd() * _IsBeta() * Writing DPI Awareness App - workaround * _AutoIt_RequiredVersion() * Chilkatsoft.au3 UDF * TeamViewer.au3 UDF * JavaManagement UDF * VIES over SOAP * WinSCP UDF * GHAPI UDF - modest begining - comunication with GitHub REST API *

My contribution to others projects or UDF based on  others projects: * _sql.au3 UDF  * POP3.au3 UDF *  RTF Printer - UDF * XML.au3 - BETA * ADO.au3 UDF SMTP Mailer UDF *

Useful links: * Forum Rules * Forum etiquette *  Forum Information and FAQs * How to post code on the forum * AutoIt Online Documentation * AutoIt Online Beta Documentation * SciTE4AutoIt3 getting started * Convert text blocks to AutoIt code * Games made in Autoit * Programming related sites * Polish AutoIt Tutorial * DllCall Code Generator * 

Wiki: Expand your knowledge - AutoIt Wiki * Collection of User Defined Functions * How to use HelpFile * Best coding practices * 

IE Related:  * How to use IE.au3  UDF with  AutoIt v3.3.14.x * Why isn't Autoit able to click a Javascript Dialog? * Clicking javascript button with no ID * IE document >> save as MHT file * IETab Switcher (by LarsJ ) * HTML Entities * _IEquerySelectorAll() (by uncommon) * 

I encourage you to read: * Global Vars * Best Coding Practices * Please explain code used in Help file for several File functions * OOP-like approach in AutoIt * UDF-Spec Questions *  EXAMPLE: How To Catch ConsoleWrite() output to a file or to CMD *

"Homo sum; humani nil a me alienum puto" - Publius Terentius Afer
"Program are meant to be read by humans and only incidentally for computers and execute" - Donald Knuth, "The Art of Computer Programming"
:naughty:  :ranting:, be  :) and       \\//_.

Anticipating Errors :  "Any program that accepts data from a user must include code to validate that data before sending it to the data store. You cannot rely on the data store, ...., or even your programming language to notify you of problems. You must check every byte entered by your users, making sure that data is the correct type for its field and that required fields are not empty."

Signature last update: 2017-06-04

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now