No, no need for a different form. From what I can see, it looks like you're trying to create a filter based on user input.
So somewhere in your form you have these 5 boxes in which the user can fill out, click a button, and the form will display results based on these selections. This a pretty standard thing for forms, maybe even the most common use, so you definitely don't need a new form.
In any case, there's several different ways to go about this. But first, let's start conceptually with why your code is doing anything.
Your machine and Access are not inherently smart by any means. They will do only what you tell it to do and only if it understands what you're asking it.
By the looks of your code, it is doing something. It's building a string based on the user input. If you put the code
right before the End Sub statement and click your button, it should show you what your code built as the string filter.
But it looks like that's all your asking the code to do, just build the string, which it probably is. The real question now becomes how do you make that string act as a filter? This where you have to make a choice on how you want your form to operate. From the looks of the way you built the string, your goal is set the filter property of a bound form. In this case, put the code supplied by Steve Kogan above to the end of your sub. You may also need to add
after that, I'm not sure. If your string is built correctly so that the form understands it, it will work.