pythonのtime関数を使用できません

Question

前提・実現したいこと

入門python3(オライリー・ジャパン)のp267において、time関数を用いてユーザのログインを監視するコードを書いていきます。その過程において、最初のユーザを追加したいです。正しく操作すれば以下のように返されます。

>>> import time
>>> now = time.time()
>>> now
1601257294.666655
>>> conn.zadd('logins', 'smeagol', now)
1

発生するエラー

以下のようにエラーが表示されます。ちなみにここではredisをimportした後にconnを定義してからtimeをimportしています。

該当のソースコード

>>> import redis
>>> conn = redis.Redis()
>>> import time
>>> now = time.time()
>>> now
1601257294.666655
>>> conn.zadd('logins', 'smeagol', now)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/kota/.pyenv/versions/3.6.5/lib/python3.6/site-packages/redis/client.py", line 2685, in zadd
    for pair in iteritems(mapping):
  File "/Users/kota/.pyenv/versions/3.6.5/lib/python3.6/site-packages/redis/_compat.py", line 159, in iteritems
    return iter(x.items())
AttributeError: 'str' object has no attribute 'items'

補足情報（FW/ツールのバージョンなど）

PCはMacを用いています。

Answer 1

エラーが起きているのは conn.zadd です。

https://github.com/andymccurdy/redis-py#user-content-mset-msetnx-and-zadd

MSET, MSETNX and ZADD
(略)
To resolve this, redis-py 3.0 has changed these three commands to all accept a single positional argument named mapping that is expected to be a dict.

redis-py 3.0からインターフェースが変わったのでそのコードは動きません。
何よりも先に公式ドキュメントを見ることです。zaddでページ内検索すれば、Upgrading from redis-py 2.X to 3.0の中にZADDの項目が見つかります。

conn.zadd('logins', {'smeagol': now})

でしょう。

Comments

[hamazon]

ご回答ありがとうございます。
ご提示頂いた通りに操作すると以下のようにエラーが出てしまいます...
>>> import redis
>>> conn = redis.Redis()
>>> import time
>>> now = time.time()
>>> now
1601264163.42672
>>> conn.zadd('logins', {'smeagol': now})
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/Users/kota/.pyenv/versions/3.6.5/lib/python3.6/site-packages/redis/client.py", line 2315, in zadd
raise RedisError("ZADD requires an equal number of "
redis.exceptions.RedisError: ZADD requires an equal number of values and scores
もしこちらの原因も分かりましたら教えて頂けますと幸いです????‍♂️

[quickquip]

このエラーは2.xにダウングレードして実行したからですね。

Answer 2

Redisのバージョンを3未満に下げないとエラーが出ます。

入門Python3 正誤表　P267- オライリー本 BillLubanovic - Qiita

Redis >= 3.0.0 not working · Issue #197 · rq/rq-scheduler

追加分: おっと、その後GitHubのスレッドを追っていたら#198で解決されていたので、バージョンを下げるのではなく、zaddの書き方を(別の回答者の方が指摘しているように)変更してください。

Update zadd calls to redis-py 3 function signature by ericatkin · Pull Request #198 · rq/rq-scheduler

Comments

[hamazon]

ご回答ありがとうございます。
zaddの書き方を変更する方は理解できませんでしたが、はじめにご提示頂いたRedisのダウングレードで解決いたしました！