Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

offsetに実際に取得できた件数を使う #130

Merged
merged 1 commit into from
Jan 14, 2024

Conversation

Nagarei
Copy link
Contributor

@Nagarei Nagarei commented Jan 14, 2024

No description provided.

@Pugma
Copy link
Collaborator

Pugma commented Jan 14, 2024

これ勉強のためにお聞きしたいんですが、今回の実装によって

  • for文は原則無限ループでoffsetの増分は1回に取得したメッセージ数
  • 最後まで到達して、その次もfor文が回ったときは0件取得のerrで止まる
    ようになったという認識であってますか?

@Nagarei
Copy link
Contributor Author

Nagarei commented Jan 14, 2024

offsetの増分は1回に取得したメッセージ数

はい。変数の意味がはっきりとすることが利点だと思っています。何かの間違いで100個以上の値が帰ってきたときもoffsetを取得した数分加算するため、重複して取得しないようになりました。
ただそういうケースは無いので、元々と挙動は同じはずです。len=100のときは全く同じですし、len<100のときはbreakするので。

for文は原則無限ループで
最後まで到達して、その次もfor文が回ったときは0件取得のerrで止まるようになった

これは元々(PR出す前)からそういう実装になっています。

@Pugma
Copy link
Collaborator

Pugma commented Jan 14, 2024

:haakusimasita:
基礎的な質問にも答えてくださってありがとうございます:pray:

大丈夫そうなのでmergeしておきます! <- 僕が言えたことじゃないですが

@Pugma Pugma merged commit ab2d980 into traP-jp:main Jan 14, 2024
2 checks passed
@Nagarei Nagarei deleted the fix-offset branch January 14, 2024 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants