If statement for inventory calculations and update is not working

(0) Share

Report

Posted on by tonykiefer

534

Based on the value of a dropdown I set a variable named transTypeMode. This works fine.

If(TransTyepDD.Selected.Value="CycleCount", Set(transTypeMode,"Update"), TransTyepDD.Selected.Value="Receipt", Set(transTypeMode,"Addition"),TransTyepDD.Selected.Value="Issue", Set(transTypeMode,"Subtraction"))

I have two text boxes and a label. Based on the transTypeMode a calculation or update will occur.

I tested each condition of the if statement separately and each one works. When I combine all three statement into one it breaks. For the life of me I cannot see the reason why. I compared my syntax to the Microsoft If and Switch functions here and I still don't see the problem.

If (transTypeMode = "Update", TransQty, transTypeMode = "Addition", TransQty + QtyOnHand, transTypeMode = "Subtraction", QtyOnHand - TransQty)

Any suggestions?

Also...is there a better, more efficient way to build this feature?

Categories:

Building Power Apps

I have the same question (0)

All responses (6)

Answers (1)

Community Power Pla... Microsoft Employee on at

Like (1)

Report
Copy link

Link copied!

@tonykiefer

The space between If and ( maybe the issue? You could also try adding an else condition eg zero

Was this reply helpful? Yes No
RandyHayes 76,299 Super User 2024 Season 1 on at

Like (1)

Report
Copy link

Link copied!
@tonykiefer

Set your Label Text property to the following formula:

Switch( TransTyepDD.Selected.Value, "CycleCount", TransQty, "Receipt", TransQty + QtyOnHand "Issue", QtyOnHand - TransQty, "unkonwn" )

The only unknowns in your formula are - what are TransQty and QtyOnHand? Are these some sort of variables, some field, or a control?

I hope this is helpful for you.

Was this reply helpful? Yes No
tonykiefer 534 on at

Like (0)

Report
Copy link

Link copied!

@RandyHayes...Thanks for the reply. TransQty and QtyOnHand are the names of the text boxes under TranQuantity and Quantity on Hand respectively.

The Test Qty Label and text box are used for testing the If/Switch statement to make sure the calculations are working before I apply the changes to the db.

Initially, when the TransType was selected I was setting a variable as described above in my initial post. From you switch statement I gather it's not necessary to set the variable because I can just use the TransTypeDD.Selected.Value. This is a cleaner approach.

The outcome should be.
IF the TransTypeDD.Selected.Value = Update THEN I'll replace the current db value.
IF the TransTypeDD.Selected.Value = Receipt THEN add the TransQty and QtyOnHand.
IF the TransTypeDD.Selected.Value = Issue THEN subtract TransQty from QtyOnHand.

Was this reply helpful? Yes No
Verified answer

RandyHayes 76,299 Super User 2024 Season 1 on at

Like (1)

Report
Copy link

Link copied!
@tonykiefer

Yes - avoid variables as much as you can. You usually don't need them except for what they were designed for.

If those TransQty and QtyOnHand are text boxes, then you need to reference them as such and the proper property that contains the value you want (.Text).

Change your formula to the following if that is the case:

Switch( TransTyepDD.Selected.Value, "CycleCount", TransQty.Text, "Receipt", Text(Value(TransQty.Text) + Value(QtyOnHand.Text)) "Issue", Text(Value(QtyOnHand.Text) - Value(TransQty.Text)), "unkonwn" )

Was this reply helpful? Yes No
tonykiefer 534 on at

Like (1)

Report
Copy link

Link copied!

@RandyHayes...Thank you very much for the insight! It seems I'm still stuck in .Net development mode and I have to simplify my approach. Overthinking is a killer when it comes to development.

Was this reply helpful? Yes No
RandyHayes 76,299 Super User 2024 Season 1 on at

Like (1)

Report
Copy link

Link copied!

@tonykiefer

Yes, PowerApps is not development. It is just like Excel in that you create formulas that reference "cells" just like you do in Excel.

Kicking the developer habit is the hardest thing to do (took me a bit of time coming from .Net dev as well). BUT, once you do, it is smooth sailing from there and you will then spend much more time designing cool things and functionality into your app as opposed to trying to "figure out" how to do something.

Keep it super simple!!

Was this reply helpful? Yes No