From 171e36997e733176cc249ebfa368a62ae3f78b75 Mon Sep 17 00:00:00 2001
From: 沈斌 <bluelazysb@hotmail.com>
Date: Wed, 20 Dec 2017 11:43:10 +0800
Subject: [PATCH] updated

---
 src/main/java/com/moral/service/impl/AccountServiceImpl.java |   62 ++++++++++++++----------------
 1 files changed, 29 insertions(+), 33 deletions(-)

diff --git a/src/main/java/com/moral/service/impl/AccountServiceImpl.java b/src/main/java/com/moral/service/impl/AccountServiceImpl.java
index 06808ff..fada657 100644
--- a/src/main/java/com/moral/service/impl/AccountServiceImpl.java
+++ b/src/main/java/com/moral/service/impl/AccountServiceImpl.java
@@ -1,46 +1,49 @@
 package com.moral.service.impl;
 
+import static com.moral.common.bean.Constants.IS_DELETE_FALSE;
+import static com.moral.common.bean.Constants.IS_DELETE_TRUE;
+import static com.moral.common.util.Crypto.md5;
+import static com.moral.common.util.ResourceUtil.getValue;
+import static org.apache.commons.lang3.StringUtils.isNumeric;
+import static org.springframework.util.ObjectUtils.isEmpty;
+
 import java.util.HashMap;
-import java.util.List;
 import java.util.Map;
 import java.util.Set;
 
-import org.apache.commons.lang3.StringUtils;
-import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.stereotype.Service;
-import org.springframework.util.ObjectUtils;
+import javax.annotation.Resource;
 
+import org.springframework.stereotype.Service;
+
+import com.moral.common.exception.BusinessException;
 import com.moral.entity.Account;
-import com.moral.entity.AccountExample;
 import com.moral.mapper.AccountMapper;
 import com.moral.service.AccountService;
 import com.moral.service.OrganizationService;
-import com.moral.util.BusinessException;
-import com.moral.util.Crypto;
-import com.moral.util.ResourceUtil;
 
 @Service
 public class AccountServiceImpl implements AccountService {
-	@Autowired
+
+	@Resource
 	private AccountMapper accountMapper;
 
-	@Autowired
+	@Resource
 	private OrganizationService organizationService;
 
 	@Override
 	public Map<String, Object> screenLogin(Map<String, Object> parameters) {
 		Map<String, Object> result = new HashMap<String, Object>();
-		AccountExample example = new AccountExample();
-		String password = Crypto.md5((String) parameters.get("account"));
-		example.or().andAccountNameEqualTo((String) parameters.get("account")).andPasswordEqualTo(password);
-		List<Account> accounts = accountMapper.selectByExample(example);
-		if (ObjectUtils.isEmpty(accounts) || accounts.size() != 1) {
+		Account account = new Account();
+		account.setAccountName((String) parameters.get("account"));
+		account.setPassword(md5((String) parameters.get("password")));
+		account = accountMapper.selectOne(account);
+		if (isEmpty(account)) {
 			result.put("msg", "���������������������������������");
 		} else {
-			Account account = accounts.get(0);
-			if ("1".equals(account.getIsDelete())) {
+			if (IS_DELETE_FALSE.equals(account.getIsDelete())) {
 				result.put("msg", "���������������");
 				result.put("accountId", account.getId());
+				result.put("orgId", account.getOrganizationId());
 			} else {
 				result.put("msg","���������������������������������������������");
 			}
@@ -49,35 +52,28 @@
 	}
 
 	@Override
-    public List<Account> getAccountLists(String accountName, String password) {
-		AccountExample example = new AccountExample();
-		example.or().andAccountNameEqualTo(accountName).andPasswordEqualTo(password);
-		return accountMapper.selectByExample(example);
-	}
-
-	@Override
-	public List<Account> getAccountList(String accountName) {
-		AccountExample example = new AccountExample();
-		example.or().andAccountNameEqualTo(accountName);
-		return accountMapper.selectByExample(example);
+	public Account getAccountByAccountName(String accountName) {
+		Account account = new Account();
+		account.setAccountName(accountName);
+		return accountMapper.selectOne(account);
 	}
 
 	@Override
 	public void setOrgIdsByAccount(Map<String, Object> parameters) {
 		String accountId = (String) parameters.get("accountId");
 		accountId = accountId.replaceFirst("-", "");
-		if (!StringUtils.isNumeric((String) parameters.get("accountId"))) {
+		if (!isNumeric((String) parameters.get("accountId"))) {
 			throw new BusinessException("accountId ������������������");
 		}
 
 		Account account = accountMapper.selectByPrimaryKey((Integer.valueOf(accountId)));
-		if (ObjectUtils.isEmpty(account) || "1".equals(account.getIsDelete())) {
-			throw new BusinessException(accountId + "���������������������");
+		if (isEmpty(account) || IS_DELETE_TRUE.equals(account.getIsDelete())) {
+			throw new BusinessException(accountId + ":���������������������");
 		}
 		Integer orgId = account.getOrganizationId();
 		// ���������������������������������������������������������������
 		
-		if (!("-1".equals(orgId) || ResourceUtil.getValue("orgId").equals(orgId))) {
+		if (!(-1 == orgId  || getValue("orgId").equals(orgId+""))) {
 			Set<Integer> orgIds = organizationService.getChildOrganizationIds(orgId);
 			parameters.put("orgIds", orgIds);
 		}

--
Gitblit v1.8.0