以前にChatGPTが作成した天気予報スクリプト。
これをBing(Copilot)が改善・修正したらどうなるかという検証です。
修正前のスクリプト
#!/bin/bash
# ユーザーに都市名を尋ねる
echo "都市名を入力してください:"
read city
# ansiweatherコマンドを実行して天気情報を表示
echo "ansiweatherの情報:"
ansiweather -l "$city"
# curlコマンドを使用してwttr.inから天気情報を表示
echo "wttr.inの情報:"
curl wttr.in/"$city?lang=ja"
行うことは単純。プロンプトで都市名を尋ね、その都市の天気予報をansiweatherとwttr.inで表示させるというもの。
これを改善案を示して欲しいですと聞いてみました。
修正後のスクリプト
- getweather.sh
#!/bin/bash
# 都市名をコマンドライン引数から取得するか、ユーザーに尋ねる
city=$1
if [[ -z "$city" ]]; then
echo "都市名を入力してください:"
read city
if [[ -z "$city" ]]; then
echo "都市名が入力されませんでした。"
exit 1
fi
fi
# ansiweatherコマンドを実行して天気情報を表示
echo "ansiweatherの情報:"
if ! ansiweather -l "$city"; then
echo "ansiweatherから情報を取得できませんでした。"
fi
# curlコマンドを使用してwttr.inから天気情報を表示
echo "wttr.inの情報:"
if ! curl -s "wttr.in/${city}?lang=ja"; then
echo "wttr.inから情報を取得できませんでした。"
fi
スクリプトの動き
対話式
./getweather.sh
![](https://manualmaton.com/wp-content/uploads/2024/03/image-19.png)
都市名を引数にする
./getweather.sh Paris
![](https://manualmaton.com/wp-content/uploads/2024/03/image-20-1024x666.png)
と、対話式でもコマンドからでも実行できるようになっています。この、都市を引数に指定することによって、他のシェルスクリプトにも転用できるのはいい改良案という形でした。
コメントを残す