[CMD] Из двоичной системы счисления в десятичную

Из двоичной системы счисления в десятичную

Скрипт переводит числа из двоичной системы счисления в десятичную. Если в переменной sub будет не бинарное число, то скрипт прекратит работу с оповещением об ошибке.

PHP код:

CMD:binary(playerid, params[]) { if (isnull(params)) return SendClientMessage(playerid, 0xAA3333AA, !"[Подсказка]: /b [двоичный код]"); static const str[] = "Result: %i"; new string[sizeof(str) + 11 - 2], result; Binary(params, result); if (!result) return SendClientMessage(playerid, -1, !"Error"); format(string, sizeof(string), str, result); SendClientMessage(playerid, -1, string); return 1; }

PHP код:

stock Binary(const sub[], &result) { static n, i, c; result = i = 0; n = strlen(sub); while ((c = sub[i++])) { switch (c) { case 48, 49: { result += (c == 49) ? (1 << n - i) : (0); } default: { return 0; } } } return result; }

Во-первых, в чём смысл команды? Просто вывести игроку, какое число он ввёл?
Если фишка в самой функции Binary, а не в команде, то и тему следовало создавать в разделе для функций.
UPD: Сначала по префиксу подумал, что тема находится в разделе "Команды", но только сейчас заметил, что на самом деле она в разделе "Скрипты", с префиксом "[CMD]", а показывается в ней работа функции. Логика, прощай.

Во-вторых, вы так любите велосипеды? Для считывания двоичного числа из строки давно существует sscanf2.

Открыть/закрыть

PHP код:

