Peter,
My first thoughts were that the following is more readable?
strIn = "in("
If Not IsNull(Me.myfield1) Then strIn = strIn & Me.myfield1 & ", "
If Not IsNull(Me.myfield2) Then strIn = strIn & Me.myfield2 & ", "
strIn = strIn & ")"
But then I slept on it and began to question the need for...