Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > comp.lang.python > #56740

Re: Why isn't this code working how I want it to?

Newsgroups comp.lang.python
Date 2013-10-12 04:03 -0700
References <f74fadac-e7c3-4419-9a65-ed6ecd5eb3fe@googlegroups.com> <mailman.1032.1381569606.18130.python-list@python.org>
Message-ID <50950ef3-9fae-41cd-8a9a-4bd3ccc418c6@googlegroups.com> (permalink)
Subject Re: Why isn't this code working how I want it to?
From reubennottage@gmail.com

Show all headers | View raw


On Saturday, October 12, 2013 10:20:24 AM UTC+1, Peter Otten wrote:
> reubennottage@gmail.com wrote:
> 
> 
> 
> > I've been working on a program and have had to halt it due a slight
> 
> > problem. Here's a basic version of the code:
> 
> > 
> 
> > a = 'filled'
> 
> > b = 'filled'
> 
> > c = 'empty'
> 
> > d = 'empty'
> 
> > e = 'filled'
> 
> > f = 'empty'
> 
> > g = 'filled'
> 
> > 
> 
> > testdict = {a : 'apple' , b : 'banana' , c : 'cake' , d : 'damson' , e :
> 
> > 'eggs' , f : 'fish' , g : 'glue'}
> 
> 
> 
> You have duplicate keys here, which becomes obvious when you spell out the 
> 
> values
> 
> 
> 
> testdict = {"filled": "apple", "filled": "banana", ...}
> 
> 
> 
> When you do that, the last value ("banana") wins, all others (e. g. "apple") 
> 
> are dropped.
> 
> 
> 
> > Now what I want to do, is if a variable is filled, print it out. This
> 
> > however isn't working how I planned. The following doesn't work.
> 
> > 
> 
> > for fillempt in testdict:
> 
> >     if fillempt == 'filled':
> 
> >         print(testdict[fillempt])
> 
> > 
> 
> > All this does though, is print glue, where I'd want it to print:
> 
> > 
> 
> > apple
> 
> > banana
> 
> > eggs
> 
> > glue
> 
> > 
> 
> > Perhaps a dictionary isn't the best way to do this.. I wonder what else I
> 
> > can do...
> 
> 
> 
> A dictionary is spot-on, but you have to use the unique "apple", 
> 
> "banana",... as keys:
> 
> 
> 
> >>> status = {"apple": "filled", "banana": "filled", "cake": "empty"}
> 
> >>> for item in status:
> 
> ...     if status[item] == "filled":
> 
> ...             print(item)
> 
> ... 
> 
> apple
> 
> banana
> 
> 
> 
> Could it be that you just confused dict keys with dict values?

This fixed it, thank you! I did think a dictionary was right; I never considered swapping the keys with the values, though. A simple 'fix, but it worked. You've been a great help. 

Back to comp.lang.python | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Why isn't this code working how I want it to? reubennottage@gmail.com - 2013-10-12 01:56 -0700
  Re: Why isn't this code working how I want it to? Marco Nawijn <nawijn@gmail.com> - 2013-10-12 02:17 -0700
  Re: Why isn't this code working how I want it to? Peter Otten <__peter__@web.de> - 2013-10-12 11:20 +0200
    Re: Why isn't this code working how I want it to? reubennottage@gmail.com - 2013-10-12 04:03 -0700
      Re: Why isn't this code working how I want it to? Mark Lawrence <breamoreboy@yahoo.co.uk> - 2013-10-12 12:13 +0100
  Re: Why isn't this code working how I want it to? Mark Lawrence <breamoreboy@yahoo.co.uk> - 2013-10-12 10:39 +0100
  Re: Why isn't this code working how I want it to? Jussi Piitulainen <jpiitula@ling.helsinki.fi> - 2013-10-12 12:50 +0300

csiph-web