Форум программистов, компьютерный форум, киберфорум
Java
Форум программистов и сисадминов Киберфорум > Форум программистов > Java
Войти
Регистрация
Восстановить пароль
Блоги Социальные группы Поиск Сообщения за день Заказать работу  
 
Рейтинг 4.78/9: Рейтинг темы: голосов - 9, средняя оценка - 4.78
169 / 66 / 15
Регистрация: 24.03.2013
Сообщений: 467
Записей в блоге: 1

Почему это говнокод?!

03.09.2013, 20:09. Показов 1989. Ответов 15
Метки нет (Все метки)
Студворк — интернет-сервис помощи студентам

Есть задачка, очень простенькая, но код ее, на мой взгляд, ужасен, что можно исправить здесь?

Метод, ну очень раздражает как написан.
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

private static boolean contains(Point p){
        if(p.getX() > 1 && p.getX() <= 2)
            if(p.getY() > 1 && p.getY() <= 2)
                return true;
        
        if(p.getX() > 2 && p.getX() <= 3)
            if(p.getY() > 2 && p.getY() <= 3)
                return true;
        
        if(p.getX() > 3 && p.getX() <= 4)
            if(p.getY() > 3 && p.getY() <= 4)
                return true;
        
        if(p.getX() > 4 && p.getX() <= 5)
            if(p.getY() > 4 && p.getY() <= 5)
                return true;
        
        if(p.getX() > 5 && p.getX() <= 6)
            if(p.getY() > 5 && p.getY() <= 6)
                return true;
        
        if(p.getX() > 6 && p.getX() <= 7)
            if(p.getY() > 6 && p.getY() <= 7)
                return true;
        
        if(p.getX() > 7 && p.getX() <= 8)
            if(p.getY() > 7 && p.getY() <= 8)
                return true;
        
        return false;
    }
Вот весь код
Кликните здесь для просмотра всего текста

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109

package ru.junior_progger;
 
import java.util.Scanner;
 
/**
 *  Задание 3
 *  Напишите функцию, которая определяет, попадает ли точка 
 *  с вещественными координатами X,Y в заданную область. 
 *  Область показана на рисунке ниже серым цветом.
 *
 */
 
public class Example_3 {
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        Point point = getPoint(sc);
        
        System.out.println("Данная точка с координатами:" +
                "\nx = " + point.getX() + "\ny = " + point.getY());
        
        if(contains(point)){
            System.out.println("попадает в заданную область");
        } else {
            System.out.println("не попадает в заданную область");
        }
    }
 
    private static boolean contains(Point p){
        if(p.getX() > 1 && p.getX() <= 2)
            if(p.getY() > 1 && p.getY() <= 2)
                return true;
        
        if(p.getX() > 2 && p.getX() <= 3)
            if(p.getY() > 2 && p.getY() <= 3)
                return true;
        
        if(p.getX() > 3 && p.getX() <= 4)
            if(p.getY() > 3 && p.getY() <= 4)
                return true;
        
        if(p.getX() > 4 && p.getX() <= 5)
            if(p.getY() > 4 && p.getY() <= 5)
                return true;
        
        if(p.getX() > 5 && p.getX() <= 6)
            if(p.getY() > 5 && p.getY() <= 6)
                return true;
        
        if(p.getX() > 6 && p.getX() <= 7)
            if(p.getY() > 6 && p.getY() <= 7)
                return true;
        
        if(p.getX() > 7 && p.getX() <= 8)
            if(p.getY() > 7 && p.getY() <= 8)
                return true;
        
        return false;
    }
    
    private static Point getPoint(Scanner sc) {
        System.out.print("Введите x = ");
        double x = getDouble(sc);
    
        System.out.print("Введите y = ");
        double y = getDouble(sc);
    
        return new Point(x, y);
    }
 
    private static double getDouble(Scanner sc) {
        while(true){
            try{
                double result = Double.parseDouble(sc.nextLine());
                return result;
            }  catch(Exception e){
                System.out.println(e.getMessage());
                System.out.print("Произошла ошибка преобразования, " +
                        "попробуйте ввести снова\n->  ");
            }  
        }
    }
 
 
    static class Point{
        private double x;
        private double y;
        
        public Point(double x, double y){
            this.x = x;
            this.y = y;
        }
 
        public double getX() {
            return x;
        }
 
        public void setX(double x) {
            this.x = x;
        }
 
        public double getY() {
            return y;
        }
 
        public void setY(double y) {
            this.y = y;
        }
    }
}

