• 0 Posts
  • 14 Comments
Joined 2 years ago
cake
Cake day: June 30th, 2023

help-circle

  • At this point I would not be surprised if steam built on top of the deck idea and the support it already provides for fairly responsive and configurable inputs, touch screen included, to launch a steam phone or something.

    I mean deck isn’t all that far from having such a device. For the actual phone network stack they would likely just partner up with someone already in the space.

    They’ve already had to tackle powering a lightweight portable device with a touch screen and adapting the UX for a small screen and non-kbd input. They’ve already established they can source parts and mass produce a competively priced device.

    But realistically I can’t see it being that much better than the recent Linux phone offerings.


  • Then you’d need to do something else.

    Precisely my point.

    And I’m not advocating for any of that. That’s just weird design, both of them, and as such a good example of something that warrants a bigger redesign in general.

    Just advocating for clear, sensible, self-documenting and most importantly, expandable and maintainable code.

    What’s idiomatic varies between languages and the conventions aren’t the same even then, when arguing across disciplines. This discussion seems to be more about different educations. I can get your point but from my personal experience in academia and working in the field it sounds undesired. But that’s just it. My, as in extremely limited, perspective. From your pov what you argue here is probably equally correct to what I think from mine is from my pov, it’s just a difference in the segment of the field we work in I suppose. Or plain old cultural differences.

    Whichever it is, I bet we both can find better use for our time. I’m thankful for the time and effort though, even if I wasn’t persuaded. Sorry to have prolonged it so.


  • That is all just external implementation details. Not sure if it was you or someone else, but the main argument in defense of the OP as in it reasonable, was that the name is wrong. That it ought to be idAdmin. None of what you just described should have anything to do with user being or not being an admin. In place of checking “isAdmin” for null, the semantical and resourcewise equivalent would be a third variable for “admin rights having been validated” or whatever. Conflating it in this one variable while renaming it to isAdmin or similar, would be even less sensical… what if somewhere else in the code you have to check whether the initial validations have been made (while the actual role or whether is admin or not is irrelevant), you’d have to check if isAdmin equals null, which in that context would be confusing, ambiguous (i.e someone reading that bit will not know this is what is being checked without additional documentation) and just a code smell in general. You do want to make the important things unambiguous and self-documenting. Even more so the bigger the codebase is and the more contributors there are across its lifetime and in parallel at any given time.

    But if we go with the original meaning of roles overall, then the union type is just a code smell that warrants a proper role thing around it.


  • That all is besides the point. There’s no real advantage to use null instead of defaulting to false there… it’s semantically more accurate and also less wasteful in that other code does not have to worry about nulls which always leads to unnecessary overhead when false is already equivalent in your proposed example.


  • I don’t really follow you there, wouldn’t it be exactly the opposite and wouldn’t checking for nulls be, as a premise, more wasteful? But doesn’t really matter, time to digress. I’m conventionally educated as an engineer so what I know and find reasonable today might be outdated and too strict for most contemporary stuff.


  • Yeah obviously with constants for the set roles per value. Some languages call them enums, but the point is that what we pass and use is always still the smallest integer type possible. With the extra bonus that if the roles ever become composable, the same value type would likely suffice for a bitflag and only thing needing refactoring would be bitshifting the constants.

    But anyway, this turns out to be the weirdest hill I find myself willing to die on.


  • Yeah, but if it is about being an admin or not, hence the bool, it’d be idiomatic and reasonable to assume it to be false if we have no data. Unless we want to try and allow admin access based on no data. Having three states for a simple binary state is weird. And if it is not about just being an admin or not, the bool is inherently a too limited choice for representation.


  • Admin is a role though, was my point. And besides, if you check for three different states, and you decide to go with a boolean to represent that, I really find it hard to believe anyone would think it reasonable. It’s valid and it’s practical, but can you really say it’s reasonable?

    I don’t do typescript, but wouldn’t a union of a null and a bool be just more resource intensive than simply using an unsigned byte-sized integer? I struggle to find reasons to ever go for that over something more reasonable and appropriate for what it attempts to represent (3 distinct states as it stands, and likely in future more than just 3 when they have a need for more granularity, as you’d often do with anything you’d need an admin role distinction in the first place), but likely I’m just not familiar with ts conventions. Happy to hear the reasoning for this though.


  • orgrinrt@lemmy.worldtoProgrammer Humor@programming.devTrue crime
    link
    fedilink
    arrow-up
    1
    arrow-down
    2
    ·
    edit-2
    4 months ago

    Yeah let’s use a union of a boolean and null to represent role, something that inherently represents more than two (…or three, I guess) different values, as opposed to something like an integer.

    Even if the name is clearly misleading in this specific case, the entire choice of using a bool here is just bad because it’s almost guaranteed you’re going to expand on that in future and then you’ll just have to entirely rewrite the logic because it simply can’t accommodate more than two values (or three with the null union… 🙈), while it gives absolute zero benefits over using something more reasonable like an integer to represent the roles, or in this case, admin, not-admin and guest. Even if you’ll end up with just admin, non-admin and guest, the integer would still work great with no disadvantages in terms of amount of code or whatever. Just increased legibility and semantical accuracy.

    Not to mention that there’s zero reason to combine the state of being logged in and the role in which you’re logged in in one variable… those are two different things. They will remain two different things in future too…

    I mean they’re already chaining elseifs (basically matching/switching, while doing it in an inefficient way to boot 🥴) as though there were an n amount of possible states. Why not just make it make sense from the start instead of whatever the hell this is?


  • This is a quote from the resolution:

    contemporary forms of racism, racial discrimination, xenophobia and related intolerance

    It’s not specifically against nazis. It’s against things like putting minorities into labour camps, jails, etc. You know, the kind of thing a lot of the places that voted green actively do this very moment.

    But let’s not let that bring our great ideals down. Surely they truly are against the very thing they do, they just can’t help themselves and need the rest of the world to make them stop. Or something?


  • Do let me know if you have the time to read it. I’ll do the same if I find some myself.

    Precisely the Russian rhetoric on and around the Ukrainian war was what got me suspicious.

    It could be that western countries just generally are not against nazis or neo-nazis and actively shoot down resolutions against negative things about nazis. That is not however my experience at all, or the de facto state of the law in many countries, such as Germany, that very strongly condemn any nazi associations or symbolism as unlawful. Do note that they also abstained for this one. There’s a reason for that, and while I could be entirely off base, I’m pretty sure it’s not that the western democracies just like nazis and Russia for example is just so nice and honorful to dare go against the western consensus on liking nazis.


  • orgrinrt@lemmy.worldtoMemes@lemmy.mlAh yes the "enlightened" democracies
    link
    fedilink
    arrow-up
    37
    arrow-down
    20
    ·
    edit-2
    7 months ago

    This seems a bit too convenient a spread to be as simple as that. The resolution very likely was phrased in a loaded way or had some bit that was dubious. Seeing as the second red one is Ukraine and all of the west is yellow, while Russia, Iran, China, India etc are green, there very likely is context that isn’t being given to us, either intentionally or by accident.

    Edit: With Russia, China, India, especially, I mean their adventures with oppression of minorities and unequality in general between cultural groups or heritages. I’m not saying the West is without fault or anything, but clearly the ones voting green are neither. They probably wouldn’t vote against their own alignments here unless it’s just word salad without meaning or responsibilities. Which is something I’m confident would lead a lot of Europe at least not accept it because it’s just a watered down version of something actually desirable.