• 5 Posts
  • 10 Comments
Joined 6 months ago
cake
Cake day: July 2nd, 2025

help-circle

  • There was a recent langdev Stack Exchange question about this very topic. It’s a bit trickier to design than it might seem at first.

    Suppose we require a keyword – say var – before all binding patterns. This results in having to write things like
    for (&(var x1, var y1, var z1), &(var x2, var y2, var z2)) in points.iter().tuple_windows() {},
    which is quite a bit more verbose than the current
    for (&(x1, y1, z1), &(x2, y2, z2)) in points.iter().tuple_windows() {}.
    Not to mention you’ll have to write let var x = 0; just to declare a variable, unless you redesign the language to allow you to just write var x = 0 (and if you do that, you’ll also have to somehow support a coherent way to express if let Some(x) = arr.pop() {} and let Some(x) = arr.pop() else {todo!()}).

    Suppose we require a keyword – say const – before all value-matching patterns that look like variables. Then, what’s currently

    match (left.next(), right.next()) {
        (Some(l), Some(r)) => {}
        (Some(l), None) => {}
        (None, Some(r)) => {}
        (None, None) => {}
    }
    

    turns into either the inconsistently ugly

    match (left.next(), right.next()) {
        (Some(l), Some(r)) => {}
        (Some(l), const None) => {}
        (const None, Some(r)) => {}
        (const None, const None) => {}
    }
    

    or the even more verbose

    match (left.next(), right.next()) {
        (const Some(l), const Some(r)) => {}
        (const Some(l), const None) => {}
        (const None, const Some(r)) => {}
        (const None, const None) => {}
    }
    

    and you always run the risk of forgetting a const and accidentally binding a new match-all variable named None – the main footgun that syntactically distinguishing binding and value-matching patterns was meant to avoid in the first place.

    Suppose we require a sigil such as $ before one type of pattern. Probably the best solution in my opinion, but that’s one symbol that can no longer be used for other things in a pattern context. Also, if you’re already using sigils before variable names for other purposes (I’ve been sketching out a language where a pointer variable $x can be auto-dereferenced by writing x), doubling up is really unpleasant.

    …So I can understand why Rust chose to give the same, most concise possible syntax for both binding and value-matching patterns. At least compiler warnings (unused, non-snake-case variables) are there to provide some protection from accidentally turning one into the other.



  • I was thinking of the three legal states as:

    • not logged in (null or {isAdmin: false, isLoggedIn: false})
    • logged in as non-admin (false or {isAdmin: false, isLoggedIn: true})
    • logged in as admin (true or {isAdmin: true, isLoggedIn: true})

    which leaves {isAdmin: true, isLoggedIn: false} as an invalid, nonsensical state. (How would you know the user’s an admin if they’re not logged in?) Of course, in a different context, all four states could potentially be distinctly meaningful.


  • My preferred way of modelling this would probably be something like
    role: "admin" | "regular" | "logged-out"
    or
    type Role = "admin" | "regular";
    role: Role | null
    depending on whether being logged out is a state on the same level as being a logged-in (non-)admin. In a language like Rust,
    enum Role {Admin, Regular}
    instead of just using strings.

    I wouldn’t consider performance here unless it clearly mattered, certainly not enough to use
    role: number,
    which is just about the least type-safe solution possible. Perhaps
    role: typeof ADMIN | typeof REGULAR | typeof LOGGED_OUT
    with appropriately defined constants might be okay, though.

    Disclaimer: neither a professional programmer nor someone who regularly writes TypeScript as of now.







  • Interesting way of handling project vs global scope:

    Some package managers (e.g. npm) use per-project scope by default, but also allow you to install packages globally using flags (npm install -g). Others (e.g. brew) use global scope.

    I like the idea of allowing both project and global scope, but I do not like the flags approach. Why don’t we apply a heuristic:

    If there is a .sqlpkg folder in the current directory, use project scope. Otherwise, use global scope.

    This way, if users don’t need separate project environments, they will just run sqlpkg as is and install packages in their home folder (e.g. ~/.sqlpkg). Otherwise, they’ll create a separate .sqlpkg for each project (we can provide a helper init command for this).

    Seems rather implicit, though, especially if the command output doesn’t specify which scope a package was installed in. If a user moves to a subdirectory, forgets they are there, and then tries to install a package, the package will unexpectedly install in global scope (though this particular version of the problem can be solved by also looking in parent directories).



  • Can’t resist pointing out how you should actually write the function in a “real” scenario (but still not handling errors properly), in case anyone wants to know.

    If the list is guaranteed to have exactly two elements:

    fn is_second_num_positive_exact(input: &str) -> bool {
        let (_, n) = input.split_once(',').unwrap();
        n.parse::<i32>().unwrap() > 0
    }
    

    If you want to test the last element:

    fn is_last_num_positive(input: &str) -> bool {
        let n = input.split(',').next_back().unwrap();
        n.parse::<i32>().unwrap() > 0
    }
    

    If you want to test the 2nd (1-indexed) element:

    fn is_second_num_positive(input: &str) -> bool {
        let n = input.split(',').nth(1).unwrap();
        n.parse::<i32>().unwrap() > 0
    }