0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
 03.09.2013, 20:09
Ответы с готовыми решениями:

Не надо писать говнокод
В общем решаю я задачки, пытаюсь импрувить свои скилы и тут я вижу что такого наворотил, подскажите нормальный код или я индус? Суть...

Это говнокод или так и делают ?
Пишу программу в которой пользователь должен угадывать столицы стран мира в Windows Form. И вот у меня вопрос: мне теперь каждую страну и...

Почему это так и как это исправить?
Почему если в паскале написать это: program test; type Operators = ( _if_,

15
 Аватар для name?
201 / 172 / 52
Регистрация: 01.06.2010
Сообщений: 371
03.09.2013, 20:17
Java
1
2
3
4
5
6
7
8
9

    private static boolean contains(Point p) {
 
        for (int index = 1; index < 8; index++)
            if (p.getX() > index && p.getX() <= index +1)
                if (p.getY() > index && p.getY() <= index +1)
                    return true;
 
        return false;
    }
0
2838 / 1647 / 254
Регистрация: 03.12.2007
Сообщений: 4,222
03.09.2013, 21:03
По-моему, этот код ещё и неправильный. (x; y) = (6; 2), например.

Добавлено через 3 минуты
Java
1

return x >= 1 && x <= 8 && Math.ceil(x) <= y;
так вроде.
0
 Аватар для mutagen
2587 / 2260 / 257
Регистрация: 14.09.2011
Сообщений: 5,185
Записей в блоге: 18
03.09.2013, 21:13
Эх ребята ...
допустим код в говнокодовой версии выдаёт правильный результат - тогда его правильный аналог будет таким:
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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75

import java.awt.Point;
 
/**
 * @author mutagen
 */
public class ContainsGovnokods {
 
    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        for (int i = 0; i <= 8; i++) {
            for (int j = 0; j <= 8; j++) {
                Point p = new Point(i, j);
                boolean govno = containsGovno(p);
                boolean well = containsWell(p);
                if (well != govno) {
                    throw new RuntimeException("нет в жизни кода без багов )))");
                }
                System.out.println("govno = " + govno);
                System.out.println("well = " + well + System.getProperty("line.separator"));
            }
        }
    }
 
    static boolean containsGovno(Point p) {
        if (p.getX() > 1 && p.getX() <= 2) {
            if (p.getY() > 1 && p.getY() <= 2) {
                return true;
            }
        }
 
        if (p.getX() > 2 && p.getX() <= 3) {
            if (p.getY() > 2 && p.getY() <= 3) {
                return true;
            }
        }
 
        if (p.getX() > 3 && p.getX() <= 4) {
            if (p.getY() > 3 && p.getY() <= 4) {
                return true;
            }
        }
 
        if (p.getX() > 4 && p.getX() <= 5) {
            if (p.getY() > 4 && p.getY() <= 5) {
                return true;
            }
        }
 
        if (p.getX() > 5 && p.getX() <= 6) {
            if (p.getY() > 5 && p.getY() <= 6) {
                return true;
            }
        }
 
        if (p.getX() > 6 && p.getX() <= 7) {
            if (p.getY() > 6 && p.getY() <= 7) {
                return true;
            }
        }
 
        if (p.getX() > 7 && p.getX() <= 8) {
            if (p.getY() > 7 && p.getY() <= 8) {
                return true;
            }
        }
 
        return false;
    }
 
    static boolean containsWell(Point p) {
        return (p.x == p.y && p.x > 1 && p.x <= 8);
    }
}
)))
0
169 / 66 / 15
Регистрация: 24.03.2013
Сообщений: 467
Записей в блоге: 1
03.09.2013, 21:22  [ТС]
mutagen, абсолютно несогласен.
Java
1
2
3

 static boolean containsWell(Point p) {
        return (p.x == p.y && p.x > 1 && p.x <= 8);
    }
