The Daily WTF 日本語 フォロー CodeSOD: 議論への追加 デビッドは、一見単純な変更、つまりJavaScript関数に1つのパラメータを追加することに焦点を当てた、広範なコメントが付いた不可解なプルリクエストに遭遇しました。元の関数にはすでに7つのパラメータがあり、オブジェクトの使用をリファクタリングする機会を示唆している可能性があります。最初の修正では、新しいパラメータnotArg8に紐付けられた条件文が導入されました。notArg8がtrueの場合、コードはarg8: !notArg8をコンテキストに渡しました。著者は、パラメータを直接追加するだけで済んだため、この複雑なアプローチを理解できませんでした。デビッドはコードの質の悪さについてコメントしましたが、元の開発者は応答しなくなりました。ジュニア開発者が混乱したコード変更を承認しましたが、マージされることはありませんでした。その後、別の開発者がソリューションを簡素化し、arg8を追加してコンテキストに渡すだけでした。簡素化されたコードは機能し、テストをパスし、皆を満足させました。元の開発者の意図と、複雑な条件の背後にある理由は何であったかは謎のままです。結末は、元の開発者のビジョンが実現されないままであることを示唆しています。 CodeSOD: Adding to the Argument thedailywtf.com The Daily WTF 日本語 RSS thenote.app
notArg8に紐付けられた条件文が導入されました。notArg8がtrueの場合、コードはarg8: !notArg8をコンテキストに渡しました。著者は、パラメータを直接追加するだけで済んだため、この複雑なアプローチを理解できませんでした。デビッドはコードの質の悪さについてコメントしましたが、元の開発者は応答しなくなりました。ジュニア開発者が混乱したコード変更を承認しましたが、マージされることはありませんでした。その後、別の開発者がソリューションを簡素化し、arg8を追加してコンテキストに渡すだけでした。簡素化されたコードは機能し、テストをパスし、皆を満足させました。元の開発者の意図と、複雑な条件の背後にある理由は何であったかは謎のままです。結末は、元の開発者のビジョンが実現されないままであることを示唆しています。