Не работает кнопка сервис. как исправить?

Question

[art_gara55555]

case WM_COMMAND: {
        if (HIWORD(wParam) == BN_CLICKED) {
            int buttonId = LOWORD(wParam);
            HWND hButton = (HWND)lParam;
            wchar_t buffer[256];
            double moneyToAdd = 0.0;
           
            switch (buttonId) {
                // ... (другие кнопки)
              case BUTTON_ID_SERVICE:
                   if(currentState != AppState::ServiceMode) {
                     ShowProductInfoDialog(hwnd, hInstance, products);
                    UpdateInfoLabel(hwnd, L"Сервисный режим");
                     currentState = AppState::ServiceMode;
                    }
                     break;
                // ... (другие кнопки)
            }
        }
        if (LOWORD(wParam) == BUTTON_ID_CLOSE_SERVICE)
            {
              currentState = AppState::WaitingForMoney;
             }
        return 0;
    }

Comments

[VoidVolker]

art_gara55555 https://qna.habr.com/help/rules#3.8

Answer 1

[Wataru]

break; в коде все портит. Выполняется первая проверка, не срабатывает, наверно, потому что currentState не тот. А может, потому что у вас там еще break между уловными проверками расставлены. До проверки на BUTTON_ID_CLOSE_SERVICE код никогда не доходит.

Break должен быть один раз в конце case блока, чтобы управление не перешло на следующий case. Switch же просто переносит управление на соответствующий case и все. Он не отключает как-то куски кода в других альтернативах.

Comments

[art_gara55555]

LRESULT CALLBACK WindowProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam) {
    HINSTANCE hInstance = (HINSTANCE)GetWindowLongPtrW(hwnd, GWLP_USERDATA);
    switch (uMsg) {
    case WM_PAINT: {
        PAINTSTRUCT ps;
        HDC hdc = BeginPaint(hwnd, &ps);
        EndPaint(hwnd, &ps);
    }
        return 0;
    case WM_COMMAND: {
        if (HIWORD(wParam) == BN_CLICKED) {
            int buttonId = LOWORD(wParam);
            HWND hButton = (HWND)lParam;
            wchar_t buffer[256];
            double moneyToAdd = 0.0;
            switch (buttonId) {
            case BUTTON_ID_ADD:
                GetWindowTextW(hEdit, buffer, 255);
                moneyToAdd = _wtof(buffer);
                if (moneyToAdd > 0) {
                    currentMoney += moneyToAdd;
                    std::wstring message = L"Внесено денег: " + std::to_wstring(currentMoney);
                    UpdateInfoLabel(hwnd, message);
                    UpdateBalanceLabel(hwnd, L"Текущий баланс: " + std::to_wstring(currentMoney) + L"р");
                    SetWindowTextW(hEdit, L"");
                }
                std::cout << "currentMoney: " << currentMoney << std::endl;
                break;
            case BUTTON_ID_PRODUCT_BASE:
            case BUTTON_ID_PRODUCT_BASE + 1:
            case BUTTON_ID_PRODUCT_BASE + 2:
            case BUTTON_ID_PRODUCT_BASE + 3:
            case BUTTON_ID_PRODUCT_BASE + 4:
                selectedProductIndex = buttonId - BUTTON_ID_PRODUCT_BASE;
                if (selectedProductIndex >= 0 && selectedProductIndex < products.size()) {
                    std::wstring message = L"Выбран лоток: " + std::to_wstring(products[selectedProductIndex].tray_number) + L". Нажмите кнопку 'Выдать'";
                    UpdateInfoLabel(hwnd, message);
                    currentState = AppState::SelectingProduct;
                }
                break;
            case BUTTON_ID_DISPENSE:
                if (selectedProductIndex >= 0 && selectedProductIndex < products.size()) {
                    Product& product = products[selectedProductIndex];
                    if (&product == nullptr) {
                        UpdateInfoLabel(hwnd, L"Не удалось получить товар.");
                        currentState = AppState::WaitingForMoney;
                    }
                    else {
                        if (product.quantity <= 0) {
                            UpdateInfoLabel(hwnd, L"Товара нет в наличии");
                            currentState = AppState::SelectingProduct;
                        } else if (currentMoney < product.price) {
                            UpdateInfoLabel(hwnd, L"Недостаточно средств");
                            currentState = AppState::SelectingProduct;
                        } else {
                            double change = currentMoney - product.price;
                            product.quantity--;
                            UpdateProductQuantityInDB(db, product.id, product.quantity);
                            currentMoney = 0.0;
                            std::wstring message = L"Заберите покупку";
                            if (change > 0) {
                                message += L" Сдача: " + std::to_wstring(change);
                            }
                            UpdateInfoLabel(hwnd, message);
                            LoadProductsFromDB(db, products);
                            currentState = AppState::WaitingForMoney;
                            UpdateBalanceLabel(hwnd, L"Текущий баланс: " + std::to_wstring(currentMoney) + L"р");
                        }
                    }
                }
                break;
            case BUTTON_ID_SERVICE:
                ShowProductInfoDialog(hwnd, hInstance, products);
                UpdateInfoLabel(hwnd, L"Сервисный режим");
                currentState = AppState::ServiceMode;
                break;
            case BUTTON_ID_INFO:
                MessageBoxW(hwnd, L"Информация о программе", L"Информация", MB_OK);
                break;
            case BUTTON_ID_RETURN:
                if (currentMoney > 0) {
                    std::wstring message = L"Возврат денег: " + std::to_wstring(currentMoney);
                    UpdateInfoLabel(hwnd, message);
                    currentMoney = 0.0;
                    selectedProductIndex = -1;
                    currentState = AppState::WaitingForMoney;
                    UpdateBalanceLabel(hwnd, L"Текущий баланс: " + std::to_wstring(currentMoney) + L"р");
                }
                break;
            }
        }
         if (LOWORD(wParam) == BUTTON_ID_CLOSE_SERVICE)
         {
             currentState = AppState::WaitingForMoney;
         }
        return 0;
    }
       case WM_TIMER:
        if (wParam == TIMER_ID) {
           CheckProductAvailability(hwnd, db, products, indicators, indicatorDCs);
       }
        return 0;
       case WM_DRAWITEM: {
           LPDRAWITEMSTRUCT pdis = (LPDRAWITEMSTRUCT)lParam;
            for (size_t i = 0; i < indicators.size(); ++i) {
                if (pdis->hwndItem == indicators[i]) {
                   HBRUSH hBrush;
                     if (products[i].quantity <= 0) {
                        hBrush = CreateSolidBrush(RGB(255, 0, 0));
                     } else {
                        hBrush = CreateSolidBrush(RGB(0, 255, 0));
                     }
                    FillRect(pdis->hDC, &pdis->rcItem, hBrush);
                     DeleteObject(hBrush);
               }
            }
       }
       return 0;
    default:
        return DefWindowProc(hwnd, uMsg, wParam, lParam);
    }
}для понимания

[Wataru]

art_gara55555, Почему у вас все кнопки обрабатываются в switch, а BUTTON_ID_CLOSE_SERVICE - отдельно в конце? Теперь этот кусок кода сработает по любому WM_COMMAND, даже если там не BN_CLICKED