views:

635

answers:

2

First off, I'm relatively new to Google App Engine, so I'm probably doing something silly.

Say I've got a model Foo:

class Foo(db.Model): name = db.StringProperty() I want to use name as a unique key for every Foo object. How is this done?

When I want to get a specific Foo object, I currently query the datastore for all Foo objects with the target unique name, but queries are slow (plus it's a pain to ensure that name is unique when each new Foo is created).

There's got to be a better way to do this!

Thanks.

+1  A: 

Here is a pretty thorough discussion of unqiueness with the AppEngine datastore: http://stackoverflow.com/questions/1185628/how-do-i-define-a-unique-property-for-a-model-in-google-app-engine

Adam Crossland
Thanks for the link, definitely going in my favourites.
Cameron
+2  A: 

I've used the code below in a project before. It will work as long as the field on which you're basing your key name on is required.

class NamedModel(db.Model):
    """A Model subclass for entities which automatically generate their own key
    names on creation. See documentation for _generate_key function for
    requirements."""

    def __init__(self, *args, **kwargs):
        kwargs['key_name'] = _generate_key(self, kwargs)
        super(NamedModel, self).__init__(*args, **kwargs)


def _generate_key(entity, kwargs):
    """Generates a key name for the given entity, which was constructed with
    the given keyword args.  The entity must have a KEY_NAME property, which
    can either be a string or a callable.

    If KEY_NAME is a string, the keyword args are interpolated into it.  If
    it's a callable, it is called, with the keyword args passed to it as a
    single dict."""

    # Make sure the class has its KEY_NAME property set
    if not hasattr(entity, 'KEY_NAME'):
        raise RuntimeError, '%s entity missing KEY_NAME property' % (
            entity.entity_type())

    # Make a copy of the kwargs dict, so any modifications down the line don't
    # hurt anything
    kwargs = dict(kwargs)

    # The KEY_NAME must either be a callable or a string.  If it's a callable,
    # we call it with the given keyword args.
    if callable(entity.KEY_NAME):
        return entity.KEY_NAME(kwargs)

    # If it's a string, we just interpolate the keyword args into the string,
    # ensuring that this results in a different string.
    elif isinstance(entity.KEY_NAME, basestring):
        # Try to create the key name, catching any key errors arising from the
        # string interpolation
        try:
            key_name = entity.KEY_NAME % kwargs
        except KeyError:
            raise RuntimeError, 'Missing keys required by %s entity\'s KEY_NAME '\
                'property (got %r)' % (entity.entity_type(), kwargs)

        # Make sure the generated key name is actually different from the
        # template
        if key_name == entity.KEY_NAME:
            raise RuntimeError, 'Key name generated for %s entity is same as '\
                'KEY_NAME template' % entity.entity_type()

        return key_name

    # Otherwise, the KEY_NAME is invalid
    else:
        raise TypeError, 'KEY_NAME of %s must be a string or callable' % (
            entity.entity_type())

You could then modify your example model like so:

class Foo(NamedModel):
    KEY_NAME = '%(name)s'
    name = db.StringProperty()

Of course, this could be dramatically simplified in your case, changing the first line of the NamedModel's __init__ method to something like:

kwargs['key_name'] = kwargs['name']
Will McCutchen
Ah, this looks cool but a bit overkill. No matter, it still led me on the right track for discovering what I was missing: knowledge about key_name! It's the key to everything :-)
Cameron
Hahaha, well, I'm glad I pointed you in the right direction, at least.
Will McCutchen
Interesting approach. Good example of how to override __init__ robustly, too.
Nick Johnson
@Nick - Thanks! I'm glad to find out that this isn't too out of line.
Will McCutchen