December 2019 - Pharo-users

Re: [Pharo-users] uses or instead of a searching literal

by Roelof Wobben

Op 27-12-2019 om 23:33 schreef Richard O'Keefe: > aString splitOn: ' -_' asSet Hello Richard, Thanks again , I find thisÂ "aString splitOn: '_- `Â asSet "Â Â much cleaner but on some way I does notÂ split for exampleÂ 'Portable Network Graphics' intoÂ " #(Portable, Network, Graphics) ".Â When I debugÂ it , it seems there is no splitting at all. So today time to dive into it why it does not work and how to solve it. Roelof

Dec. 28, 2019

Re: [Pharo-users] Hi Community

by Ben Coman

Looks cool. Thanks for sharing. cheers -ben On Sat, 28 Dec 2019 at 09:55, Pablo Navarro <pablo1n7(a)gmail.com> wrote: > Hi everyone, my name's Pablo and I'm from Argentina. I'm taking my first > steps in Pharo and created this tool ( > https://github.com/pablo1n7/Smallbook) to share with my colleagues and > show them the power of Pharo. > > > It's a simple library to create slide presentations and show them in a web > browser. > > > I used Zinc HTTP for the server, JS for the presentation controls and CSS > for the styles. > > > It's not 100% complete, I'm still working on it. I hope you find it useful > and any suggestion is welcomed > > > > Best Regards, Pablo. >

Dec. 28, 2019

Hi Community

by Pablo Navarro

Hi everyone, my name's Pablo and I'm from Argentina. I'm taking my first steps in Pharo and created this tool (https://github.com/pablo1n7/Smallbook) to share with my colleagues and show them the power of Pharo. It's a simple library to create slide presentations and show them in a web browser. I used Zinc HTTP for the server, JS for the presentation controls and CSS for the styles. It's not 100% complete, I'm still working on it. I hope you find it useful and any suggestion is welcomed Best Regards, Pablo.

Dec. 27, 2019

Re: [Pharo-users] uses or instead of a searching literal

by Richard O'Keefe

You know, this is an area where style opinions differ a lot. What the message is suggesting you try is aString splitOn: [:each | ' -_' includes: each] That's all. No need for anything more complicated. But there is a trade-off. Clarity vs efficiency. You should make your code clear and correct before you worry about efficiency, true. But you should not go out of your way to make it *less* efficient. The test you have is perfectly readable and is as fast as it gets. Using #includes: would make the code shorter, but not clearer, and certainly slower. You could also use aString splitOn: '[-_ ]' asRegex because #splitOn: is documented as treating strings as regular expressions. This is the briefest, the most confusing to read, and the slowest. It is also by far the most error-prone: - forget "asRegex" and you are in trouble - forget the square brackets and you are in trouble - put the hyphen-minus second instead of first and ... If only #split:indicesDo: were defined for Set. I cannot imagine why it is not. Here is a method definition that goes in the (new) 'splitting' category of Set: split: aSequenceableCollection indicesDo: aBlock "Perform an action specified as aBlock (with a start and end argument) to each of the indices of aSequenceableCollection that have been identified by taking the receiver as a splitter." | position | position := 1. aSequenceableCollection withIndexDo: [:element :idx | (self includes: element) ifTrue: [ aBlock value: position value: idx - 1. position := idx + 1 ]]. aBlock value: position value: aSequenceableCollection size With that in place you can use aString splitOn: ' -_' asSet On Sat, 28 Dec 2019 at 10:24, Roelof Wobben via Pharo-users <pharo-users(a)lists.pharo.org> wrote: > > Hello, > > > How can I get rid of the above error message with this code : > > > abbreviatePhrase: aString > | splitted | > splitted := aString > splitOn: [ :char | char = Character space or: [ char = $- or: [ char = $_ ] ] ]. > ^ String > streamContents: [ :stream | > splitted > reject: [ :word | word isEmpty ] > thenDo: [ :word | stream nextPut: word first uppercase ] ] > > > Regards, > > Roelof >

Dec. 27, 2019

Re: [Pharo-users] can I write this without the three if then;s

by Roelof Wobben

Hello Richard, Thanks for the feedback. I was using it because the challenge was talking about it, that yu have the use the formula a ^2Â +Â b ^2Â =Â c ^2 Â and the challenge was talking about the distance and not the distance squared. I think a better name schould be then x or xCoordinate and y or Y_Coordinate. Roelof Op 27-12-2019 om 22:48 schreef Richard O'Keefe: > Myself, I see only one issue in your code, and that is the pointless > use of sqrt. > scoreX: anInteger y: anInteger2 > | radiusSquared | > radiusSquared := anInteger squared + anInteger2 squared. > radiusSquared > 100 > ifTrue: [ ^ 0 ]. > radiusSquared > 25 > ifTrue: [ ^ 1 ]. > radiusSquared > 1 > ifTrue: [ ^ 5 ]. > ^ 10 > > Make that two issues. The names anInteger and anInteger2. > How about > scoreHorizontal: horizontal vertical: vertical > ... > > The OO dogma about "if' that you refer to is about not using > "if" to make decisions based on TYPE. Making decisions based > on NUMERIC RANGES is perfectly OK. > > You could introduce a 'HalfOpenRangeDictionary" class > [K1,K2) -> V1 > [K2,K3) -> V2 > ... > [Kn,infinity) -> Vn > But "You Ain't Gonna Need It", so don't. > > On Sat, 28 Dec 2019 at 08:18, Roelof Wobben via Pharo-users > <pharo-users(a)lists.pharo.org> wrote: >> Hello, >> >> Im trying to solve a challenge from exercism where I have to calculate the points somehow gets on a very simple darts board. >> >> I solved it like this : >> >> >> scoreX: anInteger y: anInteger2 >> | distance | >> distance := (anInteger squared + anInteger2 squared) sqrt. >> distance > 10 >> ifTrue: [ ^ 0 ]. >> distance > 5 >> ifTrue: [ ^ 1 ]. >> distance > 1 >> ifTrue: [ ^ 5 ]. >> ^ 10 >> >> >> but now I use three if then and I think it's ugly code. >> >> Is there a way I can make it more the smalltalk way ? >> >> >> Regards, >> >> Roelof >>

Dec. 27, 2019

Re: [Pharo-users] can I write this without the three if then;s

by Richard O'Keefe

Myself, I see only one issue in your code, and that is the pointless use of sqrt. scoreX: anInteger y: anInteger2 | radiusSquared | radiusSquared := anInteger squared + anInteger2 squared. radiusSquared > 100 ifTrue: [ ^ 0 ]. radiusSquared > 25 ifTrue: [ ^ 1 ]. radiusSquared > 1 ifTrue: [ ^ 5 ]. ^ 10 Make that two issues. The names anInteger and anInteger2. How about scoreHorizontal: horizontal vertical: vertical ... The OO dogma about "if' that you refer to is about not using "if" to make decisions based on TYPE. Making decisions based on NUMERIC RANGES is perfectly OK. You could introduce a 'HalfOpenRangeDictionary" class [K1,K2) -> V1 [K2,K3) -> V2 ... [Kn,infinity) -> Vn But "You Ain't Gonna Need It", so don't. On Sat, 28 Dec 2019 at 08:18, Roelof Wobben via Pharo-users <pharo-users(a)lists.pharo.org> wrote: > > Hello, > > Im trying to solve a challenge from exercism where I have to calculate the points somehow gets on a very simple darts board. > > I solved it like this : > > > scoreX: anInteger y: anInteger2 > | distance | > distance := (anInteger squared + anInteger2 squared) sqrt. > distance > 10 > ifTrue: [ ^ 0 ]. > distance > 5 > ifTrue: [ ^ 1 ]. > distance > 1 > ifTrue: [ ^ 5 ]. > ^ 10 > > > but now I use three if then and I think it's ugly code. > > Is there a way I can make it more the smalltalk way ? > > > Regards, > > Roelof >

Dec. 27, 2019

Re: [Pharo-users] uses or instead of a searching literal

by tbrunz

Since you're splitting a String, why not use a Regex to do this? -- Sent from: http://forum.world.st/Pharo-Smalltalk-Users-f1310670.html

Dec. 27, 2019

uses or instead of a searching literal

by Roelof Wobben

Dec. 27, 2019

Re: [Pharo-users] can I write this without the three if then;s

by tbrunz

You could also accomplish this with a simple 'do:' construct by creating the associations with the the ring limits & points "staggered" (by one level), so that when (each < key) you can return the 'current' association value... Which, now that I'm looking closely at your solution, Kasper, seems to be what you already proposed... Either should work. -- Sent from: http://forum.world.st/Pharo-Smalltalk-Users-f1310670.html

Dec. 27, 2019

Re: [Pharo-users] can I write this without the three if then;s

by tbrunz

Doing it the way I describe above eliminates repetitive if-then statements ('switch' or 'case' statements in other languages, which make for a maintenance mess), automatically extends for additional rings, doesn't embed "magic numbers" for defined rings within the decision logic, and takes advantage of the naturally recursive nature of this kind of graduated comparison test. -t -- Sent from: http://forum.world.st/Pharo-Smalltalk-Users-f1310670.html

Dec. 27, 2019