Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(72)
Issues Repositories
Recent Issues | Sign in with your Google Account to create issues and add comments

Delta Between Two Patch Sets: gears/database2/statement.h

Issue 717: Database2Values, argument conversion implemented (Closed) SVN Base: http://google-gears.googlecode.com/svn/contrib/dimitri.glazkov/database2/
Left Patch Set: Created 3 months, 2 weeks ago
Right Patch Set: Variant, multiple-row capability added Created 3 months, 1 week ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« gears/database2/statement.cc ('k') | gears/database2/transaction.cc » ('j') | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
LEFTRIGHT
1 // Copyright 2008, Google Inc. 1 // Copyright 2008, Google Inc.
2 // 2 //
3 // Redistribution and use in source and binary forms, with or without 3 // Redistribution and use in source and binary forms, with or without
4 // modification, are permitted provided that the following conditions are met: 4 // modification, are permitted provided that the following conditions are met:
5 // 5 //
6 // 1. Redistributions of source code must retain the above copyright notice, 6 // 1. Redistributions of source code must retain the above copyright notice,
7 // this list of conditions and the following disclaimer. 7 // this list of conditions and the following disclaimer.
8 // 2. Redistributions in binary form must reproduce the above copyright notice, 8 // 2. Redistributions in binary form must reproduce the above copyright notice,
9 // this list of conditions and the following disclaimer in the documentation 9 // this list of conditions and the following disclaimer in the documentation
10 // and/or other materials provided with the distribution. 10 // and/or other materials provided with the distribution.
11 // 3. Neither the name of Google Inc. nor the names of its contributors may be 11 // 3. Neither the name of Google Inc. nor the names of its contributors may be
12 // used to endorse or promote products derived from this software without 12 // used to endorse or promote products derived from this software without
13 // specific prior written permission. 13 // specific prior written permission.
14 // 14 //
15 // THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR IMPLIED 15 // THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR IMPLIED
16 // WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF 16 // WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
17 // MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO 17 // MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO
18 // EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 18 // EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
19 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, 19 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
20 // PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; 20 // PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS;
21 // OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, 21 // OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
22 // WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR 22 // WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR
23 // OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF 23 // OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
24 // ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 24 // ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
25 25
26 #ifndef GEARS_DATABASE2_STATEMENT_H__ 26 #ifndef GEARS_DATABASE2_STATEMENT_H__
27 #define GEARS_DATABASE2_STATEMENT_H__ 27 #define GEARS_DATABASE2_STATEMENT_H__
28 28
29 #include <vector>
30
29 #include "gears/base/common/common.h" 31 #include "gears/base/common/common.h"
30 #include "gears/base/common/js_types.h" 32 #include "gears/base/common/js_types.h"
31 #include "gears/third_party/scoped_ptr/scoped_ptr.h" 33 #include "gears/third_party/scoped_ptr/scoped_ptr.h"
32 34
33 // forward declarations 35 // forward declarations
34 class Database2; 36 class Database2;
35 class Database2Transaction; 37 class Database2Transaction;
36 class Database2Values; 38 class Database2Values;
37 39
38 // represents statement, for both synchronous and asynchronous operations 40 // represents statement, for both synchronous and asynchronous operations
39 class Database2Statement { 41 class Database2Statement {
40 public: 42 public:
41 bool HasCallback() const { 43 bool HasCallback() const {
42 assert(callback_.get()); 44 return callback_.get() != NULL;
43 return !JsTokenIsNullOrUndefined(callback_->token());
44 } 45 }
45 46
46 bool HasErrorCallback() const { 47 bool HasErrorCallback() const {
47 assert(error_callback_.get()); 48 return error_callback_.get() != NULL;
48 return !JsTokenIsNullOrUndefined(error_callback_->token());
49 } 49 }
50 50
51 void InvokeCallback(Database2Transaction *tx); 51 void InvokeCallback(Database2Transaction *tx);
52 void InvokeErrorCallback(Database2Transaction *tx, JsObject *error); 52 void InvokeErrorCallback(Database2Transaction *tx, JsObject *error);
53 53
54 // create a statement instance
55 // must passs NULL for arguments or callbacks if they are not specified
54 static bool Create(const std::string16 &sql_statement, 56 static bool Create(const std::string16 &sql_statement,
55 const JsArray &sql_arguments, 57 JsArray *sql_arguments,
56 JsRootedCallback *callback, 58 JsRootedCallback *callback,
57 JsRootedCallback *error_callback, 59 JsRootedCallback *error_callback,
58 Database2Statement **instance); 60 Database2Statement **instance);
59 61
60 std::string16 sql() const { return sql_statement_; } 62 std::string16 sql() const { return sql_statement_; }
61 Database2Values *arguments() const { return arguments_.get(); } 63 Database2Values *arguments() const { return arguments_.get(); }
62 private: 64 private:
63 Database2Statement() {} 65 Database2Statement() {}
64 // if true, the statement has invalid arguments 66 // if true, the statement has invalid arguments
65 bool bogus_; 67 bool bogus_;
66 std::string16 sql_statement_; 68 std::string16 sql_statement_;
67 scoped_ptr<Database2Values> arguments_; 69 scoped_ptr<Database2Values> arguments_;
68 scoped_ptr<JsRootedCallback> callback_; 70 scoped_ptr<JsRootedCallback> callback_;
69 scoped_ptr<JsRootedCallback> error_callback_; 71 scoped_ptr<JsRootedCallback> error_callback_;
70 72
71 DISALLOW_EVIL_CONSTRUCTORS(Database2Statement); 73 DISALLOW_EVIL_CONSTRUCTORS(Database2Statement);
72 }; 74 };
73 75
74 // used for marshaling statement arguments to the database thread and 76 // Used for marshaling statement arguments to the database thread and results to
75 // results to the main thread 77 // the main thread. A union-based structure is used to store value-type pairs.
76 class Database2Values { 78 class Database2Values {
Aaron 2008/05/22 22:16:42 The shape of this internal class is really bugging
Dimitri 2008/05/24 02:13:40 On 2008/05/22 22:16:42, Aaron wrote: > The shape o
77 public: 79 public:
78 ~Database2Values(); 80 Database2Values() {}
81 ~Database2Values() {
82 // remove all rows
83 for(unsigned int i = 0; i < rows_.size(); i++) {
84 delete[] rows_[i];
85 }
86 }
79 87
80 static bool CreateFromJsArray(const JsArray &sql_arguments, 88 static bool CreateFromJsArray(const JsArray *js_array,
81 Database2Values **instance); 89 Database2Values **instance);
82 90
91 // methods for reading values
83 int length() const { return length_; } 92 int length() const { return length_; }
93 void Reset() { current_row_ = 0; }
94 bool Next();
84 JsParamType GetType(int index) const; 95 JsParamType GetType(int index) const;
85 int GetAsInt(int index) const; 96 bool GetElementAsInt(int index, int *value);
86 double GetAsDouble(int index) const; 97 bool GetElementAsDouble(int index, double *value);
87 std::string16 &GetAsString(int index) const; 98 bool GetElementAsString(int index, std::string16 *value);
99
88 private: 100 private:
89 Database2Values() {} 101 void StartNewRow();
90 102
91 static bool SetJsParamToSend(JsArray js_array, 103 // used to store value-type pairs
92 int index, 104 struct Variant {
93 JsParamToSend *param); 105 Variant() : type(JSPARAM_NULL) {}
106 ~Variant() {
107 if (type == JSPARAM_STRING16) { delete string_value; }
108 }
94 109
95 scoped_array<JsParamToSend> arguments_; 110 JsParamType type;
111 union {
112 int int_value;
113 double double_value;
114 std::string16 *string_value;
115 };
116 };
117
118 inline Variant &current(int index) const {
119 assert(index >= 0 && index < length_);
120 assert(current_row_ < length_);
121 return rows_[current_row_][index];
122 }
123
124 friend class vector;
125 std::vector<Variant*> rows_;
96 int length_; 126 int length_;
Aaron 2008/05/11 18:11:21 Should length be passed separately? The way it is
Dimitri 2008/05/13 22:52:03 On 2008/05/11 18:11:21, Aaron wrote: > Should leng
127 int current_row_;
97 128
98 DISALLOW_EVIL_CONSTRUCTORS(Database2Values); 129 DISALLOW_EVIL_CONSTRUCTORS(Database2Values);
99 }; 130 };
100 131
101 #endif // GEARS_DATABASE2_STATEMENT_H__ 132 #endif // GEARS_DATABASE2_STATEMENT_H__
LEFTRIGHT
« gears/database2/statement.cc ('k') | gears/database2/transaction.cc » ('j') | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld r292