https://kotlinlang.org logo
Channels
Powered by
#announcements
  • c

    Czar

    12/07/2015, 1:48 PM
    @SuppresWarnings
    does not influence the behaviour in runtime, it only influences compiler and IDE warnings output.
  • y

    yole

    12/07/2015, 1:48 PM
    if an annotation is put on a class, I don’t think it’s confusing in any way to have it affect the contents of the class
  • m

    mplatvoet

    12/07/2015, 1:49 PM
    you can write new code and accidentally suppressing warnings
  • y

    yole

    12/07/2015, 1:49 PM
    it’s just normal lexical scoping
  • c

    Czar

    12/07/2015, 1:49 PM
    it can be argued that it does introduce some confusion, because you do not see it most of the time while working on methods if the class is of non-trivial length.
  • y

    yole

    12/07/2015, 1:50 PM
    well, you also don’t see the list of supertypes of the class, which is more important
  • d

    derkork

    12/07/2015, 1:50 PM
    Yes, but it still would be more pragmatic than having to type 
    open
    all over the place.
  • m

    mplatvoet

    12/07/2015, 1:50 PM
    a good reason to avoid supertypes as much as possible too 😉
  • c

    Czar

    12/07/2015, 1:52 PM
    List of supertypes is part of the conscious design of the application, developer working on the subtype sure as hell has to know what supertype this subtype belongs to simple smile
  • m

    mplatvoet

    12/07/2015, 1:52 PM
    I really like the approach of 
    val
    and 
    var
    being explicit. I wish there could be a same approach for the open closed debate
    👍 1
  • m

    mplatvoet

    12/07/2015, 1:58 PM
    so just as an idea and not a definitive naming, something like 
    val
    is to 
    var
    as 
    class
    is to 
    def
    as 
    fun
    is to 
    def
  • j

    jkbbwr

    12/07/2015, 1:59 PM
    I thjink implicitly final or closed classes are better than open, and public is better than getters.
  • m

    mplatvoet

    12/07/2015, 1:59 PM
    Copy code
    def SomeFinalClass {
    def finalMethod() {}
}
  • j

    jkbbwr

    12/07/2015, 2:00 PM
    @mplatvoet: That doesn't read nice imo.
  • j

    jkbbwr

    12/07/2015, 2:00 PM
    Suddenly 
    def
    has contextual meaning
  • m

    mplatvoet

    12/07/2015, 2:00 PM
    don’t get put of by the word 
    def
    , it’s about the idea
  • d

    derkork

    12/07/2015, 2:00 PM
    well in the end this is highly dependent on your use case @jkbbwr - for some use cases open by default is better and for some final by default is better.
  • m

    mplatvoet

    12/07/2015, 2:01 PM
    to open variant would be:
    Copy code
    class SomeOpenClass {
    fun openMethod() {}
}
  • j

    jkbbwr

    12/07/2015, 2:02 PM
    why the public decorator?
  • d

    dalexander

    12/07/2015, 2:02 PM
    I think open/closed defaults depends on the choice of problems you'd rather have. I like the idea of a <closed class>/<closed method> keyword though as a consideration.
  • m

    mplatvoet

    12/07/2015, 2:02 PM
    @jkbbwr: habit, fixed
  • d

    derkork

    12/07/2015, 2:05 PM
    or simply have the compiler force you to decide if you want 
    open class
    or 
    final class
    .. which in turn adds ceremony...
  • j

    jkbbwr

    12/07/2015, 2:05 PM
    So suddenly we are heading back to java land
  • j

    jkbbwr

    12/07/2015, 2:05 PM
    public this private that
  • d

    derkork

    12/07/2015, 2:06 PM
    well that’s basically the tradeoff you have to make - either be explicit or default on something.
  • d

    derkork

    12/07/2015, 2:07 PM
    both sides have it’s pros and cons.
  • d

    derkork

    12/07/2015, 2:07 PM
    if you default on something you’ll always find someone who doesn’t like your default for whatever reason.
  • d

    derkork

    12/07/2015, 2:08 PM
    if you’re explicit you have a lot of boilerplate.
  • j

    jkbbwr

    12/07/2015, 2:09 PM
    I argue you don't need data encapulation if you have immutability
  • d

    derkork

    12/07/2015, 2:10 PM
    i would agree
1...139914001401...3338Latest