Java 코드를 어떻게 단순화 할 수 있습니까?

Question

나는 Java에 매우 익숙하며 다음 과제에 매달렸다. 가장 높은 총 ActivityLevel을 가진 Person 객체를 반환하는 AddressBook 클래스에 메소드를 생성하라는 요청을 받았습니다.

getSocialMediaActivityLevel(), maxValue() 및 findMostSocial() 메소드를 만들었습니다.

findMostSocial 메서드는 내가 수행 한 값을 반환하지만 할당에 대한 설명에는 "간단한 알고리즘을 디자인하고 기존 클래스에 통합해야합니다."라는 문이 포함되어 있습니다. 불필요한 많은 코드를 사용한 것처럼 보이지만, 내가 한 일을 단순화하는 방법을 잘 모르겠습니다. 어떤 도움이라도 대단히 감사합니다.

public class SocialMediaAccount { 
    private String userID; 
    private String websiteName; 
    private String websiteURL; 
    private int activityLevel; 

    public SocialMediaAccount(String userID, String websiteName, String websiteURL, int activityLevel) { 
     this.userID = userID; 
     this.websiteName = websiteName; 
     this.websiteURL = websiteURL; 
     this.activityLevel = activityLevel; 
} 
    public void addSocialMediaAccount(String userID, String websiteName, String websiteURL, int activityLevel) { 
     SocialMediaAccount account1; 
     account1 = new SocialMediaAccount(userID, websiteName, websiteURL, activityLevel); 
     socialMediaAccounts.add(account1); 

}

import java.util.ArrayList; 
    public class Person { 
     private String firstName; 
     private String surname; 
     private String mobile; 
     private String email; 
     private ArrayList<SocialMediaAccount> socialMediaAccounts; 

//returns the combined ActivityLevel for all the Person's SocialMediaAccounts. 
    public int getSocialMediaActivityLevel(){ 
     int total = 0; 
      for(SocialMediaAccount e : socialMediaAccounts){ 
       total += e.getActivityLevel(); 
      } 
     return total; 

}

import java.util.ArrayList; 
import java.util.Collections; 
public class AddressBook { 
    private ArrayList<Person> contacts; 

    public AddressBook(){ 
     contacts = new ArrayList<>();  
    } 

//returns the highest combined ActivityLevel in the ArrayList contacts 
     public int maxValue(){ 
      ArrayList<Integer> maxActivityLevel = new ArrayList<>(); 
       for(Person e : contacts){ 
        maxActivityLevel.add(e.getSocialMediaActivityLevel()); 
       } 
      int maxValue = Collections.max(maxActivityLevel); 
     return maxValue; 
     } 

//returns the Person object in the contacts ArrayList with the highest combined ActivityLevel 
     public Person findMostSocial(){ 
      for(Person p: contacts){ 
       if(maxValue() == p.getSocialMediaActivityLevel()){ 
        return p; 
       } 
      } 
     return null; 
    } 

Answer 1

나는 당신이 여기에서 필요로하는 사람의 목록에서 소셜 미디어 활동의 최대 값을 결정하기 위해 여기 Quick Sort 알고리즘 생각합니다. 그래서, 귀하의 maxValue() 및 findMostSocial() 메소드를 QuickSort 구현 메소드로 대체 할 것을 제안합니다. 다음은 참고 용으로 implementation입니다. 건배