この汚いコード、明日までに書き直しちゃえ！ ? ＠IT自分戦略研究所
なんというか、書き直したわけではないのですが、新しく機能を追加してバグを生んでしまったというお話です。
機能といっても、渡された引数の中身を見て必要ならエスケープする、という単純な事なのですが。
で、書き直した時は従来の機能を損なわずに変更したつもりでした。
しかし、見事に損なっていたわけです。
エスケープするつもりの無いものまでがエスケープされてしまっていました。
敗因は、面倒くさがって自動的に判別させようとした、という事だろうと結論付けています。
エスケープが必要な場合は、スイッチになる引数を渡す、という処理をすればよかったのだと。
今はそう思っています。
…結局、そのバグは下手に触れないので、呼び出し側で何とかしましたが、いずれは直さないといけないなと思いながら、それでまたバグが…と思うとなかなか無理です。
