469,356 Members | 2,016 Online
Bytes | Developer Community
New Post

Home Posts Topics Members FAQ

Post your question to a community of 469,356 developers. It's quick & easy.

splitting one dictionary into two

Hello all,

I have to split a dict into two dicts. Depending on their values,
the items shall remain in the original dict or be moved to another
one and at the same time be removed from the original dict.

OK, this is how I do it right now:

dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {}
klist = []

for key in dict1:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]
klist.append(key)

for key in klist:
del dict1[key]

print dict1
print dict2

That means that I store the keys of the items to be removed from
the original dict in a list (klist) and subsequently remove the
items using these keys.

Is there an "even more pythonic" way?

Cheers, jsaul
Jul 18 '05 #1
7 13547
jsaul wrote:
Hello all,

I have to split a dict into two dicts. Depending on their values,
the items shall remain in the original dict or be moved to another
one and at the same time be removed from the original dict.

OK, this is how I do it right now:

dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {}
klist = []

for key in dict1:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]
klist.append(key)

for key in klist:
del dict1[key]

print dict1
print dict2

That means that I store the keys of the items to be removed from
the original dict in a list (klist) and subsequently remove the
items using these keys.

Is there an "even more pythonic" way?

Cheers, jsaul


jsaul,
I'll have a hack with small code. Nice that you can
iterate while removing.
wes
dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {}

print dict1

for key,val in dict1.items():
if val > 3: # some criterion
dict2[key] = val
del dict1[key]

print dict1
print dict2

Jul 18 '05 #2
jsaul wrote:
I have to split a dict into two dicts. Depending on their values,
the items shall remain in the original dict or be moved to another
one and at the same time be removed from the original dict.

OK, this is how I do it right now:

dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {}
klist = []

for key in dict1:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]
klist.append(key)

for key in klist:
del dict1[key]

print dict1
print dict2

That means that I store the keys of the items to be removed from
the original dict in a list (klist) and subsequently remove the
items using these keys.

Is there an "even more pythonic" way?


Only a minor change to do away with the temporary list:

for key in dict1:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]

for key in dict2:
del dict1[key]

Peter

Jul 18 '05 #3
There a quite a few different ways to do this, but
I might suggest something like:

dict1={"a":1, "b":3, "c":5, "d":4, "e":2}
dict2={}
dict3={}
[dict2.setdefault(k,v) for k,v in dict1.items() if v > 3]
[dict3.setdefault(k,v) for k,v in dict1.items() if not v > 3]
dict1=dict3.copy()

Very "Pythonic" but is 2 times slower than your original code.

Another suggestion is:

dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {}

keys=dict1.keys()
for key in keys:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]
del dict1[key]

This is a "little" faster (100000 iterations of your method
time=1.13 seconds, my method=0.94 seconds and I find easier
to read (eliminates the unneeded klist variable and second
loop).

Larry Bates
Syscon, Inc.

"jsaul" <us**********@empty.invalid> wrote in message
news:20*******************@jsaul.de...
Hello all,

I have to split a dict into two dicts. Depending on their values,
the items shall remain in the original dict or be moved to another
one and at the same time be removed from the original dict.

OK, this is how I do it right now:

dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {}
klist = []

for key in dict1:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]
klist.append(key)

for key in klist:
del dict1[key]

print dict1
print dict2

That means that I store the keys of the items to be removed from
the original dict in a list (klist) and subsequently remove the
items using these keys.

Is there an "even more pythonic" way?

Cheers, jsaul

Jul 18 '05 #4

"jsaul" <us**********@empty.invalid> wrote in message
news:20*******************@jsaul.de...
Hello all,

I have to split a dict into two dicts. Depending on their values,
the items shall remain in the original dict or be moved to another
one and at the same time be removed from the original dict.

OK, this is how I do it right now:

dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {}
klist = []

for key in dict1:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]
klist.append(key)

for key in klist:
del dict1[key]

print dict1
print dict2

That means that I store the keys of the items to be removed from
the original dict in a list (klist) and subsequently remove the
items using these keys.

Is there an "even more pythonic" way?


Delete klist stuff and do deletion with

for key in dict2: del dict1[key]

tjr


Jul 18 '05 #5
* Peter Otten [2004-04-01 18:46]:
jsaul wrote:
dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {}
klist = []

for key in dict1:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]
klist.append(key)

for key in klist:
del dict1[key]

print dict1
print dict2


Only a minor change to do away with the temporary list:

for key in dict1:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]

for key in dict2:
del dict1[key]


Hi Peter and others who responded so quickly,

I notice now that I forgot to mention an important condition,
namely that in real life dict2 is already existing and may have
become huge. That's the reason why I need to somewhere save only
those items which I most recently removed from the 1st dict, as
I want to avoid iterating over the while dict2. In real life,
dict1 contains pending jobs which, after they are done, are moved
to a second dict for post processing.

Sorry for the confusion.

I think the most "pythonic" candidate is actually the version
suggested by Larry, namely

dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {} # in real life, dict2 already exists

for key in dict1.keys():
if dict1[key] > 3:
dict2[key] = dict1.pop(key)

print dict1
print dict2

Cheers, jsaul
Jul 18 '05 #6
jsaul wrote:
I notice now that I forgot to mention an important condition,
namely that in real life dict2 is already existing and may have
become huge.


jsaul,
How huge is huge?
wes

Jul 18 '05 #7
jsaul <us**********@empty.invalid> wrote in message news:<20*******************@jsaul.de>...
Hello all,

I have to split a dict into two dicts. Depending on their values,
the items shall remain in the original dict or be moved to another
one and at the same time be removed from the original dict.

OK, this is how I do it right now:

dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 }
dict2 = {}
klist = []
klist is not necessary because key in dict1 will give you the
same but faster.

for key in dict1:
if dict1[key] > 3: # some criterion
dict2[key] = dict1[key]
klist.append(key)

for key in klist:
del dict1[key]

print dict1
print dict2

That means that I store the keys of the items to be removed from
the original dict in a list (klist) and subsequently remove the
items using these keys.

Is there an "even more pythonic" way?

Cheers, jsaul


One solution could be:
dict1 = { "a":1, "b":3, "c":5, "d":4, "e":2 } # a little transfer-function, which deletes an item (k,v) in d and
# returns (k,v) transfer=lambda d,(k,v):[d.__delitem__(k)] and (k,v) # transfer the items from one dict into a list and make a dict
# from it on the fly dict2=dict([transfer(dict1,(k,v)) for (k,v) in dict1.items() if v>3])
dict1 {'a': 1, 'b': 3, 'e': 2} dict2 {'c': 5, 'd': 4}


Regards
Peter
Jul 18 '05 #8

This discussion thread is closed

Replies have been disabled for this discussion.

Similar topics

1 post views Thread by none | last post: by
4 posts views Thread by sbucking | last post: by
2 posts views Thread by David Pratt | last post: by
7 posts views Thread by qwweeeit | last post: by
1 post views Thread by john wright | last post: by
17 posts views Thread by Qiangning Hong | last post: by
6 posts views Thread by IamIan | last post: by
2 posts views Thread by shadow_ | last post: by
reply views Thread by zhoujie | last post: by
reply views Thread by suresh191 | last post: by
1 post views Thread by Marylou17 | last post: by
By using this site, you agree to our Privacy Policy and Terms of Use.