From f14ccbc1b7fb239c5f3cdb20084be04dd85fc076 Mon Sep 17 00:00:00 2001 From: Neil Smith Date: Wed, 15 Oct 2014 23:18:12 +0100 Subject: [PATCH] Done lesson 2 --- app/src/main/AndroidManifest.xml | 2 + .../uk/me/njae/sunshine/ForecastFragment.java | 292 ++++++++++++++++++ .../uk/me/njae/sunshine/MainActivity.java | 119 +------ app/src/main/res/menu/forecastfragment.xml | 9 + app/src/main/res/values/strings.xml | 1 + 5 files changed, 305 insertions(+), 118 deletions(-) create mode 100644 app/src/main/java/uk/me/njae/sunshine/ForecastFragment.java create mode 100644 app/src/main/res/menu/forecastfragment.xml diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml index 2c5f3c8..2e791e9 100644 --- a/app/src/main/AndroidManifest.xml +++ b/app/src/main/AndroidManifest.xml @@ -2,6 +2,8 @@ + + mForecastAdapter; + + public ForecastFragment() { + } + + @Override + public void onCreate(Bundle savedInstanceState) { + super.onCreate(savedInstanceState); + setHasOptionsMenu(true); + } + + @Override + public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) { + // Inflate the menu; this adds items to the action bar if it is present. + inflater.inflate(R.menu.forecastfragment, menu); + } + + @Override + public boolean onOptionsItemSelected(MenuItem item) { + // Handle action bar item clicks here. The action bar will + // automatically handle clicks on the Home/Up button, so long + // as you specify a parent activity in AndroidManifest.xml. + int id = item.getItemId(); + if (id == R.id.action_refresh) { + FetchWeatherTask weatherTask = new FetchWeatherTask(); + weatherTask.execute("94043"); + return true; + } + return super.onOptionsItemSelected(item); + } + + @Override + public View onCreateView(LayoutInflater inflater, ViewGroup container, + Bundle savedInstanceState) { + + String[] forecastArray = { + "Today - Sunny - 10/10", + "Tomorrow - Cloudy - 11/11", + "Tuesday - Snow - 12/12", + "Wednesday - Rain - 13/13", + "Thursday - Hail - 14/14", + "Friday - Scorchio - 15/15", + "Saturday - Fog - 16/16" + }; + + List weekForecast = new ArrayList( + Arrays.asList(forecastArray)); + mForecastAdapter = new ArrayAdapter( + getActivity(), + R.layout.list_item_forecast, + R.id.list_item_forecast_textview, + weekForecast + ); + + View rootView = inflater.inflate(R.layout.fragment_main, container, false); + + ListView list_view = (ListView) rootView.findViewById(R.id.listview_forecast); + list_view.setAdapter(mForecastAdapter); + + return rootView; + } + + public class FetchWeatherTask extends AsyncTask { + + private final String LOG_TAG = FetchWeatherTask.class.getSimpleName(); + + @Override + protected void onPostExecute(String[] result) { + if (result != null) { + mForecastAdapter.clear(); +// for (String dayForecastStr: result) { +// mForecastAdapter.add(dayForecastStr); +// } + mForecastAdapter.addAll(result); + } + } + + @Override + protected String[] doInBackground(String... params) { + if (params.length == 0) { + return null; + } + + // These two need to be declared outside the try/catch + // so that they can be closed in the finally block. + HttpURLConnection urlConnection = null; + BufferedReader reader = null; + + // Will contain the raw JSON response as a string. + String forecastJsonStr; + + String format = "json"; + String units = "metric"; + int numDays = 7; + + try { + // Construct the URL for the OpenWeatherMap query + // Possible parameters are avaiable at OWM's forecast API page, at + // http://openweathermap.org/API#forecast + final String FORECAST_BASE_URL = "http://api.openweathermap.org/data/2.5/forecast/daily?"; + final String QUERY_PARAM = "q"; + final String FORMAT_PARAM = "mode"; + final String UNITS_PARAM = "units"; + final String DAYS_PARAM = "cnt"; + + Uri builtUri = Uri.parse(FORECAST_BASE_URL).buildUpon() + .appendQueryParameter(QUERY_PARAM, params[0]) + .appendQueryParameter(FORMAT_PARAM, format) + .appendQueryParameter(UNITS_PARAM, units) + .appendQueryParameter(DAYS_PARAM, Integer.toString(numDays)) + .build(); + + URL url = new URL(builtUri.toString()); +/**/ Log.v(LOG_TAG, "Built URI " + builtUri.toString()); + + // Create the request to OpenWeatherMap, and open the connection + urlConnection = (HttpURLConnection) url.openConnection(); + urlConnection.setRequestMethod("GET"); + urlConnection.connect(); + + // Read the input stream into a String + InputStream inputStream = urlConnection.getInputStream(); + StringBuffer buffer = new StringBuffer(); + if (inputStream == null) { + // Nothing to do. + return null; + } + reader = new BufferedReader(new InputStreamReader(inputStream)); + + String line; + while ((line = reader.readLine()) != null) { + // Since it's JSON, adding a newline isn't necessary (it won't affect parsing) + // But it does make debugging a *lot* easier if you print out the completed + // buffer for debugging. + buffer.append(line + "\n"); + } + + if (buffer.length() == 0) { + // Stream was empty. No point in parsing. + return null; + } + forecastJsonStr = buffer.toString(); +// Log.v(LOG_TAG, "Forecast JSON string: " + forecastJsonStr); + try { + return getWeatherDataFromJson(forecastJsonStr, numDays); + } catch (JSONException e) { + Log.e(LOG_TAG, e.getMessage(), e); + e.printStackTrace(); + } + + // This will only happen if we fail to read the forecast + return null; + + } catch (IOException e) { + Log.e(LOG_TAG, "Error ", e); + // If the code didn't successfully get the weather data, there's no point in attempting + // to parse it. + forecastJsonStr = null; + } finally { + if (urlConnection != null) { + urlConnection.disconnect(); + } + if (reader != null) { + try { + reader.close(); + } catch (final IOException e) { + Log.e(LOG_TAG, "Error closing stream", e); + } + } + } + return null; + } + + /* The date/time conversion code is going to be moved outside the asynctask later, + * so for convenience we're breaking it out into its own method now. + */ + private String getReadableDateString(long time) { + // Because the API returns a unix timestamp (measured in seconds), + // it must be converted to milliseconds in order to be converted to valid date. + Date date = new Date(time * 1000); + SimpleDateFormat format = new SimpleDateFormat("E, MMM d"); + return format.format(date).toString(); + } + + /** + * Prepare the weather high/lows for presentation. + */ + private String formatHighLows(double high, double low) { + // For presentation, assume the user doesn't care about tenths of a degree. + long roundedHigh = Math.round(high); + long roundedLow = Math.round(low); + + String highLowStr = roundedHigh + "/" + roundedLow; + return highLowStr; + } + + /** + * Take the String representing the complete forecast in JSON Format and + * pull out the data we need to construct the Strings needed for the wireframes. + *

+ * Fortunately parsing is easy: constructor takes the JSON string and converts it + * into an Object hierarchy for us. + */ + private String[] getWeatherDataFromJson(String forecastJsonStr, int numDays) + throws JSONException { + + // These are the names of the JSON objects that need to be extracted. + final String OWM_LIST = "list"; + final String OWM_WEATHER = "weather"; + final String OWM_TEMPERATURE = "temp"; + final String OWM_MAX = "max"; + final String OWM_MIN = "min"; + final String OWM_DATETIME = "dt"; + final String OWM_DESCRIPTION = "main"; + + JSONObject forecastJson = new JSONObject(forecastJsonStr); + JSONArray weatherArray = forecastJson.getJSONArray(OWM_LIST); + + String[] resultStrs = new String[numDays]; + for (int i = 0; i < weatherArray.length(); i++) { + // For now, using the format "Day, description, hi/low" + String day; + String description; + String highAndLow; + + // Get the JSON object representing the day + JSONObject dayForecast = weatherArray.getJSONObject(i); + + // The date/time is returned as a long. We need to convert that + // into something human-readable, since most people won't read "1400356800" as + // "this saturday". + long dateTime = dayForecast.getLong(OWM_DATETIME); + day = getReadableDateString(dateTime); + + // description is in a child array called "weather", which is 1 element long. + JSONObject weatherObject = dayForecast.getJSONArray(OWM_WEATHER).getJSONObject(0); + description = weatherObject.getString(OWM_DESCRIPTION); + + // Temperatures are in a child object called "temp". Try not to name variables + // "temp" when working with temperature. It confuses everybody. + JSONObject temperatureObject = dayForecast.getJSONObject(OWM_TEMPERATURE); + double high = temperatureObject.getDouble(OWM_MAX); + double low = temperatureObject.getDouble(OWM_MIN); + + highAndLow = formatHighLows(high, low); + resultStrs[i] = day + " - " + description + " - " + highAndLow; + } + +// for (String s: resultStrs) { +// Log.v(LOG_TAG, "Forecast entry: " + s); +// } + return resultStrs; + } + } +} diff --git a/app/src/main/java/uk/me/njae/sunshine/MainActivity.java b/app/src/main/java/uk/me/njae/sunshine/MainActivity.java index 3d68797..ce0d65b 100644 --- a/app/src/main/java/uk/me/njae/sunshine/MainActivity.java +++ b/app/src/main/java/uk/me/njae/sunshine/MainActivity.java @@ -1,30 +1,9 @@ package uk.me.njae.sunshine; import android.support.v7.app.ActionBarActivity; -import android.support.v7.app.ActionBar; -import android.support.v4.app.Fragment; import android.os.Bundle; -import android.util.Log; -import android.view.LayoutInflater; import android.view.Menu; import android.view.MenuItem; -import android.view.View; -import android.view.ViewGroup; -import android.os.Build; -import android.widget.ArrayAdapter; -import android.widget.ListView; - -import java.io.BufferedReader; -import java.io.IOException; -import java.io.InputStream; -import java.io.InputStreamReader; -import java.net.HttpURLConnection; -import java.net.URL; - - -import java.util.ArrayList; -import java.util.List; -import java.util.Arrays; public class MainActivity extends ActionBarActivity { @@ -35,7 +14,7 @@ public class MainActivity extends ActionBarActivity { setContentView(R.layout.activity_main); if (savedInstanceState == null) { getSupportFragmentManager().beginTransaction() - .add(R.id.container, new PlaceholderFragment()) + .add(R.id.container, new ForecastFragment()) .commit(); } } @@ -60,100 +39,4 @@ public class MainActivity extends ActionBarActivity { return super.onOptionsItemSelected(item); } - /** - * A placeholder fragment containing a simple view. - */ - public static class PlaceholderFragment extends Fragment { - - public PlaceholderFragment() { - } - - @Override - public View onCreateView(LayoutInflater inflater, ViewGroup container, - Bundle savedInstanceState) { - View rootView = inflater.inflate(R.layout.fragment_main, container, false); - - String[] forecastArray = { - "Today - Sunny - 10/10", - "Tomorrow - Cloudy - 11/11", - "Tuesday - Snow - 12/12", - "Wednesday - Rain - 13/13", - "Thursday - Hail - 14/14", - "Friday - Scorchio - 15/15", - "Saturday - Fog - 16/16" - }; - - List weekForecast = new ArrayList( - Arrays.asList(forecastArray)); - ArrayAdapter forecastAdapter = new ArrayAdapter( - getActivity(), - R.layout.list_item_forecast, - R.id.list_item_forecast_textview, - weekForecast - ); - ListView list_view = (ListView) rootView.findViewById(R.id.listview_forecast); - list_view.setAdapter(forecastAdapter); - - // These two need to be declared outside the try/catch - // so that they can be closed in the finally block. - HttpURLConnection urlConnection = null; - BufferedReader reader = null; - - // Will contain the raw JSON response as a string. - String forecastJsonStr = null; - - try { - // Construct the URL for the OpenWeatherMap query - // Possible parameters are avaiable at OWM's forecast API page, at - // http://openweathermap.org/API#forecast - URL url = new URL("http://api.openweathermap.org/data/2.5/forecast/daily?q=94043&mode=json&units=metric&cnt=7"); - - // Create the request to OpenWeatherMap, and open the connection - urlConnection = (HttpURLConnection) url.openConnection(); - urlConnection.setRequestMethod("GET"); - urlConnection.connect(); - - // Read the input stream into a String - InputStream inputStream = urlConnection.getInputStream(); - StringBuffer buffer = new StringBuffer(); - if (inputStream == null) { - // Nothing to do. - forecastJsonStr = null; - } - reader = new BufferedReader(new InputStreamReader(inputStream)); - - String line; - while ((line = reader.readLine()) != null) { - // Since it's JSON, adding a newline isn't necessary (it won't affect parsing) - // But it does make debugging a *lot* easier if you print out the completed - // buffer for debugging. - buffer.append(line + "\n"); - } - - if (buffer.length() == 0) { - // Stream was empty. No point in parsing. - forecastJsonStr = null; - } - forecastJsonStr = buffer.toString(); - } catch (IOException e) { - Log.e("PlaceholderFragment", "Error ", e); - // If the code didn't successfully get the weather data, there's no point in attemping - // to parse it. - forecastJsonStr = null; - } finally{ - if (urlConnection != null) { - urlConnection.disconnect(); - } - if (reader != null) { - try { - reader.close(); - } catch (final IOException e) { - Log.e("PlaceholderFragment", "Error closing stream", e); - } - } - } - - return rootView; - } - } } diff --git a/app/src/main/res/menu/forecastfragment.xml b/app/src/main/res/menu/forecastfragment.xml new file mode 100644 index 0000000..7ad5987 --- /dev/null +++ b/app/src/main/res/menu/forecastfragment.xml @@ -0,0 +1,9 @@ +

+ + diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index ec03bdf..e2edbfa 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -4,5 +4,6 @@ Sunshine Hello world! Settings + Refresh -- 2.34.1