Jump to content
Sign in to follow this  
13lack13lade

variable not declared?

Recommended Posts

13lack13lade

Hey guys,

not sure what im doing wrong but im getting a variable $type = _ExcelReadCell($oExcel, ^ error

#include <Excel.au3>
#include <ff.au3>

_FFConnect()
    ; open a page
    $oExcel = _ExcelBookOpen("Q:\Documents\Load Support\Tom\Projects\Current\longlats.xlsx")

    $type = _ExcelReadCell($oExcel, $i, 1)
    $city = _ExcelReadCell($oExcel, $i, 2)
    $supp = _ExcelReadCell($oExcel, $i, 3)
    $long = _ExcelReadCell($oExcel, $i, 4)
    $lati = _ExcelReadCell($oExcel, $i, 5)
    
    sleep(5000)
For $i = 1 To 4887 ;Loop
_FFOpenURL("https://www.calypso.net.au/apps/cpl/72.5/fcl/cpl.do#HdcPropertyEditPlace:" & $type & "/" & $city & "/" & $supp)
;script
sleep(5000)
send("^f")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("location")
sleep(1000)
highlightclick()
sleep(1000)
send("^f")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("edit")
sleep(1000)
highlightclick()
sleep(3000)
MouseClick ( "left" [, 928, 674 [, clicks = 1 [, speed = 10]]] )
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send($lati);latitude
MouseClick ( "left" [, 1127, 675 [, clicks = 1 [, speed = 10]]] )
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send($long);longitude
send("^f")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("{BS}")
send("save")
highlightclick()
next

Share this post


Link to post
Share on other sites
water

What's the value of $i when you call _ExcelReadCell?


My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (2018-06-01 - Version 1.4.9.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2018-09-01 - Version 1.3.4.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2017-07-21 - Version 0.4.0.1) - Download - General Help & Support - Example Scripts
PowerPoint (2017-06-06 - Version 0.0.5.0) - Download - General Help & Support
Excel - Example Scripts - Wiki
Word - Wiki
 
Tutorials:

ADO - Wiki

 

Share this post


Link to post
Share on other sites
13lack13lade

i just want it to read rows 1-4887 on a loop..

For $i = 1 To 4887 ;Loop

Edited by 13lack13lade

Share this post


Link to post
Share on other sites
water

What is the value of $i when executing this line?

$type = _ExcelReadCell($oExcel, $i, 1)

My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (2018-06-01 - Version 1.4.9.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2018-09-01 - Version 1.3.4.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2017-07-21 - Version 0.4.0.1) - Download - General Help & Support - Example Scripts
PowerPoint (2017-06-06 - Version 0.0.5.0) - Download - General Help & Support
Excel - Example Scripts - Wiki
Word - Wiki
 
Tutorials:

ADO - Wiki

 

Share this post


Link to post
Share on other sites
13lack13lade

will start at 1 then loop through to 4887 thats why i have the for $i = 1 to 4887 or is that not right:?

EDIT: never mind i see what you mean, gotta include it in the loop! lol cheers water!

Edited by 13lack13lade

Share this post


Link to post
Share on other sites
water

Fine ;)

If reading 5000 rows is too slow I suggest to have a look at my rewrite of the Excel UDF. Is about 20 to 50 times faster when reading an array of cells.


My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (2018-06-01 - Version 1.4.9.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2018-09-01 - Version 1.3.4.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2017-07-21 - Version 0.4.0.1) - Download - General Help & Support - Example Scripts
PowerPoint (2017-06-06 - Version 0.0.5.0) - Download - General Help & Support
Excel - Example Scripts - Wiki
Word - Wiki
 
Tutorials:

ADO - Wiki

 

Share this post


Link to post
Share on other sites
BrewManNH

This line is written completely wrong.

MouseClick ( "left" [, 928, 674 [, clicks = 1 [, speed = 10]]] )

It needs to be written like this.

MouseClick ( "left" , 928, 674 , 1 , 10)

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
13lack13lade

 

This line is written completely wrong.

MouseClick ( "left" [, 928, 674 [, clicks = 1 [, speed = 10]]] )

It needs to be written like this.

MouseClick ( "left" , 928, 674 , 1 , 10)

just figure that out too lol.. feel like an idiot for making rookie mistakes:/ lol  thanks guys

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
Sign in to follow this  

×