[ruby-core:122951] [Ruby Feature#17316] On memoization

Issue #17316 has been updated by Dan0042 (Daniel DeLorme).


marksiemers (Mark Siemers) wrote in #note-12:
> A big motivation for this comes from a recent change to rubocop-rails, which is now enforcing "Rails/FindByOrAssignmentMemoization" (see: https://rails.rubystyle.guide/\#find-by-memoization)
> 
> A more rails-specific example below of this proposal:
> ```ruby
> # Previously allowed (though not performant in the case of nil returned by find_by)
> def foo
>   @foo ||= Foo.find_by(id:)
> end
> 
> # The new rule enforces this syntax
> def foo
>   if instance_variable_defined?(:@foo)
>     @foo
>   else
>     @foo = Foo.find_by(id:)
>   end
> end
> ```

This is appalling. If "best practices" are going to encourage this kind of un-ruby-ish horror, this memoization issue is more urgent to solve than I had expected.

Personally I prefer a DSL like `memo def foo` but this `@||=` idea is pretty interesting in how it communicates this is a syntax that applies to instance variables.

----------------------------------------
Feature #17316: On memoization
https://bugs.ruby-lang.org/issues/17316#change-114255

* Author: sawa (Tsuyoshi Sawada)
* Status: Open
----------------------------------------
I have seen so many attempts to memoize a value in the form:

```ruby
@foo ||= some_heavy_calculation(...)
```

improperly, i.e., even when the value can potentially be falsy. This practice is wide spread, and since in most cases memoization is about efficiency and it would not be critical if it does not work correctly, people do not seem to care so much about correcting the wrong usage.

In such case, the correct form would be:

```ruby
unless instance_variable_defined?(:@foo)
  @foo = some_heavy_calculation(...)
end
```

but this looks too long, and perhaps that is keeping people away from using it.

What about allowing `Kernel#instance_variable_set` to take a block instead of the second argument, in which case the assignment should be done only when the instance variable is not defined?

```ruby
instance_variable_set(:@foo){some_heavy_calculation(...)}
```

Or, if that does not look right or seems to depart from the original usage of `instance_variable_set`, then what about having a new method?

```ruby
memoize(:foo){some_heavy_calculation(...)}
```



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- [e-mail]
 To unsubscribe send an email to [e-mail]
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/