The usage was totally inconsistant, occasionally they'd use the int parameters, but the constants were defined public static final, so sometimes they'd do the sqlObj.USER. the sql class even had a constructor that took a single int, and stored it as int selection, and then sqlObj methods would take other instances of the same class as a paramater, and use a getSelection() method.Really quite horrifying. The blank stares when I asked for the design docs when I took over the project were pretty amusing though.