Copilotは賢い!(4)

python 冗長なソースコードを探してください

Copilotの回答:

このコードにはいくつか冗長な部分がありますので、簡潔にするための提案をします。

改善ポイント
デフォルト条件の設定を関数化 serviceclient_column_c4_list の設定が複数箇所にあるため、関数化することでコードの重複を減らせます。

s_query.serviceclient_column_c4 の処理を統一 s_query.serviceclient_column_c4 の更新と保存の処理を共通化できます。

変数の適切な活用 transmission_conditions の代入が重複しているため、整理できます。

改善後の例


この修正により、可読性が向上し、無駄なコードを省略できます。 特に、条件分岐をシンプルにし、関数を使うことで冗長な処理を削減しました。

どう思いますか?さらに改善したい点があれば教えてください!

Copilotを上手に使う(3)

以下のpythonソースコードを簡易なソースコードに変換する

                if '3名以内/月' in get_data:
                    number_samples = 3
                elif '5名以内/月' in get_data:
                    number_samples = 5
                elif '10名以内/月' in get_data:
                    number_samples = 10
                elif '15名以内/月' in get_data:
                    number_samples = 15
                elif '20名以内/月' in get_data:
                    number_samples = 20
                elif '25名以内/月' in get_data:
                    number_samples = 25
                elif '30名以内/月' in get_data:
                    number_samples = 30

このコードを簡潔にするために、辞書を使ったマッピングを活用するのが良い方法です。以下のように書き換えることができます。

python
mapping = {
‘3名以内/月’: 3, ‘5名以内/月’: 5, ’10名以内/月’: 10,
’15名以内/月’: 15, ’20名以内/月’: 20, ’25名以内/月’: 25, ’30名以内/月’: 30
}
number_samples = mapping.get(get_data)

この方法では、辞書を利用して get_data の値に応じた number_samples を取得できます。辞書を使うことで、コードがスッキリし、条件分岐を減らすことができます。

いまココぼっちというマッチングアプリを開発しています。

Copilotに質問したら?

質問:python 今日の日付から年、月、日を文字形式で得る

Pythonで今日の日付から年、月、日を文字列として取得するには、datetime モジュールを使うのが一般的です。以下のように書くことで、年・月・日をそれぞれ文字列として得ることができます。

from datetime import datetime

# 今日の日付を取得

today = datetime.today()

# 年、月、日を文字列として取得

year = today.strftime(“%Y”) # 年
month = today.strftime(“%m”) # 月
day = today.strftime(“%d”) # 日

print(f”年: {year}, 月: {month}, 日: {day}”)

使える!!!