Gasman
Enthusiastic Amateur
- Local time
- Today, 01:13
- Joined
- Sep 21, 2011
- Messages
- 17,535
Code looks OK to me, but it depends on what is in the fields when that code is run.
If I was using that form, I would expect to have to select Copy and then enter my data. I would then leave it selected until I wanted to clear the fields, so I would only ever touch that field once in a while. The first time I used it, there would be nothing to copy and it would clear the fields when I deselected it.
Also you do not need two If statements, you could use the Else statement for when it chkCopy is set.
If I was using that form, I would expect to have to select Copy and then enter my data. I would then leave it selected until I wanted to clear the fields, so I would only ever touch that field once in a while. The first time I used it, there would be nothing to copy and it would clear the fields when I deselected it.
Also you do not need two If statements, you could use the Else statement for when it chkCopy is set.