Mercurial > MadButterfly
annotate src/observer.c @ 192:54fdc2a65242
Remove factory from observer APIs.
- Save factory that used to new a subject as subject's attribute.
- Get factory from subject himself instead of passing as an argument.
- It make API of observer more simple and clean.
| author | Thinker K.F. Li <thinker@branda.to> |
|---|---|
| date | Tue, 18 Nov 2008 21:42:30 +0800 |
| parents | 530bb7728546 |
| children | 45d9a1e2764d f9d507a3e1d9 |
| rev | line source |
|---|---|
| 73 | 1 #include <stdio.h> |
|
186
530bb7728546
Move header files to $(top_srcdir)/include/ and prefixed with 'mb_'.
Thinker K.F. Li <thinker@branda.to>
parents:
185
diff
changeset
|
2 #include "mb_redraw_man.h" |
|
530bb7728546
Move header files to $(top_srcdir)/include/ and prefixed with 'mb_'.
Thinker K.F. Li <thinker@branda.to>
parents:
185
diff
changeset
|
3 #include "mb_observer.h" |
|
530bb7728546
Move header files to $(top_srcdir)/include/ and prefixed with 'mb_'.
Thinker K.F. Li <thinker@branda.to>
parents:
185
diff
changeset
|
4 #include "mb_tools.h" |
| 73 | 5 |
| 126 | 6 #ifndef ASSERT |
| 7 #define ASSERT(x) | |
| 8 #endif | |
| 9 | |
| 73 | 10 subject_t *subject_new(ob_factory_t *factory, void *obj, int obj_type) { |
| 11 subject_t *subject; | |
| 12 | |
| 13 subject = factory->subject_alloc(factory); | |
| 14 if(subject == NULL) | |
| 15 return NULL; | |
| 16 | |
| 17 subject->obj = obj; | |
| 18 subject->obj_type = obj_type; | |
| 19 subject->flags = 0; | |
| 20 STAILQ_INIT(subject->observers); | |
| 21 | |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
22 subject->factory = factory; |
|
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
23 |
| 73 | 24 return subject; |
| 25 } | |
| 26 | |
|
125
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
27 /*! |
|
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
28 * \todo Keep ob_factory following subject objects. |
|
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
29 */ |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
30 void subject_free(subject_t *subject) { |
|
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
31 ob_factory_t *factory = subject->factory; |
| 73 | 32 observer_t *observer; |
| 33 | |
| 126 | 34 ASSERT(!(subject->flags & SUBF_FREE)); |
|
125
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
35 if(subject->flags & SUBF_BUSY) { |
|
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
36 /* Postpond the request until busy status been stoped. |
|
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
37 * SUBF_BUSY means in subject_notify(). |
|
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
38 */ |
|
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
39 subject->flags |= SUBF_FREE; |
|
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
40 return; |
|
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
41 } |
|
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
42 |
| 73 | 43 while((observer = STAILQ_HEAD(subject->observers))) { |
| 44 STAILQ_REMOVE(subject->observers, observer_t, next, observer); | |
| 45 factory->observer_free(factory, observer); | |
| 46 } | |
| 47 factory->subject_free(factory, subject); | |
| 48 } | |
| 49 | |
| 50 | |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
51 void subject_notify(subject_t *subject, event_t *evt) { |
|
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
52 ob_factory_t *factory = subject->factory; |
| 73 | 53 observer_t *observer; |
| 54 | |
| 77 | 55 evt->tgt = subject; |
| 73 | 56 while(subject) { |
|
127
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
57 /*! |
|
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
58 * \note What is happend when the subject is freed by observer? |
|
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
59 * Postponding the request of free until notification |
|
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
60 * been finished. (\ref SUBF_BUSY / \ref SUBF_FREE) |
|
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
61 */ |
|
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
62 subject->flags |= SUBF_BUSY; |
|
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
63 |
| 77 | 64 evt->cur_tgt = subject->obj; |
| 73 | 65 for(observer = STAILQ_HEAD(subject->observers); |
| 66 observer != NULL; | |
| 67 observer = STAILQ_NEXT(observer_t, next, observer)) { | |
| 68 observer->hdr(evt, observer->arg); | |
| 69 } | |
| 70 | |
|
127
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
71 subject->flags &= ~SUBF_BUSY; |
|
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
72 if(subject->flags & SUBF_FREE) |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
73 subject_free(subject); |
|
127
d2cc7400c971
Bug of subject_notify() when free subjects.
Thinker K.F. Li <thinker@branda.to>
parents:
126
diff
changeset
|
74 |
| 73 | 75 if(subject->flags & SUBF_STOP_PROPAGATE) |
| 76 break; | |
| 77 | |
| 78 subject = factory->get_parent_subject(factory, subject); | |
| 79 } | |
|
125
1c1f28c124c9
Postponding free request when a subject is in subject_notify
Thinker K.F. Li <thinker@branda.to>
parents:
77
diff
changeset
|
80 |
| 73 | 81 } |
| 82 | |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
83 observer_t *subject_add_observer(subject_t *subject, |
| 73 | 84 evt_handler hdr, void *arg) { |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
85 ob_factory_t *factory = subject->factory; |
| 73 | 86 observer_t *observer; |
| 87 | |
| 88 observer = factory->observer_alloc(factory); | |
| 89 if(observer == NULL) | |
| 90 return NULL; | |
| 91 observer->hdr = hdr; | |
| 92 observer->arg = arg; | |
| 93 | |
| 94 STAILQ_INS_TAIL(subject->observers, observer_t, next, observer); | |
| 95 | |
| 96 return observer; | |
| 97 } | |
| 98 | |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
99 void subject_remove_observer(subject_t *subject, |
| 73 | 100 observer_t *observer) { |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
101 ob_factory_t *factory = subject->factory; |
|
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
102 |
| 73 | 103 STAILQ_REMOVE(subject->observers, observer_t, next, observer); |
| 104 factory->observer_free(factory, observer); | |
| 105 } | |
| 106 | |
| 107 #ifdef UNITTEST | |
| 108 | |
| 109 #include <CUnit/Basic.h> | |
| 110 #include <stdlib.h> | |
| 111 | |
| 112 static subject_t *test_subject_alloc(ob_factory_t *factory) { | |
| 113 subject_t *subject; | |
| 114 | |
| 115 subject = (subject_t *)malloc(sizeof(subject_t)); | |
| 116 return subject; | |
| 117 } | |
| 118 | |
| 119 static void test_subject_free(ob_factory_t *factory, subject_t *subject) { | |
| 120 free(subject); | |
| 121 } | |
| 122 | |
| 123 static observer_t *test_observer_alloc(ob_factory_t *factory) { | |
| 124 observer_t *observer; | |
| 125 | |
| 126 observer = (observer_t *)malloc(sizeof(observer_t)); | |
| 127 return observer; | |
| 128 } | |
| 129 | |
| 130 static void test_observer_free(ob_factory_t *factory, observer_t *observer) { | |
| 131 free(observer); | |
| 132 } | |
| 133 | |
| 134 static subject_t *test_get_parent_subject(ob_factory_t *factory, | |
| 135 subject_t *subject) { | |
| 136 return NULL; | |
| 137 } | |
| 138 | |
| 139 static ob_factory_t test_factory = { | |
| 140 test_subject_alloc, | |
| 141 test_subject_free, | |
| 142 test_observer_alloc, | |
| 143 test_observer_free, | |
| 144 test_get_parent_subject | |
| 145 }; | |
| 146 | |
| 147 static void handler(event_t *evt, void *arg) { | |
| 148 int *cnt = (int *)arg; | |
| 149 | |
| 74 | 150 CU_ASSERT(evt->type == EVT_MOUSE_OUT); |
| 73 | 151 (*cnt)++; |
| 152 } | |
| 153 | |
| 154 void test_observer(void) { | |
| 155 subject_t *subject; | |
| 74 | 156 observer_t *observer[2]; |
| 73 | 157 event_t evt; |
| 158 int cnt = 0; | |
| 159 | |
| 160 subject = subject_new(&test_factory, NULL, 0); | |
| 161 subject->flags |= SUBF_STOP_PROPAGATE; | |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
162 observer[0] = subject_add_observer(subject, handler, &cnt); |
|
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
163 observer[1] = subject_add_observer(subject, handler, &cnt); |
| 73 | 164 |
| 74 | 165 evt.type = EVT_MOUSE_OUT; |
| 73 | 166 evt.tgt = NULL; |
| 167 evt.cur_tgt = NULL; | |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
168 subject_notify(subject, &evt); |
| 74 | 169 CU_ASSERT(cnt == 2); |
| 73 | 170 |
|
192
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
171 subject_remove_observer(subject, observer[0]); |
|
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
172 subject_remove_observer(subject, observer[1]); |
|
54fdc2a65242
Remove factory from observer APIs.
Thinker K.F. Li <thinker@branda.to>
parents:
186
diff
changeset
|
173 subject_free(subject); |
| 73 | 174 } |
| 175 | |
| 176 CU_pSuite get_observer_suite(void) { | |
| 177 CU_pSuite suite; | |
| 178 | |
| 179 suite = CU_add_suite("Suite_observer", NULL, NULL); | |
| 180 CU_ADD_TEST(suite, test_observer); | |
| 181 | |
| 182 return suite; | |
| 183 } | |
| 184 | |
| 185 #endif /* UNITTEST */ |