Какой код лучше?

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

Split разница. Я хотел бы сделать что-то вроде этого:

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(); 
} 

Я не думаю, что извлечение объектов типа String из ваших TextViews делает вещи любой более читаемым, так как ваши TextViews будут довольно четко названы.