BetaLeaf

[Solved] For Next Loop forgetting it's current $i position (Possible memory issue)

5 posts in this topic

#1 ·  Posted (edited)

So this is a script that reduces memory for processes stored in the ini file. The problem lies when it tries to reduce memory for steam.exe (9th on the list). It reduces it's memory, then forgets what to process next and goes straight to explorer.exe (3rd on the list). Because of this, it never reduces memory for processes after steam.exe on the ini list.

I have included my script and ini file for you to take a look at. Any help would be greatly appreciated.

Reduce Memory.au3

Reduce Memory.ini

Edited by BetaLeaf
Solved

My Scripts:

False Positive Reporter - Mass email all anti virus vendors with an attachment of your program for easy whitelisting.

PortableApps.com App Creation Wizard  - A simple GUI-based Wizard for creating PortableApps.

AutoISO  - Automatic ISO Image creation using ImgBurn.

SoundBoard - Play any song or sound you want at the press of a hotkey

Share this post


Link to post
Share on other sites



#2 ·  Posted (edited)

For $i = 1 To ...
    For $i = 1 To ...
        
    Next 
Next

Your nested for loop is reusing $i, rename to $j or some other arbitrary letter.

Edited by Tekk
1 person likes this

Share this post


Link to post
Share on other sites

I cannot believe I missed something so obvious. Man, I feel stupid. Thanks @Tekk.


My Scripts:

False Positive Reporter - Mass email all anti virus vendors with an attachment of your program for easy whitelisting.

PortableApps.com App Creation Wizard  - A simple GUI-based Wizard for creating PortableApps.

AutoISO  - Automatic ISO Image creation using ImgBurn.

SoundBoard - Play any song or sound you want at the press of a hotkey

Share this post


Link to post
Share on other sites

Just an FYI, EmptyWorkingSet isn't going to help with anything. It's a bandaid used to fix poor programming, and doesn't really reduce the memory being used by a program, it just shifts it from RAM to the much slower hard drive. It will cause an impact on the programs you're using it on, and may cause problems.


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

Nice to know. In my case though, I run out of memory and apps crash. Even if it is slower, it helps.


My Scripts:

False Positive Reporter - Mass email all anti virus vendors with an attachment of your program for easy whitelisting.

PortableApps.com App Creation Wizard  - A simple GUI-based Wizard for creating PortableApps.

AutoISO  - Automatic ISO Image creation using ImgBurn.

SoundBoard - Play any song or sound you want at the press of a hotkey

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