You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
407 lines
13 KiB
407 lines
13 KiB
<!doctype html> |
|
<html lang="en"> |
|
<head> |
|
<meta charset="utf-8"> |
|
|
|
<title>Pylint, Warnings e Correções</title> |
|
|
|
<meta name="author" content="Julio Biason"> |
|
|
|
<meta name="apple-mobile-web-app-capable" content="yes" /> |
|
<meta name="apple-mobile-web-app-status-bar-style" content="black-translucent" /> |
|
|
|
<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no"> |
|
|
|
<link rel="stylesheet" href="_external/reveal.min.css"> |
|
<link rel="stylesheet" href="_external/night.css" id="theme"> |
|
|
|
<!-- For syntax highlighting --> |
|
<link rel="stylesheet" href="_external/zenburn.css"> |
|
|
|
<!-- If the query includes 'print-pdf', include the PDF print sheet --> |
|
<script> |
|
if( window.location.search.match( /print-pdf/gi ) ) { |
|
var link = document.createElement( 'link' ); |
|
link.rel = 'stylesheet'; |
|
link.type = 'text/css'; |
|
link.href = '_external/pdf.css'; |
|
document.getElementsByTagName( 'head' )[0].appendChild( link ); |
|
} |
|
</script> |
|
|
|
<!--[if lt IE 9]> |
|
<script src="reveal.js/lib/js/html5shiv.js"></script> |
|
<![endif]--> |
|
|
|
<style> |
|
.semi-opaque { |
|
background-color: rgba(0, 0, 0, 0.7); |
|
} |
|
|
|
* { |
|
hyphens: none !important; |
|
-moz-hyphens: none !important; |
|
} |
|
|
|
div.code { |
|
font-size: 150% |
|
} |
|
</style> |
|
</head> |
|
|
|
<body> |
|
<div class="reveal"> |
|
<!-- Any section element inside of this container is displayed as a slide --> |
|
<div class="slides"> |
|
<section data-background='_images/pylint.png' class='semi-opaque'> |
|
<h1>Pylint, Warnings e Correções</h1> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>missing-docstring</code></h2> |
|
</section> |
|
|
|
<section> |
|
<h2>O que é o erro</h2> |
|
<div class='code'> |
|
<p><pre><code data-trim class='python'> |
|
def funcao(valor): |
|
return valor * 2 |
|
</code></pre></p> |
|
</div> |
|
</section> |
|
|
|
<section> |
|
<h2>Solução</h2> |
|
|
|
<p>Documente suas funções!</p> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>wildcard-import</code></h2> |
|
</section> |
|
|
|
<section> |
|
<h2>O que é o erro</h2> |
|
<div class='code'> |
|
<p><pre><code data-trim class='python'> |
|
from module import * |
|
</code></pre></p> |
|
</div> |
|
</section> |
|
|
|
<section> |
|
<h2>Solução</h2> |
|
|
|
<p>Importe apenas as funções módulos que são usados.</p> |
|
</section> |
|
|
|
<section> |
|
<h2>Por que não usar "import *"</h2> |
|
|
|
<ul> |
|
<li>O módulo pode ter código não protegido por |
|
função, classe ou <code>if __name__ == '__main__'</code></li> |
|
<li>Poluíção do namespace.</li> |
|
<li>Módulo pode redefinir uma função presente no |
|
módulo atual.</li> |
|
</ul> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>unused-import</code></h2> |
|
</section> |
|
|
|
<section> |
|
<h2>O que é o erro</h2> |
|
|
|
<div class='code'> |
|
<p><pre><code data-trim class='python'> |
|
from module import function |
|
</code></pre></p> |
|
</div> |
|
|
|
<p>(E <code>function()</code> não é usado em lugar algum.)</p> |
|
</section> |
|
|
|
<section> |
|
<h2>Solução</h2> |
|
|
|
<p>Remova o import.</p> |
|
</section> |
|
|
|
<section> |
|
<h2>Dica!</h2> |
|
|
|
<p>Quebrar imports em várias linhas facilita correção desse |
|
tipo de problema.</p> |
|
|
|
<div class='code'> |
|
<p><pre><code data-trim class='python'> |
|
from module import func1 |
|
from module import func2 |
|
</pre></code></p> |
|
</div> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>unused-argument</code></h2> |
|
</section> |
|
|
|
<section> |
|
<h2>O que é o erro</h2> |
|
|
|
<div class='code'> |
|
<p><pre><code data-trim> |
|
def func(arg1, arg2): |
|
return arg1 * 2 |
|
</pre></code></p> |
|
</div> |
|
</section> |
|
|
|
<section> |
|
<h2>Solução</h2> |
|
|
|
<ul> |
|
<li>Remova o parâmetro desnecessário</li> |
|
<li>Altere o parâmetro para "_"</li> |
|
</ul> |
|
</section> |
|
|
|
<section> |
|
<h2>Corner case</h2> |
|
|
|
<div class='code'> |
|
<p><pre><code data-trim class='python'> |
|
def addSeven(foo): # "Unused argument 'foo'" |
|
foo += [7] |
|
</code></pre></p> |
|
</div> |
|
|
|
<p>O problema é a questão de referência para objetos |
|
mutáveis. O código em si está errado.</p> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>bare-except</code></h2> |
|
</section> |
|
|
|
<section> |
|
<h2>O que é o erro</h2> |
|
|
|
<div class='code'> |
|
<p><pre><code data-trim class='python'> |
|
try: |
|
func() |
|
except Exception e: |
|
pass |
|
</code></pre></p> |
|
</div> |
|
</section> |
|
|
|
<section> |
|
<h2>Solução</h2> |
|
|
|
<p>Capture apenas as exceções que você sabe lidar.</p> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>no-self-use</code></h2> |
|
</section> |
|
|
|
<section> |
|
<h2>O que é o erro</h2> |
|
|
|
<div class='code'> |
|
<p><pre><code data-trim class='python'> |
|
class A(object): |
|
def func(self, a): |
|
return a * 2 |
|
</code></pre></p> |
|
</div> |
|
</section> |
|
|
|
<section> |
|
<h2>Solução</h2> |
|
|
|
<p>A solução mais "correta" é tirar a função da classe e deixar |
|
fora do objeto.</p> |
|
|
|
<p class='fragment'>Não sendo possível, |
|
<code># pylint:disable=no-self-use</code> antes da função. |
|
</p> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>old-style-class</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>super-on-old-class</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>no-init</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>super-init-not-called</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>attribute-defined-outside-init</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>abstract-method</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>arguments-differ</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>no-member</code> e <code>maybe-no-member</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>no-value-for-parameter</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>protected-access</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>too-few-public-methods</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>too-many-ancestors</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>too-many-arguments</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>too-many-branches</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>too-many-instance-attributes</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>too-many-lines</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>too-many-locals</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>too-many-public-methods</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>too-many-statements</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>invalid-name</code></h2> |
|
</section> |
|
</section> |
|
|
|
<section> |
|
<section> |
|
<h2><code>star-args</code></h2> |
|
</section> |
|
</section> |
|
</div> |
|
</div> |
|
|
|
<script src="_external/head.min.js"></script> |
|
<script src="_external/reveal.min.js"></script> |
|
|
|
<script> |
|
|
|
// Full list of configuration options available here: |
|
// https://github.com/hakimel/reveal.js#configuration |
|
Reveal.initialize({ |
|
controls: true, |
|
progress: true, |
|
history: true, |
|
center: true, |
|
|
|
theme: 'night', |
|
transition: 'linear', |
|
|
|
// Optional libraries used to extend on reveal.js |
|
dependencies: [ |
|
{ src: '_external/classList.js', condition: function() { return !document.body.classList; } }, |
|
{ src: '_external/marked.js', condition: function() { return !!document.querySelector( '[data-markdown]' ); } }, |
|
{ src: '_external/markdown.js', condition: function() { return !!document.querySelector( '[data-markdown]' ); } }, |
|
{ src: '_external/highlight.js', async: true, callback: function() { hljs.initHighlightingOnLoad(); } }, |
|
{ src: '_external/zoom.js', async: true, condition: function() { return !!document.body.classList; } }, |
|
{ src: '_external/notes.js', async: true, condition: function() { return !!document.body.classList; } } |
|
] |
|
}); |
|
|
|
</script> |
|
|
|
</body> |
|
</html> |
|
|
|
<!-- vim:set et ts=4 sts=4 st=4: >
|
|
|