Actually the control on the form is "cboGenus". Had it been called "Genus" the line would have worked other than a possible conflict of names as the field in the table is called "Genus"
That code segment, along with many others, will be re-used on other forms. I am not the best coder around so when something does as required I re-use it whenever possible. I have many forms performing different operations on the same table so re-using the code makes sense to me.Looking at your code, I question why you chose to make this a separate, public Sub instead of within the Form's Class Module?
There is no Form being passed into it and it is clear this Sub is specially designed to work with a certain form.
I don't think it will effect performance either way but why separate the functionality unnecessarily?