Флудилка

[exend 877]

@BlackSun, в чем по твоему различия?

[BlackSun 4 253]

@eXend, и не стыдно терминалами тут трясти? Дети увидят!

 

[exend 877]

@eXend, и не стыдно терминалами тут трясти? Дети увидят!

Пусть закаляются.

А при чем тут свойства системы? Я о разнице терминала и консоли, которые одновременно присутствуют в одной ОС.

[BlackSun 4 253]

Кажется мой оберег защиты от линуксов утратил свою силу. Придется сдать по гарантии, пока не закидали своими sh'ами, bash'ами и прочими красноглазыми шеллами с нескучными иконками.

[exend 877]

Кажется мой оберег защиты от линуксов утратил свою силу. Придется сдать по гарантии, пока не закидали своими sh'ами, bash'ами и прочими красноглазыми шеллами с нескучными иконками.

Пичалька ):

Поставь себе BolgenOS, там еще нескучные обои есть!

[SirEdvin 2 037]

 

 

Кажется мой оберег защиты от линуксов утратил свою силу. Придется сдать по гарантии, пока не закидали своими sh'ами, bash'ами и прочими красноглазыми шеллами с нескучными иконками.

Твой шелл смотрет на тебя красноглазыми глазами? 

[Florexiz 384]

@Florexiz, мод неплохой, но это уже футуристическая магия, а арса все-таки больше средневековая. Еще есть веселый мод - AcademyCraft. 

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

P.S. Почитать ранобэ - неплохая идея как потратить время...

[aliyldrm 119]

Нечего делать

[BlackSun 4 253]

"А не поставить ли мне Dark Souls?" подумал я. Первый раз установщик вылетел на 70%, забрав с собой браузер. Во второй попытке что-то пошло не так на 100% и всё установленное было удалено. Даже R.Lab'овская тулза не смогла найти свеже-удаленные файлы. Нет, я знаю, что игра не из простых, но не до такой же степени..

[Гость guineafrom]

За что Кубика и Ханеса с Модерки убрали?

И Эделя...

[SirEdvin 2 037]

@eXend, так же, в том же классе есть функция isLetter, которую можно использовать в том же цикле. И вообще без регулярок обойтись. 

 

+ по хорошому, стоит все-таки один раз вызвать message.lenght() и записать в локальную переменную. 

Изменено 24 апреля 2016 пользователем SirEdivn

[exend 877]

@eXend, так же, в том же классе есть функция isLetter, которую можно использовать в том же цикле. И вообще без регулярок обойтись.

Так-то да, но длина не уменьшится и % будет криво считать...

 

+ по хорошому, стоит все-таки один раз вызвать message.lenght() и записать в локальную переменную. 

Верно, на производительности не нужно экономить. Часто так делаю, но это на компилируемых ЯП, а тут жава, даже не знаю насколько много смысла в этом.

А в регулярном выражении вообще же можно проще написать и вызвать:

message.replaceAll("\\W|\d", "");

Не?

[SirEdvin 2 037]

 

 

Так-то да, но длина не уменьшится и % будет криво считать...

Алгоритмическая сложность станет меньше. Регулярка во много раз хуже цикла.

 

 

 

Верно, на производительности не нужно экономить. Часто так делаю, но это на компилируемых ЯП, а тут жава, даже не знаю насколько много смысла в этом.

Ну, по идее, после 100-200 вызовов оно запихнет все нормально, но зачем, если можно нормально написать? + такое даже на плюсах не прокатити ниже o2.

 

 

 

message.replaceAll("\\W|\d", "");

Можно просто \\W по идее. 

[Nuclear 834]

@Nuclear, я конечно имел очень малый опыт работы с регулярными выражениями, и уже успел забыть их, но как я понял в первой строке сначала выпиливаются только пробелы, а следом все кроме букв и пробельных символов, если не ошибся. Вопрос, зачем оставлять другие пробельные символы? Будет ли всё работать если сделать только один вызов replaceAll("[^A-Za-zа-яА-Я]")? Возможно я чего-то тут не догоняю...

Зачем крутить цикл полностью? Ведь если процент будет уже превышен, то остальную часть можно оставить в покое.

Например так:

 

short upper_сhars = 0;short caps_not_percent = plugin.config.capsPercent / 100;for(char ch : message){	if(Character.isUpperCase(ch) && (++upper_сhars / message.length() > caps_not_percent))		return Violation.CAPS;}

И как-то даже компактней.

Код писался с учётом сбора статистики. И насчет пробелов, вы забыли что это чат minecraft, тут не может быть других символов, так зачем мне делать их удаление, если их нет? Изменено 24 апреля 2016 пользователем Nuclear

[exend 877]

@eXend, так же, в том же классе есть функция isLetter, которую можно использовать в том же цикле. И вообще без регулярок обойтись.

@SirEdivn, я почитал подробнее о функции, оказалось она даже различает символы цифр и букв. В таком случае действительно подходит, только вызывать не в том же цикле, а сначала в отдельном, для подсчета букв.

 

Можно просто \\W по идее. 

Не, \W включает в себя и цифры тоже, а их нужно выпилить.

 

Я еще забыл про то, что caps_not_percent нужно было сделать float, а не short Но float медленнее целых считается, поэтому наверное лучше оставить умножение на 100.

В итоге вместо этого:

 

Должно получиться как-то так:

short value;
short upper_сhars = 0;
short letters = 0;

for(char ch : message)
	if(Character.isLetter(ch))
		++letters;

if(letters <= plugin.config.capsMinLength)
	return Violation.NONE;

value = plugin.config.capsPercent * letters / 100;

for(char ch : message)
	if(Character.isUpperCase(ch) && (++upper_сhars > value))
		return Violation.CAPS;

Хотя value можно было сделать опять же float, но тут погрешность всего-то в 1 символ. И да, тут еще отвалились всякие умножения и деления в цикле.

Код писался с учётом сбора статистики. И насчет пробелов, вы забыли что это чат minecraft, тут не может быть других символов, так зачем мне делать их удаление, если их нет?

Т.е. это такая оптимизация? Тогда способ, который предложил @SirEdivn намного лучше.

Изменено 24 апреля 2016 пользователем eXend