Showing 10 of 69 total issues
LocationFragment
has 33 methods (exceeds 20 allowed). Consider refactoring. Open
Open
public class LocationFragment extends Fragment implements OnMapReadyCallback {
private static final String TAG = LocationFragment.class.getSimpleName();
private static final int DEFAULT_ZOOM = 15;
private static final int PERMISSIONS_REQUEST_ACCESS_FINE_LOCATION = 1;
Consider simplifying this complex logical expression. Confirmed
Confirmed
if(months.get(month_index)==1 || months.get(month_index)==3 ||months.get(month_index)==5 ||months.get(month_index)==7 ||months.get(month_index)==8 ||months.get(month_index)==10 ||months.get(month_index)==12 ){
if(number_date+7<=31){
date.setText(String.valueOf((number_date+7)));
}
Avoid too many return
statements within this method. Confirmed
Confirmed
else if(month_index==6) return "June";
Avoid too many return
statements within this method. Confirmed
Confirmed
else if(month_index==4) return "April";
Avoid too many return
statements within this method. Confirmed
Confirmed
else if(month_index==3) return "March";
Avoid too many return
statements within this method. Confirmed
Confirmed
else if(month_index==2) return "February";
Avoid too many return
statements within this method. Confirmed
Confirmed
else if(month_index==8) return "August";
Avoid too many return
statements within this method. Confirmed
Confirmed
else if(month_index==7) return "July";
Avoid too many return
statements within this method. Confirmed
Confirmed
else if(month_index==5) return "May";
Avoid too many return
statements within this method. Confirmed
Confirmed
if(month_index==1) return "January";