Hi,
I have a inputbox in start of macro where user should enter a number. 5, 10 and 15 are allowed values so I want to test if user have entered any of those
This is working good enough for me but wonders if this is considered as bad coding? I have heard that GoTo shouldnt be used.
Please Login or Register to view this content.
Hi,
Yes there's a school of thought that considers GOTO stuff to be bad coding which can lead to 'spaghetti' code making de-bugging and reading code difficult.
Presumably to enter your macro the user has to click a button or something so why not just tell them what's wrong and invite them to click again. In which case something like the following is arguably better.
Please Login or Register to view this content.
Richard Buttrey
RIP - d. 06/10/2022
If any of the responses have helped then please consider rating them by clicking the small star iconbelow the post.
But I would need macro to start from line 1 if user enters wrong value. Macro runs when workbook opens so there is no button to activate macro. Thats why I liked GoTo statement.
CLng is needed to convert input to integer? 5 would otherwise be considered as a string?
Try using a while statement;
Please Login or Register to view this content.
One spreadsheet to rule them all. One spreadsheet to find them. One spreadsheet to bring them all and at corporate, bind them.
A picture is worth a thousand words, but a sample spreadsheet is more likely to be worked on.
There are currently 1 users browsing this thread. (0 members and 1 guests)
Posting Permissions
LinkBack URL
About LinkBacks
Register To Reply
Bookmarks