[Seaside] Re: A new critical blog discussing Seaside - now concrete proposals

stephane ducasse stephane.ducasse at free.fr
Sat Apr 18 10:08:30 UTC 2009


Excellent Francois thanks for the translation from cucumber dialect to  
human :)
>
> "
> Hi everybody,
>
> I've been working with Seaside on VW and it seems to me that some  
> Smalltalk coding conventions have been forgotten. It makes the code  
> hard to read (at least for me). Here are some examples:
>
> - factory methods should generally start with "new".
> WATree class root: anObject
> would be more clear if named:
> WATree class newWithRoot: anObject

Never heard about that convention.
Never used it.
even in SmalltalkWithStyle.

> - In VW, it seems that namespaces bindings are quite slow.
>
> - WARequest behaviour seems to be spread in the system. It makes  
> debbuging harder than necessary.
>
> - Usage of string instead of symbols. Example:
> WARenderContinuation -> shouldRedirect: aRequest
> ^ aRequest isGet not or: [(aRequest fields includesKey: '_n') not]
>
> The string '_n' is used in many places in the code. Why not a  
> symbol? Is there any technical reason?
>
> - ...
>
> That were some points that annoyed me in Seaside, if you want i can  
> provide a detailed list, fill some tickets and provide some patches...
>
> Cheers,
> Peter Jedermann
>
> "
>
> Easy enough isn't it? Certainly not ideal but people would certainly  
> listen and discuss about the issues you address.
>
> Have fun,
>
>
> Francois
>
> _______________________________________________
> seaside mailing list
> seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>



More information about the seaside mailing list