On 04/26/2013 02:42 AM, Avnesh Shakya wrote:
hi,
    I am trying to run my file using pychecker, but it's showing warning. I am 
unable to get these warning. Please help me, how to remove these warning. I am 
using pychecker first time.

avin@HP:~/github/UdacitySiteData$ pychecker udacity_to_jsonFinal.py
Processing module udacity_to_jsonFinal (udacity_to_jsonFinal.py)...

Warnings...

[system path]/dist-packages/bs4/__init__.py:206: Parameter (successor) not used
[system path]/dist-packages/bs4/__init__.py:209: Parameter (successor) not used
[system path]/dist-packages/bs4/__init__.py:213: Local variable (tag) not used

[system path]/dist-packages/bs4/element.py:306: Parameter (kwargs) not used
[system path]/dist-packages/bs4/element.py:507: (id) shadows builtin
[system path]/dist-packages/bs4/element.py:791: (next) shadows builtin
[system path]/dist-packages/bs4/element.py:903: Invalid arguments to 
(__repr__), got 2, expected 1

Thanks.


What's not clear about any of these? On line 206, you defined some function and one of the formal parameters was called successor. You don't use that value anywhere in the function. The fix? Omit the unused parameter in the function definition, and change all the caller sites. Sometimes this is impractical, for example in functions that are callbacks from code you don't control.

Likewise for the next 3.

id() is a built-in function, so you shouldn't use that name for your own functions or variables. rename it.

Likewise next, which is even more likely to cause confusion, since next is frequently used in real code, while id() is more likely a debugging aid.

If you want anything more detailed, perhaps you should show the context around each warning. You have the line numbers, so that should be easy.


--
DaveA
--
http://mail.python.org/mailman/listinfo/python-list

Reply via email to