Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python 3.2 FAQ example code typo? #55527

Closed
Retro mannequin opened this issue Feb 25, 2011 · 14 comments
Closed

Python 3.2 FAQ example code typo? #55527

Retro mannequin opened this issue Feb 25, 2011 · 14 comments
Labels
docs Documentation in the Doc dir type-bug An unexpected behavior, bug, or error

Comments

@Retro
Copy link
Mannequin

Retro mannequin commented Feb 25, 2011

BPO 11318
Nosy @birkenfeld, @bitdancer

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2011-02-25.17:06:21.157>
created_at = <Date 2011-02-25.12:25:14.624>
labels = ['type-bug', 'invalid', 'docs']
title = 'Python 3.2 FAQ example code typo?'
updated_at = <Date 2011-02-26.08:51:01.135>
user = 'https://bugs.python.org/Retro'

bugs.python.org fields:

activity = <Date 2011-02-26.08:51:01.135>
actor = 'Retro'
assignee = 'none'
closed = True
closed_date = <Date 2011-02-25.17:06:21.157>
closer = 'georg.brandl'
components = ['Documentation']
creation = <Date 2011-02-25.12:25:14.624>
creator = 'Retro'
dependencies = []
files = []
hgrepos = []
issue_num = 11318
keywords = []
message_count = 14.0
messages = ['129357', '129362', '129363', '129366', '129390', '129392', '129393', '129394', '129416', '129425', '129453', '129472', '129487', '129513']
nosy_count = 3.0
nosy_names = ['georg.brandl', 'Retro', 'r.david.murray']
pr_nums = []
priority = 'normal'
resolution = 'not a bug'
stage = None
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue11318'
versions = ['Python 3.2']

@Retro
Copy link
Mannequin Author

Retro mannequin commented Feb 25, 2011

I have found a possible typo in an example code of Python 3.2. It's located on page 32 in the PDF version of the FAQ document. The code is:

class C:
    count = 0 # number of times C.__init__ called

def __init__(self):
    C.count = C.count + 1

def getcount(self):
    return C.count # or return self.count

The code block of the __init__ method is the thing that has the typo. Shouldn't C.count = C.count + 1 be c.count = C.count + 1 ? Because the text after this code example says:

c.count also refers to C.count for any c such that isinstance(c, C) holds /.../.

How can c.count also refer to C.count if there is no c.count in the present code example in the FAQ of Python 3.2?

If this is a typo, please fix it for Python 3.2 and also for other Python versions with the same typo.

Else if this is not a typo, explain how can c.count refer to C.count if there is no c.count in the code example.

@Retro Retro mannequin assigned docspython Feb 25, 2011
@Retro Retro mannequin added the docs Documentation in the Doc dir label Feb 25, 2011
@bitdancer
Copy link
Member

Read a little further:

Caution: within a method of C, an assignment like ``self.count = 42`` creates a new and unrelated instance named "count" in ``self``'s own dict.

That is, c.count refers to C.count right up until the point where c.count is assigned a value. So, c.count = c.count + 1 will add one to the current value of C.count, and assign it to the *instance* variable c.count. c.count at that point no longer refers to the *class* variable C.count. Thus your change to the __init__ function would completely defeat the purpose of the example (which is to show how to use a *class* variable.

If you can suggest a concise wording that would have made this clearer to you, we can consider a doc patch.

@bitdancer bitdancer added the type-bug An unexpected behavior, bug, or error label Feb 25, 2011
@bitdancer
Copy link
Member

Hmm. Rereading your message, is seems like you just didn't understand the statement "c.count refers to C.count for any...". That is a statement about how the language behaves. If there is not yet an instance variable 'count', but a class variable 'count' exists, then the value of the class variable is used when c.count is evaluated. A class is a two level nested name space with a couple of special properties.

I don't really see any way to make that statement clearer.

@Retro
Copy link
Mannequin Author

Retro mannequin commented Feb 25, 2011

Caution: within a method of C, an assignment like self.count = 42
creates a new and unrelated instance named "count" in self's own dict.

More clear is to say Caution: within a method of class C, an assignment
like self.count = 42 creates a new and unrelated instance named "count"
in self's own dict.

*
*

@Retro
Copy link
Mannequin Author

Retro mannequin commented Feb 25, 2011

So you're saying that if a class' name is C, then c.count is the same as
C.count? I thought Python is case-sensitive. You know: c (small letter c) is
not equal to C (big letter C) in Python. I don't understand what you mean by
c.count because my intepreter always throws an exception that c is not
defined if I feed it with this code example and do   >>> c.count

@birkenfeld
Copy link
Member

Now it's clear that David is right -- what "c" refers to is defined in this very sentence.

@Retro
Copy link
Mannequin Author

Retro mannequin commented Feb 25, 2011

I don't understand it. My bad. Please explain to me exactly what "c" refers
to.

@bitdancer
Copy link
Member

Please consult the python tutor's list, the bug tracker is not the place to get introductory help with Python.

@Retro
Copy link
Mannequin Author

Retro mannequin commented Feb 25, 2011

How awful! A little pointer to the tutorial where this is explained would be
also quite smashing.

@Retro
Copy link
Mannequin Author

Retro mannequin commented Feb 25, 2011

"c.count also refers to C.count" Where in the code is "c.count"? Please
explain this for the sake of really closing this issue.

@merwok
Copy link
Member

merwok commented Feb 25, 2011

If the tutorial is not clear enough, try for example <http://diveintopython.org/object_oriented_framework/class_attributes.html\>. Please ask on a suitable venue if you don’t understand instead of this tracker.

(Please don’t send HTML email to this tracker, it creates false attachments.)

@Retro
Copy link
Mannequin Author

Retro mannequin commented Feb 25, 2011

Eric, the thing I don't understand is why is there that "c.count" thing. How is it possible for "c.count" to be the same reference as "C.count"? Where is "c" defined?

@merwok
Copy link
Member

merwok commented Feb 26, 2011

What this part means is: “If you create an instance of C named c, and get c.count, it will get the attribute count defined on C.” IOW: You can get a class variable from any instance.

In the example, the code in __init__ cannot assign to self.count, because it wants to increment the attribute on the class, not on the instance. This is very well explained in Dive Into Python.

Please experiment in a shell and ask on appropriate venues after this message.

@Retro
Copy link
Mannequin Author

Retro mannequin commented Feb 26, 2011

Understood. Now I get it. If you create an instance "c" of the class "C" (so c = C() ) and try to get the variable "count" from that class, then "c.count" is the same reference as "C.count".

Please make that clearer in the FAQ by saying "If you create an instance "c" of the class "C", then "c.count" is the same reference as "C.count".

@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

No branches or pull requests

3 participants