Skip to content
This repository has been archived by the owner on Sep 27, 2022. It is now read-only.

Кудрявцев Дмитрий 396гр Parallel and Proxy (перезаливаю) #571

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Kud8
Copy link
Contributor

@Kud8 Kud8 commented Dec 21, 2014

No description provided.

@vpavlenko vpavlenko self-assigned this Dec 21, 2014
@Erzed
Copy link
Contributor

Erzed commented Dec 21, 2014

StoreableTableProvider.java:

final-переменные должны быть в верхнем регистре

21    private final String signatureFileName = "signature.tsv";

Разве не удобнее держать это как Path? И путь потом можно не через dbdir+file.separator+..., а просто dbdir.resolve()

25     private String dbDirectory

@Erzed
Copy link
Contributor

Erzed commented Dec 21, 2014

Так же и в StoreableTable

    private final String formatOfDirectory = ".dir";
    private final String formatOfFile = ".dat";
    private final String encoding = "UTF-8";
    private final String signatureFileName = "signature.tsv";

ps signatureFileName повторяется;)

@Kud8
Copy link
Contributor Author

Kud8 commented Dec 21, 2014

Так вроде же static final должны такие быть, тем более, что часто пишут переменные, а потом думают, ставить ли final, поэтому неудобно было бы их всех переименовывать.

@penguin138
Copy link

в loggingProxy:

 if (res == null) {
     jsonObj.put("returnValue", JSONObject.NULL);
 } else {
     jsonObj.put("returnValue", res);
  }

Тут разве не получится так, что если метод возвращал void, то res будет null, т.к. он получен вот так:

res = method.invoke(obj, args);

и returnValue будет null?

@Kud8
Copy link
Contributor Author

Kud8 commented Dec 21, 2014

Насчёт signatureFileName: я его использую в цикле тут:
curDir.getCanonicalPath() + File.separator + signatureFileName
и тут:
tableDirectory + File.separator + signatureFileName
Да и у всех своё местоположение файла сигнатуры, поэтому храню просто как String

@penguin138
Copy link

Да, кстати, в тестах можно просто сделать import static org.junit.Assert.*; и писать просто assertEquals(...)

@penguin138
Copy link

и я проверяю все-таки proxy

@Kud8
Copy link
Contributor Author

Kud8 commented Dec 21, 2014

Да, просто у меня аватарка не обновилась и я думал, что это Радим написал)

@Kud8
Copy link
Contributor Author

Kud8 commented Dec 21, 2014

Насчёт void ты права.

@penguin138
Copy link

Больше вопросов к Proxy нет.Можешь сдавать.

@Erzed
Copy link
Contributor

Erzed commented Dec 21, 2014

+paralllel

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants