|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
Добрый день. Прошу вас оценить написанную программу. Мой первый опыт программирования на JAVA. Может кому-то пригодится))) https://github.com/KopanitskiyYaroslav/Query_for_Oracle Думаю, что она больше ориентирована на DBA, которые работают со большим списком БД. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 13:03 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
DarkPiligrim, а код где? ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 13:14 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
Tsyklop, прошу прощения, выложу. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 13:29 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
TsyklopDarkPiligrim, а код где? джарник же есть. декомпилируй )) ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 13:35 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
Выкладываю код. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 14:10 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
DarkPiligrim, авторpublic class LoadFile { JFileChooser IpFileChooser = new JFileChooser(); JFileChooser QueryFileChooser = new JFileChooser(); public LoadFile() {} Если не используете конструктор, то писать незачем. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 14:17 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
DarkPiligrim, автор void LoadQuery() { методы с маленькой буквы. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 14:18 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
DarkPiligrim, авторvoid LoadQuery() { QueryFileChooser.setDialogTitle("Select a file with query"); int action_selected = QueryFileChooser.showOpenDialog(null); if (action_selected == 0) { File QueryFile = QueryFileChooser.getSelectedFile(); try { BufferedReader ReaderQueryFile = new BufferedReader(new FileReader(QueryFile)); while (ReaderQueryFile.ready()) { if (MainForm.TextAreaQuery.getText().isEmpty()) { MainForm.TextAreaQuery.append(ReaderQueryFile.readLine()); } else { MainForm.TextAreaQuery.append("\n" + ReaderQueryFile.readLine()); } } } catch (Exception ex) { MainForm.TextAreaQuery.append("Query not loaded!"); } } } BufferedReader имплементирует Closeable - надо вызывать close() ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 14:21 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
mr_virtus, почему с маленькой? ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 14:25 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
mr_virtusDarkPiligrim, автор void LoadQuery() { методы с маленькой буквы. Почему с маленькой? ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 14:26 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
DarkPiligrim, авторПочему с маленькой? Потому что в документации так написано: автор https://www.oracle.com/technetwork/java/codeconventions-135099.html ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 14:28 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
mr_virtus, спасибо вам за ваши замечания! ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 14:37 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
DarkPiligrim, авторpublic class BuildStrIp { public BuildStrIp() {} void Run() { String[] StrIp = MainForm.TextAreaIp.getText().split("\n"); MainForm.TextAreaIp.setText(""); int CountStrTextAreaIp = 0; while (CountStrTextAreaIp < StrIp.length) { if (!StrIp[CountStrTextAreaIp].isEmpty()) { if (MainForm.TextAreaIp.getText().isEmpty()) { MainForm.TextAreaIp.append(StrIp[CountStrTextAreaIp]); } else { MainForm.TextAreaIp.append("\n" + StrIp[CountStrTextAreaIp]); } } CountStrTextAreaIp++; } } } Сделайте метод static - вы не используете члены класса. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 14:44 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
DarkPiligrimВыкладываю код. а на гит выложить код нет? репозитории для этого и предназначены. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 15:43 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
DarkPiligrim, я бы вот глянул код. но качать архив не буду. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 15:43 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
Непонятно что требуется оценивать? Качество кода? Это одно. Пользу использования этого приложения? Кто это оценит? Кто здесь в топике Oracle DBA? Что делает это приложение? Или нам предлагается в режиме реверс-инженеринга поиграть в угадайку? ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 15:50 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
Всем спасибо за помощь. В следующий раз буду более точно формулировать вопрос. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 17:55 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
DarkPiligrimВыкладываю код. Все напутал. Код надо в гит. Это - важно. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.04.2019, 22:25 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
mr_virtusDarkPiligrim, авторvoid LoadQuery() { QueryFileChooser.setDialogTitle("Select a file with query"); int action_selected = QueryFileChooser.showOpenDialog(null); if (action_selected == 0) { File QueryFile = QueryFileChooser.getSelectedFile(); try { BufferedReader ReaderQueryFile = new BufferedReader(new FileReader(QueryFile)); while (ReaderQueryFile.ready()) { if (MainForm.TextAreaQuery.getText().isEmpty()) { MainForm.TextAreaQuery.append(ReaderQueryFile.readLine()); } else { MainForm.TextAreaQuery.append("\n" + ReaderQueryFile.readLine()); } } } catch (Exception ex) { MainForm.TextAreaQuery.append("Query not loaded!"); } } } BufferedReader имплементирует Closeable - надо вызывать close() Можно просто try catch resource написать ,метод close вызывать необязательно ... |
|||
:
Нравится:
Не нравится:
|
|||
10.04.2019, 09:50 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
- Первым делом ябы пофиксил package names. Дефолтный пакет это не круто. - Потом ябы смигрировал проект на maven/gradle. Там надо чуть файловую структуру подфиксить. - jdbc-драйвер для оракла - это отдельная попо-боль. Я его обычно качал с сайта производителя потом ставил в свой локальный реп через mvn ... и писал инструкцию в тексте помника как использовать. возможно сейчас он уже внесен в стандартные репки. Я имею в виду восьмерку. jdbc8*.jar ... |
|||
:
Нравится:
Не нравится:
|
|||
10.04.2019, 12:41 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
mr_virtusDarkPiligrim,... Надо не в квотах исходники вставлять, а в теге src. ... |
|||
:
Нравится:
Не нравится:
|
|||
10.04.2019, 13:17 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
TsyklopDarkPiligrim, я бы вот глянул код. но качать архив не буду. Так не качай, кому нужен твой неопытный взгляд? ... |
|||
:
Нравится:
Не нравится:
|
|||
10.04.2019, 13:18 |
|
Прошу оценить программу
|
|||
---|---|---|---|
#18+
alex55555Так не качай, кому нужен твой неопытный взгляд? откуда Вы взяли что у меня нет опыта? м? всезнающий лять... умник тоже мне. ... |
|||
:
Нравится:
Не нравится:
|
|||
10.04.2019, 13:59 |
|
|
start [/forum/topic.php?fid=59&fpage=29&tid=2121378]: |
0ms |
get settings: |
10ms |
get forum list: |
13ms |
check forum access: |
3ms |
check topic access: |
3ms |
track hit: |
82ms |
get topic data: |
13ms |
get forum data: |
3ms |
get page messages: |
66ms |
get tp. blocked users: |
2ms |
others: | 235ms |
total: | 430ms |
0 / 0 |