Path: csiph.com!fu-berlin.de!uni-berlin.de!not-for-mail From: c.buhtz@posteo.jp Newsgroups: de.comp.lang.python Subject: =?utf-8?q?=5BPython-de=5D_Re=3A_Code_Style_Review?= Date: Mon, 28 Nov 2022 19:53:07 +0000 Lines: 40 Message-ID: <966557f12986f04f279fc185ac376188@posteo.de> References: <480b80d21e58d280c437b4854bfc7cf1@posteo.de> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable X-Trace: news.uni-berlin.de eD1yZyYZ7BASd4XkjHvYBwGsrfYEmJ6D7nGZvT2kkFZg== Authentication-Results: mail.python.org; dkim=pass reason="2048-bit key; unprotected key" header.d=posteo.jp header.i=@posteo.jp header.b=i7K/qRb3; dkim-adsp=pass; dkim-atps=neutral DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.jp; s=2017; t=1669665188; bh=e8bIaa/DGNzf5xGO2Ba/aUuZZaDgznx1eiKPvqrS9bI=; h=Date:From:To:Subject:From; b=i7K/qRb3MXdbdDNYwZ4SjTA9uQY7mfQNOcBPHFr7HV5xxnbe+VQegw9OxomHDsyUl hKOwzqW0BKQgLSlxcY8a3JGEbhKQ7cThoI4l1q2i+hMpKhC4gu1q/DkBKR+vdQONdw F+uN58GVXsqcRinphs0Uf2sKxpZpN7+5Ppijfa38bHDL9lwZ1zg8jiwtLwwUnWcxXo fO9drmo8E6XQLUfqiREdFCElOePI+79yPwuRFUKqlrg5GPiBJNQAwCc8OCHS6NKkhS MQ923keYXJcLbzcnkc2E6skmMw7Y2sViY8hJPzYzbbnVaxWn8ZVJ/sIhbGz/3Cyn4X +k7mV+CSBwWkA== Mail-Followup-To: python-de@python.org In-Reply-To: Message-ID-Hash: GYZ3M4EMQ65MWMEGX4AWII6C6HY2TZCF X-Message-ID-Hash: GYZ3M4EMQ65MWMEGX4AWII6C6HY2TZCF X-MailFrom: c.buhtz@posteo.jp X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-python-de.python.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.7 Precedence: list List-Id: Die Deutsche Python Mailingliste Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: Xref: csiph.com de.comp.lang.python:5866 Eigentlich wollte ich dieses Thema nicht anschneiden, um den Unmut der=20 Liste nicht auf mich zu ziehen. ;) Ich versuche es vorsichtig. Ich hatte Eingangs ja das Lob ausgesprochen, dass er sich mit dem Thema=20 so bewusst und intensiv auseinandersetzt. So lernt man nicht nur,=20 sondern verbessert auch seine Skills. Am 28.11.2022 16:48 schrieb Christopher Arndt: > Au=C3=9Ferdem k=C3=B6nntest du auch noch entweder "black" oder "yapf" ode= r ein > =C3=A4hnliches Tool benutzen, > [..] > - Sortiere die Imports mit "isort" Die hier genannten automatisierenden Tools verhindern dieses Lernen.=20 Gerade Anf=C3=A4ngern m=C3=B6chte ich dringend von solchen Tools abraten. S= orge=20 selbst daf=C3=BCr, dass dein Code sauber ist. =C3=9Cberpr=C3=BCfe deinen Co= de mit=20 Lintern, aber das Korrigieren bitte selbst machen. Linter kann man auch=20 live in einer IDE dazuschalten, so dass du schon beim Tippen durch=20 Hervorhebungen die Mitteilung erh=C3=A4ltst, dass du da etwas besser machen= =20 k=C3=B6nntest. Du m=C3=B6chtest sauberen Code im Repo haben und du m=C3=B6chtest alle Leut= e=20 deines Teams dazu bringen nur sauberen Code zu committen? Dann mach einen pre-commit-hook oder einen Unittest der Alarm schl=C3=A4gt,= =20 wenn ein Linter wie pylint/flake8 Fehler wirft. F=C3=BCr kleinere Projekte= =20 w=C3=A4hle ich gerne die unittest Variante (Ein sicher verbesserungsw=C3=BC= rdiges=20 Beispiel: [1]). [1] --=20