-1

Actually:

return all().filter("name", name).count() == 0;

May be a better way ?

Thierry
  • 5,133
  • 3
  • 25
  • 30

2 Answers2

1

Regarding performance I would go for something like:

return all().filter("name", name).get() == null;

I'm not sure about GAE but in SQL a limit 1 is several times faster than count, and obviously you don't need to know how many models have this value.

plunchete
  • 486
  • 4
  • 12
0

Your way is not bad as you filter on a field.

If you had known the key of the entity, you could have used:

return Model.getByKey(YourClass.class, id) == null

If you want to filter on a given field, you can also use the fetch keys only :

return all().filter("name", name).fetchKeys().size() == 0;

But I wonder which is the best in term of performance. It depends on performance of GAE datastore regarding "count" or "fetch keys only" operation.

mandubian
  • 4,427
  • 1
  • 22
  • 15