From f2eaa5bc6c417c9f851cb7ba2bf004c5b035c0ef Mon Sep 17 00:00:00 2001
From: fengxiang <110431245@qq.com>
Date: Wed, 29 Aug 2018 14:40:01 +0800
Subject: [PATCH] 最近24小时aqi接口完善

---
 src/main/java/com/moral/service/impl/HistoryServiceImpl.java |   34 +++++++++++++++++++++++++---------
 1 files changed, 25 insertions(+), 9 deletions(-)

diff --git a/src/main/java/com/moral/service/impl/HistoryServiceImpl.java b/src/main/java/com/moral/service/impl/HistoryServiceImpl.java
index c45cb88..f6dbae7 100644
--- a/src/main/java/com/moral/service/impl/HistoryServiceImpl.java
+++ b/src/main/java/com/moral/service/impl/HistoryServiceImpl.java
@@ -158,16 +158,20 @@
 		accountService.setOrgIdsByAccount(parameters);
 		
 		List<Map<String, Object>> result = historyMapper.getRegionRankingData(parameters);
-		
+		for (Map<String, Object> map : result) {
+			if (ObjectUtils.isEmpty(map.get("avg"))) {
+				map.put("avg", 0);
+			}
+		}
 		if ("profession".equals(parameters.get("dimension"))) {
 			List<Profession> professions = professionMapper.selectAll();
 			for (Map<String, Object> map : result) {
 				for (Profession profession : professions) {
 					if (map.get("name").equals(profession.getName())) {
 						professions.remove(profession);
-						if (ObjectUtils.isEmpty(map.get("avg"))) {
-							map.put("avg", 0);
-						}
+//						if (ObjectUtils.isEmpty(map.get("avg"))) {
+//							map.put("avg", 0);
+//						}
 						break;
 					}
 				}
@@ -183,9 +187,9 @@
 			for (Map<String, Object> map : result) {
 				if (regions.contains(map.get("name"))) {
 					regions.remove(map.get("name"));
-					if (ObjectUtils.isEmpty(map.get("avg"))) {
-						map.put("avg", 0);
-					}
+//					if (ObjectUtils.isEmpty(map.get("avg"))) {
+//						map.put("avg", 0);
+//					}
 				}
 				
 			}
@@ -221,20 +225,32 @@
 	
     @Override
 	public List<Map<String, Object>> getSensorsDataByDevice(Map<String, Object> parameters) {
+		ValidateUtil.notNull(parameters.get("deviceVersionId"), "param.is.null");
+		ValidateUtil.notNull(parameters.get("mac"), "param.is.null");
+		ValidateUtil.notNull(parameters.get("deviceId"), "param.is.null");
     	List<Map<String, Object>> sensors = sensorMapper.getSensorsByDeviceVersionId(parameters);
     	parameters.put("size", 1);
     	List<Map<String, Object>> values = historyMapper.getValueByMacAndSize(parameters);
-    	JSONObject map = JSON.parseObject(values.get(0).get("value").toString());
+    	JSONObject map = null;
+    	if (!ObjectUtils.isEmpty(values)) {
+    		map = JSON.parseObject(values.get(0).get("value").toString());
+		}
     	Device device = deviceMapper.selectByPrimaryKey(Integer.valueOf((String) parameters.get("deviceId")));
     	for (Map<String, Object> sensor : sensors) {
-    		sensor.put("value", map.remove(sensor.get("sensor_key")));
+    		if (!ObjectUtils.isEmpty(map)) {
+    			sensor.put("value", map.remove(sensor.get("sensor_key")));
+			}
     		sensor.put("state", device.getState());
+    		sensor.put("name", sensor.get("description"));
     	}
 		return sensors;
 	}
 
 	@Override
 	public List<Map<String, Object>> getSensorDataBySensorKey(Map<String, Object> parameters) {
+		ValidateUtil.notNull(parameters.get("sensorKey"), "param.is.null");
+		ValidateUtil.notNull(parameters.get("mac"), "param.is.null");
+		ValidateUtil.notNull(parameters.get("size"), "param.is.null");
 		parameters.put("size", Integer.valueOf(parameters.remove("size").toString()));
 		List<Map<String, Object>> values = historyMapper.getValueByMacAndSize(parameters);
 		for (Map<String, Object> value : values) {

--
Gitblit v1.8.0