FeatureCollection.fromJson() should also accept a 'Reader' parameter
まだ誰も着手していません。
評価
調査の方向性
services-geojson/src/main/java/com/mapbox/geojson/FeatureCollection.java と既存の fromJson(String) エントリーポイントを確認してください。示されている Gson アダプター設定を使用して Reader のオーバーロードを追加し、その後、関連するプロジェクトのテストを実行して、既存の String の動作を変更せずに Reader 入力から FeatureCollection が生成されることを確認してください。
索引モデルが issue の本文から書いたものです。
説明
Currently a FeatureCollection can not be instantiated from a Reader.
The only function provided requires a String in JSON format as can be seen here.
Such approach is inefficient when reading larger files since the entire file content has to be read and stored in memory before it can be transformed into a FeatureCollection.
If a FeatureCollection could be instantiated from a Reader aswell, only parts of the read file would have to be stored in memory which makes this the more desirable approach in the presented situation.
As the used GSON library already enables the use of a Reader instead of a String, the implementation would be very simple and could look like this:
@Nullable
public static FeatureCollection fromJson(@NonNull Reader reader) {
GsonBuilder gson = new GsonBuilder();
gson.registerTypeAdapterFactory(GeoJsonAdapterFactory.create());
gson.registerTypeAdapterFactory(GeometryAdapterFactory.create());
return gson.create().fromJson(reader, FeatureCollection.class);
}
- 主要言語
- Java
- スター
- 438
- フォーク
- 117
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
mapbox/mapbox-java のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
mapbox/mapbox-java#1640 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
mapbox/mapbox-java#1635 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
mapbox/mapbox-java#1619 ·
-
Mapbox-java incompatible with gson 2.13.0 and up - call to internal GSON classes that were renamed オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 48/100
mapbox/mapbox-java#1614 · コメント 2 件 · リアクション 1 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
mapbox/mapbox-java#1612 · リアクション 1 件 ·
mapbox/mapbox-java の 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 ·