어떤 코드가 더 좋습니까?

Question

일부 코드를 작성했지만 어느 것이 더 좋을지 잘 모르겠습니다. 한 가지 방법으로 무슨 일이 일어나는지 쉽게 읽을 수 있지만 더 많은 코드 라인이 있습니다. 다른면에서는 코드 줄이 적지 만 이해하기가 더 어려울 것이라고 생각합니다.

String imp = importance.getSelectedItem().toString(); 
String title_str = title.getText().toString(); 
String body_str = body.getText().toString(); 
String location_str = location.getText().toString(); 
int day = date.getDayOfMonth(); 
int month = date.getMonth()+1; 
int year = date.getYear(); 
int hh = time.getCurrentHour(); 
int mm = time.getCurrentMinute(); 
String date_str = year+"/"+month+"/"+day+" " + hh+":"+mm +":00"; // yyyy/MM/dd HH:mm:ss 
long dateMilliseconds = new Timeconversion().timeConversion(date_str); 

Conference conference = ConferenceBuilder.conference() 
     .id(idConf) 
     .importance(Double.parseDouble(imp)) 
     .title(title_str) 
     .body(body_str) 
     .location(location_str) 
     .timeInMilliseconds(dateMilliseconds) 
     .build(); 

또는

Conference conference2 = ConferenceBuilder.conference() 
           .id(idConf) 
           .importance(Double.parseDouble(importance.getSelectedItem().toString())) 
           .title(title.getText().toString()) 
           .body(body.getText().toString()) 
           .location(location.getText().toString()) 
           // yyyy/MM/dd HH:mm:ss 
           .timeInMilliseconds(new Timeconversion().timeConversion(date.getYear()+"/"+date.getMonth()+1+"/"+date.getDayOfMonth()+" " + time.getCurrentHour()+":"+time.getCurrentMinute() +":00")) 
           .build(); 

Answer 1

분할 차이. 당신의 textviews 꽤 명확하게 명명되기 때문에, 당신의 textviews에서 String 객체를 추출하는 일을 더 쉽게 읽을 생각하지 않습니다

Conference conference2 = ConferenceBuilder.conference() 
      .id(idConf) 
      .importance(Double.parseDouble(importance.getSelectedItem().toString())) 
      .title(title.getText().toString()) 
      .body(body.getText().toString()) 
      .location(location.getText().toString()) 
      // yyyy/MM/dd HH:mm:ss 
      .timeInMilliseconds(getTimeInMillis(datePicker, timePicker)) 
      .build(); 
} 

private long getTimeInMillis(DatePicker datePicker, TimePicker timePicker) { 
    Calendar calendar = Calendar.getInstance(); 
    calendar.set(datePicker.getYear(), datePicker.getMonth(), datePicker.getDayOfMonth(), 
    timePicker.getCurrentHour(), timePicker.getCurrentMinute(), 0); 
    return calendar.getTimeInMillis(); 
} 

: 나는 같은 것을 할 것입니다.