Random Variables not storing

Post by **racqu** » 04 Aug 2022, 13:08

Hello, im still fairly new to AHK and have come across an issue with storing variables. I run defined functions instead of the whole script at once so im thinking it has something to do with that. I am trying to store a random variable outside of the function I am running and then call it within the function. I have tried using the global tag inside the function but it did not work.

I run the Make() function but need it to call the InvX14, InvY14, InvX15, and InvY15 variables but it dosnt seem to work. I also tried creating the inv variables into their own function and placing that as the first run command within the function to try and force it to populate the variables with no success.

If anyone has any ideas or can point me in the right direction so that I can troubleshoot and figure it out myself I would greatly appreciate it!

Code: Select all

; Positions for every inventory square in OSRS. 2560x1440p resolution w/ runelite client Fullscreen and sidebar open

; 35 wide | Left Limit 2082, Right Limit 2244
; Dead space 6 pixels | 2082 - 2117 usable 

;1st - 2082, 2117 | 2nd - 2124, 2159 | 3rd - 2166, 2201 | 4th - 2208, 2244

Random, InvX1, 2086, 2112
Random, InvX2, 2129, 2153
Random, InvX3, 2170, 2197
Random, InvX4, 2212, 2240

Random, InvX5, 2086, 2112
Random, InvX6, 2129, 2153
Random, InvX7, 2170, 2197
Random, InvX8, 2212, 2240

Random, InvX9, 2086, 2112
Random, InvX10, 2129, 2153
Random, InvX11, 2170, 2197
Random, InvX12, 2212, 2240

Random, InvX13, 2086, 2112
Random, InvX14, 2129, 2153
Random, InvX15, 2170, 2197
Random, InvX16, 2212, 2240
 
Random, InvX17, 2086, 2112
Random, InvX18, 2129, 2153
Random, InvX19, 2170, 2197
Random, InvX20, 2212, 2240

Random, InvX21, 2086, 2112
Random, InvX22, 2129, 2153
Random, InvX23, 2170, 2197
Random, InvX24, 2212, 2240

Random, InvX25, 2086, 2112
Random, InvX26, 2129, 2153
Random, InvX27, 2170, 2197
Random, InvX28, 2212, 2240


; 31 tall | Upper Limit 1110, Lower Limit 1357
; Dead space 4 pixels | 1110 - 1141 usable 

;1st row | 1110, 1141
Random, InvY1, 1115, 1138
Random, InvY2, 1115, 1138
Random, InvY3, 1115, 1138
Random, InvY4, 1115, 1138

;2nd row | 1146, 1177
Random, InvY5, 1150, 1173
Random, InvY6, 1150, 1173
Random, InvY7, 1150, 1173
Random, InvY8, 1150, 1173

;3rd row | 1182, 1213
Random, InvY9, 1186, 1208
Random, InvY10, 1186, 1208
Random, InvY11, 1186, 1208
Random, InvY12, 1186, 1208

;4th row | 1218, 1249
Random, InvY13, 1222, 1243
Random, InvY14, 1222, 1243
Random, InvY15, 1222, 1243
Random, InvY16, 1222, 1243

;5th row | 1254, 1285
Random, InvY17, 1258, 1280
Random, InvY18, 1258, 1280
Random, InvY19, 1258, 1280
Random, InvY20, 1258, 1280

;6th row | 1290, 1321
Random, InvY21, 1294, 1317
Random, InvY22, 1294, 1317
Random, InvY23, 1294, 1317
Random, InvY24, 1294, 1317

;7th row | 1326, 1357
Random, InvY25, 1330, 1353
Random, InvY26, 1330, 1353
Random, InvY27, 1330, 1353
Random, InvY28, 1330, 1353


Sleep() {
Random, rand, 750, 1000
Sleep %rand%
}
SleepMake() {
Random, rand, 32000, 35000
Sleep %rand%
}


Make() {


	Loop, {
		Random, BankX, 663, 811
		Random, BankY, 687, 837
		MouseMove, BankX, BankY, 5
		Click
		Sleep()
		Random, DepositX, 1190, 1208
		Random, DepositY, 995, 1014
		MouseMove, DepositX, DepositY, 5
		Click
		Sleep()
		Random, SaltX, 933, 947
		Random, SaltY, 318, 330
		MouseMove, SaltX, SaltY, 5
		Click
		Sleep()
		Random, CompostX, 984,994
		Random, CompostY, 317, 331
		MouseMove, CompostX, CompostY, 5
		Click
		Sleep()
		Send {esc}
		Sleep()
		Mousemove, InvX14, InvY14, 5
		Click
		Sleep()
		MouseMove, InvX15, InvY15, 5
		Click
		SleepMake()
	}
}

