How to validate values when using dropdown and empty field in wpf?

From your code, it seems like you are correctly checking if the cbxCompartment is “truck-full” and cbxPurpose is “Leaving Finished Product”. If these conditions are met, you are validating whether txtRTopSeal and txtRBottomSeal are not empty.

However, you mentioned that the code is not getting into this logic. This could be due to a few reasons:

1. Mismatch in the condition values: Ensure that the values you are checking in the if condition (“truck-full” and “Leaving Finished Product”) exactly match the values in the dropdown. These checks are case-sensitive.

2. Dropdown values not being set: The selected item might not be getting set in the dropdown. You can check this by debugging and inspecting the SelectedItem property of the dropdowns.

3. Empty fields: The txtRTopSeal and txtRBottomSeal fields might be empty when the validation is happening. You can add debug statements to check their values as well.

Thanks

Hi Jayray

 

The issue still persist, when debugging it only sees the if statement but not getting into topseal and bottom seal when its empty, when compartment(truck-empty) is selected its not validating these fields.

Based on the provided code snippet, it seems that the logic for validating the topseal and bottomseal fields is not being executed correctly. Here are a few suggestions to help resolve the issue:

1. Make sure that the event handler btnSaveReturn_Click is properly wired up to the button's click event in the XAML code. Double-check that the Click attribute of the button is set to btnSaveReturn_Click.

2. Check if the conditions for the if statement are being met. Verify that the selected values of the cbxCompartment and cbxPurpose dropdowns are correctly matching the expected values ("truck-full" and "Leaving Finished Product" respectively).

3. Ensure that the txtRTopSeal and txtRBottomSeal textboxes are correctly bound to their respective properties in the XAML code. Double-check the x:Name attributes of these textboxes and make sure they match the property names used in the code-behind.

4. Consider using the SelectedItem property instead of SelectedItem.ToString() when comparing the selected values of the dropdowns. This will ensure that the comparison is done correctly.

if (cbxCompartment.SelectedItem != null && cbxCompartment.SelectedItem.ToString() == "truck-full"
    && cbxPurpose.SelectedItem != null && cbxPurpose.SelectedItem.ToString() == "Leaving Finished Product")
{
    // Validate top seal
    if (string.IsNullOrEmpty(txtRTopSeal.Text))
    {
        MessageBox.Show("TopSeal cannot be empty when truck is full", "Validation Error");
        txtRTopSeal.Focus();
        return;
    }

    // Validate bottom seal
    if (string.IsNullOrEmpty(txtRBottomSeal.Text))
    {
        MessageBox.Show("BottomSeal cannot be empty when truck is full", "Validation Error");
        txtRBottomSeal.Focus();
        return;
    }
}

Hello,

Have you checked both dropdown values in If condition,

"truck-full"  and "Leaving Finished Product" , these values are case sensitive and space also matters, so check  while comparing both combo box values in if condition.

check values for : cbxCompartment.SelectedItem.ToString(), cbxPurpose.SelectedItem.ToString()

Solution: You can convert all values to ToLower()

Example: cbxCompartment.SelectedItem.ToString().ToLower() == "leaving finished product"

It seems like your code is attempting to validate whether the top seal and bottom seal fields are empty when a specific combination of values is selected from the dropdowns (`cbxCompartment` and `cbxPurpose`). However, you mentioned that the logic is not behaving as expected.

1. You're checking if `cbxCompartment.SelectedItem` is equal to "truck-full" and `cbxPurpose.SelectedItem` is equal to "Leaving Finished Product".
2. If this condition is met, you're checking whether `txtRTopSeal.Text` and `txtRBottomSeal.Text` are empty.

First, let's confirm whether the condition to enter the if statement is being met. You can add debug statements or breakpoints to verify this.

If the condition is indeed met, then you should check if the code within the if statement is executed. Again, debug statements or breakpoints can help here.

If the condition isn't being met, ensure that the values in the dropdowns and the logic are correct. You can print out the selected values for debugging purposes.

Additionally, check if the event handler `btnSaveReturn_Click` is correctly wired up to the button click event.

If you've confirmed that the condition is being met and the code within the if statement is not executing, then there might be some other issue within the logic or with the control flow of your application. In that case, further investigation would be needed, potentially including reviewing other parts of your code.