Path: csiph.com!weretis.net!feeder4.news.weretis.net!newsreader4.netcologne.de!news.netcologne.de!newsfeed.kamp.net!newsfeed.kamp.net!fu-berlin.de!uni-berlin.de!not-for-mail From: MRAB Newsgroups: comp.lang.python Subject: Re: i cant seem to figure out the error Date: Sun, 3 Apr 2016 19:21:45 +0100 Lines: 37 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Trace: news.uni-berlin.de u6k/jLVX94AbN1e4ekhalgi0hOVNR7adt5qNx/DaY/Mg== Return-Path: X-Original-To: python-list@python.org Delivered-To: python-list@mail.python.org X-Spam-Status: OK 0.003 X-Spam-Evidence: '*H*': 0.99; '*S*': 0.00; 'else:': 0.03; 'returned.': 0.07; 'stderr': 0.09; 'subject:error': 0.11; 'accordingly.': 0.13; 'def': 0.13; '"invalid': 0.16; 'from:addr:mrabarnett.plus.com': 0.16; 'from:addr:python': 0.16; 'from:name:mrab': 0.16; 'message- id:@mrabarnett.plus.com': 0.16; 'pythonic': 0.16; 'received:192.168.1.4': 0.16; 'received:84.93': 0.16; 'received:84.93.230': 0.16; 'received:io': 0.16; 'received:psf.io': 0.16; 'withdrawal': 0.16; 'wrote:': 0.16; 'this:': 0.23; 'header:In-Reply-To:1': 0.24; 'header:User- Agent:1': 0.26; 'error': 0.27; 'checking': 0.27; 'print': 0.30; 'received:84': 0.32; 'returned': 0.32; 'worked': 0.34; 'jason': 0.35; 'something': 0.35; 'but': 0.36; 'to:addr:python-list': 0.36; 'subject:: ': 0.37; 'really': 0.37; 'expect': 0.37; 'method': 0.37; 'say': 0.37; 'skip:s 40': 0.38; 'subject:the': 0.39; 'takes': 0.39; 'received:192': 0.39; 'rather': 0.39; 'to:addr:python.org': 0.40; 'called': 0.40; 'your': 0.60; 'balance': 0.64; 'funds': 0.77; 'balance,': 0.84 X-CM-Score: 0.00 X-CNFS-Analysis: v=2.1 cv=bsGxfxui c=1 sm=1 tr=0 a=0nF1XD0wxitMEM03M9B4ZQ==:117 a=0nF1XD0wxitMEM03M9B4ZQ==:17 a=L9H7d07YOLsA:10 a=9cW_t1CCXrUA:10 a=s5jvgZ67dGcA:10 a=IkcTkHD0fZMA:10 a=6G6oRe7nBNJt0c6rjz4A:9 a=QEXdDO2ut3YA:10 X-AUTH: mrabarnett@:2500 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.7.1 In-Reply-To: X-BeenThere: python-list@python.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: General discussion list for the Python programming language List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Xref: csiph.com comp.lang.python:106386 On 2016-04-03 18:34, Jason Friedman wrote: >> >> - Create a method called `withdraw` that takes in cash withdrawal amount >> and updates the balance accordingly. if amount is greater than balance >> return `"invalid transaction"` >> >> def withdraw(self, amount): >> self.amount=amount >> if(amount > self.balance): >> return ("Amount greater than available balance.") >> else: >> self.balance -= amount >> return self.balance >> > > The instructions say to "return 'invalid transaction'" but I expect they > really want that error printed to STDERR (typically your screen) rather > than literally returned. Try this: > > def withdraw(self, amount): > self.amount=amount > if(amount > self.balance): > print "Amount greater than available balance, no funds withdrawn." > else: > self.balance -= amount > return self.balance > The instructions say "return", not "print" and the report contains: self.assertEqual(self.my_account.withdraw(1000), "invalid transaction", msg='Invalid transaction') so it _is_ checking the returned result of 'withdraw'. Not Pythonic (neither is printing to stderr! :-)), but that's something that can be worked on later.