Jump to content
Korean Random
Sign in to follow this  
Yukikaze

JS like идентификаторы в конфиге

Recommended Posts

Собственно сабж, как вы считаете, нужно ли вводить такое "улучшение" в синтаксис файлов конфигурации.

Пример:

{
    "configVersion": "99.00.9", // Тут ключи(идентификаторы) с кавычками pure JSON
    "editorVersion": "3.14",    // и тут тоже
    "definition": {
        author: "Alex Yukikaze", // а вот здесь уже без кавычек
        date: "29.04.2067",      // и являются валидной записью
        description: "Черная магия",
        game-Version: "3.7.1",   // и тут тоже
        mod_MinVersion: "23.0.4",// так же как и здесь
        url: "http://www.modxvm.com/"
    }
}

Такой подход позволяет сократить кол-во повторяющихся символов, что снизит вероятность возникновения ошибок при ручном редактировании файла. Ну и конечно стоит отметить, что это просто удобно. Обратная совместимость не теряется, все конфиги написанные до этого будут работать без каких либо доработок.

 

Ваш вердикт?

 

PS Стоит отметить, что функционал реализован и не отнимет времени у команды разработчиков

Edited by Yukikaze
  • Upvote 1
  • Downvote 1

Share this post


Link to post

Short link
Share on other sites
Такой подход позволяет сократить кол-во повторяющихся символов, что снизит вероятность возникновения ошибок при ручном редактировании файла.

Всё что мы поимеем - возможность не ставить кавычки в именах параметров? 

"login": ${"login.xc":"login"},

превращается в 

login: ${"login.xc":"login"},

или

login: ${"login.xc":login},

?

И ссылки внутри одного файла. Конструкцию, указывающую на структуру можно будет не брать в кавычки?

 

"enemy": ${"markers.ally"}

станет

 

enemy: ${markers.ally}

?

Edited by sech_92
  • Upvote 1

Share this post


Link to post

Short link
Share on other sites

@sech_92, кстати идею подкинул отличную, я почему-то сразу об этом не подумал, если комьюнити оценит то реализую.

 

ЗЫ А пока да, первый вариант login: ${"login.xc":"login"}

Edited by Yukikaze

Share this post


Link to post

Short link
Share on other sites

Denied ©

Потому что возникнет путаница, особенно у ньюфагов, типа "а почему тут кавычки надо, а тут или надо, или не надо"

Share this post


Link to post

Short link
Share on other sites

возникнет путаница, особенно у ньюфагов, типа "а почему тут кавычки надо, а тут или надо, или не надо"

Я тоже придерживаюсь этого мнения. Но в целом идея тоже неплохая.

Для себя не могу определить как лучше, 50/50.

Share this post


Link to post

Short link
Share on other sites

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

Share this post


Link to post

Short link
Share on other sites

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

Share this post


Link to post

Short link
Share on other sites

 

 

Только обдуманно, если это действительно будет полезно.
 если самый последний пример из моего поста станет рабочим - польза будет. Возможно, заработают конструкции вроде этой.

Share this post


Link to post

Short link
Share on other sites

Макросы при парсинге JSONx точно не заработают, это совершенно разные уровни обработки.

Edited by sirmax

Share this post


Link to post

Short link
Share on other sites

Я против. Это лишняя работа, толку от которой не так много.

Тем более, это потянет за собой головную боль и для разработчиков всяких конфигураторов XVM.

 

 

PS Стоит отметить, что функционал реализован и не отнимет времени у команды разработчиков

Отнимет в любом случае. Написание документации, примеров и мануалов - всё это требует времени.

Edited by Azbuka_slovensko
  • Upvote 1
  • Downvote 1

Share this post


Link to post

Short link
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.

×
×
  • Create New...