From: santiago@...
Date: 2015-04-30T18:23:14+00:00
Subject: [ruby-core:69042] [Ruby trunk - Bug #10967] Is "warning: private	attribute?" wrong?

Issue #10967 has been updated by Santiago Pastorino.


There's also a discussion going on in another PR, https://github.com/ruby/ruby/pull/889

@nobu, not sure what are the invalid use cases you refer to, and also what if I need to use let's say 2 private accessors? should I do ...

```
  attr_accessor :a, :b

  private :a, :a=, :b, :b=
```
Doesn't sound great to me.

----------------------------------------
Bug #10967: Is "warning: private attribute?" wrong?
https://bugs.ruby-lang.org/issues/10967#change-52300

* Author: Santiago Pastorino
* Status: Open
* Priority: Normal
* Assignee: 
* ruby -v: ruby 2.3.0dev (2015-03-01 trunk 49796) [x86_64-darwin14]
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN
----------------------------------------
The following code ...

```ruby
class Y
  def initialize
    @x = "ZOMG"
  end

  def print_x
    puts x
  end

  private

  attr_reader :x
end

Y.new.print_x
```

outputs ...

```
test.rb:12: warning: private attribute?
```

I tend to think this warning is wrong, I was surprised by https://github.com/rack/rack/pull/811 and I think this is a completely valid use case.

Also this code ...

```ruby
class Y
  def initialize
    @x = "ZOMG"
  end

  def print_x
    puts x
  end

  def assign_x
    self.x = "ZOMG ZOMG"
  end

  private

  attr_accessor :x
end

y = Y.new
y.assign_x
y.print_x
```

Works fine with warnings also. So a private writer works ok when the receiver is `self` because Ruby has a special case for it, this make me think that private writers were thought to be used.

So ... am I wrong thinking that the warning should be removed or the self special case shouldn't work and be removed from Ruby code?. It doesn't make sense to me to have both things.



-- 
https://bugs.ruby-lang.org/