0 
# -*- coding: utf-8 -*-
'''Please let code becomes much simpler and easier to maintain.
'''


def process(pet, action, target):
    '''
    >>> process('dog', 'eat', 'bone')
    ok
    >>> process('dog', 'eat', 'ball')
    faild
    >>> process('dog', 'play', 'ball')
    yes
    >>> process('dog', 'play', 'bone')
    ok
    >>> process('dolphin', 'play', 'ball')
    good
    >>> process('dolphin', 'play', 'bone')
    faild
    >>> process('dolphin', 'eat', 'bone')
    faild
    >>> process('dog', 'play', 'mouse')
    opps
    >>> process('cat', 'catch', 'mouse')
    Traceback (most recent call last):
        ...
    Exception
    '''
    if pet == 'dog':
        if action == 'eat':
            if target == 'bone':
                print 'ok'
            elif target == 'ball':
                print 'faild'
            else:
                raise Exception()
        elif action == 'play':
            if target == 'bone':
                print 'ok'
            elif target == 'ball':
                print 'yes'
            else:
                print 'opps'
        else:
            raise Exception()
    elif pet == 'dolphin':
        if action == 'eat':
            if target == 'bone':
                print 'faild'
            elif target == 'ball':
                print 'faild'
            else:
                raise Exception()
        elif action == 'play':
            if target == 'bone':
                print 'faild'
            elif target == 'ball':
                print 'good'
            else:
                raise Exception()
        else:
            raise Exception()
    else:
        raise Exception()

if __name__ == '__main__':
    import doctest
    doctest.testmod()

这是一段代码，它写的很丑，但不容易维护，代码不容易扩展，增加了一种新的宠物，动作，目标，需要写很多的，如何修改一下，改变示例的样子简单，很容易修改和扩展呢？

上面是一些示例代码，但是写得很丑，不容易维护，也不容易扩展——添加一个新的宠物，动作，目标。我需要写很多代码；如何重构它，使它看起来很简单，很容易修改和扩展？

4

3 回答 3

1

可能这样的事情会有所帮助。虽然它不处理“失败”的情况。

class Pet:
    def __init__(self, name, action_targets):
        self.name = name
        self.action_targets = action_targets

pets = (
            Pet('dog', (('eat','bone'), ('play','bone'), ('play', 'ball')) ),
            Pet('dolphin', (('play', 'ball')) )
        ) 

for p in pets:
    if pet == p.name and (action, target) in p.action_targets:
        return 'ok'

raise Exception()
于 2013-09-27T01:40:20.000 回答
1

这是一种无需类就可以做到这一点并处理“失败”情况的方法，尽管它更长：

from sets import Set

ALLOWABLE_ITEMS = (('dog', 'eat', 'bone'),
                   ('dog', 'play', 'bone'),
                   ('dog', 'play', 'ball'),
                   ('dolphin', 'play', 'ball'),
                  )

PETS, ACTIONS, TARGETS = Set(), Set(), Set()

for item in ALLOWABLE_ITEMS:
    PETS.add(item[0])
    ACTIONS.add(item[1])
    TARGETS.add(item[2])

if (pet, action, target) in ALLOWABLE_ITEMS:
    print 'ok'
elif  pet not in PETS or action not in ACTIONS or target not in TARGETS:
    raise Exception()
else:
    print 'faild'
于 2013-09-27T01:57:55.903 回答
0 
# -*- coding: utf-8 -*-
'''Please let code becomes much simpler and easier to maintain.
'''


def process(pet, action, target):
    '''
    >>> process('dog', 'eat', 'bone')
    ok
    >>> process('dog', 'eat', 'ball')
    faild
    >>> process('dog', 'play', 'ball')
    yes
    >>> process('dog', 'play', 'bone')
    ok
    >>> process('dolphin', 'play', 'ball')
    good
    >>> process('dolphin', 'play', 'bone')
    faild
    >>> process('dolphin', 'eat', 'bone')
    faild
    >>> process('dog', 'play', 'mouse')
    opps
    >>> process('cat', 'catch', 'mouse')
    Traceback (most recent call last):
        ...
    Exception
    '''

    def print_ok():
        print 'ok'

    def print_yes():
        print 'yes'

    def print_good():
        print 'good'

    def print_faild():
        print 'faild'

    def print_opps():
        print 'opps'

    def raise_exception():
        raise Exception()

    args_map = {}
    args_map[('dog', 'eat', 'bone')] = print_ok
    args_map[('dog', 'eat', 'ball')] = print_faild
    args_map[('dog', 'play', 'bone')] = print_ok
    args_map[('dog', 'play', 'ball')] = print_yes
    args_map[('dog', 'play', 'mouse')] = print_opps

    args_map[('dolphin', 'eat', 'bone')] = print_faild
    args_map[('dolphin', 'eat', 'ball')] = print_faild
    args_map[('dolphin', 'play', 'bone')] = print_faild
    args_map[('dolphin', 'play', 'ball')] = print_good

    func = args_map.get((pet, action, target), raise_exception)
    func()

if __name__ == '__main__':
    import doctest
    doctest.testmod()
于 2013-09-29T02:15:18.843 回答