views:

48

answers:

2

At the moment I've got this code:

name, type = meth.to_s.match(/^(.+?)([=?]?)$/)[1..-1]

But it doesn't seem to be the best solution =\

Any ideas how to make it better? Thanks.

A: 

The best option seems to be this: name, type = meth.to_s.split(/([?=])/)

Daniel
method "types" don't really exist, you're the only person in the whole history of the ruby universe to talk about them ;)
banister
i know, i just don't know how to describe it, so i thought "method types" will be better than "method suffix"
Daniel
A: 

This is roughly how I'd implement my method_missing:

def method_missing(sym, *args, &block)
  name = sym.to_s
  if name =~ /^(.*)=$/
    # Setter method with name $1.
  elsif name =~ /^(.*)\?$/
    # Predicate method with name $1.
  elsif name =~ /^(.*)!$/
    # Dangerous method with name $1.
  else
    # Getter or regular method with name $1.
  end
end

Or this version, which only evaluates one regular expression:

def method_missing(sym, *args, &block)
  name = sym.to_s
  if name =~ /^(.*)([=?!])$/
    # Special method with name $1 and suffix $2.
  else
    # Getter or regular method with name.
  end
end
Richard Cook