Обработка (2 повышения до n) -1 условия

Вы можете объединить свои условия фильтрации и иметь один оператор, который кодирует все параметры:

if( (criteria.EmpName.equals("") || criteria.EmpName.equals(emp.EmpName))
    && (criteria.OfficeName.equals("") || criteria.OfficeName.equals(emp.OfficeName))
    && (criteria.Salary.equals("") || criteria.Salary.equals(emp.Salary)))

В каждом из выражений, объединенных по И, сначала проверяется, пуст ли фильтр, если это так, то эта часть приведет к true, если нет, то проверка выполняется против соответствующего значения в emp и true только тогда, когда эта проверка true .

Есть много способов сделать это, но поскольку вы не указали один конкретный язык и поскольку я не чувствую себя вправе судить о вашем стиле кодирования, вот один, который сохраняет общую форму вашего кода, демонстрируя при этом лучшую логику:

bool bIsMatch = true;
if (criteria.EmpName != "" && criteria.EmpName != emp.EmpName) {
    bIsMatch = false;
} else if (criteria.OfficeName != "" && criteria.OffIceName != emp.OfficeName) {
    bIsMatch = false;
} /* Repeat for as many conditions as there are */

if (bIsMatch) {
    /* None of the checks above failed */
}

Начните с предположения, что у вас есть совпадение, а затем примените каждый критерий один за другим.

bIsMatch = true;

if (bIsMatch && criteria.EmpName    != "") bIsMatch = emp.EmpName    == criteria.EmpName;
if (bIsMatch && criteria.OfficeName != "") bIsMatch = emp.OfficeName == criteria.OfficeName;
// ...

Или напишите вспомогательную функцию, которая выполняет сопоставление.

bool IsMatch(String criterion, String value)
{
    return criterion == "" || criterion == value;
}

Затем вы можете сделать все в одном большом операторе if:

if (IsMatch(criteria.EmpName,    emp.EmpName)    &&
    IsMatch(criteria.OfficeName, emp.OfficeName) &&
    ...
   )

Вы можете проверять критерии по отдельности и вести подсчет совпадений. Таким образом, вам нужны только n условия:

int matches = 0;

if (criteria.EmpName != "" && emp.EmpName == criteria.EmpName)
  matches++;
// similar code for other criteria

if (matches >= 2) { // as many matches as required
  // succeeded
}

Как насчет этого? Идея хорошо масштабируется для большего количества фильтров, за исключением того, что само сопоставление основано на соглашении (имя – имя).

var map = new Dictionary<string, string>
              {
                { criteria.EmpName, emp.EmpName },       
                { criteria.OfficeName, emp.OfficeName},
                { criteria.ThirdProp, emp.ThirdProp }
              };

bIsMatch = dict.All(kvp => string.IsNullOrEmpty(kvp.Key) || kvp.Key == kvp.Value);

Я бы поставил под сомнение общий дизайн; есть что-то, что кажется неправильным. Как бы вы поступили с полем Salary, которое вы упомянули? Это точно не string? Какое дозорное значение используется в этом случае?

Прежде чем писать код, убедитесь, что вы достаточно ясно представляете бизнес-логику. Согласно вашему коду, я вижу, что вы хотите проверить, имеют ли emp и criteria одинаковые EmployeeName и OfficeName, любое из свойств считается одинаковым, если оно string.Empty. Код будет совершенно ясен после того, как вы сами станете ясны. Вот так:

public static bool EmptyOrEquals(this string one, string another)
{
    return string.IsNullOrEmpty(another) || one.Equals(another);
}
bIsMatch = emp.EmpName.EmptyOrEquals(criteria.EmpName) 
            && emp.OfficeName.EmptyOrEquals(criteria.OfficeName);

Протестируйте каждый вопрос по отдельности и используйте набор битов для кодирования комбинаций ответов.

Это приводит к более чистому коду, потому что вы тестируете каждый критерий только один раз, он компактен, но удобочитаем, и вы можете легко подключить код для обработки каждой комбинации. И это тоже быстро. O(n) для проверки всех критериев и O(1) для поиска реальной комбинации.

Для небольшого, фиксированного количества критериев вы можете перемещать биты вручную. Для многих критериев или для масштабируемого решения используйте java.util.BitSet.

Пример битового нажатия:

int bits = 0;

if (...criteria 1...) {
    bits = 1;
}

if (...criteria 2...) {
    bits |= 2;
}

if (...bits 3...) {
    bits |= 4;
}

switch (bits) {
case 0: // no criteria matched
    ;
case 1: // criteria 1 matched
    ;
case 2: // criteria 2 matched
    ;
case 3: // criteria 1 AND 2 matched
    ;
case 4: // criteria 3 matched
    ;
case 5: // criteria 1 AND 3 matched
    ;
case 6: // criteria 2 AND 3 matched
    ;
case 7: // criteria 1 AND 2 AND 3 matched
    ;
} 

Вы можете обобщить это решение, используя java.util.BitSet для управления битами для n критериев (полезно, когда n > 64!). Чтобы облегчить быстрый поиск, сохраните хэш каждой комбинации BitSet в карте, которая сопоставляет хэш-код с классом команды.