Новые сообщения [новые:0]
Дайджест
Горячие темы
Избранное [новые:0]
Форумы
Пользователи
Статистика
Статистика нагрузки
Мод. лог
Поиск
|
09.04.2019, 13:03
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
Добрый день. Прошу вас оценить написанную программу. Мой первый опыт программирования на JAVA. Может кому-то пригодится))) https://github.com/KopanitskiyYaroslav/Query_for_Oracle Думаю, что она больше ориентирована на DBA, которые работают со большим списком БД. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 13:14
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
DarkPiligrim, а код где? ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 13:29
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
Tsyklop, прошу прощения, выложу. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 13:35
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#18+
TsyklopDarkPiligrim, а код где? джарник же есть. декомпилируй )) ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 14:10
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
Выкладываю код. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 14:17
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#18+
DarkPiligrim, авторpublic class LoadFile { JFileChooser IpFileChooser = new JFileChooser(); JFileChooser QueryFileChooser = new JFileChooser(); public LoadFile() {} Если не используете конструктор, то писать незачем. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 14:18
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#18+
DarkPiligrim, автор void LoadQuery() { методы с маленькой буквы. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 14:21
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#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:25
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
mr_virtus, почему с маленькой? ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 14:26
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
mr_virtusDarkPiligrim, автор void LoadQuery() { методы с маленькой буквы. Почему с маленькой? ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 14:28
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#18+
DarkPiligrim, авторПочему с маленькой? Потому что в документации так написано: автор https://www.oracle.com/technetwork/java/codeconventions-135099.html ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 14:37
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
mr_virtus, спасибо вам за ваши замечания! ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 14:44
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#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, 15:43
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
DarkPiligrimВыкладываю код. а на гит выложить код нет? репозитории для этого и предназначены. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 15:43
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
DarkPiligrim, я бы вот глянул код. но качать архив не буду. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 15:50
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#18+
Непонятно что требуется оценивать? Качество кода? Это одно. Пользу использования этого приложения? Кто это оценит? Кто здесь в топике Oracle DBA? Что делает это приложение? Или нам предлагается в режиме реверс-инженеринга поиграть в угадайку? ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 17:55
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
Всем спасибо за помощь. В следующий раз буду более точно формулировать вопрос. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
09.04.2019, 22:25
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#18+
DarkPiligrimВыкладываю код. Все напутал. Код надо в гит. Это - важно. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
10.04.2019, 09:50
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#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, 12:41
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#18+
- Первым делом ябы пофиксил package names. Дефолтный пакет это не круто. - Потом ябы смигрировал проект на maven/gradle. Там надо чуть файловую структуру подфиксить. - jdbc-драйвер для оракла - это отдельная попо-боль. Я его обычно качал с сайта производителя потом ставил в свой локальный реп через mvn ... и писал инструкцию в тексте помника как использовать. возможно сейчас он уже внесен в стандартные репки. Я имею в виду восьмерку. jdbc8*.jar ... |
|||
:
Нравится:
Не нравится:
|
|||
|
10.04.2019, 13:17
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#18+
mr_virtusDarkPiligrim,... Надо не в квотах исходники вставлять, а в теге src. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
10.04.2019, 13:18
|
|||
---|---|---|---|
Прошу оценить программу |
|||
#18+
TsyklopDarkPiligrim, я бы вот глянул код. но качать архив не буду. Так не качай, кому нужен твой неопытный взгляд? ... |
|||
:
Нравится:
Не нравится:
|
|||
|
10.04.2019, 13:59
|
|||
---|---|---|---|
|
|||
Прошу оценить программу |
|||
#18+
alex55555Так не качай, кому нужен твой неопытный взгляд? откуда Вы взяли что у меня нет опыта? м? всезнающий лять... умник тоже мне. ... |
|||
:
Нравится:
Не нравится:
|
|||
|
|
start [/forum/moderation_log.php?user_name=%D0%9B%D1%8E%D0%B1%D0%BE%D0%BF%D1%8B%D1%82%D0%BD%D1%8B%D0%B9+%D0%B3%D0%BE%D1%81%D1%82%D1%8C]: |
0ms |
get settings: |
9ms |
get forum list: |
13ms |
get settings: |
10ms |
get forum list: |
16ms |
check forum access: |
4ms |
check topic access: |
4ms |
track hit: |
152ms |
get topic data: |
12ms |
get forum data: |
3ms |
get page messages: |
57ms |
get tp. blocked users: |
2ms |
others: | 633ms |
total: | 915ms |
0 / 0 |