Queries with no results return a null array
まだ誰も着手していません。
評価
調査の方向性
StatementBuilder と InventoryServiceInterface#getAdUnitsByStatement() のパスから始め、次に #214 の関連する議論を確認してください。他のクエリ実装で、空のクエリ結果に対して XYPage#getResults() がどのように生成されるかを確認してください。対象となるすべての実装で null ではなく空の配列が返されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Hello!
This library allows you to query via StatementBuilder. When you run a query that returns no results, the XYPage#getResults() returns null. On my side, this has only been confirmed to be true with InventoryServiceInterface#getAdUnitsByStatement(), however it would be right to assume it is true for other query implementations as well.
Returning null arrays is an anti-pattern. Can this be fixed to return empty arrays instead? This has already been mentioned in #214.
- 主要言語
- Java
- スター
- 234
- フォーク
- 360
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
googleads/googleads-java-lib のほかの issue
-
googleads/googleads-java-lib#284 · コメント 3 件 · 担当者 1 名 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 30/100
googleads/googleads-java-lib#273 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
googleads/googleads-java-lib#269 ·
-
enhancement P2
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
googleads/googleads-java-lib#268 · コメント 2 件 · リアクション 2 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
googleads/googleads-java-lib#262 ·
googleads/googleads-java-lib の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
elastic/gradle-plugins#157 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
cryptomator/hub#497 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
johanhaleby/occurrent#1120 ·