- это не есть аналог моего говнокода.

Допустим, если взять точку (5,4; 5,5) - совсем разные результаты.
0
 Аватар для mutagen
2587 / 2260 / 257
Регистрация: 14.09.2011
Сообщений: 5,185
Записей в блоге: 18
03.09.2013, 21:31
Цитата Сообщение от Almaz_1993 Посмотреть сообщение
mutagen, абсолютно несогласен.
а вы видели тест в main, там все варианты между 1 и 8 перебираются и ни на одном результаты не расходятся, такшо нефиг )))

Не по теме:

прежде чем чёнибуть утверждать - хорошо бы убедиться при помощи проверки

0
169 / 66 / 15
Регистрация: 24.03.2013
Сообщений: 467
Записей в блоге: 1
03.09.2013, 21:32  [ТС]
Пока остановился на этом, но всеравно чувствуется запах говна=))

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

private static boolean contains(Point p){
        if (p.getX() >= 1 && p.getY() >= 1) {
            if (p.getX() >= 1 && p.getX() < 2 && p.getY() <= 2)
                return true;
 
            if (p.getX() >= 2 && p.getX() < 3 && p.getY() <= 3)
                return true;
 
            if (p.getX() >= 3 && p.getX() < 4 && p.getY() <= 4)
                return true;
 
            if (p.getX() >= 4 && p.getX() < 5 && p.getY() <= 5)
                return true;
 
            if (p.getX() >= 5 && p.getX() < 6 && p.getY() <= 6)
                return true;
 
            if (p.getX() >= 6 && p.getX() < 7 && p.getY() <= 7)
                return true;
 
            if (p.getX() >= 7 && p.getX() <= 8 && p.getY() <= 8)
                return true;
        }
        return false;
    }
0
 Аватар для mutagen
2587 / 2260 / 257
Регистрация: 14.09.2011
Сообщений: 5,185
Записей в блоге: 18
03.09.2013, 21:36
Цитата Сообщение от Almaz_1993 Посмотреть сообщение
Допустим, если взять точку (5,4; 5,5) - совсем разные результаты.
почитайте документацию http://docs.oracle.com/javase/... Point.html
там про doube - ни слова, всё в int

Добавлено через 2 минуты
Цитата Сообщение от Almaz_1993 Посмотреть сообщение
всеравно чувствуется запах говна=))
запах чувствуется когда выполняется операция сравнения int c double
0
38 / 38 / 11
Регистрация: 21.05.2013
Сообщений: 128
03.09.2013, 21:47
Almaz_1993, вам же написали выше
Java
1

return (p.getX() > 1 && p.getX() < 8 && p.getY() > 1 && Math.ceil(p.getX()) < p.getY());
0
169 / 66 / 15
Регистрация: 24.03.2013
Сообщений: 467
Записей в блоге: 1
03.09.2013, 22:03  [ТС]
MaxNevermind, у вас не выполняется условия задачи.
0
 Аватар для verylazy
462 / 462 / 71
Регистрация: 26.02.2013
Сообщений: 1,263
03.09.2013, 22:33
Цитата Сообщение от Almaz_1993 Посмотреть сообщение
Пока остановился на этом, но всеравно чувствуется запах говна=))

Кликните здесь для просмотра всего текста
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

private static boolean contains(Point p){
        if (p.getX() >= 1 && p.getY() >= 1) {
            if (p.getX() >= 1 && p.getX() < 2 && p.getY() <= 2)
                return true;
 
            if (p.getX() >= 2 && p.getX() < 3 && p.getY() <= 3)
                return true;
 
            if (p.getX() >= 3 && p.getX() < 4 && p.getY() <= 4)
                return true;
 
            if (p.getX() >= 4 && p.getX() < 5 && p.getY() <= 5)
                return true;
 
            if (p.getX() >= 5 && p.getX() < 6 && p.getY() <= 6)
                return true;
 
            if (p.getX() >= 6 && p.getX() < 7 && p.getY() <= 7)
                return true;
 
            if (p.getX() >= 7 && p.getX() <= 8 && p.getY() <= 8)
                return true;
        }
        return false;
    }
