нужен реинжиниринг короткого кода на java / Java

ReSQL.ru

Мобильная версия Контакт Правила FAQ Помощь

Гость

Войти | Регистрация | Профиль | Очистить

Новые сообщения | Избранное

Форумы | Пользователи | Статистика | Мод. лог | Поиск

Цитировать

Написать

Автор*:

Ввести пароль для входа

Тема*:

Сообщение

Данное сообщение тематическое

Сообщение содержит картинки или видео 18+

Автор:

ВНИМАНИЕ! На данном подфоруме действуют строгие правила. Удостоверьтесь, что ваше сообщение соответствует им!

Форум или тема закрыты для гостей. Необходима авторизация!

Загрузить последнюю сохраненную версию

Вложение:

Вставить как галерею

Максимальный размер вложений: 4,0 МБ, аудио/видео: 8,0 МБ. Картинки большего размера ужимаются, если возможно.

Введите код, изображенный на картинке. Если код нечитаемый, кликните картинку, чтобы загрузить другой вариант.

Отправляя сообщение, я выражаю свое согласие с правилами форума и принимаю пользовательское соглашение.

Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр

Форумы / Java [игнор отключен] [закрыт для гостей] / нужен реинжиниринг короткого кода на java

25 сообщений из 249, страница 8 из 10

все

нужен реинжиниринг короткого кода на java

#40058059

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

Сергей Лалов,
Какое будет сообщение юзверю если путь ошибочный или прав нету или... 1500 случаев?

Код: java

1.
2.
3.
4.
5.
6.
7.

File file = new File(PATH_TO_FOLDER+fileName);
        try {
            if (file.createNewFile()) {
                System.out.println("Файл с именем " + fileName + " ,благополучно создан!");
            } else {
                System.out.println("Файл с таким именем уже существует по указанному пути, мы перезаписали его новыми значениями!");
            }

ЗЫ. Сообщения прикольные)))

...

Рейтинг:

0 / 0

30.03.2021, 07:32

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058060

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

// наконец то добрались до генерации
))))
Вот весь код выше убрать из main функции.

...

Рейтинг:

0 / 0

30.03.2021, 07:40

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058061

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

Сергей Лалов,
Выше было предложение от мембера. Все параметры взять в pojo класс.
И потом по ООП и инкапсуляции, в отдельном пакете провалидировать.
И послать юзвера подальше при ошибке: "попробуйте ка еще разок"))
Имхо

...

Рейтинг:

0 / 0

30.03.2021, 07:44

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058064

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

Про валидацию входных.
Если в ТЗ сказано что конкретизировать ошибку юзверю, то выводить конкретную ошибку.
Если нет, то ограничится Одной фразой:
"параметры не соответствуют формату"
.....
Справка
...
...
...
Так делают большинство утилит в линукс

...

Рейтинг:

0 / 0

30.03.2021, 07:59

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058204

pesets

Гость

PetroNotC Sharp

pesets,
Завтра разберитесь где варнинги в IDE

привет всем.
Сгенерировал варнинги в отдельный файл

...

Рейтинг:

0 / 0

30.03.2021, 13:55

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058213

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

pesets,
Доброе утро.
Сделай НАМ доброе дело.
Сюда скрин варнингов из твоего IDE
Window - show view - marrkers

...

Рейтинг:

0 / 0

30.03.2021, 14:09

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058230

pesets

Гость

PetroNotC Sharp

pesets,
Доброе утро.
Сделай НАМ доброе дело.
Сюда скрин варнингов из твоего IDE
Window - show view - marrkers

оно?

...

Рейтинг:

0 / 0

30.03.2021, 14:37

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058235

pesets

Гость

или это ?

...

Рейтинг:

0 / 0

30.03.2021, 14:46

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058242

mayton

Участник

Откуда: loopback

Сообщения: 53 422

Рейтинг: 2 / 0

Мне кажется топик пошел не туда. За деревьями не видно леса. Нет приоритетов и непонятно вообще где конечная цель.

Так code-review не делается.

Надо что-то менять.

...

Рейтинг:

0 / 0

30.03.2021, 14:54

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058244

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

mayton,
Как это.
У меня иклипс даст варнинг что вызывать статику у экземпляра гупость.
А у тебя кто это говорит? Попугай кеша?

...

Рейтинг:

0 / 0

30.03.2021, 15:00

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058246

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

pesets,
Ждем профи твоего IDE. У меня компактное окно с несколькими ошибками в твоем коде.
Их и надо исправлять.
А потом уже смотрит профи.... За 100 баксов в час
)))

...

Рейтинг:

0 / 0

30.03.2021, 15:02

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058247

pesets

Гость

PetroNotC Sharp

изменил в main
String [] argumentsForVerify = args;
number = verifyArgs(argumentsForVerify,numParams);

