The Daily WTF 日本語
フォロー
CodeSOD:チェック、チェック
提供されたReactコードスニペットは、ユーザーの権限に基づいて管理者オプションを表示します。
論理AND演算子を使用した条件付きレンダリングアプローチを使用しています。
ユーザーが管理者であるか、結果を表示する権限がある場合、「Admin Actions」ヘッダーがレンダリングされます。
ヘッダーに続いて、「Show Results」ボタンも条件付きで表示されます。
このボタンは、ユーザーが管理者であるか、結果を表示できるという同じ権限基準を満たした場合にのみ表示されます。
著者はこの実装を「belts and braces」アプローチに例え、冗長性を示唆しています。
この重複した条件は、セキュリティや機能性を向上させないと考えています。
このコードは、機密性の高い管理者機能へのアクセスを制限することを目的としています。
しかし、繰り返されるチェックは不要または非効率であると指摘されています。
中心的な考え方は、権限のあるユーザーが管理者関連のコンテンツを見るということですが、その実装の繰り返しが疑問視されています。