Форум программистов, компьютерный форум, киберфорум
Java SE (J2SE)
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.88/8: Рейтинг темы: голосов - 8, средняя оценка - 4.88
1 / 1 / 0
Регистрация: 16.08.2017
Сообщений: 44

Не надо писать говнокод

16.11.2018, 02:09. Показов 1770. Ответов 7

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

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
public static String showSequence(int value) {
 
        StringBuffer buffer = new StringBuffer();
        
        int count = 0;
        String string = "";
 
        if (value < 0) return string = buffer.append(value).toString() + " < 0";
        else if (value == 0) return "0 = 0";
 
        for (int i = 0; i <= value; i++) {
            count += i;
            string = buffer.append("+").append(i).toString();
 
        } return string = buffer.deleteCharAt(0).toString() + " = " + count;
    }
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
16.11.2018, 02:09
Ответы с готовыми решениями:

Почему это говнокод?!
:-| Есть задачка, очень простенькая, но код ее, на мой взгляд, ужасен, что можно исправить здесь? Метод, ну очень раздражает как...

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

Что такое говнокод, и как перестать его писать?)
Может вопрос покажется и глупым, но думаю многим хотелось бы получить на него ответ)

7
528 / 263 / 70
Регистрация: 11.12.2016
Сообщений: 1,223
16.11.2018, 02:34
Цитата Сообщение от JuniorDp Посмотреть сообщение
подскажите нормальный код или я индус?
Плохой.
Цитата Сообщение от JuniorDp Посмотреть сообщение
считаем суму
сумма чего?
Цитата Сообщение от JuniorDp Посмотреть сообщение
записываем решение в строку + результат - выводим
какое решение, какой результат, тут вообще ничего не выводится.

Напишите что именно вам надо и думаю что вам помогут.
1
1 / 1 / 0
Регистрация: 16.08.2017
Сообщений: 44
16.11.2018, 02:37  [ТС]
Description:
We want to generate a function that computes the series starting from 0 and ending until the given number following the sequence:
0 1 3 6 10 15 21 28 36 45 55 ....
which is created by
0, 0+1, 0+1+2, 0+1+2+3, 0+1+2+3+4, 0+1+2+3+4+5, 0+1+2+3+4+5+6, 0+1+2+3+4+5+6+7 etc..

Input: LastNumber
Output: series and result

Example:

Input: 6
Output: 0+1+2+3+4+5+6 = 21

Input: -15
Output: -15 < 0

Input: > 0
Output: 0 = 0

Добавлено через 1 минуту
Тесты проходит, результат правленый, но вот реализация мне не нравится
0
528 / 263 / 70
Регистрация: 11.12.2016
Сообщений: 1,223
16.11.2018, 02:38
JuniorDp,
Зачем вы используете StringBuffer (уж лучше StringBuilder, но не суть) и в тоже время все время "перезаписываете" строку?
1
1 / 1 / 0
Регистрация: 16.08.2017
Сообщений: 44
16.11.2018, 02:44  [ТС]
Суть понял, сейчас попробую сам исправить, если что сюда еще напишу, спасибо)

Добавлено через 4 минуты
Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
public static String showSequence(int value) {
 
        StringBuilder builder = new StringBuilder();
 
        int count = 0;
 
        if (value < 0) return builder.append(value).toString() + " < 0";
        else if (value == 0) return "0 = 0";
 
        for (int i = 0; i <= value; i++) {
            count += i;
            builder.append("+").append(i).toString();
 
        } return builder.deleteCharAt(0).toString() + " = " + count;
    }
0
Эксперт Java
3639 / 2971 / 918
Регистрация: 05.07.2013
Сообщений: 14,220
16.11.2018, 02:58
builder.append(value).toString() + " < 0"
зачем так то?
1
528 / 263 / 70
Регистрация: 11.12.2016
Сообщений: 1,223
16.11.2018, 03:04
JuniorDp, Так тут похожая тема прамо "на верху", практически такое же задание.
Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
import java.util.StringJoiner; 
 
public class MyClass {
    public static void main(String[] args){
        MyClass m = new MyClass();
        System.out.println(m.myMethod(20));
    }
    public String myMethod(int n) {
        System.out.println("Input : "+n);
        if(n == 0)
            return "0 = 0";
        if(n < 0) 
            return ""+n+" < 0";
        StringJoiner sj = new StringJoiner(" +", "Output : ", " = ");
        int sum = 0;
        for(int i = 0; i <= n; sum += i, i++) 
            sj.add(" "+i);
        return sj.toString() + sum;
    }
}
Code
1
2
Input : 20
Output :  0 + 1 + 2 + 3 + 4 + 5 + 6 + 7 + 8 + 9 + 10 + 11 + 12 + 13 + 14 + 15 + 16 + 17 + 18 + 19 + 20 = 210
1
1 / 1 / 0
Регистрация: 16.08.2017
Сообщений: 44
16.11.2018, 03:11  [ТС]
Блин, исправил, спасибо)
Нужно бооольше билдеров

Добавлено через 7 минут
Ну как вариант, только вместо Билдера - Джоинер
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
16.11.2018, 03:11
Помогаю со студенческими работами здесь

Помогите писать на С++ через шаблоны. Консуле я писал, но надо писать исползуя шаблоны
В одномерном массиве, состоящем из п вещественных элементов, вычислить: 1) количество элементов массива, равных 0; 2) сумму элементов...

В VB.NEТ надо писать Nothing а не Null
Система ругается в коде Cache.Insert(&quot;proverbs&quot;, ds, Null, System.Web.Caching.Cache.NoAbsoluteExpiration, TimeSpan.FromMinutes(5))на...

надо было не писать в поддержку
Написала в яндекс письмо, что раньше сайт имел неуникальный контент, но теперь пишем свои интересные статьи. Просила пересмотреть статус...

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

Что надо писать в пояснительную записку БД
Пояснительная к курсовому &quot;Художественная галерея&quot; (обязательно &quot;Введение&quot;) срочно, заранее спасибо)))


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

Или воспользуйтесь поиском по форуму:
8
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer
8Observer8 08.02.2026
Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .
SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств
8Observer8 07.02.2026
Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве
8Observer8 02.02.2026
Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .
Философия технологии
iceja 01.02.2026
На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf
8Observer8 01.02.2026
Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .
SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли
8Observer8 30.01.2026
Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm
8Observer8 30.01.2026
Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .
SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 29.01.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru