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

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

public LoadFile() {}

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

автор void LoadQuery()
{

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

автор void LoadQuery()
{

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

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

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


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

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

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

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

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


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