img - GitHub Pages

3.5. Implementations-Sicht 52
157
158 Community.find({ where: { name: communityData.name }})
159 .success(function findResult(community) {
160 // ... more callbacks (see cited file)
161 })
162 .error(function findError() {
163 return res.send(500);
164 });
165 })
166 .error(function getError() {
167 return res.send(500);
168 });
169 });
170 }
Quelltext 3.8: Ausschnitt aus Community Controller mit Callback Hell [AJWf]
Quelltext 3.8 zeigt einen Ausschnitt der createCommunity-Funktion. Es ist offensichtlich,
dass dieser Quelltext aufgrund der Verschachtlung von Funktionen sehr schwer
lesbar ist. Diese Situation wird treffend als “Callback hell” bezeichnet.
Für ein Refactoring können folgende zwei Ansätze angewendet werden:
• Funktional
• Promises / Futures
Funktional
Der funktionale Ansatz versucht Callbacks in eigene Funktionen auszulagern und dabei
Seiteneffekte zu vermeiden.
Die Vermeidung von Seiteneffekten ist wichtig und wünschenswert, bedingt aber meist
die Übergabe vieler einzelner Argumente beim Aufruf der ausgelagerten Funktion.
Quelltext 3.9 zeigt einen Ausschnitt der überarbeiteten Version von Quelltext 3.8. Es
wurde der funktionale Ansatz für das Refactoring gewählt.
192 function createCommunity(success, error, data) {
193 var // ...
194 , forwardError = function forwardError(err) {
195 // ...
196 }
197 , afterCommunitySearch = function afterCommunitySearch(community) {
198 // ...
199 communityDao.create(communityData)
200 .success(afterCommunityCreate)
201 .error(forwardError);
202 }
203 , afterCommunityCreate = function afterCommunityCreate(community) {
204 // ...
205 createUniqueSlug(db, community.name, community.id