Блог

У меня есть следующий код (для Country City классов и key_name — числовой идентификатор с добавлением «i» в начале):

 def add_country(country, country_name):
  if country and country_name 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
 

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

Вам не нужно добавлять к своим идентификаторам символ — просто преобразуйте их в строку и используйте как есть. В противном случае, это, вероятно, нормально, хотя трудно понять, не видя, как вы его используете и как вы извлекаете объекты.