-
-
/**
- * ユーザーのパスワードリセットリクエストの受信者
- */
- function requestResetPassword() {
- //ユーザーが存在するかどうかを確認します
- if( !checkUserExists( $_GET['userid'] ) ) {
- exit ( '申し訳ありませんが、ユーザーは存在しません。ユーザー アカウントを確認してください。');
- }
- resetPassword( $_GET['userid'] );
- //最後にユーザーにメールを送信します
- sendEmail( $_GET[' userid'], 'パスワードのリセットに成功しました', '新しいパスワードは xxxx です' );
- exit('新しいパスワードがメールに送信されました。');
- }
/* *
- * ユーザーがパスワードをリセットできるようにします
- */
- function restartPassword( $userid ) {
- //ユーザーが存在するかどうかを確認します
- if( !checkUserExists( $userid ) ) {
- return false;
- }
/ /ユーザーパスワードのリセット操作
- //短い...
- return true;
- }
/**
- * ユーザーにメールを送信します
- */
- function sendEmail( $userid, $title, $content ) {
- / /ユーザーが存在するか確認
- if( !checkUserExists( $userid ) ) {
- return false;
- }
//メール送信操作
- //省略…
- return true;
- }
/**
- * ユーザーが存在するかどうかを確認します
- */
- function checkUserExists( $userid ) {
- $user = getUserInfo( $userid );
- return !empty( $user );
- } p>
/**
- * ユーザーのデータを取得します
- */
- function getUserInfo( $userid ) {
- //データベースをクエリしてデータを返すために使用されるクエリ関数があるとします
- $ user = query( "SELECT * FROM `user` WHERE `uid`=" . intval( $userid ) );
- return is_array( $user ) ;
- }
-
コードをコピー
問題: これら 3 つの関数はすべて、ユーザーが存在しないことを確認するために checkUserExists 関数を同時に使用するため、追加のオーバーヘッドが発生します。
3つの中のcheckUserExistsを削除したい場合は可能と思われます。ただし、後から一部の関数がresetPasswordまたはsendEmailを呼び出し、ユーザーが存在しない場合、システムでエラーが発生する可能性があります。
もう 1 つの解決策は、resetPassword のロジックを requestResetPassword に書き込み、少し後に sendEmail のロジックも同様に書き込むことです。このようにして、関数呼び出しが減り、データベース クエリが 1 回になり、パフォーマンスが向上します。ただし、パスワードのリセットと電子メールの送信の機能は再利用できず、単一責任の原則に違反し、コードの複雑さも増加します。
ただし、関数の分離と再利用性は非常に優れているため、実際のパフォーマンスに影響がある場合は、キャッシュの使用を検討してデータベース クエリを減らし、共有の checkUserExists 関数を変更することを検討できます。
-
-
/**
- * ユーザーが存在するかどうかを確認します
- */
- function checkUserExists( $userid ) {
- //ユーザーのチェック結果を記録するキャッシュを追加します
- static $cache = array() < ;/p>
//現在のユーザーが一度チェックしたかどうかを確認します
- if( isset( $cache[ $userid ] ) ) {
- return $cache[ $userid ];
- }
$user = getUserInfo( $userid );
- // 結果をキャッシュに記録します
- $cache[ $userid ] = !empty( $user );
[ $userid ];
- }
-
-
コードをコピーします
同じ方法を使用して getUserInfo 関数を変更することもできます。
コードの再利用性が高まると、パフォーマンスを向上させるのが非常に簡単になり、パフォーマンスのボトルネックを見つけて変更するのも簡単になります。
たとえば、データを走査するとき、再利用のために走査を関数にカプセル化し、それを複数回使用することがあります。
これらの費用は予想されたほどプロジェクトに大きな影響を与えず、最小限で済みました。
したがって、パフォーマンスを少しでも無駄にすることにこだわるのではなく、コードの再利用性と保守性を向上させることに重点を置くことができます。
実際のパフォーマンスが要件を満たしていない場合は、ハードウェア構成を増やすことも検討できます。
|