The Daily WTF 日本語
フォロー
代表的なセリフ:Sort This Out
匿名の投稿者が、以前のコンサルタントが残した問題を修正しています。彼らは最もクリティカルなチケットから着手しました。それらをクリアした後、現在は問題ないものの将来的な問題の可能性を示唆するコードの調査を開始しました。JavaScriptをレビュー中に、カスタムソート関数を発見し、即座に懸念が生じました。この関数の最初の行、obj[x._id.account_id] = x.count_totalは特に懸念されました。このコードは、MongoDBのユニーク識別子であるべきものに対して、誤ってaccount_idフィールドにアクセスしています。これは、任意のオブジェクトがデータベース内でユニーク識別子として使用されていることを示しており、問題のあるプラクティスと見なされています。識別子の問題を超えて、ソート関数内でのこのコードの目的は不明です。この行は、ソートを実行するのではなく、{"id0": 5}のようなオブジェクトを構築しているように見えます。この状況は、コードがまだ失敗を引き起こしていないため、大きな謎となっています。主な疑問は、なぜ壊れていないのか、そして最終的に壊れたときに何が起こるのかということです。