trying to shorten code to get values from userform

[smokebreak]

Below I have posted my code. The commented out section works just fine, but it is very lengthy.... The part that is not commented out is what I need to fix.

here is my setup: I have a userform, each of the fields is custom named. I have those names listed across the top of a spreadsheet. My thought is to put those field names from the spreadsheet "QS" into an array "fieldval()" and then use the names in that array to return a value from the userform into a cell on the same spreadsheet.

When I run the code it tells me that fieldval() is an invalid qualifier. How do I fix this?

Please Login or Register  to view this content.

[Leith Ross]

Hello smokebreak,

There is nothing in the posted code to suggest that fieldval is a qualifier. This would be easier to sort out if everything where available. Can you post the workbook?

[smokebreak]

Leith,
The sheet is attached.

The code is on the userform module, on the btn_Submit_Click() event.

It's kind of a lot of code, I hope you will not have an issue making sense of it.

Thanks,
Kevin

[Leith Ross]

Hello smokebreak,

Copying the values from so many controls is going to make your code long no matter want you do. You can speed the process by using an array. Once the array is loaded it can be assigned directly to a range, so no loop is needed.

There were a couple of problems in the code, specifically with variable typing. The types for firstempty, lastcol, and fieldval() were incorrect. Those have been corrected. Here is the corrected code.

Please Login or Register  to view this content.