Re: Respect the Built-in Names ~ ੩తίʔυղੳπʔϧͷ׆༻ ~ PyCon JP 2017 Day2 LT (2017/9/9) @momijiame

͓લɺ୭Α • Twitter: @momijiame • Blog: http://blog.amedama.jp/ • Book: εϚʔτPythonϓϩάϥϛϯά • Proposal: ࣮ફSQLAlchemy (མબ)

͜ͷ LT ͷ͖͔͚ͬ • PyCon JP 2017 Day1 LT • @CardinalXaro ͞Μ • λΠτϧ: Respect the Built-in Names • ಺༰: ϏϧτΠϯؔ਺ʹ΋ਓݖ͸͋Δʂ

ਓݖ৵֐ͷྫ def max(elements): … def f(id, name): … str = 'Hello, World!'

͋Δ͋Δ…

ͦΜͳʹ͋Δ͋ΔͳΒ ࣗಈͰνΣοΫ͍ͨ͠

ຊ೔ͷϝΠϯςʔϚ

੩తίʔυղੳπʔϧͷ׆༻

੩తίʔυղੳπʔϧͱ͸ʁ • ιʔείʔυΛ੩తʹղੳ͢Δπʔϧͷ͜ͱ • ੩త: ιʔείʔυΛ࣮ࡍʹ͸ಈ͔͞ͳ͍ • యܕతʹ͸ AST (ந৅ߏ଄໦) Λղੳ͢Δ • ಈత: ιʔείʔυΛ࣮ࡍʹಈ͔ͯ͠ΈΔ

πʔϧΛ࢖ͬͨͱ͖ͷخ͠͞ • νΣοΫͯ͘͠ΕΔ͜ͱͷྫ • ίʔσΟϯάελΠϧʹԊ͍ͬͯΔ͔ʁ • όάͷݪҼͱͳΔΑ͏ͳهड़͸ͳ͍͔ʁ • ਓݖ৵֐͍ͯ͠ͳ͍͔ʁ

Pylint • Ϡό͍ॻ͖ํͯ͠Δͱڭ͑ͯ͘ΕΔ • ͘͢͝ྑ͍πʔϧ͚ͩͲӨ͕ബ͍ (ؾ͕͢Δ) $ pip install pylint

࣮ߦྫ • ઌ΄Ͳͷਓݖ৵֐͸શͯ Pylint Ͱर͑Δ $ pylint --disable=R,C overwrite.py No config file found, using default configuration ************* Module overwrite W: 1, 0: Redefining built-in 'str' (redefined-builtin) …(লུ)

ͱ͜ΖͰɺਓݖ৵֐Λ ݟ͚ͭͨͱ͖͸Ͳ͏͢Ε͹ʁ

࣮͸ରॲํ๏͕ PEP8 ʹʂ single_trailing_underscore_: used by convention to avoid conflicts with Python keyword, e.g. Tkinter.Toplevel(master, class_='ClassName') https://www.python.org/dev/peps/pep-0008/

ࣦΘΕͨਓݖΛऔΓ໭ͨ͠ྫ def max_(elements): … def f(id_, name): … str_ = 'Hello, World!'

·ͩ·ͩ͋ΔΑʂ • ศརͳ੩తղੳπʔϧͨͪ • pycodestyle (چ pep8 / ίʔσΟϯάελΠϧνΣοΧ) • pydocstyle (چ pep257 / ίʔσΟϯάελΠϧνΣοΧ) • Flake8 (ϓϥΨϒϧͳઃܭͷ൚༻੩తίʔυղੳπʔϧ) • Radon (ίʔυͷෳࡶ౓Λఆྔతʹଌఆ) • autopep8, yapf (ίʔυϑΥʔϚολ)

πʔϧΛ࢖͏্ͰͷϙΠϯτ • ॏཁ: ࣗಈͰνΣοΫ͕࣮ߦ͞ΕΔΑ͏ʹ͢Δ • ΤσΟλͱ౷߹͢Δ • อଘ࣌ʹࣗಈͰνΣοΫ͞ΕΔͱ͔ • CI ʹ૊ΈࠐΉ • ϓογϡ͢ΔͱࣗಈͰνΣοΫ͞ΕΔͱ͔

·ͱΊ • ੩తίʔυղੳπʔϧΛຯํʹ͚ͭΑ͏ • ਓݖ৵֐͕ى͖ͦ͏ͳͱ͖͸ޙΖʹ '_' ͚ͭΔ • ৭ʑͳ੩తղੳπʔϧ͕͋Δ • ҙࣝ͠ͳͯ͘΋πʔϧ͕૸Δ࢓૊Έ͕ॏཁ

͋Γ͕ͱ͏͍͟͝·ͨ͠