Форум программистов, компьютерный форум, киберфорум
Java SE (J2SE)
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.67/3: Рейтинг темы: голосов - 3, средняя оценка - 4.67
9 / 9 / 4
Регистрация: 07.11.2013
Сообщений: 63

Программа работает, но не получается реорганизовать код

01.04.2015, 20:14. Показов 609. Ответов 1
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Собственно говоря, само задание(см. скрин)
Я хочу убрать метод и поле NOK с класса Number, так как нецелесообразно получается, хочу перекинуть это все в класс NumberComparator, но ничего не выходит, ошибки вылетают, вроде, нету такой переменной, хотя я ее объявляю. А также я не совсем понял принцип сортировки здесь, а если я захочу, чтобы не по возрастанию выводились пары, а по убыванию, тогда что нужно будет поменять?
Код:
Главный класс:

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
package MainClass;
 
import java.util.Scanner;
 
import Number.Number;
import Comparator.NumberComparator;
import java.util.ArrayList;
 
public class MainClass {
static int gcd(int x, int y) {
        return (y==0)  ? x : gcd(y, x%y);
    }
    static int nok(int x, int y) {
        return (x*y)/(gcd(x,y));
    }
    public static void main(String[] args) {
                Scanner sc = new Scanner(System.in);
        ArrayList<Number> numbers = new ArrayList<>(); //использование оператора diamond
        for (int i = 0;i<2;i++)
        {
        System.out.println("Ввод первого числа пары <x,y>");
        int x = sc.nextInt();
        System.out.println("Ввод второго числа пары <x,y>");
        int y = sc.nextInt();
 
            //int NOD = gcd(x,y);
        int NOK=nok(x,y);
        numbers.add(new Number(x, y, NOK));
        }
 
        numbers.sort(new NumberComparator());
        numbers.stream().forEach((obj) -> { //использование функциональной операции
            System.out.println(obj);
    });
    }
}
Компаратор:


Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
package Comparator;
 
import java.util.Comparator;
 
import Number.Number;
 
 
public class NumberComparator implements   Comparator<Number>{
   
    public int compare(Number n1, Number n2) {
        int res = new Integer(n1.getNOK()).compareTo(n2.getNOK());
        if (res != 0) {
             return res;
        } else {
            res = new Integer(n1.getX()).compareTo(n2.getX());
            if (res != 0) {
 
                return res;
            } else {
                return new Integer(n1.getY()).compareTo(n2.getY());
            }
 
        }
    }
}
И класс, методы, считывающие числа:

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
package Number;
 
public class Number {
     private final  int x;
    private final  int y;
    private final  int NOK;
 
    public Number(int x, int y, int NOK ) {
        this.x = x;
        this.y = y;
        this.NOK = NOK;
 
    }
    public int getX () {
        return x;
    }
    public int getY () {
        return y;
    }
    public int getNOK () {
        return NOK;
    }
 
    @Override
    public String toString() { 
        return x + " " + y + "  " + NOK;
    }
}
Миниатюры
Программа работает, но не получается реорганизовать код  
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
01.04.2015, 20:14
Ответы с готовыми решениями:

Реорганизовать код: вызовы методов в зависимости от условий
Здравствуйте Опыт в программировании небольшой, код получается достаточно раздутый, написал конструкцию достаточно простую, но кажется...

Программа работает, но получается не тот график
Программа работает, но получается не тот график который должен. Где я накосячил:scratch:? В уравнении или в функциях? Скрин то что должно...

Не получается нарисовать нужную фигуру в C++ Builder. Код написан но работает некорректно
Подскажите, пожалуйста! Пишу лабораторную работу по компьютерной графике. Дано задание написать программу в билдере, где нужно...

1
 Аватар для lemegeton
4903 / 2696 / 921
Регистрация: 29.11.2010
Сообщений: 5,783
02.04.2015, 21:28
Очень даже целесообразно у вас сделано. Иначе нок будет пересчитываться при каждой проверке, что отрицательно скажется на производительности.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
02.04.2015, 21:28
Помогаю со студенческими работами здесь

Программа не работает посмотрите код
Помогите пожалуйста с кодом программы Вычислить сумму 1-1\2+1\3-…+(-1)sqr(n+1)*(1\n) //#include &lt;stdafx.h&gt; #include...

Не работает программа(рекурсия)(код в нутри)
#include &quot;stdafx.h&quot; #include &lt;iostream&gt; using namespace std; char shest; int i; void vvod() {

Программа из методички про наследование - не работает код
В методичке приведён такой код в качестве примера к главе про Наследование. Задание там такое: Но код не работает, и не...

С++ Borland 6.0, написал код, ошибки нет, но программа не работает
в общем надо лабу сделать, уже 2 пары маюсь и не могу понять где шибка код написал, ошибок не пишет, но прога не работает. Задание...

Как записать правильно? Программа работает но код наверно не верен.
#include &lt;iostream&gt; using namespace std; int main() { long sek; cout &lt;&lt; &quot;Enter the number of senonds: &quot;; ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
2
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 12.02.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru