regex query implementation

Question

I am trying to check a search box and this is my query which works but has a couple of problems.

 // =================================
 var check = new Regex("[^A-Za-z]");
 Match m = check.Match(searchQuery);
 if (m.Equals(check) != true)
 {
     pnl_Message.Visible = true;
     pnl_Message.CssClass = "messageTable";
     lbl_message.Text = " * Only Letters are allowed";
  }

The If statement Equals says Suspicious comparison !
but works!
the thing is it always goes in the if statement no matter what I set the expression too.

How should I be Querying the searchQuery?

check this answer stackoverflow.com/questions/273141/regex-for-numbers-only look at the "^[0-9]+$" note — dinnouti
– dinnouti, Commented Feb 15, 2013 at 11:31

Rawling · Accepted Answer · 2013-02-15 11:29:33Z

1

Your Equals is comparing a Match to a Regex - they're different classes, so it's never true, and you always go into your if block.

You probably want if (m.Success) - that checks whether the value returned by check.Match(...) was a successful match (i.e. the input contained a non-letter) or not.

answered Feb 15, 2013 at 11:29

Rawling

50.3k7 gold badges94 silver badges131 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Anirudha · Accepted Answer · 2013-02-15 11:33:53Z

0

You are using Object's Equals method which is the base class for Regex

Use IsMatch method of Regex class i.e check.IsMatch(searchQuery)

edited Feb 15, 2013 at 11:33

answered Feb 15, 2013 at 11:28

Anirudha

32.9k8 gold badges71 silver badges90 bronze badges

Collectives™ on Stack Overflow

regex query implementation

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related