Siema.
Ostatnio z mojego kursu na udemy nauczyłem się obsługiwać pliki JSON, co uznałem za ciekawe rozwiazanie. W ramach ćwiczeń napisałem bardzo prosty program, który z pliku pobiera dane na temat Billboard hot 100 i wypisuje podana liczbę piosenek zgodnie z miejscami w rankingu. Czy jest w nim coś, co można poprawić?
Tutaj link: https://github.com/MaciekSm19/MusicChart
Z góry dziękuję za odpowiedź!
Java for (int i = 0; i < data.length(); i++) {
String title = data.getJSONObject(i).getString("song");
String artist = data.getJSONObject(i).getString("artist");
Integer thisWeek = data.getJSONObject(i).getInt("this_week");
Integer lastWeek;
if (data.getJSONObject(i).get("last_week").equals(null)) lastWeek = null;
else lastWeek = data.getJSONObject(i).getInt("last_week");
Integer peakPosition = data.getJSONObject(i).getInt("peak_position");
Integer weeksOnChart = data.getJSONObject(i).getInt("weeks_on_chart");
songs.add(new Song(title, artist, thisWeek, lastWeek, peakPosition, weeksOnChart));
}
Raczej tak się nie robi. Twoje podejście jest w pełni mechaniczne co nie jest jakąś wielką wadą, ale większość woli pół-automatyczne rozwiązania, gdzie mapowanie klucz -> pole w strukturze dzieje się automatycznie + jakieś andnotacje. Poczytaj jak działa jackson albo gson
Maciek_SK8 napisał(a):
Czy jest w nim coś, co można poprawić?
Tak - JSON jak sama nazwa wskazuje to powstał na potrzeby JSa a nie Javy.
W Javie używaj modeli Javowych a nie JSONa.
Jackson też jest spoko:
https://www.baeldung.com/jackson-object-mapper-tutorial
Wady projektowe:
DatabaseOperations wygląda jakby jej głównym zadaniem było wczytanie JSON'a z zewnętrznego serwisu - jeśli tak, to dobrze żeby zwróciła String. Chyba że powinna tez parsować treść, to powinna zwrócić Song. Zwrócenie JSONObject to najgorsze wyjście.DatabaseOperations jest używane w taki sposób databaseOperations.getJSONObject(databaseOperations.connect()). Co nie ma specjalnego sensu. Czemu nie po prostu String json = databaseOperations.fetchJson()?getTopChartedSongs() w ogóle nie pasuje do klasy ChartOperations. Należałoby ją z tamtąd zabrać i umieścić w innym miejscu.ChartOperations wygląda jakby jego głównym zadaniem było parsowanie jsona i zbudowanie Song. Ja zmieniłbym nazwę tej klasy na SongFactory albo coś w tym stylu.Ja bym to napisał jakoś tak:
public class Main {
public static void main(String[] args) {
SongsResponse allSongs = new SongsProvider().fetch();
System.out.println("The chart was last updated on " + allSongs.publishDate());
System.out.print("How many top charted songs would you like to see? ");
int amountOfSongs = new Scanner(System.in).nextInt();
response.songs()
.stream()
.limit(amountOfSongs)
.map(Main::formatSongListItem)
.forEach(System.out::println);
}
private static String formatSongListItem(Song song) {
return song.thisWeek() + ". " + song.title() + " - " + song.artist();
}
}
record SongsResponse(
String publishDate,
List<Song> songs) {}
record Song(
String title,
String artist,
Integer thisWeek,
Integer lastWeek,
Integer peakPosition,
Integer weeksOnChart) {}
class SongsProvider {
private final HttpClient client;
public SongsProvider() {
this.client = new HttpClient("https://raw.githubusercontent.com/mhollingshead/billboard-hot-100/main/recent.json");
}
public SongsResponse fetch() {
return createResponse(new JSONObject(this.client.fetchJson()));
}
private SongsResponse createResponse(JSONObject jsonObject) {
return new SongsResponse(
jsonObject.getString("date"),
parseSongs(jsonObject.getJSONArray("data")));
}
private List<Song> parseSongs(JSONArray jsonSongs) {
List<Song> songs = new ArrayList<>();
for (int i = 0; i < jsonSongs.length(); i++) {
songs.add(createSong(jsonSongs.getJSONObject(i)));
}
return songs;
}
private Song createSong(JSONObject song) {
return new Song(
song.getString("song"),
song.getString("artist"),
song.getInt("this_week"),
getIntOrNull(song, "last_week"),
song.getInt("peak_position"),
song.getInt("weeks_on_chart"));
}
private Integer getIntOrNull(JSONObject object, String fieldName) {
if (object.isNull(fieldName)) {
return null;
}
return object.getInt(fieldName);
}
}
public class HttpClient {
private final String url;
public HttpClient(String url) {
this.url = url;
}
String fetchJson() {
try {
return readInputStream(connect().getInputStream());
} catch (IOException e) {
throw new RuntimeException(e);
}
}
private static String readInputStream(InputStream inputStream) throws IOException {
BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream));
String line;
StringBuilder stringBuilder = new StringBuilder();
while ((line = reader.readLine()) != null) stringBuilder.append(line);
reader.close();
return stringBuilder.toString();
}
private HttpURLConnection connect() {
try {
URI uri = new URI(this.url);
HttpURLConnection connection = (HttpURLConnection) uri.toURL().openConnection();
if (connection.getResponseCode() != 200) {
throw new RuntimeException();
}
return connection;
} catch (IOException | URISyntaxException exception) {
throw new RuntimeException(exception);
}
}
}