eclipse-sailing-analytics/sailing-analytics
1
0 Fork 0
mirror of https://github.com/eclipse-sailing-analytics/sailing-analytics.git synced 2026-10-19 02:29:58 +00:00
Code Activity

always use two decimal digits in UI and let NumberFormat do the rounding

Browse Source
This commit is contained in:
Axel Uhl committed 2012-07-20 00:09:37 +02:00
1 parent 24de5865fb
commit 60ce4d7097
2 files changed
+11 -7
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboard/LeaderboardPanel.java
+10 -6
View File
@@ -19,6 +19,7 @@ import com.google.gwt.cell.client.TextCell;
import com.google.gwt.core.client.GWT;
import com.google.gwt.event.dom.client.ClickEvent;
import com.google.gwt.event.dom.client.ClickHandler;
import com.google.gwt.i18n.client.NumberFormat;
import com.google.gwt.resources.client.ImageResource;
import com.google.gwt.safehtml.shared.SafeHtml;
import com.google.gwt.safehtml.shared.SafeHtmlBuilder;
@@ -92,6 +93,8 @@ public class LeaderboardPanel extends FormPanel implements TimeListener, PlaySta
private static final int SAIL_ID_COLUMN_INDEX = 1;
private static final int CARRY_COLUMN_INDEX = 3;
protected static final NumberFormat scoreFormat = NumberFormat.getFormat("0.00");
private final SailingServiceAsync sailingService;
@@ -489,15 +492,16 @@ public class LeaderboardPanel extends FormPanel implements TimeListener, PlaySta
if (entry.reasonForMaxPoints == null || entry.reasonForMaxPoints == MaxPointsReason.NONE) {
if (!entry.discarded) {
html.appendHtmlConstant("<span style=\"font-weight: bold;\">");
html.appendHtmlConstant(entry.totalPoints == 0 ? "" : ""+entry.totalPoints);
html.appendHtmlConstant(entry.totalPoints == 0 ? "" : scoreFormat.format(entry.totalPoints));
html.appendHtmlConstant("</span>");
} else {
html.appendHtmlConstant(" <span style=\"opacity: 0.5;\"><del>");
html.appendHtmlConstant(entry.netPoints == 0 ? "" : ""+entry.netPoints);
html.appendHtmlConstant(entry.netPoints == 0 ? "" : scoreFormat.format(entry.netPoints));
html.appendHtmlConstant("</del></span>");
}
} else {
html.appendHtmlConstant(" <span title=\""+entry.netPoints+"/"+entry.totalPoints+"\" style=\"opacity: 0.5;\">");
html.appendHtmlConstant(" <span title=\""+scoreFormat.format(entry.netPoints)+"/"+
scoreFormat.format(entry.totalPoints)+"\" style=\"opacity: 0.5;\">");
if (entry.discarded) {
html.appendHtmlConstant("<del>");
}
@@ -565,7 +569,7 @@ public class LeaderboardPanel extends FormPanel implements TimeListener, PlaySta
@Override
public String getValue(LeaderboardRowDTO object) {
final double totalPoints = object.fieldsByRaceColumnName.get(getRaceColumnName()).totalPoints;
return "" + (totalPoints == 0 ? "" : totalPoints);
return "" + (totalPoints == 0 ? "" : scoreFormat.format(totalPoints));
}
@Override
@@ -836,7 +840,7 @@ public class LeaderboardPanel extends FormPanel implements TimeListener, PlaySta
@Override
public String getValue(LeaderboardRowDTO object) {
double totalPoints = getLeaderboard().getTotalPoints(object);
return "" + (totalPoints==0 ? "" : totalPoints);
return "" + (totalPoints==0 ? "" : scoreFormat.format(totalPoints));
}
@Override
@@ -880,7 +884,7 @@ public class LeaderboardPanel extends FormPanel implements TimeListener, PlaySta
@Override
public String getValue(LeaderboardRowDTO object) {
return object.carriedPoints == null ? "" : "" + object.carriedPoints;
return object.carriedPoints == null ? "" : scoreFormat.format(object.carriedPoints);
}
@Override
java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboardedit/EditableLeaderboardPanel.java
+1 -1
View File
@@ -354,7 +354,7 @@ public class EditableLeaderboardPanel extends LeaderboardPanel {
LeaderboardEntryDTO leaderboardEntryDTO = object.fieldsByRaceColumnName.get(raceName);
String result = "";
if (leaderboardEntryDTO != null) {
result = result+leaderboardEntryDTO.netPoints;
result = scoreFormat.format(leaderboardEntryDTO.netPoints);
}
return result;
}