GetVariable() {
MsgBox "Variable is" %InvX14%
}



^1::Make()
^2::Reload
^3::ExitApp
^4::GetVariable()
^!p::Pause

Post by **BoBo** » 04 Aug 2022, 15:38

Code: Select all

; Positions for every inventory square in OSRS. 2560x1440p resolution w/ runelite client Fullscreen and sidebar open

; 35 wide | Left Limit 2082, Right Limit 2244
; Dead space 6 pixels | 2082 - 2117 usable 

;1st - 2082, 2117 | 2nd - 2124, 2159 | 3rd - 2166, 2201 | 4th - 2208, 2244

Global InvX14, InvY14, InvX15, InvY15

Random, InvX1, 2086, 2112
Random, InvX2, 2129, 2153
Random, InvX3, 2170, 2197
Random, InvX4, 2212, 2240

Random, InvX5, 2086, 2112
Random, InvX6, 2129, 2153
Random, InvX7, 2170, 2197
Random, InvX8, 2212, 2240

Random, InvX9, 2086, 2112
Random, InvX10, 2129, 2153
Random, InvX11, 2170, 2197
Random, InvX12, 2212, 2240

Random, InvX13, 2086, 2112
Random, InvX14, 2129, 2153
Random, InvX15, 2170, 2197
Random, InvX16, 2212, 2240
 
Random, InvX17, 2086, 2112
Random, InvX18, 2129, 2153
Random, InvX19, 2170, 2197
Random, InvX20, 2212, 2240

Random, InvX21, 2086, 2112
Random, InvX22, 2129, 2153
Random, InvX23, 2170, 2197
Random, InvX24, 2212, 2240

Random, InvX25, 2086, 2112
Random, InvX26, 2129, 2153
Random, InvX27, 2170, 2197
Random, InvX28, 2212, 2240


; 31 tall | Upper Limit 1110, Lower Limit 1357
; Dead space 4 pixels | 1110 - 1141 usable 

;1st row | 1110, 1141
Random, InvY1, 1115, 1138
Random, InvY2, 1115, 1138
Random, InvY3, 1115, 1138
Random, InvY4, 1115, 1138

;2nd row | 1146, 1177
Random, InvY5, 1150, 1173
Random, InvY6, 1150, 1173
Random, InvY7, 1150, 1173
Random, InvY8, 1150, 1173

;3rd row | 1182, 1213
Random, InvY9, 1186, 1208
Random, InvY10, 1186, 1208
Random, InvY11, 1186, 1208
Random, InvY12, 1186, 1208

;4th row | 1218, 1249
Random, InvY13, 1222, 1243
Random, InvY14, 1222, 1243
Random, InvY15, 1222, 1243
Random, InvY16, 1222, 1243

;5th row | 1254, 1285
Random, InvY17, 1258, 1280
Random, InvY18, 1258, 1280
Random, InvY19, 1258, 1280
Random, InvY20, 1258, 1280

;6th row | 1290, 1321
Random, InvY21, 1294, 1317
Random, InvY22, 1294, 1317
Random, InvY23, 1294, 1317
Random, InvY24, 1294, 1317

;7th row | 1326, 1357
Random, InvY25, 1330, 1353
Random, InvY26, 1330, 1353
Random, InvY27, 1330, 1353
Random, InvY28, 1330, 1353


Sleep() {
	Random, rand, 750, 1000
	Sleep %rand%
	}
	
SleepMake() {
	Random, rand, 32000, 35000
	Sleep %rand%
	}


Make() {
	Loop
		{
		Random, BankX, 663, 811
		Random, BankY, 687, 837
		MouseMove, BankX, BankY, 5
		Click
		Sleep()
		Random, DepositX, 1190, 1208
		Random, DepositY, 995, 1014
		MouseMove, DepositX, DepositY, 5
		Click
		Sleep()
		Random, SaltX, 933, 947
		Random, SaltY, 318, 330
		MouseMove, SaltX, SaltY, 5
		Click
		Sleep()
		Random, CompostX, 984,994
		Random, CompostY, 317, 331
		MouseMove, CompostX, CompostY, 5
		Click
		Sleep()
		Send {esc}
		Sleep()
		Mousemove, InvX14, InvY14, 5
		Click
		Sleep()
		MouseMove, InvX15, InvY15, 5
		Click
		SleepMake()
		}
	}

GetVariable() {
	return "Variable is " . InvX14
	}



^1::Make()
^2::Reload
^3::ExitApp
^4::MsgBox % GetVariable()
^!p::Pause

HTH. Not tested.

AutoHotkey Community

Random Variables not storing

Random Variables not storing

Re: Random Variables not storing