-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CS2 Discussion: Output: Destructuring assignment #4919
Comments
From @carlsmith on 2016-08-03 01:59 Thank you for getting involved. No discussion has taken place here yet on destructuring assignments, so it's good that you've started that conversation. My only thought at this stage is that CoffeeScript destructuring syntax is too cryptic, so it would be nice to simplify things. I'd love to see a comprehensive proposal for what the exact syntax should be. If it's simple enough to be a good spec, it shouldn't be that difficult to explain. As for what you're specifically asking for, to be honest, I don't understand the proposal or what this code aught to do: b = {}
b.{poet: {name, address: [street, city]}} = futurists I'd rather force assignments to be broken up across a few lines than have this super cryptic syntax. It's impossible to figure out what the code does from just looking at it and knowing CoffeeScript. |
From @rattrayalex on 2016-08-05 04:43 Yes, welcome @Asc2011 ! Personally I find destructuring assignment an exciting feature, and one I've been missing in ES6. I think nested destructrucing assignment might be too complex, as @carlsmith perhaps alludes to; I can't tell if the example is setting That said,
seems quite clear and useful to me. I'd also consider
Any other ideas? Thoughts? |
From @JimPanic on 2016-08-05 06:03 I love what ES calls destructuring assignment. In Elixir it's called pattern matching and it is the utter most awesome feature of the language. Destructured assignment should stay simple, it mimics mathematical equations and are quite intuitive to follow as-is. I think nested to-property assignments like this should be implemented at runtime, not in the syntax. |
From @Asc2011 on 2016-08-05 16:07 sry for being late on this :) I did some homework and found we are almost there. Most examples work in CS as expected :
not working :
to watch out for because the syntax is dangerously similar :
I'm happy with how things work right now and hey this is not scala. In the future maybe 'computed-property.keys' make sense ? For now i can do that with backticks. I'd like to see destructuring on Iterators/Sets someday. But more important would be a 'harmonized' for-loop on iterators, as already proposed here. greets andreas |
From @Asc2011 on 2016-08-05 16:15 The idea was to end up with the poet greets andreas |
From @rattrayalex on 2016-08-06 15:09 Thanks Andreas! I'm not quite sure I've followed everything here; where does this leave us? |
From @Asc2011 on 2016-08-07 08:57 Hello alex @rattrayalex, The syntax i had in mind as well as your example from above
represent new language-features. You can't do a assignments BTW: we should fix this forum, so that one can put in indented code-examples ?! So i see this issue moving down the prio-list :
I'll keep an eye on destructuring-assignments and will maintain the 'compatabillity-chart' based on the tests from the attached-file above. As a CS-user i'd welcome a chart that shows which features of ES-specs are fully/partially/not supported. Like the browser-vendors do it. greets andreas |
From @mrmowgli on 2016-08-20 23:26 This is an interesting area for me as well. I like the way ES6 has implemented the destructuring and the spread and rest facilities, and I think CS6 should make an attempt to adopt the functionality if not directly the syntax. The rules for destructuring should match the ES6 versions. It's a shame they used portions of CS for language features, because it does break a lot of the code. Particularly the splats. Perhaps an alternate with the Realistically though this might need to be a breaking change for CS6. |
From @JimPanic on 2016-08-22 11:44 We will not be able to keep up with ES201x without introducing breaking changes into CS, I think that is quite certain. This is not inherently a bad thing. A lot of CS code that is out there right now might still work, but the ecosystem has changed and continues to change quite rapidly. I see a lot of former must-have libraries being replaced by the standard library and ES201x language features; just like with CS before. |
From @carlsmith on 2016-08-22 22:21 On a technical level, if we fixed classes, we would have everything we need. Everything else can be added to CoffeeScript without breaking anything, or isn't required for interop with JavaScript. It's just a nice to have. As long as CoffeeScript has a good set of features in its own right, and can continue to use the entire JavaScript ecosystem, it can continue as its own language, with its own destructuring features. There's no fundamental reason why CoffeeScript needs the same feature set as ES6 (or syntax). On a pragmatic level, CoffeeScript still needs a lot of features it could otherwise live without, just to stem the exodus to ES6, and semicolons. I'm personally backing a novel approach, where we define domain specific JavaScript dialects for each project, adding features to the dialect as you go along. I have a working compiler, but there's lots left to do, so not much point going into details, but it works, and it makes adding features much easier than using macros, and a lot more efficient. Once you trivialise writing parsers, and target Babel, there's no reason why users need to pick a language, and deal with the compromises. They can just pick a small, core dialect, then add features to it, as they need them. All the common features will already be implemented, so they would literally just select what they want from a list of features that is pretty much a superset of every altJS language. If you need something a bit different, there's an API that is still easier to use than macros. Sorry. I didn't mean to plug my library. It's not even out yet. I just wanted to say that it is possible to trivialise parsing (without resorting to clunky parser generators), and is a better approach long term than creating six hundred similar languages, each with its own benefits, and requiring each project to pick one of them. |
From @rattrayalex on 2016-08-23 02:21 @carlsmith sounds interesting. Do you have a repo you can share? |
From @carlsmith on 2016-08-23 15:34 I do have it online here, just for backup, but the code is really sketchy at the moment. I'm still figuring some things out, and then I'll tidy everything up. You can probably figure out how to use it from looking at the code at the bottom of the file. If you do have a look and have any questions, just open an issue on that repo mate. I think it was your idea @rattrayalex to target Babel. That was a really good idea. Thanks. |
From @rattrayalex on 2016-08-23 16:58 Wow looks quite interesting. Looking forward to following along on the development @carlsmith |
From @Asc2011 on 2016-08-24 11:53 @carlsmith just to make sure i understand your concept in 'shalt' :
and in effect, the community-/core-developers make sure that
I believe this to be a strong concept :) It's truly novel and en-vogue - looking at D3js-V4. It could become a game-changer. As it simplifies decisions and seems truly liberal. I welcome such a solution in every way. The discussion about it should continue on coffeescript6/discuss#25. |
From @carlsmith on 2016-08-24 14:26 @Asc2011 - Thanks man, and you totally get the concept, but my library doesn't parse CoffeeScript. Sorry - I should have made that clearer. It parses custom dialects of a language similar to JavaScript. Most of the features are up to the user, but I hardcoded the grammar for primitives and whitespace, and used Brendan Eich's Paren Free Mode. WhitespaceThere's no Automatic Semicolon Insertion. Implicit terminators are based on significant newlines, like in Python, so you can omit semicolons safely. The logic is consistent for statements, and expressions, so you can omit trailing commas according to the precise same logic as you can omit semicolons. array = [
foo, bar
spam, eggs
]
if x > y {
do spam
do eggs
} That also gets rid of the git-blame problem, without allowing trailing commas. Indentation and trailing whitespace (outside of string literals) are totally insignificant. Paren FreeIn all the predefined grammar, parens around predicates are optional, but blocks must be braced, even if they contain a single statement. The parser assumes you'll do your dialect this way, so it's essentially required, if you want an easy life. if foo { do spam } else { do eggs }
while foo > bar {
// do something here...
} It works the same with functions and generators too: square = function x { return x * x }
counter = generator {
i = 0
loop { yield i++ }
} These examples use the grammar I would opt for, but you may prefer something different. It's only the primitive literals, whitespace and Paren-Free that you're basically stuck with (and wrapping expressions in parens to create an expression group, but who would ever change that anyway??). Sorry for a long, off topic reply. Just thought people seem interested enough, and I don't want to disappoint anyone when it turns out it's not a CoffeeScript parser. It's really just a general purpose altJS parser, but I figured, if we're transpiling to JS, I may as well take a rare opportunity make JavaScript clean and simple, but wanted to avoid really adding anything. Innovation can happen around the library. Again, the source is well sketchy. Please don't jump to any conclusions based on it. I'm trying to work out the best API before tightening any bolts too much. |
From @rattrayalex on 2016-08-24 15:31 This is definitely awesome stuff, but also definitely the wrong thread for it 😄 Let's open a new one and move these comments there? Personally I think it's 👍 to chat about coffee-cousins on this repo, and it might be valuable to @carlsmith to be able to gather feedback and share ideas here. (sidenote, pardon the hypocrisy: here's the link to Brendan Eich's "Paren Free Mode" post for the lazy – and this post by Douglas Crockford seems to be another source of inspiration behind Shalt and is quite interesting). EDIT: to clarify, I actually think coffeescript6/discuss#25 might not be the right place for this thread to move, since @carlsmith doesn't expect his tool to be used for CSNext directly. |
From @carlsmith on 2016-08-25 00:39 Thanks @rattrayalex. Appreciate it. Sounds good. We could move that discussion to the shalt repo, as I'll eventually use that once the code is ready, but it would be nice to keep things on this repo, closer to where the discussion is taking place too. Hmm. I'll could open an issue for it here, once it's a bit further along, or anyone can as soon as they want to discuss it really. |
From @GeoffreyBooth on 2016-09-06 01:34 Getting back to destructuring assignment, what exactly is the ask here? CoffeeScript already has a feature it calls destructuring assignment; is it missing something that ES2015’s version of destructuring assignment provides? If CoffeeScript’s version is missing something, then it sounds like this is a feature request—some new syntax that outputs ES2015’s syntax, so that we get whatever missing feature ES2015 added. If the features are functionally equivalent, then the request seems to be just that CoffeeScript output the ES2015 syntax for simplicity and readability’s sake; similar to transpiling Nothing against the latter, but I want to understand which it is so that we can prioritize. Missing features should take priority over improving output. |
From @lydell on 2016-09-06 06:02 @GeoffreyBooth the only thing I know that ES destructuring can do but CS destructuring can't is plucking values from a generator: function* countToFive() {
let i = 1
while (i <= 5) {
yield i
i++
}
}
const counter = countToFive()
const [a, b, c, d, e, f, g] = counter
console.log('results:', a, b, c, d, e, f, g)
//> results: 1 2 3 4 5 undefined undefined However, that's not because of CoffeeScript's syntax, it's because of what CoffeeScript compiles to. Outputting ES2015+ would solve that. (But requires |
From @rattrayalex on 2016-09-09 15:40 Closing as this has been marked as "no action" in https://github.com/coffeescript6/discuss/blob/master/Features.md I'm personally not opposed to re-opening in the future. |
From @rattrayalex on 2016-09-09 15:42 Given the long-running confusion around what exactly is being asked for here (I know I personally misinterpreted), I suggest that any ask for related features occur in a new thread. Any comments clarifying the purpose here would certainly be welcome to occur on this thread. |
From @Asc2011 on 2016-07-31 12:36
Destructuring assignment
the example from the CS-hompage run with CS 1.10 / browser(chrome) / latest
I'm bringing this up, 'cos i can't figue out the current the status of 'destructuring assignments'.
I'm not sure if that is expected behaviour. I'd expect 'poet' to be avail in local-scope.
There has been some discussions during 2013 at CS-github.issues.page eg. Issue 3500, Issue 3225
I'd like to be able to assign to a local var :
This has been proposed before - the regarding issue(s) is/are solved | closed. The status is unclear to me ?
So i vote for it in CS6.
The text was updated successfully, but these errors were encountered: