Грамотный код - Printable Version

Грамотный код - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: Non-English (https://sampforum.blast.hk/forumdisplay.php?fid=9)
+--- Forum: Languages (https://sampforum.blast.hk/forumdisplay.php?fid=33)
+---- Forum: Русский/Russian (https://sampforum.blast.hk/forumdisplay.php?fid=32)
+---- Thread: Грамотный код (/showthread.php?tid=651647)

Грамотный код - NikitaFoxze - 24.03.2018

Я тут хотел просить как грамотно и правильно писать код, ведь существует много способов
Например №1:
stock PayDay(playerid)
{
GivePlayerMoney(playerid, 600);
SendClientMessage(playerid, -1, "Вы получили 600 кредитов");
return 1;
}
Например №2:
stock PayDay(playerid) {
GivePlayerMoney(playerid, 600);
SendClientMessage(playerid, -1, "Вы получили 600 кредитов");
return 1;}

В основном я хотел спросить про скобочки, в каких вариантах их писать.

Re: Грамотный код - Bombo - 24.03.2018

Ну.. скобочки не помешало бы открывать в функции GivePlayerMoney(). Это для начала.

Re: Грамотный код - NikitaFoxze - 24.03.2018

Quote:

Originally Posted by Bombo

Ну.. скобочки не помешало бы открывать в функции GivePlayerMoney(). Это для начала.

Исправил

Re: Грамотный код - TriskeliS - 24.03.2018

Code:

stock PayDay(playerid) {
  GivePlayerMoney(playerid, 600);
  SendClientMessage(playerid, -1, "Вы получили 600 кредитов");
  return 1;
}

Re: Грамотный код - Stepashka - 25.03.2018

PSR вам в помощь:
PHP Standard Recommendation
Для "ну там всё на аглицком"

Re: Грамотный код - ForCop - 25.03.2018

Code:

stock PayDay(playerid) 
{
    GivePlayerMoney(playerid, 600);
    return SendClientMessage(playerid, -1, "Вы получили 600 кредитов");
}

PayDay(playerid) 
{
     GivePlayerMoney(playerid, 600);
     return SendClientMessage(playerid, -1, "Вы получили 600 кредитов");
}

stock PayDay(playerid) return GivePlayerMoney(playerid, 600),SendClientMessage(playerid, -1, "Вы получили 600 кредитов");

PayDay(playerid) return GivePlayerMoney(playerid, 600),SendClientMessage(playerid, -1, "Вы получили 600 кредитов");

Re: Грамотный код - OKStyle - 26.03.2018

Имхо. Так более читабельно и понятно. Ретурнить функции не комильфо, т.к. может произойти сбой и вернёт не то, что планировалось.

PHP Code:



stock PayDay(playerid) 
{
    GivePlayerMoney(playerid, 600);
    SendClientMessage(playerid, 0xFFFFFFFF, "Вы получили 600 кредитов");
    return 1;
}

Re: Грамотный код - Eims - 26.03.2018

Quote:

Originally Posted by OKStyle

Имхо. Так более читабельно и понятно. Ретурнить функции не комильфо, т.к. может произойти сбой и вернёт не то, что планировалось.

PHP Code:



stock PayDay(playerid) 

{

    GivePlayerMoney(playerid, 600);

    SendClientMessage(playerid, 0xFFFFFFFF, "Вы получили 600 кредитов");

    return 1;

}

Далеко не везде важно что возвращает функция, но вообще соглашусь. Мало того, что чётко не ясно какое именно значение вернёт функция, так ещё и если вдруг захочешь переписать код функции, придётся тратить лишнее время на вычленение "SendClientMessage" из return.

Касаемо оформления скобок.
Вообще в SA-MP закрепился стиль Олмана, так что если ты не хочешь получить недопонимание со стороны сообщества, стоит придерживаться именно его.
Ну а можешь пойти другим путём и начать использовать стиль K&R, который в SA-MP пришёл, вероятнее всего, от тех, кто привык кодить на других языках с этим стилем, а потом всякие понтокодеры решили слепо повторять, чтоб быть "нитакимикаквсе".
Хотя вообще, если ты пишешь код исключительно для себя, то можно использовать любой стиль, который тебе удобен. Но вот когда ты публикуешь код в паблик, лучше, чтоб этот код придерживался распространённого в сообществе (не важно каком) стиля. Тогда все будут счастливы

Re: Грамотный код - NikitaFoxze - 26.03.2018

Quote:

Originally Posted by Eims

Спасибо

Re: Грамотный код - OKStyle - 27.03.2018

Quote:

Originally Posted by Eims

Далеко не везде важно что возвращает функция

Везде. 0 вообще наш худший враг. Так что внимательно надо к ретурнам.