pythonでのソースコードの書き方について

Question

前提・実現したいこと

python3.7でプログラムの勉強をしています。
Python 3.7 (64-bit)のコマンドの画面を使って
ファイル名の変更するようなものを書いてみたのですが、
もっとコンパクトに書けますでしょうか。
また、関数にしたり、例外に関して記述してあるようなサイトが
あれば教えていただけないでしょうか。

■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

import os
import glob
print(os.getcwd())
os.chdir(r"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX")
print(os.getcwd())
os.listdir(os.getcwd())
files=glob.glob('*.pdf')
for oldname in files:
	if '-1' in oldname[-6:]:
		if oldname.count('-1') == 2:
			newname=oldname.replace('-1','',2)
			os.rename(oldname,newname)
		elif oldname.count('-1') == 1:
			newname=oldname.replace('-1','',1)
			os.rename(oldname,newname)
	if '-2' in oldname[-6:]:
		if oldname.count('-2') == 2:
			newname=oldname.replace('-2','',2)
			os.rename(oldname,newname)
		elif oldname.count('-2') == 1:
			newname=oldname.replace('2','',1)
			os.rename(oldname,newname)
	if '-3' in oldname[-6:]:
		if oldname.count('-3') == 2:
			newname=oldname.replace('-3','',2)
			os.rename(oldname,newname)
		elif oldname.count('-3') == 1:
			newname=oldname.replace('-3','',1)
			os.rename(oldname,newname)
			

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

どういう入力データ(ファイル名）をどのように加工して出力（変更）したいかを(プログラムの仕様と制約事項）を質問文に記載すると、もっとよい回答がつきやすくなります。

考え方のヒントだけ。
1,3回同じコードを記述したら関数化できないかを考えてみてくださいな。
今回の場合は以下の箇所です。

    if '-1' in oldname[-6:]:
        if oldname.count('-1') == 2:
            newname=oldname.replace('-1','',2)
            os.rename(oldname,newname)
        elif oldname.count('-1') == 1:
            newname=oldname.replace('-1','',1)
            os.rename(oldname,newname)
    if '-2' in oldname[-6:]:
        if oldname.count('-2') == 2:
            newname=oldname.replace('-2','',2)
            os.rename(oldname,newname)
        elif oldname.count('-2') == 1:
            newname=oldname.replace('2','',1)
            os.rename(oldname,newname)
    if '-3' in oldname[-6:]:
        if oldname.count('-3') == 2:
            newname=oldname.replace('-3','',2)
            os.rename(oldname,newname)
        elif oldname.count('-3') == 1:
            newname=oldname.replace('-3','',1)
            os.rename(oldname,newname)

2,os.listdir(os.getcwd())の戻り値を使っていないように見えますが、使っていない場合はos.listdirの呼び出し自体が不要なためソースコードから削除してくださいな。
3,glob.globはglob.iglobの呼び出しに変更して、イテレータを返す形にしてください。
4,ファイルパスの操作はpathlibライブラリを使うとシンプルにコードを記述できる事が多いです。
5,windowsの場合はos.renameはアトミック操作ではないため、os.replaceを使ってください。

Comments

[yutaka_1213]

ありがとうございます。

コメントの内容、いろいろ調べてみます。