Cant get desired output!

[lapor1at]

I need three different outputs for this function.. I cant figure out why it gives me the same answer no matter what I enter!

>>> def DSValue(bina, binb, binc):
               if bina < 16 and binb < 13 and binc < 10:
                         dsv=0
                         print "bina DSV = '%s', binb DSV = '%s', binc DSV = '%s'" % (dsv, dsv, dsv)
               elif bina < 19 and binb < 16 and binc < 13:
                         dsv=1
                         print "bina DSV = '%s', binb DSV = '%s', binc DSV = '%s'" % (dsv, dsv, dsv)
               elif bina < 22 and binb < 19 and binc < 16:
                          dsv=2
                          print "bina DSV = '%s', binb DSV = '%s', binc DSV = '%s'" % (dsv, dsv, dsv)
               elif bina < 25 and binb < 22 and binc < 19:
                           dsv=3
                           print "bina DSV = '%s', binb DSV = '%s', binc DSV = '%s'" % (dsv, dsv, dsv)
               elif bina <26 and binb < 25 and binc < 22:
                            dsv=4
                            print "bina DSV = '%s', binb DSV = '%s', binc DSV = '%s'" % (dsv, dsv, dsv)
               else:
                            print "WRONGGGG!!!"


>>> DSValue(15, 15, 15)
bina DSV = '2', binb DSV = '2', binc DSV = '2'
>>> # The answer should be
>>> # bina DSV = '0', binb DSV = '1', binc DSV = '2'


If anyone has any suggestions please let me know!!!

[Ofnuts]

When your code is:

    dsv=2
    print "bina DSV = '%s', binb DSV = '%s', binc DSV = '%s'" % (dsv, dsv, dsv)

... you insert three times the same value 2 in bina DSV = '%s', binb DSV = '%s', binc DSV = '%s

[lapor1at]

Is their a way to change it so that it can have an output that would be the "dsv" of all the variables depending on the numbers inserted into the function?

[Ofnuts]

If you need a separate "dsv" value for each of the input variables (bina, binb, binc), then you need three variables (say: dsva, dsvb, dsvc).

And since each of these variables will have independent values, you need 3 groups of if/elif/else, one for each input variable.

If you want to print a "wrong input" message, then initialize each of the dsv variables to something recognizable such as None,
and at the end of your code, test if any is still None, if so print the error message else print the three dsv values.

[lapor1at]

Thank you!!!!

[Ofnuts]

Also, you could handle the the problem differently. Assume you have variables:

thresholdsA=[19,22,25,26]
thresholdsB=[16,19,22,25]
thresholdsC=[13,16,19,22]

Then you can scan the array until you hit something bigger than your input, and the index where you stop is your dsv value.

And then you notice that you are doing the same thing for the three inputs, just using a different list of thresholds, then instead of duplicating the code you can use a function that takes the input value and the lists, and return the dsv (or None).