471,337 Members | 887 Online
Bytes | Software Development & Data Engineering Community
Post +

Home Posts Topics Members FAQ

Join Bytes to post your question to a community of 471,337 software developers and data experts.

executing list of methods (and collecting results)

Hi all. Im in this situation: I want to perform several kind of
(validating) methods to a given value.
Lets say i have a class named Number, and the following methods:
is_really_a_number(),
is_even(),
is_greater_than_zero(),
and so on. All of them returning booleans.

I want the collect_validators() method is to execute any of the above
methods, and collect their names as items of a list (wich will be the
collect_validators() return value).

My first approach is:

Expand|Select|Wrap|Line Numbers
  1. def collect_validators(self):
  2. v_dict = { 'is_really_a_number': is_really_a_number,
  3. 'is_even': is_even,
  4. 'is_greater_than_zero', is_greater_than_zero
  5. }
  6.  
  7. for name, meth in v_dict.items():
  8. result = meth()
  9. if result: yield name
  10.  
I wondering if is this a good pattern to apply, i like the way it looks
like, at least to me it looks `natural', but...im calling every method
twice here? One in v_dict and again on the dict iteration?

Any suggestion will be great!

Thanks!
Gerardo
Sep 20 '07 #1
5 1312
Gerardo Herzig wrote:
Hi all. Im in this situation: I want to perform several kind of
(validating) methods to a given value.
Lets say i have a class named Number, and the following methods:
is_really_a_number(),
is_even(),
is_greater_than_zero(),
and so on. All of them returning booleans.

I want the collect_validators() method is to execute any of the above
methods, and collect their names as items of a list (wich will be the
collect_validators() return value).

My first approach is:

Expand|Select|Wrap|Line Numbers
  1. def collect_validators(self):
  2.    v_dict = { 'is_really_a_number': is_really_a_number,
  3.                      'is_even': is_even,
  4.                      'is_greater_than_zero', is_greater_than_zero
  5.                   }
  6.       for name, meth in v_dict.items():
  7.          result = meth()
  8.          if result: yield name
  9.  

I wondering if is this a good pattern to apply, i like the way it looks
like, at least to me it looks `natural', but...im calling every method
twice here? One in v_dict and again on the dict iteration?

Any suggestion will be great!

Thanks!
Gerardo
You are not calling every method twice. You are painstakingly typing out
their names twice. But fortunately, functions and methods have a
__name__ attribute which makes a this typing redundant. I would give
your class instances a _validators attribute which is a list of
validating methods, then make the generator like this:

def collect_validators(self):
for v in self._validators:
if v():
yield v.__name__

James
Sep 21 '07 #2
Gerardo Herzig wrote:
I want the collect_validators() method is to execute any of the
above methods, and collect their names as items of a list (wich
will be the collect_validators() return value).
(inside class definition -- untested)
validators = {"is a number": is_really_a_number,
"is even": is_even,
"is greater than zero": is_greater_than_zero}

def collect_validators(self):
return [desc for desc, func in self.validators.items() if func()]
My first approach is:
.... no method, but a generator. Executing it will give you a
generator object instead of a result list.
Expand|Select|Wrap|Line Numbers
  1. def collect_validators(self):
  2.     v_dict = { 'is_really_a_number': is_really_a_number,
  3.                       'is_even': is_even,
  4.                       'is_greater_than_zero', is_greater_than_zero
  5.                    }
  6.        for name, meth in v_dict.items():
  7.           result = meth()
  8.           if result: yield name
  9.  

I wondering if is this a good pattern to apply, i like the way it
looks like, at least to me it looks `natural',
IMHO, it doesn't look natural. It depends on what you want to
achieve. This generator will need to be iterated over until it
is "exhausted".
but...im calling every method twice here?
No. Methods are only called if you apply the function call
operator, "()".

BTW, I hope you don't really want to test a number to be greater
than zero, or even, by using an own method, respectively, just to
test this.

Regards,
Björn

--
BOFH excuse #321:

Scheduled global CPU outage

Sep 21 '07 #3
Gerardo Herzig wrote:
>
>I want the collect_validators() method is to execute any of the
above methods, and collect their names as items of a list (wich
will be the collect_validators() return value).

(inside class definition -- untested)
validators = {"is a number": is_really_a_number,
"is even": is_even,
"is greater than zero": is_greater_than_zero}

def collect_validators(self):
return [desc for desc, func in self.validators.items() if func()]
Excelent!!!
>My first approach is:

... no method, but a generator. Executing it will give you a
generator object instead of a result list.
>
Expand|Select|Wrap|Line Numbers
  1. def collect_validators(self):
  2.     v_dict = { 'is_really_a_number': is_really_a_number,
  3.                       'is_even': is_even,
  4.                       'is_greater_than_zero', is_greater_than_zero
  5.                    }
  6.        for name, meth in v_dict.items():
  7.           result = meth()
  8.           if result: yield name

I wondering if is this a good pattern to apply, i like the way it
looks like, at least to me it looks `natural',

IMHO, it doesn't look natural. It depends on what you want to
achieve. This generator will need to be iterated over until it
is "exhausted".
Im having some fun doing a mail filter. A master thread will fire several
threads (each one returning a list with the matched validators) and
collect the results of each one of them.
>
>but...im calling every method twice here?

No. Methods are only called if you apply the function call
operator, "()".

BTW, I hope you don't really want to test a number to be greater
than zero, or even, by using an own method, respectively, just to
test this.
Haha, no, the actual methods do other kind of things.
Thanks Björn!!!

Cheers.
Gerardo
Sep 21 '07 #4
On Sep 21, 12:26 am, Gerardo Herzig <gher...@fmed.uba.arwrote:
def collect_validators(self):
v_dict = { 'is_really_a_number': is_really_a_number,
'is_even': is_even,
'is_greater_than_zero', is_greater_than_zero
}

for name, meth in v_dict.items():
result = meth()
if result: yield name
Are these validators actually methods rather than functions? If so,
you should write something like this:

def collect_validators(self):
methods = ['is_really_a_number', 'is_even',
'is_greater_than_zero']
return (m for m in methods if getattr(self, m)())

--
Paul Hankin

Sep 21 '07 #5
gh*****@fmed.uba.ar wrote:
Haha, no, the actual methods do other kind of things.
Thanks Björn!!!
Okay, so I hoped. Glad to be of help.

Regards,
Björn

--
BOFH excuse #233:

TCP/IP UDP alarm threshold is set too low.

Sep 21 '07 #6

This discussion thread is closed

Replies have been disabled for this discussion.

Similar topics

14 posts views Thread by Jay O'Connor | last post: by
6 posts views Thread by ahart | last post: by
2 posts views Thread by Jon Slaughter | last post: by
6 posts views Thread by HMS Surprise | last post: by
2 posts views Thread by =?Utf-8?B?Um9nZXIgTWFydGlu?= | last post: by
7 posts views Thread by Karlo Lozovina | last post: by
reply views Thread by rosydwin | last post: by

By using Bytes.com and it's services, you agree to our Privacy Policy and Terms of Use.

To disable or enable advertisements and analytics tracking please visit the manage ads & tracking page.