python locals() redundant usage

Question

Hi I am kinda new to using locals(), I did research that locals() means treat a variable within a function as a dictionary. We have a bunch if this function with the same function of locals() below. I am man of DRY principle but in this area its not. How will I simplify this or any alternative way?

def home_creation(self, property, item):
    def date(data):
        return data
    def rate(data):          
        return data

    if property in locals():
        return locals()[property](item)
    else:
        return None

Don't use `locals` for this, just define your own dictionary, or use `if-elif-else`. — juanpa.arrivillaga, May 15 '17 at 10:45
@juanpa.arrivillaga, why dont use locals? How can you do it alternatively? — Bry, May 15 '17 at 22:18
@juanpa.arrivillaga also seen this http://stackoverflow.com/questions/9168340/using-a-dictionary-to-select-function-to-execute — Bry, May 15 '17 at 22:25

Uriel · Answer 1 · 2017-05-15T10:50:32.380

0

You can define your own dictionary to limit the permitted methods for use:

def home_creation(self, property, item):
    def date(data):
        return data
    def rate(data):          
        return data

    functions = {
        'date': date,
        'rate': rate
    }

    return functions.get(property, lambda: None)(item)

edited May 15 '17 at 10:50

answered May 15 '17 at 10:37

Uriel

15,579
6
25
46

Sorry dont quite get what you mean. This function is under a class by the way. – Bry May 15 '17 at 10:44

python locals() redundant usage

1 Answers1