там же все время true будет возвращаться если если X Y от 1 до 8 включительно
или это говнокод в квадрате или я чего-то недопонял
0
 Аватар для mutagen
2587 / 2260 / 257
Регистрация: 14.09.2011
Сообщений: 5,185
Записей в блоге: 18
03.09.2013, 22:53
Цитата Сообщение от Almaz_1993 Посмотреть сообщение
MaxNevermind, у вас не выполняется условия задачи.
Ну допустим, но у меня же всё выполняется(даже тесты прилагаются - я прямо как чувствовал что будут попытки голословно чёнить утверждать), но вы всё равно несогласны )))
Или вы уже довольны своей версией? )))

у меня к вам встречный вопрос

Java
1

p.getX() >= 4 && p.getX() < 5
эквивалентно
Java
1

p.getX() == 4
или нет ???? ))))
1
04.09.2013, 16:31

Не по теме:

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

0
2838 / 1647 / 254
Регистрация: 03.12.2007
Сообщений: 4,222
04.09.2013, 17:20
Цитата Сообщение от Gibby Посмотреть сообщение
Попробовал сделать проверку на принадлежность точки к произвольному многоугольнику
Если сумма углов - 0, то снаружи, если 360°, то внутри, если 180° - на границе.
Миниатюры
Почему это говнокод?!  
0
 Аватар для Skipy
2000 / 1427 / 92
Регистрация: 25.11.2010
Сообщений: 3,611
04.09.2013, 17:26
Цитата Сообщение от Gibby Посмотреть сообщение
Попробовал сделать проверку на принадлежность точки к произвольному многоугольнику.
Чертова линейная алгебра.
Пока сдался.
А что там сложного? Берете произвольную точку многоугольника. Начало координат помещаете в Вашу точку, которую проверяете. И потом начинаете перебирать по очереди все точки многоугольника, начиная с выбранной, вычисляя угол между направлением на соседние (через скалярное и векторное произведение однозначно определите угол). Углы суммируете. Если при возвращении в исходную точку сумма получилась 360 градусов - точка внутри. Если 0 - снаружи. Если точка на ребре - будет неопределенность, но это легко вычисляется.
0
 Аватар для Gibby
154 / 154 / 10
Регистрация: 16.10.2012
Сообщений: 354
Записей в блоге: 1
04.09.2013, 17:37
Понятно. Спасибо за науку
Я пошел своим путем, который лежал через триангуляцию многоугольника и опеределение лежит ли точка в каком-нибудь из треугольников. Со вторым я справился через переход к аффинным координатам, первое забороть не смог.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
 04.09.2013, 17:37
Помогаю со студенческими работами здесь

Скажите почему например флешка на 4 ГБ а на ней ёмкость всего 3.73 ГБ почему это так?
Скажите почему например флешка на 4 ГБ а на ней ёмкость всего 3.73 ГБ почему это так? Почему не сделают ровно 4 ГБ ну или приближённо к...

Быдлокодер. Что это и почему это?
Ребята, мне нравится этот форум и я пользуюсь им, что бы научиться чему то, а по возможности других научить. Но это выражение меня уже...

То ли баг, то ли я говнокод написал
Попросил знакомый написать прогу. Тема заезженная, про ряд Тейлора. Дана функция: y=\left(1 + 2{x}^{2} \right){e}^{{x}^{2}} Промежуток:...

Что значит говнокод в php?
Очень часто слышу, что пхп рождает говнокодеров. И что на самом деле на нем можно хорошо писать, но 90% пишут ужасный код и.т.д. Что значит...

Как правильно писать говнокод?
теорию я знаю, а практики маловато( поделитесь опытом


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

Или воспользуйтесь поиском по форуму:
16
Ответ Создать тему
Опции темы

Новые блоги и статьи
Символьное дифференцирование
igorrr37 13.02.2026
/ * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .
Камера 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. Пошагово создадим проект для загрузки изображения. . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image
8Observer8 10.02.2026
Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .
Установка Qt-версии Lazarus IDE в Debian Trixie Xfce
volvo 10.02.2026
В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Поддержать - Реклама - Условия использования - Обратная связь
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru