while loopとbreakの使い方について

while loopとbreakの使い方について質問です。
この下の一つ目のパラメーター(long_word)はstringで中にアルファベットがたくさん入っています
そして、二つ目のパラメーター(char)もstringで一つだけアルファベットが入っています。
その時に、一つ目のパラメーターのアルファベットをwhile loopを使って一つずつ調べて二つ目のパラメーターと同じアルファベットが出たらその前までのアルファベットをreturnさせたいです。
私は下のような式を作ったけれどうまくいきませんでした。
どこが間違っているのか教えていただけないでしょうか？

python
1def letters_up_to_char(long_word, char):
2    new_string = ''
3    i = 0
4    while long_word[i] != char:
5        long_word[i] += new_string
6        if long_word[i] == char:
7            break
8        i += 1
9    return new_string

　P.S

ちなみにこれを実行したら下のようなエラーが出ました。

long_word[i] += new_string
TypeError: 'str' object does not support item assignment

行動規範の内容に同意します

回答3件

質問に対する回答ではありませんが、文字列のfindメソッドを使えば、ループを使わずに作れるので、読んでおくことをお勧めします。

投稿2021/07/16 21:55

ppaul

総合スコア24670

Miyu_Oyama

2021/07/16 22:29

新しいやり方のアドバイスありがとうございます(^-^) もしこのやり方を使う場合、対象の文字を見つけられることはわかったのですがどのように上のプログラムのように答えが出るのでしょうか？何番目に対象の文字があると表す式ができてその前までの文字を出せませんでした。 findメゾットだけでは足りないのでしょうか？重ねて質問してしまってすみません…

行動規範の内容に同意します

ベストアンサー

最初のは

python
1        long_word[i] += new_string

を

python
1        new_string += long_word[i]

にすればいけるんじゃね?

って思ったけど、ヒットしなかったときがまずいスね。
--- 追記 ---
少し違うやり方をするとしたら

python
1def letters_up_to_char(long_word, char):
2    new_string = ''
3    for i in range(len(long_word)):
4        if long_word[i] == char:
5            return new_string
6        new_string += long_word[i]
7    return ''

--- 追記 ---
さらに違うやり方をすると

python
1def letters_up_to_char(long_word, char):
2    for i in range(len(long_word)):
3        if long_word[i] == char:
4            return long_word[0:i]
5    return ''