Как оптимальнее написать функции по выборке из БД, если они отличаются ток дополнительным параметром?

Question

[yarovikov]

Привет всем.
Как оптимальнее сделать - использовать две разных функции для выборки по одну и двум параметрам или одну с условием, что второй может отсутствовать?

Например:

public static function get_all_responses( $task_id = '' ) {

	$get_results = $wpdb->get_results(
		"
		SELECT * FROM $table_name
		WHERE task_id = $task_id				
		", ARRAY_A
	); 
}

public static function get_responses_by_user( $user_id = '', $task_id = '' ) {
			
	$get_results = $wpdb->get_results(
		"
		SELECT * FROM $table_name
		WHERE task_id = $task_id
		AND sender_id = $user_id			
		", ARRAY_A
	);	
}

Какая хорошая практика в данном случае? Плодить код или проверки?

Answer 1

[Вова Дружаев]

$user_id = '', $task_id = '' - поменяйте местами. Первой $task_id

В функции не пустой строкой а $user_id = false делайте
И проверяйте - есть есть $user_id - то добавляйте в sql

function get_responses_by_user( $task_id = '', $user_id = false ) {
    $and = '';
    if ( $user_id )
        $and = " AND sender_id = $user_id";

    $get_results = $wpdb->get_results(
        "
    SELECT * FROM $table_name
    WHERE task_id = $task_id
    $and
    ", ARRAY_A
    );
}

Comments

[yarovikov]

Забыл упомянуть - есть и третья функция с выборкой только по юзеру))

[Вова Дружаев]

yarovikov, включаем логику - и делаем по моему примеру.

Или вы думаете что 3 лишних if - офигенная нагрузка на сайт и замедление вашего кода?

[yarovikov]

Владимир Дружаев, речь не про нагрузку, а красоту) мне попадались решения с кучей условий, и с кучей функций. Больше интересует, с каким подходом при расширении проекта я словлю меньше проблем с поддерживаемостью кода

[yarovikov]

Владимир Дружаев, пока сделал так, а там видно будет)

if ( $user_id && !$task_id ) {
				$get_results = $wpdb->get_results( "SELECT * FROM $table_name sender_id = $user_id", ARRAY_A );
			} elseif ( !$user_id && $task_id) {
				$get_results = $wpdb->get_results( "SELECT * FROM $table_name WHERE task_id = $task_id", ARRAY_A );
			} else {
				$get_results = $wpdb->get_results( "SELECT * FROM $table_name WHERE task_id = $task_id AND sender_id = $user_id", ARRAY_A );
			}
			      
            return $get_results;

[Вова Дружаев]

yarovikov, а вы посмотрите как в вордпресс реализованы - тот же wpdb - не красиво? Может быть для вас и так. Но все согласно кодексу и wpcs

[Вова Дружаев]

yarovikov, как вы сделали - нарушение принципа DRY - вы повторяетесь - ваш код "с душком"

[yarovikov]

Владимир Дружаев, не думаю, что три условия сократят мой вариант до вашего