I get why removing global state is a good thing in general, but here you are replacing it with a whole lot of complexity for not a particularly clear reason. Is tracking the user count with an integer dangerous in this case (Have you run into problems with it)? Is your solution not using global state anyway, just wrapped in a library?
All in all, are the abstractions (a bunch of new concepts plus a new library) worth it to just replace an integer?
function timerFunc() {
if (connectionCounter) get results();
}
socket.on('connect', function() { connectCounter++; });
socket.on('disconnect', function() { connectCounter--; });
The real problem is you want to start and stop the timer when someone connects. Just keep it simple and keep it going, but not doing anything if there is nobody connected, like above.
That always depends on whether whatever is contained within the abstraction is something that might change. Sigh, humor.
Leaving it as a primitive basically says 'you may have to hack the programming language or framework in order to modify code that depends on this piece of the logic'. Global state is something you can manage in terms of modular components. Allows for dynamic orchestration of components.
It's more the long term issue that keeps repeating (more humor) than it is just replacing an integer. Being able to decide 'is this component something that needs to be reasoned about in it's own independent logical layer from the rest of the system or is it not' or 'does this component relate to other components in a well defined way'.
Connect is an action, an observable action has occurred. An abstraction can trigger a behavior to other abstractions through that predefined relation. That's the point.
"The server always asks for new location data, every 30 seconds, even when no one is listening."
...
"Our task is to pause the timer when there are no websocket connections and start the timer when a client connects."
That sounds like asking for trouble. Maybe there's something I am not understanding, but why not have the timer run continuously and have the timer action check whether it needs to request location data?
Yes, that will cause the process to wake up every 30 seconds even if there is no action, but unless circumstances are very special that won't be a significant problem.
The central problem of this post is not whether the timer is running continually. The central problem is how does the timer know when to stop/start making requests.
It's a problem because knowing this depends on access to information that is trapped in a different context. For example, the number of closed connections is only knowable within the context of the connection event handler. So how do you share this information with the timer?
One approach is to mutate global state. You move the information into a shared context where it's accessible everywhere. A second option, which is the focus of this post, is to look for better abstractions. In this case, those are abstractions over time. And when you have those abstractions, you can eliminate the global state.
Very nice explanation. Gives a good insight how you can "think in observables". Two small things 1. instead of merging zero$ into connectionClosed$ you can use the startWith() method. 2. Instead of using the head method to pluck the first item of an array you could use array destructuring. (But I can understand that you wouldn't do it in this to limit the amount of "new" concepts)
[+] [-] orf|7 years ago|reply
All in all, are the abstractions (a bunch of new concepts plus a new library) worth it to just replace an integer?
The real problem is you want to start and stop the timer when someone connects. Just keep it simple and keep it going, but not doing anything if there is nobody connected, like above.[+] [-] s-shellfish|7 years ago|reply
Leaving it as a primitive basically says 'you may have to hack the programming language or framework in order to modify code that depends on this piece of the logic'. Global state is something you can manage in terms of modular components. Allows for dynamic orchestration of components.
It's more the long term issue that keeps repeating (more humor) than it is just replacing an integer. Being able to decide 'is this component something that needs to be reasoned about in it's own independent logical layer from the rest of the system or is it not' or 'does this component relate to other components in a well defined way'.
Connect is an action, an observable action has occurred. An abstraction can trigger a behavior to other abstractions through that predefined relation. That's the point.
[+] [-] mpweiher|7 years ago|reply
"The server always asks for new location data, every 30 seconds, even when no one is listening."
...
"Our task is to pause the timer when there are no websocket connections and start the timer when a client connects."
That sounds like asking for trouble. Maybe there's something I am not understanding, but why not have the timer run continuously and have the timer action check whether it needs to request location data?
Yes, that will cause the process to wake up every 30 seconds even if there is no action, but unless circumstances are very special that won't be a significant problem.
[+] [-] maxhallinan|7 years ago|reply
The central problem of this post is not whether the timer is running continually. The central problem is how does the timer know when to stop/start making requests.
It's a problem because knowing this depends on access to information that is trapped in a different context. For example, the number of closed connections is only knowable within the context of the connection event handler. So how do you share this information with the timer?
One approach is to mutate global state. You move the information into a shared context where it's accessible everywhere. A second option, which is the focus of this post, is to look for better abstractions. In this case, those are abstractions over time. And when you have those abstractions, you can eliminate the global state.
[+] [-] the8472|7 years ago|reply
[+] [-] unknown|7 years ago|reply
[deleted]
[+] [-] wilgertvelinga|7 years ago|reply
[+] [-] fleitz|7 years ago|reply
Or put your server in front of a CDN / Nginx, set cache headers for 30 seconds ;)