по соглашению наименованиям теперь нормально или нет ?

сделал verifyArgs функцией, потому что не смог ранее сделать number видимой в main
почему сделать статическую переменную в main это неправильно ?
пусть неправильно , но почему ?

...

Рейтинг:

0 / 0

30.03.2021, 15:05

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058248

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

mayton,
Где окно ошибок внизу у него?

...

Рейтинг:

0 / 0

30.03.2021, 15:06

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058249

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

pesets

PetroNotC Sharp

изменил в main
String [] argumentsForVerify = args;
number = verifyArgs(argumentsForVerify,numParams);

по соглашению наименованиям теперь нормально или нет ?

первая строка - да. Вторая непонятен возврат..
Имхо

...

Рейтинг:

0 / 0

30.03.2021, 15:08

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058253

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

if (! classVerify.isOK())
exit(-1)

...

Рейтинг:

0 / 0

30.03.2021, 15:11

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058254

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

numParams зачем если массив имеет размер?

...

Рейтинг:

0 / 0

30.03.2021, 15:12

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058256

pesets

Гость

PetroNotC Sharp,

...

Рейтинг:

0 / 0

30.03.2021, 15:13

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058257

mayton

Участник

Откуда: loopback

Сообщения: 53 422

Рейтинг: 2 / 0

PetroNotC Sharp

Вообще не имеет значения что говорит IDE. Сначала должны быть зелеными модульные тесты.
И как-то вручную хотя-бы раз проверить что софт работает.

А то за этими рефакторингами можно безконтрольно просидеть неделю и сломать то что уже работало.
Учитывая нулевую подготовку автора - сломанный код после рефакторинга - это очень даже реально.

...

Рейтинг:

0 / 0

30.03.2021, 15:13

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058258

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

mayton,
Какие тесты если у него первый урок. И по ТЗ они есть?
Говорил работодатель - за написание лишнего на работу не берут.

...

Рейтинг:

0 / 0

30.03.2021, 15:15

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058259

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

mayton,
Как ты ВРУЧНУЮ проверишь варнинги.
Автор errors прошел 5 страниц назад

...

Рейтинг:

0 / 0

30.03.2021, 15:17

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058260

pesets

Гость

PetroNotC Sharp

pesets

пропущено...

изменил в main
String [] argumentsForVerify = args;
number = verifyArgs(argumentsForVerify,numParams);

по соглашению наименованиям теперь нормально или нет ?

первая строка - да. Вторая непонятен возврат..
Имхо

мне нужно как то вернуть из verifyArgs количество операций number , которое содержит 2ой аргумент командной строки, который проверяется в verifyArgs

если объявить static number тогда можно не возвращать, но статик переменную в main жестко забраковали

...

Рейтинг:

0 / 0

30.03.2021, 15:18

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058261

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

pesets

PetroNotC Sharp,

а я тебя хвалил.
Поехали сначала.
Все Errors решай сам.
У тебя работало всё?
Как решишь - приходи.

...

Рейтинг:

0 / 0

30.03.2021, 15:19

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058262

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

pesets

PetroNotC Sharp

пропущено...
первая строка - да. Вторая непонятен возврат..
Имхо

отвечаю.
Отдельный класс ВходныеАргументы осилишь?
Тогда создай и ему отдай все переменные скопом. Он все проверит.

...

Рейтинг:

0 / 0

30.03.2021, 15:21

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058263

mayton

Участник

Откуда: loopback

Сообщения: 53 422

Рейтинг: 2 / 0

PetroNotC Sharp

mayton,
Как ты ВРУЧНУЮ проверишь варнинги.
Автор errors прошел 5 страниц назад

Что он прошел? Он их снова заинтродьюсил.

...

Рейтинг:

0 / 0

30.03.2021, 15:22

| Ответить | Цитировать | Написать

нужен реинжиниринг короткого кода на java

#40058264

PetroNotC Sharp

Участник

Сообщения: 10 098

Рейтинг: 0 / 0

>если объявить static number тогда можно не возвращать, но статик переменную в main жестко забраковали
Выше класс будет тебе давать все что надо. Без статик

...

Рейтинг:

0 / 0

30.03.2021, 15:25

| Ответить | Цитировать | Написать

25 сообщений из 249, страница 8 из 10

все

Форумы / Java [игнор отключен] [закрыт для гостей] / нужен реинжиниринг короткого кода на java

Читали тему (0):

Читали форум (0):

Пользователи онлайн (0):

start [/forum/topic.php?fid=59&msg=40058257&tid=2120494]:	0ms
get settings:	9ms
get forum list:	10ms
check forum access:	3ms
check topic access:	3ms
track hit:	66ms
get topic data:	9ms
get forum data:	2ms
get page messages:	53ms
get tp. blocked users:	1ms
others:	249ms

total:	405ms