我刚刚转换到PyCharm，我非常满意它提供的所有警告和提示来改善我的代码。除了这个我不理解的:
此检查检测到在外部作用域中定义的重复名称。
我知道从外部作用域访问变量是不好的做法，但是影响外部作用域有什么问题呢？
这里有一个示例，PyCharm 给出了警告信息：
data = [4, 5, 6]
def print_data(data): # <-- Warning: "Shadows 'data' from outer scope
print data
print_data(data)
在上面的代码片段中并没有什么大不了的问题，但是想象一下一个有更多参数和更多代码行的函数。然后你决定将data参数重命名为yadda，但是错过了函数体中其中一个使用它的地方...现在data指向全局，你开始出现奇怪的行为 - 如果没有全局名称data，你会有一个更明显的NameError。
还要记住的是，在Python中，所有内容都是对象（包括模块、类和函数），因此函数、模块或类之间没有不同的命名空间。另一个场景是，在模块顶部导入函数foo，并在函数体中的某个地方使用它。然后你给函数添加了一个新的参数，并将其命名为-倒霉的是-foo。
最后，内置的函数和类型也存在于同一个命名空间中，可以以相同的方式被影响。
如果你的函数很短，命名合理并且具有良好的单元测试覆盖率，那么这些都不是大问题，但是有时候你必须维护不太完美的代码，并且被警告此类可能出现的问题可能会有所帮助。
nonlocal关键字来明确地使外部得分引用（就像在闭包中一样）。请注意，这与遮蔽不同，因为它明确地不会遮蔽来自外部的变量。 - Felix D.目前投票最高且被接受的答案以及大部分回答都没有抓住重点。
你的函数有多长，你的变量名称是否具有描述性（以希望最小化可能的名称冲突的机会）并不重要。
事实上，你的函数的局部变量或其参数恰好在全局范围内共享一个名称是完全无关紧要的。而且实际上，无论你选择地多么仔细，你的本地变量名都不能预见“我的酷炫名称yadda将来也会用作全局变量吗？”。解决方法？只需不必担心！正确的想法是设计你的函数从其签名参数中获取输入。这样你就不需要关心全局范围内是什么（或将会是什么），阴影问题就根本不是问题了。
换句话说，当你的函数需要使用相同名称的局部变量和全局变量时，阴影问题才真正有意义。但你应该避免这样的设计。OP的代码并没有真正存在这样的设计问题。只是PyCharm不够聪明，并在必要时发出警告。因此，只是为了让PyCharm高兴，也为了让我们的代码更干净，可以查看来自silyevsk的答案的解决方案，将全局变量完全删除。
def print_data(data):
print data
def main():
data = [4, 5, 6]
print_data(data)
main()
这是正确的方式来“解决”这个问题，通过修复/删除你的全局事件，而不是调整你当前的本地函数。
print_data 是一个全局变量。想一想…… - bruno desthuilliers在某些情况下，一个好的解决方法可能是将变量和代码移动到另一个函数中：
def print_data(data):
print data
def main():
data = [4, 5, 6]
print_data(data)
main()
我希望在PyCharm的右上角看到一个绿色的勾号。我会在变量名后附加下划线以清除此警告，以便我可以集中注意力处理重要的警告。
data = [4, 5, 6]
def print_data(data_):
print(data_)
print_data(data)
# noinspection PyShadowingNames - Christopher Galpin这取决于函数的长度。函数越长，未来进行修改的人写data以为它指的是全局变量的可能性就越大。实际上，它指的是局部变量，但因为该函数非常长，他们不知道存在一个同名的局部变量。
对于您的示例函数，我认为覆盖全局变量并不算坏事。
看起来这是100%的 pytest 代码模式。
参见:
我也遇到了同样的问题，这就是我找到这篇文章的原因 ;)
# ./tests/test_twitter1.py
import os
import pytest
from mylib import db
# ...
@pytest.fixture
def twitter():
twitter_ = db.Twitter()
twitter_._debug = True
return twitter_
@pytest.mark.parametrize("query,expected", [
("BANCO PROVINCIAL", 8),
("name", 6),
("castlabs", 42),
])
def test_search(twitter: db.Twitter, query: str, expected: int):
for query in queries:
res = twitter.search(query)
print(res)
assert res
它将使用This inspection detects shadowing names defined in outer scopes.警告。
要解决这个问题，只需将twitter fixture移动到./tests/conftest.py中即可。
# ./tests/conftest.py
import pytest
from syntropy import db
@pytest.fixture
def twitter():
twitter_ = db.Twitter()
twitter_._debug = True
return twitter_
请删除 twitter fixture，就像在 ./tests/test_twitter2.py 中所做的那样：
# ./tests/test_twitter2.py
import os
import pytest
from mylib import db
# ...
@pytest.mark.parametrize("query,expected", [
("BANCO PROVINCIAL", 8),
("name", 6),
("castlabs", 42),
])
def test_search(twitter: db.Twitter, query: str, expected: int):
for query in queries:
res = twitter.search(query)
print(res)
assert res
这将让QA、PyCharm和所有人都感到高兴。
data = [4, 5, 6] # Your global variable
def print_data(data): # <-- Pass in a parameter called "data"
print data # <-- Note: You can access global variable inside your function, BUT for now, which is which? the parameter or the global variable? Confused, huh?
print_data(data)
data，都深埋在几百行的代码之中呢？ - John Colanduonidata是此函数中的本地名称，因此我甚至不会费心去检查/记住是否存在同名的全局变量，更不用说它包含了什么内容了。 - user395760data，会在不同的作用域中查找直到找到一个变量，因此它会找到全局变量data，并返回False。data = [1, 2, 3]; def foo(): print(data); foo() - dwanderson我认为这个规则并没有太大的帮助。我只是通过进入 设置 -> 编辑器 -> 检查，然后取消勾选这个规则来禁用它：
Shadowing names from outer scope
请执行以下操作：
data = [4, 5, 6]
def print_data():
global data
print(data)
print_data()
global 声明一个全局变量。global。 - OBu要忽略这个警告，就像Christopher在评论中说的那样，你可以在它上面加注释
# noinspection PyShadowingNames