ruby-style-guide icon indicating copy to clipboard operation
ruby-style-guide copied to clipboard

DIscourage rescue error message checking

Open simulk opened this issue 1 year ago • 1 comments

I proposed PR https://github.com/rubocop/rubocop/pull/13352 to add a new Lint/NoRescueErrorMessageChecking cop that discourages rescue error matching. We built a custom cop at my company and thought it might be useful to roll it out as built in cop.

I was suggested by @dvandersluis to reach out to the community for feedback whether its a useful cop or not.

# bad
begin
  something
rescue => e
  if e.message.match?(/Duplicate entry/)
    handle_error
  end
end

# bad
begin
  something
rescue => e
  unless e.message.match?(/Duplicate entry/)
    handle_error
  end
end

# good
begin
  something
rescue ActiveRecord::RecordNotUnique => e
  handle_error
end

simulk avatar Oct 17 '24 14:10 simulk