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

Question

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

def letters_up_to_char(long_word, char):
    new_string = ''
    i = 0
    while long_word[i] != char:
        long_word[i] += new_string
        if long_word[i] == char:
            break
        i += 1
    return new_string

　P.S

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

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

Answer 1

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

Comments

[Miyu_Oyama]

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

Answer 2

最初のは

        long_word[i] += new_string

を

        new_string += long_word[i]

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

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

def letters_up_to_char(long_word, char):
    new_string = ''
    for i in range(len(long_word)):
        if long_word[i] == char:
            return new_string
        new_string += long_word[i]
    return ''

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

def letters_up_to_char(long_word, char):
    for i in range(len(long_word)):
        if long_word[i] == char:
            return long_word[0:i]
    return ''

で、ここでいう i を見つける簡単な方法があるよってのが ppaulさんのアドバイスですね。

Comments

[Miyu_Oyama]

返信ありがとうございます！！
どちらのアドバイスも試してみたところ成功しました。
もやもやが晴れました
的確なアドバイスとても勉強になりました(^^)

[takasima20]

(･∀･)b

[Miyu_Oyama]

なるほど！
そんな方法もあるんですね。
この方法もマスター出来るようにプログラムを見直します。
追記までつけてくださりありがとうございます????

Answer 3

def letters_up_to_char(long_word, char):
new_string = ''
i = 0
while True:
if long_word[i] == char:
break
new_string += long_word[i]
i += 1

return new_string

すみません。無事解決できました。
でもこのやり方は例を調べて当てはめていった答えのたどり着き方なので、最初に挙げたやり方で成功する方法があれば教えてほしいです。
よろしくお願いします。