> ## Documentation Index > Fetch the complete documentation index at: https://docs.codeant.ai/llms.txt > Use this file to discover all available pages before exploring further. # Best practice Comparison to boolean. Just use `$X` Comparison to boolean. Just use `not $X` Use instead `Str.first_chars` Use instead `Str.string_after` Use instead `Str.last_chars` Useless sprintf You should not re-raise exceptions using 'raise' because it loses track of where the exception was raised originally, leading to a useless and possibly confusing stack trace. Instead, you should obtain a stack backtrace as soon as the exception is caught using 'try ... with exn -> let trace = Printexc.get\_raw\_backtrace () in ...', and keep it around until you re-raise the exception using 'Printexc.raise\_with\_backtrace exn trace'. You must collect the stack backtrace before calling another function which might internally raise and catch exceptions. To avoid false positives from Semgrep, write 'raise (Foo args)' instead of 'let e = Foo args in raise e'. Useless else. Just remove the else branch; Backwards if. Rewrite the code as 'if not $E then $E2'. You should not use Hashtbl.find outside of a try, or you should use Hashtbl.find\_opt You should not use List.find outside of a try, or you should use List.find\_opt You should use `incr` You should use `decr`