Can you check my code?

[qwerty]

Is my coding correct,

If Not Form_002_Criteria.TxtPop.Value = "" Then


' Me.Filter = "[Criteria].[PopMin]='" & [Criteria].[PopMax]='" & Form_002_Criteria.TxtPop.Value & "'"
' Me.FilterOn = True

[CJ_London]

two rows are commented out and you have an if without an end, so not much to go on but I would say no, not correct.

[pbaldy]

And your filter doesn't make sense. Plus your test doesn't account for Null. This would be better:

If Len(Form_002_Criteria.TxtPop.Value & vbNullString) > 0 Then

[qwerty]

What would be the corrected version?

[CJ_London]

If you ask one sentence questions with minimal code and descriptions, don't expect anything more than a minimal response.

Syntactically, your code is confusing so you need to explain what the filter supposed to do (other than 'filter') i.e. describe as a sentence and provide some example data and what outcome is required. Also what are the datatypes involved? where is the code? on a form? on a button/some other control click event.

[qwerty]

If you ask one sentence questions with minimal code and descriptions, don't expect anything more than a minimal response.

Syntactically, your code is confusing so you need to explain what the filter supposed to do (other than 'filter') i.e. describe as a sentence and provide some example data and what outcome is required. Also what are the datatypes involved? where is the code? on a form? on a button/some other control click event.

Why are you so aggressive? I am just what needs to be corrected that's, all. I did provide everything in that code. I just need to know what needs to be corrected.

[qwerty]

Why are you so aggressive? I am just what needs to be corrected that's, all. I did provide everything in that code. I just need to know what needs to be corrected.

The comment is a big help, but I don't understand where they begin and end. Can you correct the comments?

[CJ_London]

no - because I don't know what the code is supposed to do.

[qwerty]

no - because I don't know what the code is supposed to do.

I thought you are a PRO? The text box suppose to filter based on a values from two field.

Text box name = TxTPop
Field names = PopMin and PopMax

[CJ_London]

I am a PRO, and if I don't understand something, I ask to get clarification. I provide my help for free and don't like to see my time wasted so I am going to disconnect from this thread since you are reluctant to provide the answers. If you want to pay me as a professional to get something done, happy to provide you with a quote.

[pbaldy]

I'm also a PRO and I also couldn't understand what you were trying to accomplish. Being a pro doesn't mean being a mind reader. I gave you an answer here 10 days ago:

https://www.accessforums.net/showthr...240#post413240

[apr pillai]

Me.Filter = "[Criteria].[PopMin]='" & [Criteria].[PopMax]='" & Form_002_Criteria.TxtPop.Value & "'"

Me.Filter = "[Criteria].[PopMin]='" & [NEED VALUE FROM FIELD HERE] & "' OR " & "[Criteria].[PopMax]='" & Form_002_Criteria.TxtPop.Value & "'"