Как оптимизировать код для сохранения данных в хранилище GAE?


У меня есть следующий код (для страны и города - классы key_name - это числовой идентификатор, с добавлением " я " в начале):

def add_country(country, country_name):
  if country and country_namef and country_name != '':
    return Country.get_or_insert('i'+str(country), country_name=country_name)
  else:
    return None  

def add_city(city, city_name, country):
  if country and city and city_name and city_name != '':
    return City.get_or_insert('i'+str(city), city_name=city_name, parent=country)
  else:
    return None

Это правильный код или можно как-то оптимизировать?



149
0
задан 26 октября 2011 в 07:10 Источник Поделиться
Комментарии
1 ответ

def add_country(country, country_name):
if country and country_namef and country_name != '':

Я предполагаю, что country_namef опечатка. Если это так, то нет смысла в проверке country_name != " потому что пустые строки уже считаются ложными

    return Country.get_or_insert('i'+str(country), country_name=country_name)
else:
return None

def add_city(city, city_name, country):
if country and city and city_name and city_name != '':

Как и раньше, пустые строки являются ложными, так вы проверяете, что в два раза

    return City.get_or_insert('i'+str(city), city_name=city_name, parent=country)
else:
return None

2
ответ дан 26 октября 2011 в 08:10 Источник Поделиться