delete
Вид для печати
delete
Нескромный вопрос к автору темы: вы вообще проверяли свой код?
PHP код:CMD:id(playerid, params[])
{
if(sscanf(params,"u",params[0])) return SendClientMessage(playerid, 0xAFAFAFFF, "Используйте: /id [ид игрока/часть ника]");
if(!IsPlayerConnected(params[0])) return SendClientMessage(playerid, 0xFFFF00FF, "Совпадении не найдено ");
new name[MAX_PLAYER_NAME], str[13+MAX_PLAYER_NAME];
GetPlayerName(params[0], name, sizeof(name));
format(str, sizeof(str), "%s - ID: %d", name, params[0]);
SendClientMessage(playerid, 0xFFFFFFFF, str);
return 1;
}
Вы не совсем поняли суть команды.Цитата:
Сообщение от Excel_Smit.
Да.Цитата:
Сообщение от Daniel_Cortez
Про название массива fmt_str, пожалуй, не буду ничего говорить.Цитата:
Сообщение от Sparks
Цитата:
Сообщение от Daniel_Cortez
1. Если на сервере будет один единственный игрок под ID 0, то GetPlayerPoolSize() вернёт 0 (подробнее здесь).Цитата:
Сообщение от Sparks
В цикле for условие проверяется перед выполнением тела цикла. Первая же проверка условия (--i >= 0) выдаст false (переменная i окажется равна -1) и тело цикла не выполнится ни разу.
2. Как уже подметили выше, вместо получения никнейма каждого i-того игрока берётся ник игрока, который ввёл команду.
Поэтому если на сервере будет 5 или более человек и игрок введёт часть своего ника, команда профлудит сообщениями с его ником 5 раз.
3.
Что мешает выйти из цикла при инкременте переменной found_id на предыдущей итерации?PHP код:if(found_id > 5)
break;
found_id++;
Можно было сделать примерно так:
Но нет, мы лёгких путей не выбираем: если достигнут лимит, проведём поиск ещё раз - и плевать, что найденный результат не будет использоваться.PHP код:if(++found_id >= 5)
break;
4.
Зачем эти действия вообще нужны в цикле, если они должны выполняться однократно?PHP код:else if(found_id == -1)
return SendClientMessage(playerid, 0xB4B5B7FF, "Совпадений не найдено");
else if(found_id >= 4)
SendClientMessage(playerid, 0xB4B5B7FF, "Показаны первые 5 совпадений");
5. Сообщение "Показаны первые 5 совпадений" будет выведено 2 раза, если на сервере будут хотя бы 5 игроков.
6. Если игрок введёт подстроку, которой нет в его никнейме, сообщение "Совпадений не найдено" будет показано столько раз, сколько игроков играют на сервере.
Из всего этого напрашивается вопрос: каким образом вы тестировали свою команду, если упустили столько ошибок, которые даже без тестирования бросаются в глаза? Тяп-ляп и в паблик? Not cool.
Daniel, вы полностью правы. Люди не очень понимают то, что они делают. Каждый программист должен понимать, все что он делает, он так же должен понимать как его программа работает, и что в ней происходит.
Если бы были такие смайлы как : Задница и Язык, вы бы их видели вместо этого сообщения.Цитата:
Сообщение от VVWVV
Ясно, но зачем так грубо? Нельзя быть вежливым со всеми? Простите, если что-то сделал не так. Я просто высказал свое мнение. Спасибо.Цитата:
Сообщение от Salvacore
Тут нету оскорбления )Цитата:
Сообщение от VVWVV
Но как же так? Хорошо... У вас на форуме нельзя выражать свое мнение, а так же быть вежливым? Мое мнение о вас меняется...Цитата:
Сообщение от gangzone.ini
Мое мнение было очень похоже с сообщением Daniel'a.
«... Задница и Язык, вы бы их видели вместо этого сообщения.» — большой намек.
Обновил.
не оскорбление а грубость скорее)Цитата:
Сообщение от gangzone.ini
как раз таки на этом форуме в большей части можно высказывать свои мнения,проверено)Цитата:
Сообщение от VVWVV
а на счет вежливости,то,как-бы сказать,если и встретишь то оочень редко человека с такой вежливостью что-ли,да и по сути на любом форуме много маленьких быдло кодеров которые только чужих мамок в кино умеют водить,на счет Сальвакора я честно говоря не знаю,видимо со стороны ваш комментарий был поход на "лизоблюдство",но,на это у каждого свой взгляд,проще не обращай внимание.
p.s. а по сути твоя манера общения мне даже нравится)
Это здорово, что вы согласны с чьей-то точкой зрения, но в чём смысл оставлять пост, если вы просто пересказываете чужую идею?Цитата:
Сообщение от VVWVV
Не следует путать конструктивное обсуждение с набиванием постов. В конце концов, если вы согласны с чьим-то мнением, но вам нечего добавить, и вы просто хотите как-то выразить своё согласие - под каждым постом есть кнопка "Спасибо".
Когда я оставил свой предыдущий пост, я не заметил, что в found_id начинается с -1, а не с 0.Цитата:
Сообщение от Sparks
Могу поинтересоваться, почему вы сделали именно так? Вы как-нибудь представляете себе -1 игроков? Или -1 метр?
К тому же, в функцию format приходится передавать не found_id, а found_id + 1.
Кроме того, вы ищете до 5 игроков, но в условии прерывания поиска приходится указывать 4, что только ещё больше запутывает ваш код.
И да, ещё. Что послужило причиной того, что вы сделали foreach.inc обязательным?
Потому что я считаю его актуальным.Цитата:
Сообщение от Daniel_Cortez