Coming back to formatting error with brackets

I mentioned this some time ago and Mariano wasn't able to reproduce it.

OK, try this one.

the method so far is as follows:

nextTrainNumber

| next |

next :=  (CCIni getNextTrain) asNumber.

 

I have a list called RandomTrainList and if next exceeds the size of this list, it should be reset to 1.

So, the next line I want to add is 

next > self RandomTrainList size ifTrue:[] ifFalse:[].

I partially type the line and it gets reformatted to:

(next > self) RandomTrainList .

David

TotallyObjects

I mentioned this some time ago and Mariano wasn't able to reproduce it.

OK, try this one.

the method so far is as follows:

nextTrainNumber

| next |

next :=  (CCIni getNextTrain) asNumber.

 

I have a list called RandomTrainList and if next exceeds the size of this list, it should be reset to 1.

So, the next line I want to add is 

next > self RandomTrainList size ifTrue:[] ifFalse:[].

I partially type the line and it gets reformatted to:

(next > self) RandomTrainList .

David

TotallyObjects

Seth, I am just typing into a standard coding browser under standard 9.2. As far as I am aware, I haven't configured anything, included anything or run anything to change the way typing code in managed.

as you can see in the image, just simply typing the text results in the inline formatting and the spurious brackets being included.

David

On Thursday, May 14, 2020 at 3:01:57 PM UTC+1, Totally Objects wrote:

I mentioned this some time ago and Mariano wasn't able to reproduce it.

OK, try this one.

the method so far is as follows:

nextTrainNumber

| next |

next :=  (CCIni getNextTrain) asNumber.

 

I have a list called RandomTrainList and if next exceeds the size of this list, it should be reset to 1.

So, the next line I want to add is 

next > self RandomTrainList size ifTrue:[] ifFalse:[].

I partially type the line and it gets reformatted to:

(next > self) RandomTrainList .

David

TotallyObjects

Seth, David,

like I mentioned in this other thread, I get the same result using the code assist more often than I'd like. It seems at least for my thinking, the "ordering" of options in code assist is wrong for many cases. I often end up deleting these wrong brackets.

But this is such a short and quick moment that it is hard to explain. I mostly just press enter when the suggested keyword is the one I am looking for and I forget to think about the icons. I do look at the implemtning classes, but fail to pay attention to this yellow backpointing arrow...

I never had this simply when typing, however. I am absolutely sure this only happens to me when I accept code assist suggestions. From my experience, the code assist suggestions reliably appear in the foreground.

Joachim

Hi David,

I don't really know how to explain that if you don't see a code assist box in this scenario.
It's possible that the code assist box somehow shows up behind the text pane and you don't see it..but I'm not sure.

Code Assist will inject parens around expressions where there is ambiguity.
In this case, if it suggests "RandomTrainList", there will be 2 entries in the code assist box.

One for the receiver "self" and one for the receiver "next > self".  It has to offer both because
it can't decide for you which one would be correct.  And there are other scenarios (i.e. ifTrue:ifFalse:)

where it gets incredibly annoying if it doesn't inject the parens since you almost never want to apply
ifTrue:ifFalse to the inner expression...and this is obviously used a lot.

The apparent formatting behavior comes if you select the entry for "next > self", then
it knows that it has to add parens to make that work.

This is outlined in more detail in a section called "Method Suggestions:" in the doc
<a href="https://www.instantiations.com/docs/921/wwhelp/wwhimpl/js/html/wwhelp.htm#href=vaast/vaast-3cc.html" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fwww.instantiations.com%2Fdocs%2F921%2Fwwhelp%2Fwwhimpl%2Fjs%2Fhtml%2Fwwhelp.htm%23href%3Dvaast%2Fvaast-3cc.html\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEhpt6303a5iMyhFWiPLBfTtJRJEw&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fwww.instantiations.com%2Fdocs%2F921%2Fwwhelp%2Fwwhimpl%2Fjs%2Fhtml%2Fwwhelp.htm%23href%3Dvaast%2Fvaast-3cc.html\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEhpt6303a5iMyhFWiPLBfTtJRJEw&#39;;return true;">https://www.instantiations.com/docs/921/wwhelp/wwhimpl/js/html/wwhelp.htm#href=vaast/vaast-3cc.html

- Seth

On Thursday, May 14, 2020 at 4:01:09 PM UTC-4, Totally Objects wrote:

