-
Notifications
You must be signed in to change notification settings - Fork 253
Expand file tree
/
Copy pathusing_the_global_statement.html
More file actions
256 lines (169 loc) · 14.6 KB
/
using_the_global_statement.html
File metadata and controls
256 lines (169 loc) · 14.6 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
<!DOCTYPE html>
<!--[if IE 8]><html class="no-js lt-ie9" lang="en" > <![endif]-->
<!--[if gt IE 8]><!--> <html class="no-js" lang="en" > <!--<![endif]-->
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Using the global statement — Python Anti-Patterns documentation</title>
<link rel="stylesheet" href="../_static/css/theme.css" type="text/css" />
<link rel="stylesheet" href="../_static/css/ribbon.css" type="text/css" />
<link rel="stylesheet" href="../_static/css/font-awesome-4.1.0/css/font-awesome.min.css" type="text/css" />
<link rel="stylesheet" href="../_static/css/menu.css" type="text/css" />
<link rel="index" title="Index"
href="../genindex.html"/>
<link rel="search" title="Search" href="../search.html"/>
<link rel="top" title="Python Anti-Patterns documentation" href="../index.html"/>
<link rel="up" title="Maintainability" href="index.html"/>
<link rel="next" title="Using single letter to name your variables" href="using_single_letter_as_variable_name.html"/>
<link rel="prev" title="Returning more than one variable type from function call" href="returning_more_than_one_variable_type_from_function_call.html"/>
</head>
<body class="wy-body-for-nav" role="document">
<div class="wy-grid-for-nav">
<nav data-toggle="wy-nav-shift" class="wy-nav-side">
<div class="wy-side-nav-search">
<a href="../index.html"> Python Anti-Patterns</a>
<div role="search">
<form id ="rtd-search-form" class="wy-form" action="../search.html" method="get">
<input type="text" name="q" placeholder="Search docs" />
<input type="hidden" name="check_keywords" value="yes" />
<input type="hidden" name="area" value="default" />
</form>
</div>
</div>
<div class="wy-menu wy-menu-vertical" data-spy="affix" role="navigation" aria-label="main navigation">
<ul class="current">
<li class="toctree-l1"><a class="reference internal" href="../correctness/index.html"><i class="fa fa-check"></i> Correctness</a></li>
<li class="toctree-l1 current"><a class="reference internal" href="index.html"><i class="fa fa-puzzle-piece"></i> Maintainability</a><ul class="current">
<li class="toctree-l2"><a class="reference internal" href="from_module_import_all_used.html">using wildcard imports (<cite>from … import *</cite>)</a></li>
<li class="toctree-l2"><a class="reference internal" href="not_using_with_to_open_files.html">Not using <code class="docutils literal notranslate"><span class="pre">with</span></code> to open files</a></li>
<li class="toctree-l2"><a class="reference internal" href="returning_more_than_one_variable_type_from_function_call.html">Returning more than one variable type from function call</a></li>
<li class="toctree-l2 current"><a class="current reference internal" href="#">Using the <code class="docutils literal notranslate"><span class="pre">global</span></code> statement</a></li>
<li class="toctree-l2"><a class="reference internal" href="using_single_letter_as_variable_name.html">Using single letter to name your variables</a></li>
<li class="toctree-l2"><a class="reference internal" href="dynamically_creating_names.html">Dynamically creating variable/method/function names</a></li>
</ul>
</li>
<li class="toctree-l1"><a class="reference internal" href="../readability/index.html"><i class="fa fa-eye"></i> Readability</a></li>
<li class="toctree-l1"><a class="reference internal" href="../security/index.html"><i class="fa fa-lock"></i> Security</a></li>
<li class="toctree-l1"><a class="reference internal" href="../performance/index.html"><i class="fa fa-dashboard"></i> Performance</a></li>
<li class="toctree-l1"><a class="reference internal" href="../django/index.html"><i class="fa fa-book"></i> Django</a></li>
</ul>
</div>
</nav>
<section data-toggle="wy-nav-shift" class="wy-nav-content-wrap">
<nav class="wy-nav-top" role="navigation" aria-label="top navigation">
<i data-toggle="wy-nav-top" class="fa fa-bars"></i>
<a href="../index.html">Python Anti-Patterns</a>
</nav>
<div class="wy-nav-content" id="signup-box" >
<div class="rst-content">
<div class="navigation" role="navigation" aria-label="breadcrumbs navigation">
<ul class="wy-breadcrumbs">
<li><a href="../index.html">Documentation</a> »</li>
<li><a href="index.html"><i class="fa fa-puzzle-piece"></i> Maintainability</a> »</li>
<li>Using the <code class="docutils literal notranslate"><span class="pre">global</span></code> statement</li>
<li class="wy-breadcrumbs-aside">
<a href="../_sources/maintainability/using_the_global_statement.rst.txt" rel="nofollow"> View page source</a>
</li>
</ul>
<hr/>
</div>
<div role="main">
<div class="section" id="using-the-global-statement">
<h1>Using the <code class="docutils literal notranslate"><span class="pre">global</span></code> statement<a class="headerlink" href="#using-the-global-statement" title="Permalink to this headline">¶</a></h1>
<p>Global variables are dangerous because they can be simultaneously accessed from multiple sections of a program. This frequently results in bugs. Most bugs involving global variables arise from one function reading and acting on the value of a global variable before another function has the chance to set it to an appropriate value.</p>
<p>Global variables also make code difficult to read, because they force you to search through multiple functions or even modules just to understand all the different locations where the global variable is used and modified.</p>
<div class="section" id="examples">
<h2>Examples<a class="headerlink" href="#examples" title="Permalink to this headline">¶</a></h2>
<p>The code below uses global variables and a function to compute the area and perimeter of a rectangle. As you can see, even with two functions it becomes difficult to keep track of how the global variables are used and modified.</p>
<div class="highlight-python notranslate"><div class="highlight"><pre><span></span><span class="n">WIDTH</span> <span class="o">=</span> <span class="mi">0</span> <span class="c1"># global variable</span>
<span class="n">HEIGHT</span> <span class="o">=</span> <span class="mi">0</span> <span class="c1"># global variable</span>
<span class="k">def</span> <span class="nf">area</span><span class="p">(</span><span class="n">w</span><span class="p">,</span> <span class="n">h</span><span class="p">):</span>
<span class="k">global</span> <span class="n">WIDTH</span> <span class="c1"># global statement</span>
<span class="k">global</span> <span class="n">HEIGHT</span> <span class="c1"># global statement</span>
<span class="n">WIDTH</span> <span class="o">=</span> <span class="n">w</span>
<span class="n">HEIGHT</span> <span class="o">=</span> <span class="n">h</span>
<span class="k">return</span> <span class="n">WIDTH</span> <span class="o">*</span> <span class="n">HEIGHT</span>
<span class="k">def</span> <span class="nf">perimeter</span><span class="p">(</span><span class="n">w</span><span class="p">,</span> <span class="n">h</span><span class="p">):</span>
<span class="k">global</span> <span class="n">WIDTH</span> <span class="c1"># global statement</span>
<span class="k">global</span> <span class="n">HEIGHT</span> <span class="c1"># global statement</span>
<span class="n">WIDTH</span> <span class="o">=</span> <span class="n">w</span>
<span class="n">HEIGHT</span> <span class="o">=</span> <span class="n">h</span>
<span class="k">return</span> <span class="p">((</span><span class="n">WIDTH</span> <span class="o">*</span> <span class="mi">2</span><span class="p">)</span> <span class="o">+</span> <span class="p">(</span><span class="n">HEIGHT</span> <span class="o">*</span> <span class="mi">2</span><span class="p">))</span>
<span class="nb">print</span><span class="p">(</span><span class="s2">"WIDTH:"</span> <span class="p">,</span> <span class="n">WIDTH</span><span class="p">)</span> <span class="c1"># "WIDTH: 0"</span>
<span class="nb">print</span><span class="p">(</span><span class="s2">"HEIGHT:"</span> <span class="p">,</span> <span class="n">HEIGHT</span><span class="p">)</span> <span class="c1"># "HEIGHT: 0"</span>
<span class="nb">print</span><span class="p">(</span><span class="s2">"area():"</span> <span class="p">,</span> <span class="n">area</span><span class="p">(</span><span class="mi">3</span><span class="p">,</span> <span class="mi">4</span><span class="p">))</span> <span class="c1"># "area(): 12"</span>
<span class="nb">print</span><span class="p">(</span><span class="s2">"WIDTH:"</span> <span class="p">,</span> <span class="n">WIDTH</span><span class="p">)</span> <span class="c1"># "WIDTH: 3"</span>
<span class="nb">print</span><span class="p">(</span><span class="s2">"HEIGHT:"</span> <span class="p">,</span> <span class="n">HEIGHT</span><span class="p">)</span> <span class="c1"># "HEIGHT: 4"</span>
</pre></div>
</div>
</div>
<div class="section" id="solutions">
<h2>Solutions<a class="headerlink" href="#solutions" title="Permalink to this headline">¶</a></h2>
<div class="section" id="encapsulate-the-global-variables-into-objects">
<h3>Encapsulate the global variables into objects<a class="headerlink" href="#encapsulate-the-global-variables-into-objects" title="Permalink to this headline">¶</a></h3>
<p>One common solution for avoiding global variables is to create a class and store related global variables as members of an instantiated object of that class. This results in more compact and safer code.</p>
<p>In the modified code below, the author eliminates the need for the global variables <code class="docutils literal notranslate"><span class="pre">WIDTH</span></code> and <code class="docutils literal notranslate"><span class="pre">HEIGHT</span></code> by encapsulating this data into a class called <code class="docutils literal notranslate"><span class="pre">Rectangle</span></code>.</p>
<div class="highlight-python notranslate"><div class="highlight"><pre><span></span><span class="k">class</span> <span class="nc">Rectangle</span><span class="p">:</span>
<span class="k">def</span> <span class="fm">__init__</span><span class="p">(</span><span class="bp">self</span><span class="p">,</span> <span class="n">width</span><span class="p">,</span> <span class="n">height</span><span class="p">):</span>
<span class="bp">self</span><span class="o">.</span><span class="n">width</span> <span class="o">=</span> <span class="n">width</span>
<span class="bp">self</span><span class="o">.</span><span class="n">height</span> <span class="o">=</span> <span class="n">height</span>
<span class="k">def</span> <span class="nf">area</span><span class="p">(</span><span class="bp">self</span><span class="p">):</span>
<span class="k">return</span> <span class="bp">self</span><span class="o">.</span><span class="n">width</span> <span class="o">*</span> <span class="bp">self</span><span class="o">.</span><span class="n">height</span>
<span class="k">def</span> <span class="nf">circumference</span><span class="p">(</span><span class="bp">self</span><span class="p">):</span>
<span class="k">return</span> <span class="p">((</span><span class="bp">self</span><span class="o">.</span><span class="n">width</span> <span class="o">*</span> <span class="mi">2</span><span class="p">)</span> <span class="o">+</span> <span class="p">(</span><span class="bp">self</span><span class="o">.</span><span class="n">height</span> <span class="o">*</span> <span class="mi">2</span><span class="p">))</span>
<span class="n">r</span> <span class="o">=</span> <span class="n">Rectangle</span><span class="p">(</span><span class="mi">3</span><span class="p">,</span> <span class="mi">4</span><span class="p">)</span>
<span class="nb">print</span><span class="p">(</span><span class="s2">"area():"</span> <span class="p">,</span> <span class="n">r</span><span class="o">.</span><span class="n">area</span><span class="p">())</span>
</pre></div>
</div>
</div>
</div>
<div class="section" id="references">
<h2>References<a class="headerlink" href="#references" title="Permalink to this headline">¶</a></h2>
<ul class="simple">
<li><p><a class="reference external" href="http://c2.com/cgi/wiki?GlobalVariablesAreBad">Cunningham & Cunningham, Inc. - Global Variables Are Bad</a></p></li>
<li><p><a class="reference external" href="https://pylint.readthedocs.io/en/latest/user_guide/messages/warning/global-statement.html">PyLint - W0603, global-statement</a></p></li>
</ul>
</div>
</div>
</div>
<footer>
<div class="rst-footer-buttons" role="navigation" aria-label="footer navigation">
<a href="using_single_letter_as_variable_name.html" class="btn btn-neutral float-right" title="Using single letter to name your variables"/>Next <span class="fa fa-arrow-circle-right"></span></a>
<a href="returning_more_than_one_variable_type_from_function_call.html" class="btn btn-neutral" title="Returning more than one variable type from function call"><span class="fa fa-arrow-circle-left"></span> Previous</a>
</div>
<hr/>
<div role="contentinfo">
<p>
</p>
</div>
<!--End mc_embed_signup-->
<a href="https://github.com/snide/sphinx_rtd_theme">Sphinx theme</a> provided by <a href="https://readthedocs.org">Read the Docs</a> - Last updated: Sep 29, 2020
</footer>
</div>
</div>
</section>
</div>
<script type="text/javascript">
var DOCUMENTATION_OPTIONS = {
URL_ROOT:'../',
VERSION:'',
COLLAPSE_INDEX:false,
FILE_SUFFIX:'.html',
HAS_SOURCE: true
};
</script>
<script type="text/javascript" src="../_static/jquery.js"></script>
<script type="text/javascript" src="../_static/underscore.js"></script>
<script type="text/javascript" src="../_static/doctools.js"></script>
<script type="text/javascript" src="../_static/language_data.js"></script>
<script type="text/javascript" src="../_static/js/theme.js"></script>
<script type="text/javascript">
jQuery(function () {
SphinxRtdTheme.StickyNav.enable();
});
</script>
<!-- Place this tag right after the last button or just before your close body tag. -->
<script async defer id="github-bjs" src="https://buttons.github.io/buttons.js"></script>
</body>
</html>