Timeline for Golf Me a Random Dubstep of the Day List
Current License: CC BY-SA 3.0
16 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Feb 11, 2016 at 12:52 | history | edited | Connor Bell | CC BY-SA 3.0 |
Left my key in...
|
| Feb 11, 2016 at 11:16 | history | edited | Connor Bell | CC BY-SA 3.0 |
Removed a variable I didn't need
|
| Feb 11, 2016 at 11:08 | history | edited | Connor Bell | CC BY-SA 3.0 |
Golfed a tiny little bit more.
|
| Feb 11, 2016 at 10:23 | history | edited | Connor Bell | CC BY-SA 3.0 |
Clarified some things
|
| Feb 11, 2016 at 10:13 | comment | added | Connor Bell |
@ӍѲꝆΛҐӍΛПҒЦꝆ Thanks. I implemented most of those - And I'm so embarrassed I forgot to change re to something else! Unfortunately I use d twice - Once to call the function and once for the recursion, though I may look around meta since I don't think I actually need to call the function for it to be considered legitimate in JS...
|
|
| Feb 11, 2016 at 10:11 | history | edited | Connor Bell | CC BY-SA 3.0 |
Golfed a little more, added suggestions
|
| Feb 11, 2016 at 2:19 | comment | added | Mama Fun Roll |
h.toString().replace(/,/g, "\n") could become h.join`\n`, if I'm not mistaken (correct me if I'm wrong). Replace \n with an actual newline.
|
|
| Feb 11, 2016 at 2:16 | comment | added | Mama Fun Roll |
.split('"') -> .split`"`
|
|
| Feb 11, 2016 at 2:15 | comment | added | Mama Fun Roll |
Oh, change a.indexOf(z)>-1?h.push(a.split(z)[1]):h.push(a) to h.push(~a.indexOf(z)?a.split(z)[1]:a). Also, since you don't seem to be using d anywhere else in your code, try moving that string to your o call at the end.
|
|
| Feb 11, 2016 at 2:10 | comment | added | Mama Fun Roll |
Also, re is a 2-character variable. Why not use R? And try keeping only m = re.exec(v) in the for loop condition.
|
|
| Feb 11, 2016 at 2:08 | comment | added | Mama Fun Roll | You have some accidentally unnecessary whitespace in there. Look around arrows. | |
| Feb 11, 2016 at 1:14 | comment | added | Quill |
j.indexOf(y) > -1 -> j.includes(y). "nextPageToken\"\: \"" into `nextPageToken": "`
|
|
| Feb 10, 2016 at 23:51 | history | edited | Alex A. | CC BY-SA 3.0 |
Tweaked header capitalization and added syntax highlighting
|
| Feb 10, 2016 at 23:41 | comment | added | Conor O'Brien | As a JavaScript developer who bears your name, I forgive you. ;) Nice post, and welcome to PPCG! | |
| Feb 10, 2016 at 22:36 | review | First posts | |||
| Feb 10, 2016 at 23:03 | |||||
| Feb 10, 2016 at 22:34 | history | answered | Connor Bell | CC BY-SA 3.0 |