Please scrutinize this simple working function and give your suggestions

[onmyway]

Hi guys,

My i ask that you scrutinize this function for me. Please give me your input and suggestions.

Although the code works just fine, I am scared that I am teaching myself methods of doing things/writing code that are not entirely wholesome and helpful. I.e., I am using the wrong methods.

This is what I wanted to achieve (working but to what end?):

Loop through a named range "Locations" (I ended up using the starting row (130)), and find the next empty row in this range. Save my new data to this empty row in this range.

I am making use of ActiveCell, and don't know if this is the right thing. Should one not be using EmptyRow or something like that?

Please Login or Register  to view this content.

thank you!

[davesexcel]

possibly>

Please Login or Register  to view this content.

[onmyway]

Hi Dave

I have a specific range I need to do the last row in: A130:E180

I believe the method you show above, is to find the next row, but make use of xlUp method, and will therefore not take my parameters into account?

[davesexcel]

Changing rows.count to 180 should do it.

Or you can work with this.

Please Login or Register  to view this content.

[onmyway]

Changing rows.count to 180 should do it.

Or you can work with this.

Please Login or Register  to view this content.

Thanks Dave!

Note: I closed this post, as I posted an updated version: http://www.excelforum.com/excel-prog...ml#post4002740