質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

解決済

2回答

1250閲覧

Pythonのコードの改善点を教えてください。。

Fujikochan

総合スコア1

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

0グッド

0クリップ

投稿2020/06/09 15:11

編集2020/06/09 15:22

Pythonを初めて3週間の初心者です。指定範囲の文字列だけを大文字にするプログラムで、うまく動作しなくて困っています。どうしても理由がわからないので、改善点を教えてください。また、できたら、うまく作動するコードの例を教えてくれたら幸いです。。。

入力例:(スペース区切りで与えられた二つの整数の範囲を、大文字にします。)
2 6
this is a pen

出力(正解)
tHIS Is a pen

私の出力
tHIS IS a pen

私が書いたコード↓

python

1 2target_range = input() 3string = input() 4range_list = target_range.split() 5string_list = [] 6result_list = [] 7 8for i in range(len(string)): 9 string_list.append(string[i]) 10 11for i in string_list: 12 if string_list.index(i) in range(int(range_list[0]) - 1, int(range_list[1]) - 1): 13 j = i.upper() 14 result_list.append(j) 15 16 else: 17 result_list.append(i) 18 19 20for i in result_list: 21 print(i, end = "")

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

list.indexは、要素のインデックスのうち常に最小のものを返します。

Python

1>>> lst = [1, 2, 3, 1, 2, 3] 2>>> lst.index(1) 30 4>>> lst.index(2) 51 6>>> lst.index(3) 72 8>>> lst.index(1) 90 10>>> lst.index(2) 111 12>>> lst.index(3) 132

この方法だと、"this is a pen" の一つ目のsと二つ目のsを見分けられません。
"this is his shop" "this is her shop" なども試してみて下さい。面白い結果になります。


添え字付きでループするのが確実でしょう。
enumerateについて調べてみて下さい。簡単なサンプルも置いておきます。

Python

1>>> lst = ['a', 'b', 'c'] 2>>> for i, e in enumerate(lst): 3... print(i, e) 4... 50 a 61 b 72 c

別解

おそらく一番簡単なのは、文字列のスライスを利用することです。二、三行で書けます。
これも調べておくと勉強になるでしょう。

コードの貼り方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押して下さい。
Python

特にPythonの場合、インデントが崩れるとコードの意味が変わってしまいます。

投稿2020/06/09 15:17

編集2020/06/09 15:23
LouiS0616

総合スコア35668

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

pythonは文字列をイミュータブルなオブジェクトとして保持するので文字列をスライスしてやるのが一番スマートな解だと思います。その他の解としては出力を1文字ずつ行う方法が考えられます。Cのputcharのような出力方法です。この方法ではコード内でループを行う必要がありますがオブジェクトを多く作らなくてよいというメリットがあります。(→Pythonでは文字型はなく、文字列を1文字ずつ切り出しても文字列型であるのでこの利点はありません)

Python3

1[start, end] = [ int(s) for s in input().split() ] 2 3count = 1 4for c in input(): 5 if count >= start and count <= end: 6 print(c.upper(), end="") 7 else: 8 print(c, end="") 9 count += 1 10 11print() # 改行 12

投稿2020/06/09 16:15

編集2020/06/09 16:26
anndonut

総合スコア667

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問