Python初心者です。 TypeError: get() takes 2 positional arguments but 3 were givenで困ってます。

Question

はじめまして。
Python3.6でtwitterのユーザーストリームを取得するコードを書いていますが、
納得のいかないエラーが出て困っています。
下記ソースコードです。

from requests_oauthlib import  OAuth1Session

USER_TL = "https://api.twitter.com/1.1/statuses/user_timeline.json"
API_KEY = "*"
API_SECRET_KEY = "*"
ACCESS_TOKEN = "*"
ACCESS_TOKEN_SECRET = "*"

class Twitterclass:
    def __init__(self,key,secret_key,token,secret_token):
        self.twitter = OAuth1Session(key, secret_key, token, secret_token)
        self.param = {
            "screen_name": "apple",
            "count": 200,
            "include_entities": True,
            "exclude_replies": False,
            "include_rts": False,
        }

    def get_user_timeline(self):
        self.twitter.get(USER_TL,self.param)
# Twitterclass END

twitter = Twitterclass(API_KEY,
                       API_SECRET_KEY,
                       ACCESS_TOKEN,
                       ACCESS_TOKEN_SECRET)
twitter.get_user_timeline()

下記エラーメッセージです。

TypeError: get() takes 2 positional arguments but 3 were given

エラー箇所は、下記コード部分です。

        self.twitter.get(USER_TL,self.param)

他サイト様で調べたところ、
Class内メソッドでClass内変数を呼び出す際、暗黙的にSelfが引数が呼ばれている、
との記載がありましたがいまいち理解が及ばずにいます。

下記コードのように、selfが暗黙的に引数として呼ばれていることは理解しています。

        self.twitter.get(self,USER_TL,self.param)

何故エラーが発生しているか、正しい書き方はどのような書き方か、
ご教授頂ければ幸いです。

また、Classの使い方が間違っている等ありましたら、
そちらもご指摘頂ければ幸いです。

長々と失礼致しました。

追記:
can110様
ご返信、ご回答頂きました方々、ありがとうございました。
ご回答、ご指摘頂きましたとおり、変更したコードを下記に示します。

from requests_oauthlib import  OAuth1Session

USER_TL = "https://api.twitter.com/1.1/statuses/user_timeline.json"
API_KEY = "*"
API_SECRET_KEY = "*"
ACCESS_TOKEN = "*"
ACCESS_TOKEN_SECRET = "*"

class Twitterclass:
    def __init__(self,key,secret_key,token,secret_token):
        self.twitter = OAuth1Session(key, secret_key, token, secret_token)
        self.param = {
            "screen_name": "apple",
            "count": 200,
            "include_entities": True,
            "exclude_replies": False,
            "include_rts": False,
        }

    def get_user_timeline(self):
        return self.twitter.get(url = USER_TL,params = self.param)
# Twitterclass END

twitter = Twitterclass(API_KEY,
                       API_SECRET_KEY,
                       ACCESS_TOKEN,
                       ACCESS_TOKEN_SECRET)
user_timeline = twitter.get_user_timeline()

Comments

[quickquip]

OAuth1Session がどこから来ているか(おそらくimportでしょうけど)、ソースに書いてあるとよいかと思いました。

[t_obara]

OAuth1Sessionのgetメソッドのマニュアルをご覧になり、どのような定義になっているかを確認した上で、ご提示のような記述を行なったのでしょうか？その場合、どのドキュメントをご覧になったのか、URLなどをご提示されるとよろしいかと思います。

[退会済みユーザー]

@quiqui様
ご指摘ありがとうございます。
追記致しました。
requests-oauthlibでOAuth認証を利用しています。 requests-oauthlibのバージョンは1.0.0です。

[退会済みユーザー]

@t_obara様
ご返信ありがとうございます。
確かにその通りでございます。クラスメソッドの振る舞いについて私の理解が及ばなかった事がエラーの原因である断定してしまい、 ライブラリのドキュメントを読むことを疎かにしていました。
当初参考にしていたのは下記の記事です。
https://qiita.com/ogrew/items/0b267f57b8aaa24f1b73

[t_obara]

であれば、ドキュメントを読む前に、貴殿が提示したコードは参考にした記事の通りになっていませんよね。まぁ答えはすでに出ていますが。

[退会済みユーザー]