CMD:binary(playerid, params[]) { extract params -> new bin:x; else // new x; if (sscanf(params, "b", x)) return SendClientMessage(playerid, -1, !"Использование: /binary [число в двоичной системе]"); static const fmt_str[] = "Вы ввели число %d."; new string[sizeof(fmt_str) - 2 + 11]; format(string, sizeof(string), fmt_str, x); return SendClientMessage(playerid, -1, string); }

И только не нужно оправдываться, что ваша реализация сделана специально, "чтобы не подключать плагин ради одной задачи".
На дворе 2016 год и sscanf2 используется почти повсеместно, кроме лишь некоторых "гипотетических" модов, которые не используют команд и не работают со строками.
Я называю их "гипотетическими" потому, что обычно такие моды либо имеют неудобный пользовательский интерфейс (на одних лишь текстдравах и диалогах далеко не уедешь), либо они недалеки от простого "hello world", либо их создатели - нездоровые люди, одержимые идеей отказа от всех плагинов.
(Но я всё же надеюсь, что вы просто не знали о спецификаторе "b", ибо то оправдание в случае со sscanf2 выглядело бы очень глупо.)

В-третьих, вы сами не находите странным, что функция, предназначенная для преобразования из строки в число, показывает игрокам сообщения?
Реализация алгоритма должна быть изолирована от взаимодействия с пользователем. Как говорится, компот отдельно, мухи отдельно. Очевидно же.

Если размышлять замкнуто, то да.
Не раз видел, что люди не используют sscanf2. Расизм? Я использовал cmd с sscanf, чтобы просто показать стока. Это можно было расписать и на команду и диалог, но у меня на это времени нет.
Вот это я не понял. Выводит всего 2 сообщения, о конечном результате или ошибке.

Цитата:

Сообщение от m1n1vv

3. Вот это я не понял. Выводит всего 2 сообщения, о конечном результате или ошибке.

Цитата:

Реализация алгоритма должна быть изолирована от взаимодействия с пользователем.

Алгоритм - функция; Пользователь - вывод сообщений.

Вы могли сделать так:

PHP код:

CMD:b(playerid, params[]) { if (isnull(params)) return SendClientMessage(playerid, 0xAA3333AA, !"[Подсказка]: /b [двоичный код]"); new result = Binary(params); if (result) { new str[32 /* магия... */]; format(str, sizeof str, "%d", result); SendClientMessage(playerid, -1, str); } else SendClientMessage(playerid, -1, !"Error"); return 1; }

PHP код:

stock Binary(playerid, sub[]) { new result; for (new i = 0, n = strlen(sub) - 1; i <= n; i++) { switch (sub[i]) { case '0', '1': { result += (sub[i] == '1') ? (1 << n - i) : (0); } default: { return 0; } } } return result; }

Цитата:

Сообщение от m1n1vv

Если размышлять замкнуто, то да.

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

Цитата:

Сообщение от m1n1vv

Не раз видел, что люди не используют sscanf2. Расизм?

Забыл ещё упомянуть про RLS (коих будет больше, чем всех остальных перечисленных вместе взятых) и им подобные примеры устаревшего кода. Усугубляет ситуацию ещё и то, что начинающие учатся на таком старомодном коде, а потом сами же усложняют себе жизнь отказом от современных решений, которые и удобнее (как в написании, так и в плане чтения кода), и производительнее прежних. И да, такой отказ можно классифицировать как болезнь: синдром утёнка.
sscanf2 - это своего рода швейцарский нож, только для скриптинга. Если знать, как им пользоваться, то ему найдётся применение в любом моде (кроме уже упомянутых мной выше хелловорлдов).
Разумеется, без sscanf2 можно обойтись, но во многих случаях это будет в ущерб читаемости и эффективности работы кода.

Цитата:

Сообщение от m1n1vv

Вот это я не понял. Выводит всего 2 сообщения, о конечном результате или ошибке.

Вы там что-то говорили о "замкнутом мышлении"? Ладно, очередной спор всё равно ни к чему не приведёт, поэтому просто попробую помочь.
Если название функции гласит о том, что функция преобразует строку в число, то она должна выполнять эту и только эту задачу.
Что, если вам потребуется, чтобы функция не показывала никаких сообщений об ошибке? Будете передавать неправильный ID игрока, чтобы функция SendClientMessage работала вхолостую? Оно и ежу понятно, что это дурной тон.

Если нужно взаимодействовать с пользователем, это должно быть в коде, вызывающем ту функцию, т.е. в данном случае, за это должен отвечать обработчик команды.
При ошибке ввода можно сделать так, чтобы функция вернула 0, но этим можно создать неопределённость, поскольку 0 - это правильное число, которое может ввести пользователь. Отрицательные числа тоже правильные: допустим, игрок ввёл число из 32 двоичных разрядов - в этом случае самый старший разряд будет отвечать за знак.
Логично было бы сделать отдельный аргумент, передаваемый по ссылке, в который функция записывала бы статус успеха.

PHP код:

stock BinaryToInt(const str[], &bool:success) {     success = false;     new result;     result = 0;     // ...     if (...) // Если что-то пошло не так         return 0; // В success уже записано false, просто выходим из функции     // ...     success = true;     return result; }

Использовать эту функцию следует так:

PHP код:

new bool:success; new x = BinaryToInt(params, success); if (!success)     return SendClientMessage(playerid, -1, ...);

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

PHP код:

stock bool:BinaryToInt(const str[], &result) {     result = 0;     // ...     if (...) // Если что-то пошло не так         return false;     // ...     success = true;     return true; }

Использование:

PHP код:

new x; if (!BinaryToInt(params, x))     return SendClientMessage(playerid, -1, ...);

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

что означает символ перед result в шапке функции:

PHP код:

stock bool:BinaryToInt(const str[], &result)

PHP код:

&result

Цитата:

Сообщение от BadPawn

что означает символ перед result в шапке функции:

PHP код:

stock bool:BinaryToInt(const str[], &result)

PHP код:

&result

Вот тут, в самом конце, я давал описание:
[Урок] Всё о Stock

PHP код:

stock bool:BinaryToInt(const str[], &result) { result = 0; // ... if (...) // Если что-то пошло не так return false; // ... success = true; return true; }

А откуда тут взялось success?

PHP код:

&result

Но куда оно вернет?

Цитата:

Сообщение от m1n1vv

PHP код:

stock bool:BinaryToInt(const str[], &result) { result = 0; // ... if (...) // Если что-то пошло не так return false; // ... success = true; return true; }

А откуда тут взялось success?

Явно обычная опечатка же. Можно было понять

Цитата:

Сообщение от m1n1vv

PHP код:

&result

Но куда оно вернет?

В переменную, которую ты укажешь при вызове?

Daniel_Cortez, давно хотел написать... Не забывай о максимальном количестве цифр в числе. sscanf это не переведет в десятичную систему:

PHP код:

1010110111001011010

Цитата:

Сообщение от m1n1vv

Daniel_Cortez, давно хотел написать... Не забывай о максимальном количестве цифр в числе. sscanf это не переведет в десятичную систему:

PHP код:

1010110111001011010

Как и твоя, например, это

PHP код:

1001010100000010111110001111111111

Имхо, если взялся создавать "универсальный" (не знаю как ещё назвать) конвертер, стоит возвращать строку. Тогда можно действительно любое число перевести

Цитата:

Сообщение от DeimoS

Как и твоя, например, это

PHP код:

1001010100000010111110001111111111

Имхо, если взялся создавать "универсальный" (не знаю как ещё назвать) конвертер, стоит возвращать строку. Тогда можно действительно любое число перевести

Убери 3 символа и сможет. У меня более работоспособней, так-как может перевести код длиннее. Число с количеством цифр больше 10 не выведешь. О значении и сам знаешь ).

Цитата:

Сообщение от m1n1vv

Daniel_Cortez, давно хотел написать... Не забывай о максимальном количестве цифр в числе. sscanf это не переведет в десятичную систему:

PHP код:

1010110111001011010

PHP код:

#include <a_samp> #include "../include/sscanf2.inc" main() { static const binary_str[] = "1010110111001011010"; // Считывание числа extract binary_str -> new bin:x; else // new x; if(sscanf(binary_str,"b",x)) printf("Failed to read the number"); printf("%s -> %d", binary_str, x); // Проверка new output[sizeof(binary_str)]; format(output, sizeof(output), "%b", x); printf( "%d -> %s - %s", x, output, (0 == strcmp(binary_str, output)) ? ("ok") : ("test failed") ); }

Код:

1010110111001011010 -> 355930 355930 -> 1010110111001011010 - ok Number of vehicle models: 0

...

И куда этот код пригодится?))
Работу хакера на РП придумать разве что и присунуть код для красоты)