[FilterScript] Calculator
#2

  • Well done.
  • Why do you try to include something that all user has it ( i'm talking about #tryinclude )
  • this can be merged in one, because it does return the same value.
    Код:
                    if !response *then return true;
                            if !strlen(inputtext) *then return true;
  • Someplace you use binary(+0b10 ) and in some you use hex, keep with one for readability.
  • static a, b, c; must be new ( i guess ).
  • Here in this code, is there any reason for checking the length and adding true + == 0 ? its default the true & the == 0
    Код:
    if (strcmp("/calc", cmdtext, true, 10) == 0)
  • Good job.
Reply


Messages In This Thread
Calculator - by FanHamMer - 02.09.2015, 17:43
Re: Calculator - by Freezo - 02.09.2015, 17:54
Re: Calculator - by kaban4ik - 02.09.2015, 17:55
Re: Calculator - by FanHamMer - 02.09.2015, 17:58
Re: Calculator - by Freezo - 02.09.2015, 18:05
Re: Calculator - by FanHamMer - 02.09.2015, 18:10
Re: Calculator - by Michael B - 02.09.2015, 18:14
Re: Calculator - by kaban4ik - 02.09.2015, 18:14
Re: Calculator - by FanHamMer - 02.09.2015, 18:17
Re: Calculator - by Freezo - 02.09.2015, 18:23
Re: Calculator - by Crayder - 02.09.2015, 20:09
Re: Calculator - by Mariciuc223 - 03.09.2015, 06:24
Re: Calculator - by Freezo - 03.09.2015, 19:28

Forum Jump:


Users browsing this thread: 1 Guest(s)