[swift-evolution] What about a VBA style with Statement?

[Vladimir.S]

Yes, I agree, that "implicit" "with", where we can't say if method belongs 
to the target instance or not, is very bad idea. I strongly agains it.

But the proposal is for "explicit" "with", where it is clear what 
method/property belongs to target instance, and which don't. So most of 
your notes regarding problems with "with" are not related to "external" 
with, but to "internal" "with", that was not even introduced by the proposal.

As for your notes regarding implicit "self." when you writing code inside 
class. No, you not always know what exact methods is in class and what is 
external functions: in case you are extending some class from some library 
or inherit from such class. And I think we actually have this problem when 
writing class code : I believe we need "explicit" marker that some 
method/property belongs to our class. Personally I think "self." should be 
necessarily to use in our code.

 >Even if we use .bar() instead of just bar() when calling from within.....

It is not about saving the space in code. It is about more readable and (I 
insist) more stable(with less errors) code.

Let's get your example with br. Let's say you have additionally bt 
instance. Take into account code-completion feature of your editor. Take 
into account copy-paste errors(when you copy code of br, wants to use for bt).

br = ..
br.one = ..
br.two = ...
br.three = ...
some1 = br.four()

bt = ..
bt.one = ..
bt.two = ...
br.three = ...
some2 = bt.four()

oops.. "br.three = ..." instead of "bt.three"

but let's compare :

with br {
   .one = ..
   .two = ...
   .three = ...
   some1 = .four()
}

with bt {
   .one = ..
   .two = ...
   .three = ...
   some2 = .four()
}

Much less possibilities for copy-paste errors. Wrong code completion 
suggestion(by editor) can not produce error. It is explicit and clear, it 
has much less noise in code. IMO it just better in any ways.


 > IMHO, saving sourecode-bits is not everything; readable code is
 > everything. Therefore, a -1 from me.

This proposal is not about saving bits. And it is about more readable code. 
As I understand you are -1 for "implicit" "where". I'm too.

On 13.04.2016 22:48, Michael Peternell via swift-evolution wrote:
> I think the idea is good, but I think it would be bad for the language overall.
>
> -1
>
> As a plus, it makes code easier to write, sometimes; or at least it seems so. On the other hand, I think it makes code harder to comprehend. A `with` statement introduces a new scope in which it is not obvious what exactly happens. For example:
>
> with(someObject) {
>      foo()
> }
>
> what does it do? The mere fact that I wrote `with(someObject)` suggests that `someObject.foo()` is what should happen. But it could also mean `self.foo()` or just the global function `foo()`.
>
> Usually, you can see from the lexical surrounding of a call, what is happening. `foo()` means either `self.foo()`, or it means `foo()`. It's okay that I, as the class author know if my class has a `foo()` method or not. But do I have to know the API of someObject completely, including any future changes to its API?
>
> For example: Suppose I have a class Foo, which has a method foo(). A remote co-worker wrote class Bar, which has a method bar(). Because I need bar a lot recently, I decided to use the with() form:
>
> with(someBarObject) {
>     ...
>     foo()
>     ...
>     bar()
>     ...
>     bar()
>     ...
>     foo()
> }
>
> All is fine, it's clear that foo() calls my method and bar() calls that method on the someBarObject object. Then, suddenly, my co-worker had a great idea! He also implements foo() - it cannot break any existing code doing so, because existing code will not call foo(), because up until now class Bar did not implement foo(), after all...
>
> Suddenly, my Foo class breaks, because my co-worker implemented foo() in his own class :-o
>
> Such a thing really shouldn't happen. I am arguing that `with` is an inherently unsafe language feature! I also think it doesn't make the code actually easier to read. It may be an advantage if you have some reallyReallyLongAndFunnyVariableNames_withUnderscores_andSpecialTypeAnnotations, but even then I think it isn't worth it. I prefer easy-to-read over easy-to-write.
>
> I like it when I can know what some code does, just by looking at it locally. These are syntactic guarantees that I can depend on. E.g. when I read something like
>
>      x.bar()
>      foo()
>
> I know that there is an x variable somewhere (either in the current instance, or globally, or a local variable; a global variable is unlikely, because no one should dare to call a local variable just x.)
> I know that the bar() method is called on that variable x, so x implements the bar() method. Either itself, or through an extension.
> foo() is either a global method, or an instance method, or a static method.
> All of these questions can be answered if I just know the class that I am currently implementing. If I know that my class didn't implement foo(), then foo() must be global. If x is a local var, an iVar or a static var, I can tell just by looking at the surrounding code. If it isn't => it must be global.
> This (let me call it) "lexical decidability" is reduced by a `with`-operator, and this is the primary reason why I don't like it. For exactly this reason, I use a convention when writing Objective C code, that iVars should always be prefixed by an underscore, static and global vars always start with an uppercase letter, and all local vars and parameters start with a lowercase letter.
>
> ***
>
> Even if we use .bar() instead of just bar() when calling from within a `with` block, I still don't like it. How many keystrokes are we sparing? If the variable has two letters, e.g. it's named `br`, we have `with(br){` (9 characters, 1 newline) and `}` (1 character, 1 newline) compared to 2 keystrokes for each time you may omit `br`. If you count the newlines as characters, you have to use the variable br 7 times in order for the `with` statement to pay off.
>
> ***
>
> Using { $0.bar() } is interesting too. But why?? Just give the variable a short name, call it x0, for example, and you are saving even more keystrokes.
>
> ***
>
> IMHO, saving sourecode-bits is not everything; readable code is everything. Therefore, a -1 from me.
>
> -Michael
>
> _______________________________________________
> swift-evolution mailing list
> swift-evolution at swift.org
> https://lists.swift.org/mailman/listinfo/swift-evolution
>