Yes, someone actually did this and I found it running on our server

  • @[email protected]
    link
    fedilink
    74
    edit-2
    7 months ago

    As a Real Programmer™ I have developed such a deep fear of anything time and date related that I would fully endorse dispatching an API call to the tz_database instead of attempting any fucking part of this.

    Kids, it’s fine to meme about silly stuff… but date and time is deadly serious, regardless of how careful you think you’re being you are wrong.

    Do you know how many timezones there are in Indiana? No? Look it up and scream in horror.

    • @[email protected]
      link
      fedilink
      267 months ago

      What if I told you that weekend days are locale dependent?!

      Time and date is the black hole where optimistic programmers go to die. Nothing is simply with localisation and if you think it is, you mustn’t have worked enough with it.

      Source: Run a system that schedules millions of interactions across the world and deeply depend on this. The amount of code to manage and/or call out to external services to give us information about time zones, summer time, locale specific settings, day names, calendar systems, week numbers etc etc.

    • @[email protected]
      link
      fedilink
      97 months ago

      Here’s a fun thought experiment: What gregorian year and date will the spacian date value of zero correlate to? Trick question.

      The atomic clock on the moon and every other celestial body colonized will simply start at zero, and thanks to relativity it will not actually be the same rate of time passing as on earth.

      Enjoy your nightmares.

    • @coloredgrayscale
      link
      77 months ago

      Luckily we won’t colonize the moon or another planet anytime soon…

    • Kogasa
      link
      5
      edit-2
      7 months ago

      IMO every datetime should be in utc, and variables for datetimes should either be suffixed “Utc” or have a type indicating their time zone (DateTimeOffset or UtcDateTime etc). Conversion to local time happens at the last possible second (e.g. in the view model or an outbound http request parameter). Of course that doesn’t solve the problem of interoperating with other morons programmers who don’t follow these rules, but it keeps things a lot neater locally.

      Scheduling based on regional time conventions (holidays, weekends, etc) is just not great though.

      • @[email protected]
        link
        fedilink
        27 months ago

        Throwing UTC everywhere doesn’t solve comparisons around leap seconds. I’m sure they’re other issues with this method, but this is kinda the point of “just use a library”. Then it’s someone else’s problem.

        • Kogasa
          link
          27 months ago

          I’m a .NET dev, I don’t have a concept of “just use a library.” Everything is a library. I don’t mean “using int for datetimes is ok as long as you label it utc,” I just mean “don’t deal with time zones.”

        • @v9CYKjLeia10dZpz88iU
          link
          17 months ago

          Unix is the easiest format I’ve used. It’s easy to parse, it’s consistent, there’s not usually competing unix like formats, it converts perfectly to other time formats, most file explorers can immediately sort it correctly, and it’s clearly the date from which the universe spawned into existence.

            • @v9CYKjLeia10dZpz88iU
              link
              0
              edit-2
              7 months ago

              I also really like the Bitcoin block number. It will likely be one of the most provable records of time passing, but not as convienent for tracking or converting time.

  • @[email protected]
    link
    fedilink
    567 months ago

    You want to expand your business to Europe. Bam, your code is broken, in Europe the week starts on Monday.

    Than you want to expand to the middle east. Bam, broken again… Because in arab countries and Israel, the weekend is on Friday and Saturday.

    Then you want to expand to Mexico and India. Bam, broken again, their weekend is only on Sunday.

    • Kogasa
      link
      237 months ago

      The obvious solution is to inject an IWeekendDaysOfWeekProvider service in the inversion of control container. In your, uh, javascript web app.

    • @RonSijm
      cake
      link
      77 months ago

      Not using CultureInfo.InvariantCulture for basically everything

    • @[email protected]
      link
      fedilink
      57 months ago

      I was wondering why the second example returned monday and tuesday. I had no idea the week could start any day other than monday

  • @eluvatar
    link
    507 months ago

    Honestly the first one is the only one that works when people define the first day of the week differently. On the other hand, it does make you wonder. If Sunday is the first day of the week (as it is in many places) then how is it also part of the weekend?

    • @[email protected]
      link
      fedilink
      157 months ago

      But if you’re worried about locale, you can’t assume people use the string “Saturday” to describe Saturday either. That solution only works in English.

      • @[email protected]
        link
        fedilink
        77 months ago

        I assume this is in a language where the first day of the week is a fixed part of the language (like JS).

    • subignition
      link
      fedilink
      127 months ago

      Yeah it’s the front end of the week and Saturday is the rear end

      • @Schrolli97
        link
        37 months ago

        Thats not really one weekend then though, is it? Its more like last week’s post-week weekend and this week’s pre-week weekend

  • @[email protected]
    link
    fedilink
    347 months ago

    Ok another US local units are retarded rant: it’s called weekEND! why do you start your week at sunday and not monday! Sunday is part of the weekEND!

    • @[email protected]
      link
      fedilink
      97 months ago

      If you’re referring to an “end” of an object, it can refer to the extreme of a side of it. For example, aglets are at either end of a shoelace.

      • @[email protected]
        link
        fedilink
        4
        edit-2
        7 months ago

        I’m refering to end in a temporal sense because we are talking about a time context here. There is a clear direction so going backwards brings you to the begin.

    • MrScottyTay
      link
      fedilink
      English
      37 months ago

      I’m English, not American but I see it as Saturday and Sunday are the two ends of the week. Like how a string has two ends. The weekend is both the start and the finishing end of the week.

      • Chrissie
        link
        fedilink
        107 months ago

        So, when someone asks if you are free the next two weekends, you assume they’re talking about the next Saturday (tail weekend) and the next Sunday (front weekend)?

      • @[email protected]
        link
        fedilink
        17 months ago

        since we are in a temporal context here i would argue that there is a clear distrinction between beginning and end here

        • MrScottyTay
          link
          fedilink
          English
          2
          edit-2
          7 months ago

          End doesn’t always have to be the latter side of something though like I said earlier with the string analogy. The start is also an end.

  • @[email protected]
    link
    fedilink
    30
    edit-2
    7 months ago
    weekend = day_of_week in (“sat”, “sun”)
    

    As a bonus this completely sidesteps the issue of what day is 0 or 1.

    • @[email protected]
      link
      fedilink
      17 months ago

      yeah I like having an array of days that are weekend days then testing if the day is in the array. can change what days are considered weekend if we go to a three day weekend and it reads really well. I hate massive if statements

    • @[email protected]
      link
      fedilink
      17 months ago

      dayOfWeek is clearly represented by 1-7 in the example, with Sunday being 1.

      So, I guess the answer is “depends on what date library you’re linking against”

    • DroneRights [it/its]OP
      link
      fedilink
      English
      -10
      edit-2
      7 months ago

      Sunday is 1 0 and Saturday is 7 6. You just made Friday part of the weekend You forgot Sunday

        • Zagorath
          link
          fedilink
          11
          edit-2
          7 months ago

          Both Monday and Sunday are used as the first day of the week with quite some regularity. It’s a completely arbitrary standard no different to "the tenth month is the one called “October”. Or dividing a day into 24 segments which are each broken into 60 smaller segments of 60 even smaller segments. You can’t say either is “wrong” per se.

          Personally, I was brought up learning Sunday is the first day of the week, but at some point decided that was bullshit partly because it’s the week end. But also just from a practical standpoint when looking at a calendar, it’s useful to have the weekend days grouped together.

          • Cosmicomical
            link
            fedilink
            37 months ago

            Funny thing, september comes from the number 7, october from 8 and november and december from 9 and 10, as the year in ancient rome was starting around march. This problem is timeless.

            • Zagorath
              link
              fedilink
              English
              27 months ago

              Huh. I knew about the problem (that’s why I used October as my example, rather than, say, February), but I was mistaken as to the cause. The way I had always heard it told, September–December don’t match their current place in the year because of the addition of July and August. But I just looked it up and it seems you’re right. Those months are merely renamings of Quintilis and Sextilis, and the numbering issue comes from moving the start of the year from March to January.

          • @[email protected]
            link
            fedilink
            197 months ago

            Ah the same place that uses feet and inches, and puts the month before the day, and cannot read a 24 hour clock?

            • DroneRights [it/its]OP
              link
              fedilink
              English
              27 months ago

              That’s nice, you can run weekend = dayOfWeek > 5 and then explain to the boss why clients are getting work texts on a sunday

                • Cosmicomical
                  link
                  fedilink
                  27 months ago

                  Yo nodejs is just plain amazing. We should just keep improving on js and replace all other languages. Js is already on all browsers, by adopting it on the server you get huge efficiency as you can move code AND coders between backend and frontend. Of course you must make the right choices of practices and frameworks for this to be possible

                • DroneRights [it/its]OP
                  link
                  fedilink
                  English
                  27 months ago

                  I’ll be sure to tell my boss to throw away all the work he already paid for and start over in a different language. I’m sure he’ll be very understanding

            • DroneRights [it/its]OP
              link
              fedilink
              English
              -1
              edit-2
              7 months ago

              That’s nice, you can run weekend = dayOfWeek > 5 and then explain to the boss why clients are getting work emails on a sunday

        • @[email protected]
          link
          fedilink
          1
          edit-2
          7 months ago

          Depending on how you’re counting your integers, Monday is 0, being the first day of the week.

  • @MagicShel
    link
    237 months ago

    I’d make it a named function for clarity and testability and proceed to give zero shits how it is implemented. I would unironically write this code if it worked, but I wouldn’t inline it to reduce the cognitive load of reading it.

  • mac
    link
    187 months ago

    Image Transcription: Meme


    [Paneled meme with a brain that gets increasingly glowing]


    [The brain is smaller than the skull]

    if dayOfWeek.name == "Sunday" || dayOfWeek.name == "Saturday"
        weekend = true
    

    [The brain is glowing in some areas]

    if dayOfWeek < 2 || dayOfWeek > 6
        weekend = true
    

    [The brain is shooting out rays of light]

    weekend = !((dayOfWeek - 1) % 6)
    

    I am a human volunteer who transcribes posts to improve accessibility on programming.dev and you could be one too! [email protected]

  • ISometimesAdmin
    link
    fedilink
    17 months ago

    Depending on whether this code is in a hotpath (and considering how “elementary” it is, I figure that’s a possibility), this could very well be a significant speed improvement.

    Though I’d say that only excuses it if it’s truly an elementary function (and not one line as part of a larger function), as otherwise it’s unreadable garbage. But on its own it:

    • has a clear purpose
    • (presumably) isn’t reimplementing functionality
    • is easily tested
    • can be modified with no side effects (besides breaking your calendar, but that’s beside the point)
    • DroneRights [it/its]OP
      link
      fedilink
      English
      1
      edit-2
      7 months ago

      It’s one line as part of a larger function. Also, it doesn’t actually say weekend, it just executes some other functionality if !(day % 6). I made it more readable so that everyone here could understand what it does