【まず1嫁】くだすれPython(超初心者用) その59

**デフォルトの名無しさん** · 2023/07/13(木) 18:48:41.56

当スレに★Python以外のプログラミング言語での回答類を書くべからず★
派生言語はそれぞれ専スレがある（この板にあるとは限らない）ので、そっち池。
「Ruby」「Rails」「某言語では」「クソチョンw」をNGワード登録推奨。荒らしは、完全スルー放置が一番きらいです。

このスレッドは「お勉強」スレのほうには書き込めない超低レベル、
もしくは質問者自身何が何だか分からない質問を、勇気を持って書き込むスレッドです。
へび使いが優しくコメントを返しますので、お礼は Python の布教と初心者の救済をお願いします。

★エラーを解決したい場合★は、
　　表示されたエラーの全文（勝手に省略やスクショうｐ等の横着クソ行為禁止）と、
　　実行環境（Pythonのバージョン・OSとIDEの種類およびバージョン）をシッカリ書くこと。

Pythonの★ソースコードをそのまま５ちゃんに貼るとインデントが崩れてチヌ★
【【【複数の連続半角スペースはなにもなかったことにされる＆タブは普通には入れられない】】】掲示板の仕様なので、
プログラム文は↓等の、いわゆるコードうｐ用サイトに貼ってこいください。
ttps://techiedelight.com/compiler/　　Run Code機能あり。
ttp://ideone.com/　　　　　　デフォ設定はC用のため、言語選択ボタン押下がﾋﾟｺ手間かも。
ttp://codepad.org/　　　　　ほぼ直感的に使える。Run codeボタンあり。
ttp://pastebin.com/　　　　　まずまずシンプル。
ttp://dpaste.com/　　　　　　とてもシンプル。消えるまでの日数は十分長ーく指定のこと。

◇Python公式◇ http://www.python.org/
◇まとめwiki◇ ttp://python.rdy.jp/wiki.cgi

〇前スレ〇
【まず1嫁】くだすれPython(超初心者用) その58
https://mevius.5ch.net/test/read.cgi/tech/1662514433/

次スレたては >>985 あたりが挑戦する。
### END OF TEMPLATE ###

**デフォルトの名無しさん** · 2023/07/13(木) 19:23:06.59

>>1
まず1乙

**デフォルトの名無しさん** · 2023/07/14(金) 12:11:43.42

>>1
O2

**デフォルトの名無しさん** · 2023/07/17(月) 18:37:08.11

うまいプログラムの構造がさっぱりわかりません…

入門書を終えて早速何か作ってみようと思い、画像を拡張子を変換するソフトを書いています
リーダブルコードにはシンプルであれとか単一責任うんぬんと書いて納得したつもりでしたが、いざ実践しようとするとわからんわからん

「画像を読み込む関数」、「画像を変換する関数」、「画像を保存する関数」
と作成してきましたが、pillowいじくって1、2行なので車輪の再発明どころか傷つけてる感じです
1つの関数にすべて集約してconvertと名付けたら良くないのでしょうか…

**デフォルトの名無しさん** · 2023/07/17(月) 20:35:13.91

>>4
標準ライブラリを見るとわかると思うけど、1, 2行だけの関数は別に悪いことではないし、珍しい事でもない
その関数のテストを書いたり、メンテしたりしてると実感できる
他の人の書いたコードとか特にそうだと思う

ていうか、色々詰め込んだ関数作ってしまってから、リファクタリングというものを知って悩むことも多いよね
最初から書き直した方が早いんじゃねみたいな

**デフォルトの名無しさん** · 2023/07/17(月) 20:36:34.03

良いです

**デフォルトの名無しさん** · 2023/07/17(月) 20:49:31.09

convert関数でそれぞれの関数を呼び出せるようにした方が楽になりそう

**デフォルトの名無しさん** · 2023/07/17(月) 20:52:17.81

忘れてたり知らない関数を触ろうとするとき、ドキュメントとかコメントも頼るけど実際たどるのはコードだから一つの関数に盛沢山だったら結構大変

**デフォルトの名無しさん** · 2023/07/17(月) 21:52:54.20

極端な話、外からの役割がconvertなら内部はどうなっててもいい
最初から良質なコードは書けなくて当然
いつでもリファクタできる体制でいることが大事かな

**デフォルトの名無しさん** · 2023/07/18(火) 16:13:32.68

画像処理の勉強してるんなら
pillow(PIL)使ったら負け

**デフォルトの名無しさん** · 2023/07/18(火) 20:03:13.98

病∞∞∞∞！
漠∞∞∞∞∞！
向∞∞∞∞∞∞！
大∞∞∞∞∞∞∞！
度∞∞∞∞∞∞∞∞！
蔵∞∞∞∞∞∞∞∞∞！

**デフォルトの名無しさん** · 2023/07/18(火) 23:01:13.81

ご意見ありがとうございます
元画像を消去するかとか保存先はどうするかとか考えてたら複雑になってきたのでクラスを設けました
パスの扱いと画像の操作で2つ役割があるけどもう面倒なのでこのまま突き進みます
未来の修正は未来の自分に託す、ということで…

**デフォルトの名無しさん** · 2023/07/18(火) 23:04:24.96

今の自分が把握できないコードを
記憶が曖昧になった未来の自分がどうにかできると思うのか…

**デフォルトの名無しさん** · 2023/07/19(水) 11:43:21.48

成長した場合「だれだこんなバカなコード描いたのは」

成長してない場合「何描いてたか判んね」

**デフォルトの名無しさん** · 2023/07/19(水) 15:09:29.58

【まず1嫁】くだすれPython(超初心者用) その58
https://mevius.5ch.net/test/read.cgi/tech/1662514433/
まず前スレ消化を

**デフォルトの名無しさん** · 2023/07/21(金) 08:51:32.52

pythonは関数が全部外から見られてしまうから
プライベートメソッドにする代わりに関数内関数にして隠すってやり方もある
なお速度は犠牲になる

**デフォルトの名無しさん** · 2023/07/21(金) 09:17:50.55

見えるからって使うほうがどうかしてる
リフレクション系がお手軽だからこんだけ周辺ツール充実してんだし

**デフォルトの名無しさん** · 2023/07/21(金) 09:48:08.75

>>16
setter getterでそんなに速度落ちるの……