@t_obara様
記事に記載されているコードでは動作することを既に確認しておりまして、
Pythonの学習も兼ねて私の方で改悪したものが質問内容に提示したコードとなります。
失礼致しました。

Answer 1

getメソッドの仕様を調べるために、とりあえずドキュメントを読みに行きました。

https://requests-oauthlib.readthedocs.io/en/latest/api.html#oauth-1-0-session

書いてなかったので、親クラスで定義されているのだろうと思いました。親クラスがドキュメントではわからなかったので（どこかに書いてあるんだろうけど、このページにはぱっと見なかった）、ソースを読みに行きました。

https://github.com/requests/requests-oauthlib/blob/master/requests_oauthlib/oauth1_session.py#L54

requests.Sessionを親にしていることがわかったので、こちらのドキュメントを見に行きました。

http://docs.python-requests.org/en/master/api/#requests.Session.get

get(url, **kwargs)[source]
    Sends a GET request. Returns Response object.

    Parameters:  url – URL for the new Request object.
                 **kwargs – Optional arguments that request takes.
    Return type: requests.Response

あとはcan110さんと同様の結論になりますが、**kwargsになにを渡したら良いのかわかりませんね。「requestの取るのと同じキーワード引数を取れる」ということなので、これを見に行きます。

http://docs.python-requests.org/en/master/api/#requests.request

きっとparamsで渡せば大丈夫だろうな、ということがわかります。

以上を踏まえて修正すると、

        self.twitter.get(USER_TL, params=self.param)

になると思います。

---

全部書いてから、同じことやってるページを見つけた・・・

https://qiita.com/sugurunatsuno/items/55a46a63112ad378dca2
https://qiita.com/sugurunatsuno/items/2faa069b475bb45da552

Comments

[退会済みユーザー]

hayataka2049様
詳しく調べていただきまして、ありがとうございます！！
普通でしたら私の方がそのようにドキュメントや内部コードを辿って答えを探し出さなければいけないのですが…。
私自身、クラスの使い方ですら曖昧で理解に及ばない部分があり、説明不足でした。申し訳ありません。
次回、エラーメッセージに困惑することがありましたら、ご回答のようにドキュメントや内部コードを辿るようにしてみます。

Answer 2

何故エラーが発生しているか、正しい書き方はどのような書き方か、

.get関数が以下のように定義されているためだと思われます。
以下では先頭２つself,param1が位置引数、それ以降がキーワード引数になっています。
この場合にget(1,'hoge')のように２つの位置引数を渡すとselfと合わせて３つの位置引数を渡したことになり提示エラーが発生します。

class Hoge():
    def __init(self):
        pass
    def get(self,param1,**kwargs):
        print(param1,kwargs)

h = Hoge()
h.get(1,param2='hoge') # 1 {'param2': 'hoge'}
h.get(1,'hoge')        # TypeError: get() takes 2 positional arguments but 3 were given

対処方法としては.get(1,param2='hoge')のように２つ目をキーワード引数で渡せばよいです。

また、Classの使い方が間違っている等ありましたら、

後の処理なりで.getした結果が必要になると思われるので、戻り値として返すほうがよいと思います。

    def get_user_timeline(self):
        #self.twitter.get(USER_TL,self.param)
        return self.twitter.get(USER_TL,params=self.param)

Comments

[退会済みユーザー]

can110様
的確なご回答ありがとうございます。
クラス変数に位置引数で引数を指定してしまうと、selfが溢れてしまい、引数の数が合わなくなるエラーだったのですね。大変勉強になりました。
質問内容に提示させて頂きましたコードの場合、確かにurl,paramsのキーワード引数が必要でした。

クラスについてのご指摘もありがとうございます。
getと関数名に明示にしているにも関わらず、何も返さない関数になっていました。

ご丁寧に深く掘り下げて説明いただきましてありがとうございました。

Answer 3

エラーメッセージを読みましょうよ。

google翻訳
TypeError：get（）は2つの位置引数を取りますが、3つが指定されました

Comments

[退会済みユーザー]

y_waiwai様
説明不足で申し訳ありません。
引数が多すぎるのは理解しておりまして、
self.twitter.get()の引数に暗黙的にselfが渡されていることが原因であると把握しております。
それを踏まえて、クラス変数に外部関数を使う時にエラーとなってしまうロジックを知りたく思い、
質問させていただいた次第です。