Spirin [group 597] - Task 4 #323
Conversation
ignorer
left a comment
ignorer
left a comment
There was a problem hiding this comment.
Что-то я не увидел, чтобы пароль использовался где-то, кроме регистрации.
| import org.springframework.context.annotation.Configuration; | ||
| import ru.mipt.java2016.homework.base.task1.Calculator; | ||
|
|
||
| /** |
There was a problem hiding this comment.
Ну хоть бы коммент удалил при копипасте :)
| public class BillingDatabaseConfiguration { | ||
| @Bean | ||
| public DataSource billingDataSource( | ||
| @Value("jdbc:h2:mem:testdb") String jdbcUrl, |
There was a problem hiding this comment.
Можно создать в main/resources файл application.properties, в который положить вот это значение. То же касается и следующих двух строк, только там выставлены пустые значения по умолчанию.
|
|
||
| // Update initial (username, password) | ||
| jdbcTemplate.execute("DELETE FROM billing.users WHERE username = 'username'"); | ||
| jdbcTemplate.execute("INSERT INTO billing.users VALUES ('username', 'password')"); |
There was a problem hiding this comment.
Можно было бы использовать MERGE
| name = "world"; | ||
| } | ||
| return "<html>" + | ||
| "<head><title>FediqApp</title></head>" + |
| String username = authentication.getName(); | ||
| double result = calculator.calculate(expression); | ||
|
|
||
| LOG.trace("Result: " + result); |
There was a problem hiding this comment.
Лучше бы объединить это с предыдущим сообщением логгера, потому как если 100 пользователей одновременно захотят посчитать, то получится каша из сообщений, непонятно, какой результат к какому выражению относится.
| String password = arguments[1]; | ||
|
|
||
| LOG.trace(username); | ||
| LOG.trace(password); |
There was a problem hiding this comment.
Ещё один пример неудачного логирования.
| } | ||
|
|
||
| // Calculate the value of call function(a, b) | ||
| private double performBinaryFunction(String function, double a, double b) throws ParsingException { |
There was a problem hiding this comment.
Хм, а если я добавлю в список базовых функцию от 3 аргументов?
There was a problem hiding this comment.
@ignorer В условиях задания, пока не предусмотрены функции от 3 аргументов.
А если вдруг они и возникнут, то грамматика спокойно обобщается и на них. :)
| .logout().disable() | ||
| .csrf().disable() | ||
| .authorizeRequests() | ||
| .antMatchers("/eval/**").authenticated() |
2 participants
in progress