Text class: changed __str__ method and added __repr__ method

[jonchar]

This should address #6042

According to the Python docs for __repr__ and __str__: print() will output what is returned by __str__, while __repr__ should return a string that looks like a valid python expression. The text class currently has a __str__ but no __repr__. This changes the existing __str__ method into __repr__ (which returns Text(x_coord, y_coord, "string")) and adds a new __str__ method that returns self._text, which achieves the behavior requested in #6042 .

This would result in the following behavior:

>>> text
Text(x_coord,y_coord,"String in self._text")
>>> print(text)
String in self._text

Thoughts?

[tacaswell]

👍 on the change __repr__, 50/50 on __str__ just returning the text. What is the use case for wanting to print a Text instance and have the fact that it is a Text instance hidden?

[jonchar]

I was going on what was requested in #6042, however I can't think of when it would be useful for print to hide the fact that it's a Text instance and not a string. Maybe just keep __repr__ and leave out __str__? If there's no __str__ defined __repr__ will be used instead.

[tacaswell]

I like the option of just dropping __str__.

[jonchar]

__str__ method has been dropped in favor of __repr__

[tacaswell]

Thanks!

On Sun, Feb 28, 2016, 10:35 Jon Charest notifications@github.com wrote:

str method has been dropped in favor of repr

—
Reply to this email directly or view it on GitHub
#6068 (comment)
.