Seth, I am just typing into a standard coding browser under standard 9.2. As far as I am aware, I haven't configured anything, included anything or run anything to change the way typing code in managed.

as you can see in the image, just simply typing the text results in the inline formatting and the spurious brackets being included.

David

On Thursday, May 14, 2020 at 3:01:57 PM UTC+1, Totally Objects wrote:

I mentioned this some time ago and Mariano wasn't able to reproduce it.

OK, try this one.

the method so far is as follows:

nextTrainNumber

| next |

next :=  (CCIni getNextTrain) asNumber.

 

I have a list called RandomTrainList and if next exceeds the size of this list, it should be reset to 1.

So, the next line I want to add is 

next > self RandomTrainList size ifTrue:[] ifFalse:[].

I partially type the line and it gets reformatted to:

(next > self) RandomTrainList .

David

TotallyObjects

Sorry - as you Americans say "my bad". Yes, if I select RandomTrainList from the code assist drop down, it goes wrong but if I continue typing the complete method name it is fine.

I didn't see that as part of the problem! Anyway, I assume from all of this that this is something I will have to live with as it is the only time that it gets in the way. Thanks for all the input.

It does appear that I am only offered the one choice (see the image) (well two but the same method name but one with a colon.) but, again, I am not a deep thinker in these things so maybe I don't understand your answers terribly well.

David

On Thursday, May 14, 2020 at 9:19:50 PM UTC+1, Seth Berman wrote:

Hi David,

I don't really know how to explain that if you don't see a code assist box in this scenario.
It's possible that the code assist box somehow shows up behind the text pane and you don't see it..but I'm not sure.

Code Assist will inject parens around expressions where there is ambiguity.
In this case, if it suggests "RandomTrainList", there will be 2 entries in the code assist box.

One for the receiver "self" and one for the receiver "next > self".  It has to offer both because
it can't decide for you which one would be correct.  And there are other scenarios (i.e. ifTrue:ifFalse:)

where it gets incredibly annoying if it doesn't inject the parens since you almost never want to apply
ifTrue:ifFalse to the inner expression...and this is obviously used a lot.

The apparent formatting behavior comes if you select the entry for "next > self", then
it knows that it has to add parens to make that work.

This is outlined in more detail in a section called "Method Suggestions:" in the doc
<a href="https://www.instantiations.com/docs/921/wwhelp/wwhimpl/js/html/wwhelp.htm#href=vaast/vaast-3cc.html" rel="nofollow" target="_blank" onmousedown="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fwww.instantiations.com%2Fdocs%2F921%2Fwwhelp%2Fwwhimpl%2Fjs%2Fhtml%2Fwwhelp.htm%23href%3Dvaast%2Fvaast-3cc.html\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEhpt6303a5iMyhFWiPLBfTtJRJEw&#39;;return true;" onclick="this.href=&#39;https://www.google.com/url?q\x3dhttps%3A%2F%2Fwww.instantiations.com%2Fdocs%2F921%2Fwwhelp%2Fwwhimpl%2Fjs%2Fhtml%2Fwwhelp.htm%23href%3Dvaast%2Fvaast-3cc.html\x26sa\x3dD\x26sntz\x3d1\x26usg\x3dAFQjCNEhpt6303a5iMyhFWiPLBfTtJRJEw&#39;;return true;">https://www.instantiations.com/docs/921/wwhelp/wwhimpl/js/html/wwhelp.htm#href=vaast/vaast-3cc.html

- Seth

On Thursday, May 14, 2020 at 4:01:09 PM UTC-4, Totally Objects wrote:

Seth, I am just typing into a standard coding browser under standard 9.2. As far as I am aware, I haven't configured anything, included anything or run anything to change the way typing code in managed.

as you can see in the image, just simply typing the text results in the inline formatting and the spurious brackets being included.

David

On Thursday, May 14, 2020 at 3:01:57 PM UTC+1, Totally Objects wrote:

I mentioned this some time ago and Mariano wasn't able to reproduce it.

OK, try this one.

the method so far is as follows:

nextTrainNumber

| next |

next :=  (CCIni getNextTrain) asNumber.

 

I have a list called RandomTrainList and if next exceeds the size of this list, it should be reset to 1.

So, the next line I want to add is 

next > self RandomTrainList size ifTrue:[] ifFalse:[].

I partially type the line and it gets reformatted to:

(next > self) RandomTrainList .

David

TotallyObjects