Originally Posted By: Glenn Barnas
If you're using my IniArray UDF, then the IniSections and IniKeys UDFs are redundant - you already have all of the data in the master array that was loaded. By using these extra UDFs, you're losing efficiency by loading the INI file into the array, then enumerating the INI file to locate data that's in your array.

I'm not sure what you mean here. I'm only reading the file once. Those two functions just take the default delimited string return from readiniarray function and puts it back into an array.


 Quote:
This UDF was updated just today, so you should pull the latest copy after 10pm EST tonight when it gets uploaded to our web site. The update improves performance for larger files and resoves a recently discovered bug that will appear when a section is defined in the file without any keys associated with it.

I put a link in my post the the UDF I'm using. It says it hasn't been updated since 8-2011.


 Quote:
You shouldn't have to do all the "UCase" commands as text comparisons are case insensitive unless you use a SetOption call to enable case sensitivity.

Yea, there are a lot of little things I do that are unnecessary, I know. Sometimes it's just for readability, but often it's because I write code primarily in PHP where it does matter a lot and it's just habit.


 Quote:
It's generally good practice to leave the headers in the UDFs as they contain important reference material.

I left links to all the UDFs in my post. The headers are quite long and I was already making a pretty lengthy post \:\)


 Quote:
You should never declare a GLOBAL var inside a function. Doing so will cause the var to be declared each time the function is called causing a Duplicate Definition error.

Good to know, I'll move it out.


 Quote:
I haven't reviewed the code in depth, but first blush it seems more complicated than necessary. If your computer name identifies the school and classroom along with a unique PC identifier, then a single config file could be used with one section per location and one key per classroom.

See my my reply to Lonkero above. I had only pasted a snipit of my full INI. The goal was to have as few sections in the INI as possible.