Гость
Форумы / Java [игнор отключен] [закрыт для гостей] / Прошу оценить программу / 24 сообщений из 24, страница 1 из 1
09.04.2019, 13:03
    #39798702
DarkPiligrim
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
Добрый день. Прошу вас оценить написанную программу. Мой первый опыт программирования на JAVA. Может кому-то пригодится)))
https://github.com/KopanitskiyYaroslav/Query_for_Oracle
Думаю, что она больше ориентирована на DBA, которые работают со большим списком БД.
...
Рейтинг: 0 / 0
09.04.2019, 13:14
    #39798717
Tsyklop
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
DarkPiligrim, а код где?
...
Рейтинг: 0 / 0
09.04.2019, 13:29
    #39798732
DarkPiligrim
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
Tsyklop, прошу прощения, выложу.
...
Рейтинг: 0 / 0
09.04.2019, 13:35
    #39798739
andreykaT
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
TsyklopDarkPiligrim, а код где?
джарник же есть. декомпилируй ))
...
Рейтинг: 0 / 0
09.04.2019, 14:10
    #39798767
DarkPiligrim
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
Выкладываю код.
...
Рейтинг: 0 / 0
09.04.2019, 14:17
    #39798770
mr_virtus
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
DarkPiligrim,

авторpublic class LoadFile
{
JFileChooser IpFileChooser = new JFileChooser();
JFileChooser QueryFileChooser = new JFileChooser();

public LoadFile() {}

Если не используете конструктор, то писать незачем.
...
Рейтинг: 0 / 0
09.04.2019, 14:18
    #39798773
mr_virtus
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
DarkPiligrim,

автор void LoadQuery()
{

методы с маленькой буквы.
...
Рейтинг: 0 / 0
09.04.2019, 14:21
    #39798775
mr_virtus
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
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()
...
Рейтинг: 0 / 0
09.04.2019, 14:25
    #39798779
DarkPiligrim
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
mr_virtus, почему с маленькой?
...
Рейтинг: 0 / 0
09.04.2019, 14:26
    #39798780
DarkPiligrim
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
mr_virtusDarkPiligrim,

автор void LoadQuery()
{

методы с маленькой буквы.

Почему с маленькой?
...
Рейтинг: 0 / 0
09.04.2019, 14:28
    #39798786
mr_virtus
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
DarkPiligrim,

авторПочему с маленькой?


Потому что в документации так написано:

автор https://www.oracle.com/technetwork/java/codeconventions-135099.html
...
Рейтинг: 0 / 0
09.04.2019, 14:37
    #39798791
DarkPiligrim
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
mr_virtus, спасибо вам за ваши замечания!
...
Рейтинг: 0 / 0
09.04.2019, 14:44
    #39798794
mr_virtus
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
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 - вы не используете члены класса.
...
Рейтинг: 0 / 0
09.04.2019, 15:43
    #39798844
Tsyklop
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
DarkPiligrimВыкладываю код.

а на гит выложить код нет? репозитории для этого и предназначены.
...
Рейтинг: 0 / 0
09.04.2019, 15:43
    #39798845
Tsyklop
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
DarkPiligrim, я бы вот глянул код. но качать архив не буду.
...
Рейтинг: 0 / 0
09.04.2019, 15:50
    #39798854
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
Непонятно что требуется оценивать? Качество кода? Это одно.

Пользу использования этого приложения? Кто это оценит? Кто здесь в топике Oracle DBA?
Что делает это приложение? Или нам предлагается в режиме реверс-инженеринга
поиграть в угадайку?
...
Рейтинг: 0 / 0
09.04.2019, 17:55
    #39798976
DarkPiligrim
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
Всем спасибо за помощь. В следующий раз буду более точно формулировать вопрос.
...
Рейтинг: 0 / 0
09.04.2019, 22:25
    #39799046
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
DarkPiligrimВыкладываю код.
Все напутал. Код надо в гит. Это - важно.
...
Рейтинг: 0 / 0
10.04.2019, 09:50
    #39799176
asv79
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
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 вызывать необязательно
...
Рейтинг: 0 / 0
10.04.2019, 12:41
    #39799267
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
- Первым делом ябы пофиксил package names. Дефолтный пакет это не круто.
- Потом ябы смигрировал проект на maven/gradle. Там надо чуть файловую структуру подфиксить.
- jdbc-драйвер для оракла - это отдельная попо-боль. Я его обычно качал с сайта производителя
потом ставил в свой локальный реп через mvn ... и писал инструкцию в тексте помника как использовать.
возможно сейчас он уже внесен в стандартные репки. Я имею в виду восьмерку. jdbc8*.jar
...
Рейтинг: 0 / 0
10.04.2019, 13:17
    #39799299
alex55555
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
mr_virtusDarkPiligrim,...
Надо не в квотах исходники вставлять, а в теге src.
...
Рейтинг: 0 / 0
10.04.2019, 13:18
    #39799302
alex55555
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
TsyklopDarkPiligrim, я бы вот глянул код. но качать архив не буду.
Так не качай, кому нужен твой неопытный взгляд?
...
Рейтинг: 0 / 0
10.04.2019, 13:59
    #39799342
Tsyklop
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
alex55555Так не качай, кому нужен твой неопытный взгляд?

откуда Вы взяли что у меня нет опыта? м? всезнающий лять... умник тоже мне.
...
Рейтинг: 0 / 0
10.04.2019, 14:00
    #39799344
Tsyklop
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Прошу оценить программу
alex55555, Как же бесят такие. трепло.
...
Рейтинг: 0 / 0
Форумы / Java [игнор отключен] [закрыт для гостей] / Прошу оценить программу / 24 сообщений из 24, страница 1 из 1
Целевая тема:
Создать новую тему:
Автор:
Найденые пользователи ...
Разблокировать пользователей ...
Читали форум (0):
Пользователи онлайн (0):
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]