Упрощение условного оператора android

Question

У меня есть метод с несколькими условными операторами. Я добавил условия if, чтобы избежать аварии NPE. Есть ли способ упростить условия if, в которых я добавил ненулевую проверку?

private void addComplementaryProductToBasket(String productId, String comboName) { 
     Product product = ProductComboUtils.getProductById(productId); 
     if (null == product) { 
      LOG.info("Product does not exist in ProductComboUtils, fetching from DB"); 
      product = getProduct(productId); 
     } 
     if (product != null){ 
      product.setPrice(BigDecimal.ZERO); 
      product.setCurrencyCode(currentBasket.getCurrencyCode()); 
      // Ensure to add comboName with purchase, this 
      currentBasket.addToPurchase(product, comboName); 
      int numberOfComplementaryProducts = 
        currentBasket.getPurchaseLineItemForProduct(product.getProductId()).getNumberOfComplementaryProducts(); 
      currentBasket.getPurchaseLineItemForProduct(product.getProductId()).setNumberOfComplementaryProducts(
        ++numberOfComplementaryProducts); 
     }else 
     { 
      CrashUtil.logNonFatalException("Product instance is null for productId: " + productId); 
     } 
    } 

Answer 1

Там нет никакого способа, чтобы упростить, если условия в этом примере, если вы хотите сохранить логику проверки ProductComboUtils первого, а затем выборки из БД.

Answer 2

Поскольку вы получаете значение из двух источников, я просто создаю методы для этого с помощью проверок (Utlis или DB).

public Product getProductTest(String productId){ 
    Product product = ProductComboUtils.getProductById(productId); 
    if (null == product) { 
     LOG.info("Product does not exist in ProductComboUtils, fetching from DB"); 
     product = getProduct(productId); 
    } 
    return product; 
} 

Тогда вам просто нужно получить значение и проверить его значение. Если это null, возникает проблема, и вы увидите ошибку.

private void addComplementaryProductToBasket(String productId, String comboName) { 
     Product product = getProductTest(productId); 
     if (product != null){ 
      .... 
     } else ... 
} 

Answer 3

Вы можете оставить отдельную колонку для информации о продукте/деталях.

Добавить дополнительный столбец isUpdated в таблице и сохранить в качестве значения по умолчанию.

При достижении достаточно серьёзного обновления требования значения столбца

public void isProdUpdated(int id) { 
     SQLiteDatabase db = this.getWritableDatabase(); 
     try { 
     Cursor cursor = db.rawQuery(selectQuery, null); // selectQuery is your complete query 
     if (cursor.getCount() != 0 
       && cursor.moveToFirst()) { 
      int status = cursor.getInt(cursor 
        .getColumnIndex(Const.KEY_IS_UPDATED)); 
      cursor.close(); 
      return status; 
     } 
     cursor.close(); 
    } catch (Exception e) { 
     return 0; 
    } 
    return 0; 
    } 

Пример

public Product getTestProduct(String productId){ 
    Product product = null; 
    if (ProductComboUtils.isProdUpdated(productId) == 1) { 
     return product = ProductComboUtils.getProductById(productId); 
    }else{ 
     LOG.info("Product does not exist in ProductComboUtils, fetching from DB"); 
     return product = getProduct(productId); 
} 
} 

Перед тем, как информация о продукте вызвать вышеописанный метод с вашим основным и проверить значение. Если он вернется 1, то вперед, иначе запросите пользователя, чтобы он не был доступен.