2016-10-24 12 views
-2

私は昨日Pythonを学習し始めました。私が使っているサイトの課題の1つは、この問題を解決することです。私は問題を首尾よく解決しましたが、非常にアマチュアに見えました。スマートな人がより良い解決策を示したり、関連するアドバイスを提供したりすることができたら嬉しいです。ありがとう。Python初心者 - リファクタリング文字列関数

問題: 2つの文字列textおよびwordを入力として受け取るcensorという関数を記述します。選択した単語をアスタリスクに置き換えてテキストを返します。

マイソリューション:

​​
+1

これはスタックオーバーフローの*トピック外*なのでdownvotedになっています。作業コードを持っているが改善方法を知りたい場合は、[Code Review](http://codereview.stackexchange.com/)を使用してください。 –

答えて

3
def censor(text, word): 
    return text.replace(word, '*' * len(word)) 

単純な文字列関数を置き換える、ほとんどすべての標準ライブラリで見つかりました。

+1

それはあなたが分割して参加することができると言った変わったことですが、これは簡単です。唯一のことは、単語を1つのアスタリスクで置き換えることですが、 '*' * len(単語)である必要があります。あなたの答えを更新するなら、それを受け入れるよ、ありがとう。 –

+0

@AshtonMorganあなたは、コメント内の置換文字列として述べたように ''*' * len(word)'を使うことができます – ritesht93

関連する問題