tomasz
/
urban-explorer
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Replaced loggers for slf4j in HomeFragment.

master
Tomasz Półgrabia 2016-09-17 13:17:44 +02:00
parent bbc7666f95
commit bb6a274e2e
1 changed files with 33 additions and 32 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

65
app/src/main/java/pl/tpolgrabia/urbanexplorer/fragments/HomeFragment.java
View File

@ -9,11 +9,12 @@ import android.preference.PreferenceManager;
import android.support.annotation.Nullable; import android.support.annotation.Nullable;
import android.support.v4.app.Fragment; import android.support.v4.app.Fragment;
import android.support.v4.app.FragmentActivity; import android.support.v4.app.FragmentActivity;
import android.util.Log;
import android.view.LayoutInflater; import android.view.LayoutInflater;
import android.view.View; import android.view.View;
import android.view.ViewGroup; import android.view.ViewGroup;
import android.widget.*; import android.widget.*;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import pl.tpolgrabia.urbanexplorer.AppConstants; import pl.tpolgrabia.urbanexplorer.AppConstants;
import pl.tpolgrabia.urbanexplorer.MainActivity; import pl.tpolgrabia.urbanexplorer.MainActivity;
import pl.tpolgrabia.urbanexplorer.R; import pl.tpolgrabia.urbanexplorer.R;
@ -34,6 +35,7 @@ import java.util.concurrent.Semaphore;
public class HomeFragment extends Fragment { public class HomeFragment extends Fragment {
private static final String CLASS_TAG = HomeFragment.class.getSimpleName(); private static final String CLASS_TAG = HomeFragment.class.getSimpleName();
private static final Logger lg = LoggerFactory.getLogger(HomeFragment.class);
private static final int PANORAMIA_BULK_DATA_SIZE = 10; private static final int PANORAMIA_BULK_DATA_SIZE = 10;
public static final String TAG = HomeFragment.class.getSimpleName(); public static final String TAG = HomeFragment.class.getSimpleName();
@ -54,7 +56,7 @@ public class HomeFragment extends Fragment {
@Override @Override
public void onCreate(Bundle savedInstanceState) { public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState); super.onCreate(savedInstanceState);
Log.v(CLASS_TAG, "onCreate"); lg.trace("onCreate");
pageId = 1L; pageId = 1L;
loading = new Semaphore(1, true); loading = new Semaphore(1, true);
photos = new ArrayList<>(); photos = new ArrayList<>();
@ -80,7 +82,7 @@ public class HomeFragment extends Fragment {
try { try {
fetchAdditionalPhotos(); fetchAdditionalPhotos();
} catch (InterruptedException e) { } catch (InterruptedException e) {
Log.e(CLASS_TAG, "Failed trying acquring lock to load photos", e); lg.error("Failed trying acquring lock to load photos", e);
} }
} }
}); });
@ -94,7 +96,7 @@ public class HomeFragment extends Fragment {
try { try {
return Double.parseDouble(text.toString()); return Double.parseDouble(text.toString());
} catch (NumberFormatException e) { } catch (NumberFormatException e) {
Log.w(CLASS_TAG, "Wrong number format", e); lg.warn("Wrong number format", e);
return null; return null;
} }
} }
@ -134,15 +136,15 @@ public class HomeFragment extends Fragment {
if (firstVisibleItem <= 0) { if (firstVisibleItem <= 0) {
// scrolled to the top // scrolled to the top
Log.v(CLASS_TAG, "Scrolled to the top"); lg.trace("Scrolled to the top");
} }
if (firstVisibleItem + visibleItemCount >= totalItemCount) { if (firstVisibleItem + visibleItemCount >= totalItemCount) {
Log.v(CLASS_TAG, "Scrolled to the bottom"); lg.trace("Scrolled to the bottom");
// scrolled to the bottom // scrolled to the bottom
final View fragView = getView(); final View fragView = getView();
if (fragView == null) { if (fragView == null) {
Log.v(CLASS_TAG, "Frag still not initialized"); lg.trace("Frag still not initialized");
return; return;
} }
fetchAdditionalPhotos(); fetchAdditionalPhotos();
@ -150,7 +152,7 @@ public class HomeFragment extends Fragment {
} }
} catch (InterruptedException e) { } catch (InterruptedException e) {
Log.e(CLASS_TAG, "Aquiring lock interrupted exception", e); lg.error("Aquiring lock interrupted exception", e);
} }
} }
@ -164,44 +166,44 @@ public class HomeFragment extends Fragment {
private void fetchAdditionalPhotos() throws InterruptedException { private void fetchAdditionalPhotos() throws InterruptedException {
if (noMorePhotos) { if (noMorePhotos) {
Log.v(CLASS_TAG, "No more photos - last query was zero result"); lg.trace("No more photos - last query was zero result");
return; return;
} }
if (!initialized) { if (!initialized) {
Log.v(CLASS_TAG, "Fetching additional photos blocked till system is initialized"); lg.trace("Fetching additional photos blocked till system is initialized");
return; return;
} }
if (getView() == null) { if (getView() == null) {
Log.v(CLASS_TAG, "Application still not initialized"); lg.trace("Application still not initialized");
return; return;
} }
final FragmentActivity activity = getActivity(); final FragmentActivity activity = getActivity();
if (activity == null) { if (activity == null) {
Log.w(CLASS_TAG, "Activity shouldn't be null. No headless fragment"); lg.trace("Activity shouldn't be null. No headless fragment");
return; return;
} }
final Location location = locationService.getLastKnownLocation(LocationUtils.getDefaultLocation(activity)); final Location location = locationService.getLastKnownLocation(LocationUtils.getDefaultLocation(activity));
if (location == null) { if (location == null) {
Log.i(CLASS_TAG, "Location still not available"); lg.info("Location still not available");
Toast.makeText(activity, "Location still not available", Toast.LENGTH_SHORT).show(); Toast.makeText(activity, "Location still not available", Toast.LENGTH_SHORT).show();
return; return;
} }
Log.v(CLASS_TAG, "Fetching additional photos. Trying loading acquirng lock"); lg.trace("Fetching additional photos. Trying loading acquirng lock");
if (!loading.tryAcquire()) { if (!loading.tryAcquire()) {
Log.v(CLASS_TAG, "Fetching additional photos. Loading in progress"); lg.info("Fetching additional photos. Loading in progress");
return; return;
} }
int offset = photos.size(); int offset = photos.size();
Log.v(CLASS_TAG, "Fetching additional photos offset: " + offset + ", count: " + PANORAMIA_BULK_DATA_SIZE); lg.debug("Fetching additional photos offset: {}, count: {}", offset, PANORAMIA_BULK_DATA_SIZE);
PanoramioUtils.fetchPanoramioImages( PanoramioUtils.fetchPanoramioImages(
activity, activity,
@ -214,8 +216,7 @@ public class HomeFragment extends Fragment {
new PanoramioResponseCallback() { new PanoramioResponseCallback() {
@Override @Override
public void callback(PanoramioResponseStatus status, List<PanoramioImageInfo> images, Long imagesCount) { public void callback(PanoramioResponseStatus status, List<PanoramioImageInfo> images, Long imagesCount) {
Log.v(CLASS_TAG, "Fetched with status: " + status + ", images: " + images + ", count: " + lg.debug("Fetched with status: {}, images: {}, count: {}", status, images, imagesCount);
imagesCount);
if (status != PanoramioResponseStatus.SUCCESS) { if (status != PanoramioResponseStatus.SUCCESS) {
return; return;
} }
@ -236,7 +237,7 @@ public class HomeFragment extends Fragment {
// TODO we can think about removing first items also and last if the number // TODO we can think about removing first items also and last if the number
// TODO of items exceeds the limit (to save the memory) // TODO of items exceeds the limit (to save the memory)
Log.v(CLASS_TAG, "Finished Fetching additional photos count: " + photos.size()); lg.debug("Finished Fetching additional photos count: {}", photos.size());
loading.release(); loading.release();
@ -247,16 +248,16 @@ public class HomeFragment extends Fragment {
} }
public void fetchPanoramioPhotos() { public void fetchPanoramioPhotos() {
Log.v(CLASS_TAG, "Fetch panoramio photos"); lg.trace("Fetch panoramio photos");
final FragmentActivity activity = getActivity(); final FragmentActivity activity = getActivity();
if (activity == null) { if (activity == null) {
Log.w(CLASS_TAG, "Activity shouldn't be null. It isn't headless fragment"); lg.warn("Activity shouldn't be null. It isn't headless fragment");
return; return;
} }
final Location location = locationService.getLastKnownLocation(LocationUtils.getDefaultLocation(activity)); final Location location = locationService.getLastKnownLocation(LocationUtils.getDefaultLocation(activity));
if (location == null) { if (location == null) {
Log.i(CLASS_TAG, "Location is still not available"); lg.info("Location is still not available");
Toast.makeText(getActivity(), "Location is still not available", Toast.LENGTH_SHORT).show(); Toast.makeText(getActivity(), "Location is still not available", Toast.LENGTH_SHORT).show();
return; return;
} }
@ -306,7 +307,7 @@ public class HomeFragment extends Fragment {
final String pref_panoramio_radiusx = sharedPreferences.getString( final String pref_panoramio_radiusx = sharedPreferences.getString(
"pref_panoramio_radiusx", "pref_panoramio_radiusx",
String.valueOf(AppConstants.PAMNORAMIO_DEF_RADIUSX)); String.valueOf(AppConstants.PAMNORAMIO_DEF_RADIUSX));
Log.d(CLASS_TAG, "Panoramio radiusx pref equals " + pref_panoramio_radiusx); lg.debug("Panoramio radiusx pref equals {}", pref_panoramio_radiusx);
return Double.parseDouble( return Double.parseDouble(
pref_panoramio_radiusx); pref_panoramio_radiusx);
} }
@ -316,7 +317,7 @@ public class HomeFragment extends Fragment {
final String pref_panoramio_radiusy = sharedPreferences.getString( final String pref_panoramio_radiusy = sharedPreferences.getString(
"pref_panoramio_radiusy", "pref_panoramio_radiusy",
String.valueOf(AppConstants.PAMNORAMIO_DEF_RADIUSY)); String.valueOf(AppConstants.PAMNORAMIO_DEF_RADIUSY));
Log.d(CLASS_TAG, "Panoramio radiusy pref equals " + pref_panoramio_radiusy); lg.debug("Panoramio radiusy pref equals {}", pref_panoramio_radiusy);
return Double.parseDouble( return Double.parseDouble(
pref_panoramio_radiusy); pref_panoramio_radiusy);
} }
@ -325,26 +326,26 @@ public class HomeFragment extends Fragment {
public void onResume() { public void onResume() {
super.onResume(); super.onResume();
getActivity().setTitle("Panoramio search"); getActivity().setTitle("Panoramio search");
Log.v(CLASS_TAG, "onResume"); lg.trace("onResume");
updateLocationInfo(); updateLocationInfo();
} }
public void updateLocationInfo() { public void updateLocationInfo() {
Log.v(CLASS_TAG, "Update locations info"); lg.trace("Update locations info");
final View view = getView(); final View view = getView();
if (view == null) { if (view == null) {
Log.wtf(CLASS_TAG, "Fragment has no view"); lg.warn("Fragment has no view");
return; return;
} }
TextView locationInfo = (TextView) view.findViewById(R.id.locationInfo); TextView locationInfo = (TextView) view.findViewById(R.id.locationInfo);
final FragmentActivity activity = getActivity(); final FragmentActivity activity = getActivity();
if (activity == null) { if (activity == null) {
Log.w(CLASS_TAG, "Activity should'nt be null. No headless fragment"); lg.warn("Activity should'nt be null. No headless fragment");
return; return;
} }
locationService = (LocationManager) activity.getSystemService(Context.LOCATION_SERVICE); locationService = (LocationManager) activity.getSystemService(Context.LOCATION_SERVICE);
Location currLocation = locationService.getLastKnownLocation(LocationUtils.getDefaultLocation(activity)); Location currLocation = locationService.getLastKnownLocation(LocationUtils.getDefaultLocation(activity));
Log.v(CLASS_TAG, "Current location: " + currLocation + ", locationInfo: " + locationInfo); lg.trace("Current location: {}, locationInfo: {}", currLocation, locationInfo);
if (currLocation != null && locationInfo != null) { if (currLocation != null && locationInfo != null) {
// update home fragment's location info // update home fragment's location info
locationInfo.setText("Your current location: (" locationInfo.setText("Your current location: ("
@ -357,20 +358,20 @@ public class HomeFragment extends Fragment {
@Override @Override
public void onPause() { public void onPause() {
super.onPause(); super.onPause();
Log.v(CLASS_TAG, "onPause"); lg.trace("onPause");
} }
@Override @Override
public void onDestroy() { public void onDestroy() {
super.onDestroy(); super.onDestroy();
Log.v(CLASS_TAG, "onDestroy"); lg.trace("onDestroy");
} }
@Override @Override
public void onSaveInstanceState(Bundle outState) { public void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState); super.onSaveInstanceState(outState);
Log.v(CLASS_TAG, "Saving state"); lg.trace("Saving state");
} }
} }