<!DOCTYPE html> <html> <head> <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"> </head> <body> I would reformat it like this: <pre>handleDog: aRequest "handle an HTTP request based on its method" | id parts | aRequest method = 'GET' ifTrue: [ | accept | accept := aRequest headerAt: 'Accept'. (accept includesSubString: 'application/json') ifTrue: [ ^self getDogsAsJson: aRequest]. (accept includesSubString: 'text/html') ifTrue: [ ^self getDogsAsHtml: aRequest]]. aRequest method = 'POST' ifTrue: [ ^self createDog: aRequest]. "Get the path parameter value." "TODO: Is this really the best way to do it?" parts := aRequest url prefixAndSuffix: $/. id := parts last asNumber. aRequest method = 'DELETE' ifTrue: [ ^self deleteDog: aRequest id: id]. aRequest method = 'PUT' ifTrue: [ ^self updateDog: aRequest id: id] </pre> In my opinion, Smalltalk is not very good for this kind of multiple conditions stuff. I know it can be refactored to involve and dispatch using several objects, but sometimes (like in this case) a multiple conditional solution is fine. In some parts I think my code would benefit from having a Lisp/Scheme COND type of language construct. Actually, it is easy to implement, but I don't use it because it is not part of Cuis Core, the multiple returns works for most cases. Mariano <div class="moz-cite-prefix">El 17/6/24 a las 18:20, Mark Volkmann via Cuis-dev escribió: </div> <blockquote type="cite" cite="mid:CAFfRWnV2P=_nswxBTEZHbMx0i8JOxLe4MStqOwwar+xSH4vLqQ@mail.gmail.com"> <div>handleDog: aRequest "handle an HTTP request based on its method" | id parts | aRequest method = 'GET' ifTrue: [ | accept |</div> <div> accept := aRequest headerAt: 'Accept'.</div> <div> (accept includesSubString: 'application/json') ifTrue: [</div> <div> self getDogsAsJson: aRequest.</div> <div> ^nil</div> <div> ].</div> <div> </div> <div> (accept includesSubString: 'text/html') ifTrue: [</div> <div> self getDogsAsHtml: aRequest.</div> <div> ^nil</div> <div> ]</div> <div> ]. aRequest method = 'POST' ifTrue: [ self createDog: aRequest.</div> <div> ^nil</div> <div> ]. "Get the path parameter value." "TODO: Is this really the best way to do it?"</div> <div> parts := aRequest url prefixAndSuffix: $/. id := parts last asNumber. aRequest method = 'DELETE' ifTrue: [ self deleteDog: aRequest id: id.</div> <div> ^nil</div> <div> ]. aRequest method = 'PUT' ifTrue: [ self updateDog: aRequest id: id.</div> <div> ^nil</div> <div> ]</div> </blockquote> </body> </html>