Issue with Exception Handling in Python

Question

I am having following issue with my code in propagating an Exception:

What the code should do:

It takes an input for number a of combinations or Test Cases to be run.

It takes n = number and its power = p. (i.e 2^3=8)

If p and n are positive, it needs to print 2^3 = 8 and return it - this is working.

If either n or p is negative, it needs to print "n and p should be non-negative" - this is not working.

My code:

class Calculator():
    def power(self,n,p):
        self.n=n
        self.p=p
        try:
            raise NameError("n and p should be non-negative")

        except NameError:
            a=pow(n,p)
            return a
myCalculator=Calculator()
T=int(raw_input())
for i in range(T):
    n,p = map(int, raw_input().split())
    try:
        ans=myCalculator.power(n,p)
        print ans
    except Exception,e:
        print e

Input given

Output received:

However, I should have received "n and p should be non-negative" for 3rd and 4th Test Case as last 2 Test Cases(Test Case number 3 and 4) had either of the values set to negative.

What is wrong?

You're not actually testing for the values to be positive..

L3viathan
– L3viathan

2016-05-17 17:55:23 +00:00
Commented May 17, 2016 at 17:55 — L3viathan
– L3viathan, Commented May 17, 2016 at 17:55

L3viathan · Accepted Answer · 2016-05-17 17:57:20Z

1

You're not actually checking the values. Do you want something like that?

class Calculator():
    def power(self,n,p):
        self.n=n
        self.p=p
        if n<0 or p<0:
            raise NameError("n and p should be non-negative")
        a=pow(n,p)
        return a

A NameError doesn't make much sense though, replace it with ValueError.

answered May 17, 2016 at 17:57

L3viathan

27.5k2 gold badges63 silver badges84 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Diwakar SHARMA Over a year ago

I tried the other way round as well earlier but that didn't worked either , please forgive as i am a new-bee: class Calculator(): def power(self,n,p): self.n=n self.p=p try: if n>0 and p>0: a=pow(n,p) return a except ValueError,e: print "n and p should be non negative" : I will try your code and let you know

Diwakar SHARMA Over a year ago

yes you are absolutely right , i figured my mistake and found out what i was doing wrong . Kudos to you

Collectives™ on Stack Overflow

Issue with Exception Handling in Python

1 Answer 1

2 Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Related