SeTSeR off
Так лучше?
aNNiMON off
let live
SeTSeR, места, говоришь, в прошлый раз не хватило? А на каждую фигурную скобку по несколько пробелов и перенос строки уходит.
Вот именно, что расстановка скобок не джавовская ни разу. Вопрос религии, конечно, но стандарт есть стандарт.
А что в той строке хорошего? Обрамляющие всё выражение скобки излишни, да и выглядит дико.
  1. final int sign = (oldlook == '+') ? 1 : -1;
  2. a = a + getTerm() * sign;
Или
  1. a = (oldlook == '+') ? a + getTerm() : a - getTerm();
SeTSeR off
aNNiMON, фигурные скобки а-ля C#? Этот стиль ещё во времена C/Unix был распростанён, почитай Кернигана. Отступы такие, потому что в прошлый раз мне места не хватило, заново решил на грабли не наступать. Про децентрализованную логику, скинь всё в один метод и посмотри, какие костыли нужны для нормальной реализации приоритетов.
А что не так в той строке, что ты привёл?
oldlook - это символ перед числом, плюс или минус, ничего криминального не вижу. Кстати говоря, ты пропустил открывающую скобку.
Bogdan-G off
aNNiMON, Скобки как скобки, все вроде бы расставлены. Название переменных, вроде бы не совсем китай, но понятно, отступы немножко криво.
aNNiMON off
let live
Bogdan-G, rly? Фигурные скобки а-ля C#, отступы в 1-2 пробела, переменные названы непонятно, логика децентрализована (в одном методе сложение/вычитание, в другом умножение/деление, в третьем скобки). За эту строку вообще убить надо:
  1. ((oldlook=='+')?(a+=getTerm()):(a-=getTerm());
Bogdan-G off
Да нормально оформлено О_о
SeTSeR off
DaGGeR16, спасибо, так и сделаем
DaGGeR off
Удалить код,и заново добавить
SeTSeR off
Ладно, убедили, буду с компа, поменяю. Кстати, как это сделать?
DaGGeR off
Соглашусь с Витей,название переменных кривоватые какие-то
Всего: 14
1 2 >>

Реклама

Мы в соцсетях

tw tg yt gt