From: "matz (Yukihiro Matsumoto) via ruby-core" <ruby-core@...>
Date: 2024-03-14T09:04:06+00:00
Subject: [ruby-core:117146] [Ruby master Feature#20202] Memoized endless method definitions

Issue #20202 has been updated by matz (Yukihiro Matsumoto).

Status changed from Open to Rejected

I don't accept this proposal for several reasons:

* I don't see significant use-case
* method set for a class should be stable for consistency/understandability
* fragile method set could be bad for object shape and method caching

Matz.


----------------------------------------
Feature #20202: Memoized endless method definitions
https://bugs.ruby-lang.org/issues/20202#change-107227

* Author: matheusrich (Matheus Richard)
* Status: Rejected
----------------------------------------
I propose introducing a shorthand for memoized endless method definitions:

```rb
class Foo
  def bar ||= :memoized_value

  # It should behave like
  def bar = (@bar ||= :memoized_value)
end
```

Not only is this shorter and (IMO) a natural follow-up for endless method definitions, but it's also a common pattern on Ruby codebases. It's very useful to decompose responsibilities into several objects:

```rb
class User
  def notifications_enabled? = settings.notifications?
  def enable_notifications = (settings.notifications = true)
  def disable_notifications = (settings.notifications = false)

  private

  def settings = @settings ||= Settings.new(self)
end

class User::Settings
  attr_writer :notifications

  def initialize(user)
    @user = user
    @notifications = false
  end

  def notifications? = @notifications
end

u = User.new
u.notifications_enabled? # => false
u.enable_notifications
u.notifications_enabled? # => true
```

In the example, the `settings` method could be rewritten as

```rb
  def settings ||= Settings.new(self)
```



-- 
https://bugs.ruby-lang.org/
 ______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/postorius/lists/ruby-core.ml.ruby-lang.org/