@ProLua

Страница 175 из 307
TriΔng3l
22.09.2017
20:35:54
И кусочек кода Quake портировал

Шо есть сент
SENT = scripted entity (игровой объект на Lua), SWEP = scripted weapon

Кусочки Half-Life: Source туда ещё впиливал

http://steamcommunity.com/id/Triang3l/screenshots/?appid=4010&sort=newestfirst&browsefilter=myfiles&view=imagewall

Google
TriΔng3l
22.09.2017
20:37:35
http://steamcommunity.com/id/Triang3l/screenshots/?appid=4000&sort=newestfirst&browsefilter=myfiles&view=imagewall

Philipp
23.09.2017
15:35:05
Ребята, кто-нибудь у себя имеет написанную либу для взаимодействия с файловой системой?

Мне просто лень своё писать

Типа, чтобы можно было просто определять директория/файл/что-то еще

Philipp
23.09.2017
15:42:26
Гляну, спасибо

Просто я помню Лёша свой прелоадер для библиотек писал, там кажись нужный функционал был

Vadim
23.09.2017
16:27:52
@mva_name
что?

Vadim
23.09.2017
16:29:01
и чем там код страшный?

пара метатаблиц, да матчи в цикле

Aydar
23.09.2017
16:31:47
и чем там код страшный?
Лишние глобалки, форматирование

Google
Vadim
23.09.2017
16:33:10
1) что не так с форматированием?

2) что не так с глобалами?

Philipp
23.09.2017
16:35:13
Не, ну глобалы всё-таки моветон

Aydar
23.09.2017
16:35:28
Не, ну глобалы всё-таки моветон
Ты смотрел код lua-htmlparser?

Philipp
23.09.2017
16:35:39
Нет

Aydar
23.09.2017
16:36:20
Нет
А что тогда говоришь?

Philipp
23.09.2017
16:36:26
Ты значение слова моветон знаешь?

Vadim
23.09.2017
16:37:24
покажи мне там глобалы

Aydar
23.09.2017
16:39:28
покажи мне там глобалы
Сообщение про моветон не заметил. Я просто в total overdose играю чат фоном открыт. А глобалы например pattern, htmparser_limit и так далее

Vadim
23.09.2017
16:40:13
limit - намеренно глобал

и он не объявляется в коде

он проверяется на наличие

и ты можешь его не использовать, ВНЕЗАПНО

Aydar
23.09.2017
16:41:10
limit - намеренно глобал
А его спокойно можно локалом сделать

Vadim
23.09.2017
16:41:23
а паттерн - видимо, по недосмотру. Не должен был быть глобалом

А его спокойно можно локалом сделать
ты совсем не понял его суть

local limit = limit or htmlparser_looplimit or 1000

сама переменная локальная

Philipp
23.09.2017
16:42:18
limit - намеренно глобал
Недавно как раз фиксил в luaposix именно такой же случай

Google
Vadim
23.09.2017
16:42:24
а ты там как хочешь так и объявляй

Philipp
23.09.2017
16:42:26
Что забавно

Типа, некоторые стрикт-чекеры будут ругаться на него и падать

Типа attemp to call undefined global

Aydar
23.09.2017
16:43:19
а ты там как хочешь так и объявляй
А смысл самому это делать?

Vadim
23.09.2017
16:43:19
там не такой случай

Philipp
23.09.2017
16:43:31
там не такой случай
У нас чекер на таких выражениях падает

Vadim
23.09.2017
16:43:37
а для некоторых юзеров он есть

Aydar
23.09.2017
16:43:43
тебе - никакого
А всем остальным?

Vadim
23.09.2017
16:43:50
А всем остальным?
есть юзеры которым удобно задефайнить в конфиге приложения переменную с общим лимитом, а не дрочить на кажом вызове функции

ещё раз говорю, тебя никто не заставляет её дефайнить

Vadim
23.09.2017
16:45:57
штоблеать?

внутри htmlparser'а переменная локальная

а снаружи ты как хочешь так и дефайни

хоть метатаблицей

Google
Vadim
23.09.2017
16:47:02
(кстати, надо tonumber добавить)

Так а что если он локальным станет его поменять нельзя?
итого только один косяк - pattern по недогляду глобальный. Один. И где же страшнота кода?

Aydar
23.09.2017
16:48:28
(кстати, надо tonumber добавить)
А то что там две функции в начале неиспользуемые функции задефайнены ?

Philipp
23.09.2017
16:48:36
А если мне нужна свой глобальный limit для своих целей?

Vadim
23.09.2017
16:49:36
А то что там две функции в начале неиспользуемые функции задефайнены ?
а чем ЭТО страшно? Они есть не просят // и не используются они *сейчас*. А в следующем релизе будут.

Philipp
23.09.2017
16:50:02
ты читай внимательнее, ага
Я тебе не грубил. Давай-ка и ты не перегибай

Vadim
23.09.2017
16:50:08
внимательно

Admin


Vadim
23.09.2017
16:50:20
пожалуйста

Philipp
23.09.2017
16:50:58
Следи за руками: config.lua: limit = 100 -- limit for internal calls main.lua require'config.lua' require'htmlparser'

Vadim
23.09.2017
16:51:13
и хрен тебе (не получится, в смысле)

Aydar
23.09.2017
16:51:23
Vadim
23.09.2017
16:51:39
она будет nil

Aydar
23.09.2017
16:52:56
Ты спросил где глобалки я ответил, а там еще и unused код, форматирование табами

Philipp
23.09.2017
16:53:31
потому что limit там локальная от вызова функци
Стоп. Значит я тупой. htmlparser_looplimit – вот это глобалка?

Google
Philipp
23.09.2017
16:53:52
Aydar
23.09.2017
16:54:05
Vadim
23.09.2017
16:54:11
Ты спросил где глобалки я ответил, а там еще и unused код, форматирование табами
ты нашёл только одну глобалку. В unused функциях и форматировании табами нет ничего страшного. Так в чём же страшность. В одной клобальной pattern? // лучше б пуллреквест сделал

Philipp
23.09.2017
16:54:39
Айдар, код там на 99% адекватный

Vadim
23.09.2017
16:54:48
Стоп. Значит я тупой. htmlparser_looplimit – вот это глобалка?
да. Глобалка, которая проверяется на существование (хотя надо бы ещё на бытие числом проверять)

Philipp
23.09.2017
16:55:27
А почему нельзя вынести htmlparser_looplimit в скоуп htmlparser'a?

Типа, htmlparser.limit = 100

И все были бы довольны

Aydar
23.09.2017
16:55:58
Philipp
23.09.2017
16:56:14
Я по секрету скажу – 100% идеального кода не существует

Philipp
23.09.2017
16:57:05
99.9 надо
У тебя чекер падает на проверке глобальной неопределенной переменной?

Vadim
23.09.2017
16:57:17
А почему нельзя вынести htmlparser_looplimit в скоуп htmlparser'a?
ну, наверное, можно. Но что-то я не уверен, что будет удобнее. Но сделаю и так :)

Aydar
23.09.2017
16:57:48
Philipp
23.09.2017
16:57:59
Vadim
23.09.2017
16:58:03
ну, почти

я его подобрал

Aydar
23.09.2017
16:58:13
Так это его проект, нет?
Я не смотрел профиль

Vadim
23.09.2017
16:58:19
изначально он не мой, но рерутнули на меня и я допиливаю и фикшу

Philipp
23.09.2017
16:58:19
Ну я же не просто так его в тред призвал :)

Страница 175 из 307