Conversation
| BigDecimal id; | ||
| String first_name; | ||
| String last_name; | ||
| String phone_number; |
| "first_name: " + this.first_name + '\n' + | ||
| "last_name: " + this.last_name + '\n' + | ||
| "phone_number: " + this.phone_number + '\n'); | ||
| } |
There was a problem hiding this comment.
Обычно переопределяют toString, Потом его используют при печати.
| } | ||
| } | ||
|
|
||
| public void printUserWithMaxIdJava() { |
| } | ||
| } | ||
|
|
||
| private User initUserFromResultSet(ResultSet user) throws SQLException { |
There was a problem hiding this comment.
Лучше назвать эту переменную userResultSet или что-то такое. Без чтения сигнатуры не сразу понимается
|
|
||
| import com.sun.org.apache.bcel.internal.generic.ARETURN; | ||
|
|
||
| import javax.jws.soap.SOAPBinding; |
There was a problem hiding this comment.
Это два странных импорта, вряд ли они тебе нужны.
К слову, они и не используются.
ctrl + alt + O = оптимизация импортов, их схлопывание, удаление неиспользуемых
| return dbEntries; | ||
| } | ||
|
|
||
| public boolean equalityTest() throws SQLException { |
There was a problem hiding this comment.
Рекомендуется именовать методы так, чтобы первым словом в имени был глагол
|
|
||
| public boolean equalityTest() throws SQLException { | ||
| ResultSet count1 = db1Connection.prepareStatement("select count(*) from " + table1Name).executeQuery(); | ||
| ResultSet count2 = db2Connection.prepareStatement("select count(*) from " + table2Name).executeQuery(); |
There was a problem hiding this comment.
Снова плохое имя для resultSet.
Имя должно быть таким, чтобы при одном взгляде на него было понятно, за что оно отвечает. Про count1 кажется, что он должен быть интом с количеством чего-то. Я бы лучше назвал countResultSet1
| count1.next(); | ||
| count2.next(); | ||
| int size1 = Integer.parseInt(count1.getString(1)); | ||
| int size2 = Integer.parseInt(count2.getString(1)); |
There was a problem hiding this comment.
Почему не getLong какой-нибудь?
| IntStream.range(1, columnsNum).reduce(0, (acc, j) -> acc == 0 ? t1.get(j).compareTo(t2.get(j - 1)) : acc) | ||
| ); | ||
| db2Entries.sort((t1, t2) -> | ||
| IntStream.range(1, columnsNum).reduce(0, (acc, j) -> acc == 0 ? t1.get(j).compareTo(t2.get(j - 1)) : acc) |
| IntStream.range(0, columnsNum).reduce(0, | ||
| (innerAcc, j) -> innerAcc == 0 ? | ||
| db1Entries.get(i).get(j).compareTo(db2Entries.get(i).get(j)) : innerAcc | ||
| ) : acc |
There was a problem hiding this comment.
Операции над стримами принято писать с новой строки.
myList.stream()
.map(..)
.filter(...)
.collect();
2 participants
No description provided.