some refactoring

pull/77/head
Denis 10 years ago
parent 8bde5159c8
commit ba7a1174a8

@ -183,12 +183,17 @@ public class DateHelper
* *
* So we have {@code wdaysIndex = dayNumber % 7} * So we have {@code wdaysIndex = dayNumber % 7}
* *
* @return array with names from Saturday to Friday according to the current locale * @return weekday number in the internal interpretation
* *
* @see #getWeekday(long) * @see #getWeekday(long)
* @see java.util.Calendar#SUNDAY * @see java.util.Calendar#SUNDAY
* *
*/ */
public static int weekDayNumber2wdays(int number)
{
return number % 7;
}
public static String[] getDayNames(int format) public static String[] getDayNames(int format)
{ {
String[] wdays = new String[7]; String[] wdays = new String[7];

@ -65,7 +65,6 @@ public class HabitFrequencyView extends ScrollableDataView implements HabitDataV
private HashMap<Long, Integer[]> frequency; private HashMap<Long, Integer[]> frequency;
private String wdays[]; private String wdays[];
private Integer[] localeWeekdayList; private Integer[] localeWeekdayList;
private Map<Integer, Integer> number2wdays;
public HabitFrequencyView(Context context) public HabitFrequencyView(Context context)
{ {
@ -94,12 +93,7 @@ public class HabitFrequencyView extends ScrollableDataView implements HabitDataV
wdays = DateHelper.getShortDayNames(); wdays = DateHelper.getShortDayNames();
/**
* here we create the mapping of week days numbers into the "wdays"-indices
* @see DateHelper#getDayNames(int)
*/
localeWeekdayList = DateHelper.getLocaleWeekdayList(); localeWeekdayList = DateHelper.getLocaleWeekdayList();
number2wdays = DateHelper.getWeekdayMap();
dfMonth = DateHelper.getDateFormat("MMM"); dfMonth = DateHelper.getDateFormat("MMM");
dfYear = DateHelper.getDateFormat("yyyy"); dfYear = DateHelper.getDateFormat("yyyy");
@ -245,7 +239,7 @@ public class HabitFrequencyView extends ScrollableDataView implements HabitDataV
rect.set(0, 0, baseSize, baseSize); rect.set(0, 0, baseSize, baseSize);
rect.offset(prevRect.left, prevRect.top + columnWidth * j); rect.offset(prevRect.left, prevRect.top + columnWidth * j);
int i = number2wdays.get(localeWeekdayList[j]); int i = DateHelper.weekDayNumber2wdays(localeWeekdayList[j]);
if(values != null) if(values != null)
drawMarker(canvas, rect, values[i]); drawMarker(canvas, rect, values[i]);
@ -284,7 +278,7 @@ public class HabitFrequencyView extends ScrollableDataView implements HabitDataV
pGrid.setColor(dimmedTextColor); pGrid.setColor(dimmedTextColor);
for (Integer dayNumber : localeWeekdayList) { for (Integer dayNumber : localeWeekdayList) {
canvas.drawText(wdays[number2wdays.get(dayNumber)], rGrid.right - columnWidth, canvas.drawText(wdays[DateHelper.weekDayNumber2wdays(dayNumber)], rGrid.right - columnWidth,
rGrid.top + rowHeight / 2 + 0.25f * em, pText); rGrid.top + rowHeight / 2 + 0.25f * em, pText);
pGrid.setStrokeWidth(1f); pGrid.setStrokeWidth(1f);

@ -59,8 +59,6 @@ public class HabitHistoryView extends ScrollableDataView implements HabitDataVie
private int nColumns; private int nColumns;
private String wdays[]; private String wdays[];
private Integer[] localeWeekdayList;
private Map<Integer, Integer> number2wdays;
private SimpleDateFormat dfMonth; private SimpleDateFormat dfMonth;
private SimpleDateFormat dfYear; private SimpleDateFormat dfYear;
@ -106,9 +104,6 @@ public class HabitHistoryView extends ScrollableDataView implements HabitDataVie
dfMonth = DateHelper.getDateFormat("MMM"); dfMonth = DateHelper.getDateFormat("MMM");
dfYear = DateHelper.getDateFormat("yyyy"); dfYear = DateHelper.getDateFormat("yyyy");
localeWeekdayList = DateHelper.getLocaleWeekdayList();
number2wdays = DateHelper.getWeekdayMap();
baseLocation = new RectF(); baseLocation = new RectF();
} }
@ -306,10 +301,10 @@ public class HabitHistoryView extends ScrollableDataView implements HabitDataVie
private void drawAxis(Canvas canvas, RectF location) private void drawAxis(Canvas canvas, RectF location)
{ {
for (Integer dayNumber : localeWeekdayList) for (Integer dayNumber : DateHelper.getLocaleWeekdayList())
{ {
location.offset(0, columnWidth); location.offset(0, columnWidth);
canvas.drawText(wdays[number2wdays.get(dayNumber)], location.left + headerTextOffset, canvas.drawText(wdays[DateHelper.weekDayNumber2wdays(dayNumber)], location.left + headerTextOffset,
location.bottom - headerTextOffset, pTextHeader); location.bottom - headerTextOffset, pTextHeader);
} }
} }

Loading…
Cancel
Save