{"id":404258,"date":"2024-06-29T17:36:06","date_gmt":"2024-06-29T17:36:06","guid":{"rendered":"http:\/\/savepearlharbor.com\/?p=404258"},"modified":"-0001-11-30T00:00:00","modified_gmt":"-0001-11-29T21:00:00","slug":"","status":"publish","type":"post","link":"https:\/\/savepearlharbor.com\/?p=404258","title":{"rendered":"<span>RavenDB \u0438 PVS-Studio: \u043a\u043e\u043b\u043b\u0430\u0431\u043e\u0440\u0430\u0446\u0438\u044f, \u043e\u0442 \u043a\u043e\u0442\u043e\u0440\u043e\u0439 \u0432\u044b\u0438\u0433\u0440\u044b\u0432\u0430\u044e\u0442 \u0432\u0441\u0435<\/span>"},"content":{"rendered":"<div><!--[--><!--]--><\/div>\n<div id=\"post-content-body\">\n<div>\n<div class=\"article-formatted-body article-formatted-body article-formatted-body_version-2\">\n<div xmlns=\"http:\/\/www.w3.org\/1999\/xhtml\">\n<p>\u041d\u0435\u0431\u043e\u043b\u044c\u0448\u0430\u044f \u0438\u0441\u0442\u043e\u0440\u0438\u044f \u043f\u0440\u043e \u0441\u043e\u0442\u0440\u0443\u0434\u043d\u0438\u0447\u0435\u0441\u0442\u0432\u043e PVS-Studio \u0438 RavenDB. PVS-Studio \u2014 \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0439 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 \u0434\u043b\u044f \u0443\u043b\u0443\u0447\u0448\u0435\u043d\u0438\u044f \u043a\u043e\u0434\u0430. RavenDB \u2014 Open Source \u0431\u0430\u0437\u0430 \u0434\u0430\u043d\u043d\u044b\u0445. \u041a\u0430\u043a \u043f\u043e\u0438\u0441\u043a \u043e\u0448\u0438\u0431\u043e\u043a \u0432 \u043e\u0434\u043d\u043e\u043c \u043f\u0440\u043e\u0435\u043a\u0442\u0435 \u043f\u0440\u0438\u0432\u043e\u0434\u0438\u0442 \u043a \u0443\u043b\u0443\u0447\u0448\u0435\u043d\u0438\u044e \u0441\u0440\u0430\u0437\u0443 \u0434\u0432\u0443\u0445? \u041e\u0442\u0432\u0435\u0442\u0438\u043c \u043d\u0430 \u044d\u0442\u043e\u0442 \u0432\u043e\u043f\u0440\u043e\u0441, \u0430 \u0442\u0430\u043a\u0436\u0435 \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0438\u043c \u043d\u0430 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f \u043d\u0430\u0439\u0434\u0435\u043d\u043d\u044b\u0445 \u043f\u0440\u043e\u0431\u043b\u0435\u043c \u0438 \u043a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0438 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u043e\u0432.<\/p>\n<figure class=\"full-width\"><img loading=\"lazy\" decoding=\"async\" src=\"https:\/\/habrastorage.org\/r\/w1560\/getpro\/habr\/upload_files\/e9e\/fa8\/6cc\/e9efa86cce1444ccaa605475a4755615.png\" alt=\"1052_Collaboration_of_RavenDB_and_PVSStudio_ru\/image1.png\" title=\"\" width=\"1600\" height=\"902\" data-src=\"https:\/\/habrastorage.org\/getpro\/habr\/upload_files\/e9e\/fa8\/6cc\/e9efa86cce1444ccaa605475a4755615.png\"\/><\/figure>\n<h3>\u041d\u0430\u0447\u0430\u043b\u043e \u043a\u043e\u043b\u043b\u0430\u0431\u043e\u0440\u0430\u0446\u0438\u0438<\/h3>\n<p>\u041f\u0435\u0440\u0435\u0434 \u043d\u0430\u0447\u0430\u043b\u043e\u043c \u0438\u0441\u0442\u043e\u0440\u0438\u0438 \u043d\u0430\u043f\u043e\u043c\u043d\u044e \u043f\u0440\u043e <a href=\"https:\/\/ravendb.net\/\">RavenDB<\/a> \u0438 <a href=\"https:\/\/pvs-studio.ru\/ru\/pvs-studio\/\">PVS-Studio<\/a>. RavenDB \u2014 \u044d\u0442\u043e Open Source \u0434\u043e\u043a\u0443\u043c\u0435\u043d\u0442\u043d\u043e\u043e\u0440\u0438\u0435\u043d\u0442\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u0430\u044f <a href=\"https:\/\/en.wikipedia.org\/wiki\/ACID\">ACID<\/a> \u0431\u0430\u0437\u0430 \u0434\u0430\u043d\u043d\u044b\u0445, \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u044e\u0449\u0430\u044f \u043f\u043e\u0434\u0445\u043e\u0434 NoSQL. PVS-Studio \u2014 \u044d\u0442\u043e \u0438\u043d\u0441\u0442\u0440\u0443\u043c\u0435\u043d\u0442 \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u0433\u043e \u0430\u043d\u0430\u043b\u0438\u0437\u0430, \u043d\u0430\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u043d\u044b\u0439 \u043d\u0430 \u0443\u043b\u0443\u0447\u0448\u0435\u043d\u0438\u0435 \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0430, \u0437\u0430\u0449\u0438\u0449\u0451\u043d\u043d\u043e\u0441\u0442\u0438 (SAST) \u0438 \u0431\u0435\u0437\u043e\u043f\u0430\u0441\u043d\u043e\u0441\u0442\u0438 \u043a\u043e\u0434\u0430 \u0434\u043b\u044f C, C++, C# \u0438 Java.<\/p>\n<p>\u041a\u043e\u043c\u0430\u043d\u0434\u0430 RavenDB \u0440\u0435\u0448\u0438\u043b\u0430 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c PVS-Studio \u0434\u043b\u044f \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 \u043a\u043e\u0434\u0430 \u043f\u0440\u043e\u0435\u043a\u0442\u0430. \u041c\u044b \u0432\u044b\u0441\u0442\u0443\u043f\u0438\u043b\u0438 \u0441\u043e \u0432\u0441\u0442\u0440\u0435\u0447\u043d\u044b\u043c \u043f\u0440\u0435\u0434\u043b\u043e\u0436\u0435\u043d\u0438\u0435\u043c: \u0441\u043d\u0430\u0447\u0430\u043b\u0430 \u043f\u0440\u043e\u0432\u0435\u0441\u0442\u0438 \u0440\u0430\u0437\u043e\u0432\u044b\u0439 \u0430\u043d\u0430\u043b\u0438\u0437, \u0440\u0430\u0437\u043e\u0431\u0440\u0430\u0442\u044c \u043d\u0435\u0441\u043a\u043e\u043b\u044c\u043a\u043e \u043d\u0430\u0438\u0431\u043e\u043b\u0435\u0435 \u0438\u043d\u0442\u0435\u0440\u0435\u0441\u043d\u044b\u0445 \u043e\u0448\u0438\u0431\u043e\u043a \u0438 \u0440\u0430\u0441\u0441\u043a\u0430\u0437\u0430\u0442\u044c \u043e \u043d\u0438\u0445. \u0412 \u0438\u0442\u043e\u0433\u0435 \u0442\u0430\u043a \u0438 \u0440\u0435\u0448\u0438\u043b\u0438 \u0441\u0434\u0435\u043b\u0430\u0442\u044c.<\/p>\n<p>\u041c\u044b \u0432\u044b\u043a\u0430\u0447\u0430\u043b\u0438 <a href=\"https:\/\/github.com\/ravendb\/ravendb\/tree\/e1f32a6b234bcd9e9ea27b1c431dcd3156044aed\">\u0438\u0441\u0445\u043e\u0434\u043d\u0438\u043a\u0438<\/a> \u0438 \u043f\u0440\u0438\u043d\u044f\u043b\u0438\u0441\u044c \u0430\u043d\u0430\u043b\u0438\u0437\u0438\u0440\u043e\u0432\u0430\u0442\u044c. \u0421\u0442\u043e\u0438\u0442 \u0441\u043a\u0430\u0437\u0430\u0442\u044c, \u0447\u0442\u043e \u043a\u043e\u0434 \u043d\u0430\u043f\u0438\u0441\u0430\u043d \u0434\u043e\u0432\u043e\u043b\u044c\u043d\u043e \u0445\u043e\u0440\u043e\u0448\u043e \u0438 \u0447\u0438\u0442\u0430\u0435\u043c\u043e. \u0420\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u043e\u0432 \u0437\u0430 \u044d\u0442\u043e \u043c\u043e\u0436\u043d\u043e \u0442\u043e\u043b\u044c\u043a\u043e \u043f\u043e\u0445\u0432\u0430\u043b\u0438\u0442\u044c. \u041f\u043e\u0441\u043b\u0435 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 \u043c\u044b \u043e\u0442\u043f\u0440\u0430\u0432\u0438\u043b\u0438 \u043a\u043e\u043c\u0430\u043d\u0434\u0435 RavenDB 10 \u043d\u0430\u0438\u0431\u043e\u043b\u0435\u0435 \u043f\u043e\u0434\u043e\u0437\u0440\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u0445, \u043f\u043e \u043d\u0430\u0448\u0435\u043c\u0443 \u043c\u043d\u0435\u043d\u0438\u044e, \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u043e\u0432 \u043a\u043e\u0434\u0430.<\/p>\n<p>\u041e\u0442\u0432\u0435\u0442 \u043d\u0435 \u0437\u0430\u0441\u0442\u0430\u0432\u0438\u043b \u0441\u0435\u0431\u044f \u0434\u043e\u043b\u0433\u043e \u0436\u0434\u0430\u0442\u044c. Head of Development <a href=\"https:\/\/github.com\/ppekrol\">\u041f\u0430\u0432\u0435\u043b \u041f\u0435\u043a\u0440\u043e\u043b<\/a> \u043f\u0440\u0438\u0441\u043b\u0430\u043b \u043d\u0430\u043c \u0444\u0438\u0434\u0431\u044d\u043a \u043f\u043e \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u044f\u043c PVS-Studio \u0438 \u0441\u043e\u0437\u0434\u0430\u043b <a href=\"https:\/\/github.com\/ravendb\/ravendb\/pull\/16234\">pull request<\/a> c \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f\u043c\u0438 \u043e\u043f\u0438\u0441\u0430\u043d\u043d\u044b\u0445 \u043f\u0440\u043e\u0431\u043b\u0435\u043c.<\/p>\n<p>\u0420\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 RavenDB \u043e\u0447\u0435\u043d\u044c \u0431\u044b\u0441\u0442\u0440\u043e \u043e\u0442\u0440\u0435\u0430\u0433\u0438\u0440\u043e\u0432\u0430\u043b\u0438 \u043d\u0430 \u043f\u043e\u0434\u043e\u0437\u0440\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u0435 \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u044b \u043a\u043e\u0434\u0430 \u0438 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438 \u043d\u0430\u0439\u0434\u0435\u043d\u043d\u044b\u0435 \u043e\u0448\u0438\u0431\u043a\u0438. \u0414\u0430\u0432\u0430\u0439\u0442\u0435 \u043f\u0440\u043e\u0439\u0434\u0451\u043c\u0441\u044f \u043f\u043e \u0441\u0430\u043c\u044b\u043c \u0438\u043d\u0442\u0435\u0440\u0435\u0441\u043d\u044b\u043c \u0438\u0437 \u043d\u0438\u0445, \u0440\u0430\u0437\u0431\u0435\u0440\u0451\u043c \u043e\u0448\u0438\u0431\u043a\u0438 \u0438 \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0438\u043c, \u043a\u0430\u043a \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438 \u043a\u043e\u0434.<\/p>\n<h3>\u041e\u0448\u0438\u0431\u043a\u0438 \u0438 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f<\/h3>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 1<\/h4>\n<pre><code class=\"cs\">[Flags] public enum TypeAttributes {   ....   NotPublic = 0,   .... }  private static bool CheckIfAnonymousType(Type type) {   \/\/ hack: the only way to detect anonymous types right now   return type.IsDefined(typeof(CompilerGeneratedAttribute), false)       &amp;&amp; type.IsGenericType &amp;&amp; type.Name.Contains(\"AnonymousType\")       &amp;&amp; (type.Name.StartsWith(\"&lt;>\") || type.Name.StartsWith(\"VB$\"))       &amp;&amp; type.Attributes.HasFlag(TypeAttributes.NotPublic);        \/\/ &lt;= } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3180\/\">V3180<\/a> The &#8216;HasFlag&#8217; method always returns &#8216;true&#8217; because the value of &#8216;TypeAttributes.NotPublic&#8217; is &#8216;0&#8217;, and it is passed as the method&#8217;s argument. ExpressionStringBuilder.cs 2117<\/p>\n<p>\u041c\u0435\u0442\u043e\u0434 <em>HasFlag<\/em> \u0432\u0441\u0435\u0433\u0434\u0430 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 <em>true<\/em>, \u0435\u0441\u043b\u0438 \u0432 \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0435 \u0430\u0440\u0433\u0443\u043c\u0435\u043d\u0442\u0430 \u043f\u0440\u0438\u043d\u0438\u043c\u0430\u0435\u0442 \u0444\u043b\u0430\u0433 \u0441\u043e \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435\u043c 0 (\u0441\u043c. <a href=\"https:\/\/learn.microsoft.com\/en-us\/dotnet\/api\/system.enum.hasflag?view=net-8.0#remarks\">\u0434\u043e\u043a\u0443\u043c\u0435\u043d\u0442\u0430\u0446\u0438\u044e<\/a>). \u041f\u043e\u043b\u0443\u0447\u0430\u0435\u0442\u0441\u044f, \u0447\u0442\u043e \u0447\u0430\u0441\u0442\u044c \u0432\u044b\u0440\u0430\u0436\u0435\u043d\u0438\u044f \u0432\u0441\u0435\u0433\u0434\u0430 \u0440\u0430\u0432\u043d\u0430 <em>true<\/em> \u0438 \u043d\u0435 \u0438\u043c\u0435\u0435\u0442 \u0441\u043c\u044b\u0441\u043b\u0430.<\/p>\n<p>\u041a\u043e\u0434 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438, \u0443\u0434\u0430\u043b\u0438\u0432 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0443 \u0441 <em>HasFlag<\/em>.<\/p>\n<p><a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/d01ba0c60e51f8733ae5cdf1fb8a608b52056b1e\">\u041a\u043e\u043c\u043c\u0438\u0442 \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c<\/a>.<\/p>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 2<\/h4>\n<pre><code class=\"cs\">private JsValue Spatial_Distance(JsValue self, JsValue[] args) {   if (args.Length &lt; 4 &amp;&amp; args.Length > 5) \/\/ &lt;=     throw new ArgumentException(\"....\"); } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3022\/\">V3022<\/a> Expression &#8216;args.Length &lt; 4 &amp;&amp; args.Length > 5&#8217; is always false. Probably the &#8216;||&#8217; operator should be used here. ScriptRunner.cs 930<\/p>\n<p>\u0417\u043d\u0430\u0447\u0435\u043d\u0438\u0435 \u0432\u044b\u0440\u0430\u0436\u0435\u043d\u0438\u044f <em>args.Length<\/em> \u043d\u0435 \u043c\u043e\u0436\u0435\u0442 \u0431\u044b\u0442\u044c \u043e\u0434\u043d\u043e\u0432\u0440\u0435\u043c\u0435\u043d\u043d\u043e \u043c\u0435\u043d\u044c\u0448\u0435 4 \u0438 \u0431\u043e\u043b\u044c\u0448\u0435 5, \u0441\u043b\u0435\u0434\u043e\u0432\u0430\u0442\u0435\u043b\u044c\u043d\u043e, \u044d\u0442\u043e \u0443\u0441\u043b\u043e\u0432\u0438\u0435 \u0432\u0441\u0435\u0433\u0434\u0430 \u043b\u043e\u0436\u043d\u043e. \u0418\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435 \u043e\u0447\u0435\u0432\u0438\u0434\u043d\u043e \u2014 \u043f\u043e\u043c\u0435\u043d\u044f\u0442\u044c &#8216;&amp;&amp;&#8217; \u043d\u0430 &#8216;||&#8217;.<\/p>\n<p>\u041e\u0434\u043d\u0430\u043a\u043e \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 RavenDB \u0437\u0430\u043c\u0435\u0442\u0438\u043b\u0438 \u0435\u0449\u0451 \u043e\u0434\u043d\u0443 \u043e\u0448\u0438\u0431\u043a\u0443. \u041a\u0430\u043a \u043e\u043a\u0430\u0437\u0430\u043b\u043e\u0441\u044c, \u0432\u0435\u0440\u0445\u043d\u0435\u0439 \u0433\u0440\u0430\u043d\u0438\u0446\u0435\u0439 \u0434\u043b\u044f \u043a\u043e\u043b\u0438\u0447\u0435\u0441\u0442\u0432\u0430 \u0430\u0440\u0433\u0443\u043c\u0435\u043d\u0442\u043e\u0432 \u0434\u043e\u043b\u0436\u043d\u043e \u0431\u044b\u0442\u044c \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 6, \u0430 \u043d\u0435 5.<\/p>\n<p>\u0418\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u043d\u044b\u0439 \u043a\u043e\u0434 \u0432\u044b\u0433\u043b\u044f\u0434\u0438\u0442 \u0442\u0430\u043a:<\/p>\n<pre><code class=\"cs\">private static JsValue Spatial_Distance(JsValue self, JsValue[] args) {   if (args.Length is &lt; 4 or > 6)     throw new ArgumentException(\"....\"); } <\/code><\/pre>\n<p>\u041a\u043e\u043c\u043c\u0438\u0442\u044b \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f\u043c\u0438: <a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/bd487885bce7c3e1a1a441533c2cd106d7bce304\">\u0440\u0430\u0437<\/a>, <a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/0e5185dfbf367ee14ad4c647aa85f557b899f90d\">\u0434\u0432\u0430<\/a>.<\/p>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 3<\/h4>\n<pre><code class=\"cs\">public override DynamicJsonValue ToJson() {   var json = base.ToJson();   json[nameof(DataDirectory)] = DataDirectory;   json[nameof(JournalStoragePath)] = DataDirectory;      \/\/ &lt;=   json[nameof(SnapshotRestore)] = SnapshotRestore.ToJson();   return json; } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3127\/\">V3127<\/a> Two similar code fragments were found. Perhaps, this is a typo and &#8216;JournalStoragePath&#8217; variable should be used instead of &#8216;DataDirectory&#8217; RestoreResult.cs 24<\/p>\n<p>\u0418\u0441\u0445\u043e\u0434\u044f \u0438\u0437 \u0442\u043e\u0433\u043e, \u0447\u0442\u043e \u0432 <em>json[nameof(DataDirectory)]<\/em> \u0437\u0430\u043f\u0438\u0441\u044b\u0432\u0430\u0435\u0442\u0441\u044f \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 \u0441\u0432\u043e\u0439\u0441\u0442\u0432\u0430 <em>DataDirectory<\/em>, \u0432 <em>json[nameof(JournalStoragePath)]<\/em> \u0434\u043e\u043b\u0436\u043d\u043e \u0437\u0430\u043f\u0438\u0441\u044b\u0432\u0430\u0442\u044c\u0441\u044f \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 \u0441\u0432\u043e\u0439\u0441\u0442\u0432\u0430 <em>JournalStoragePath<\/em>. \u0418\u043c\u0435\u043d\u043d\u043e \u0442\u0430\u043a \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 \u0438 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438 \u043e\u0448\u0438\u0431\u043a\u0443:<\/p>\n<pre><code class=\"cs\">json[nameof(JournalStoragePath)] = JournalStoragePath; <\/code><\/pre>\n<p><a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/c6c14145b97b74db353e7f00480d75a168839634\">\u041a\u043e\u043c\u043c\u0438\u0442 \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c<\/a>.<\/p>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 4<\/h4>\n<pre><code class=\"cs\">public override int GetHashCode() {   unchecked   {     var hashCode = MinimumRevisionsToKeep?.GetHashCode() ?? 0;     hashCode = (hashCode * 397) ^        MinimumRevisionAgeToKeep?.GetHashCode() ?? 0; \/\/ &lt;=     hashCode = (hashCode * 397) ^        Disabled.GetHashCode();     hashCode = (hashCode * 397) ^       PurgeOnDelete.GetHashCode();     return hashCode;   } } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3123\/\">V3123<\/a> Perhaps the &#8216;??&#8217; operator works in a different way than it was expected. Its priority is lower than priority of other operators in its left part. RevisionsCollectionConfiguration.cs 40<\/p>\n<p>\u0417\u0434\u0435\u0441\u044c PVS-Studio \u0443\u043a\u0430\u0437\u044b\u0432\u0430\u0435\u0442 \u043d\u0430 \u0442\u043e, \u0447\u0442\u043e \u0432\u043e\u0437\u043c\u043e\u0436\u043d\u0430 \u043e\u0448\u0438\u0431\u043a\u0430 \u0441 \u043f\u0440\u0438\u043e\u0440\u0438\u0442\u0435\u0442\u043e\u043c \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u0430 &#8216;??&#8217;. \u0418 \u0434\u0435\u0439\u0441\u0442\u0432\u0438\u0442\u0435\u043b\u044c\u043d\u043e, \u043f\u0440\u0438\u043e\u0440\u0438\u0442\u0435\u0442 \u0443 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u0430 &#8216;^&#8217; \u0432\u044b\u0448\u0435, \u0447\u0435\u043c\u0443 \u0443 &#8216;??&#8217;. \u0415\u0441\u043b\u0438 <em>MinimumRevisionAgeToKeep<\/em> \u0431\u0443\u0434\u0435\u0442 \u0438\u043c\u0435\u0442\u044c \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 <em>null<\/em>, \u0442\u043e \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442 \u0432\u044b\u0447\u0438\u0441\u043b\u0435\u043d\u0438\u044f <em>hashCode<\/em> \u0432 \u0432\u044b\u0434\u0435\u043b\u0435\u043d\u043d\u043e\u0439 \u0441\u0442\u0440\u043e\u043a\u0435 \u0432\u0441\u0435\u0433\u0434\u0430 \u0431\u0443\u0434\u0435\u0442 \u0440\u0430\u0432\u0435\u043d 0. \u0414\u043b\u044f \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f \u0434\u043e\u0441\u0442\u0430\u0442\u043e\u0447\u043d\u043e \u0432\u0437\u044f\u0442\u044c \u0432\u044b\u0440\u0430\u0436\u0435\u043d\u0438\u0435 \u0441 &#8216;??&#8217; \u0432 \u0441\u043a\u043e\u0431\u043a\u0438.<\/p>\n<p>\u041a\u0441\u0442\u0430\u0442\u0438, \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 \u0437\u0430\u043c\u0435\u0442\u0438\u043b\u0438 \u0435\u0449\u0451 \u043e\u0434\u043d\u0443 \u043e\u0448\u0438\u0431\u043a\u0443. \u041f\u0440\u0438 \u0432\u044b\u0447\u0438\u0441\u043b\u0435\u043d\u0438\u0438 <em>hashCode<\/em> \u0437\u0430\u0431\u044b\u043b\u0438 \u043f\u0440\u043e \u0441\u0432\u043e\u0439\u0441\u0442\u0432\u043e <em>MaximumRevisionsToDeleteUponDocumentUpdate<\/em>.<\/p>\n<p>\u0418\u0442\u043e\u0433\u043e\u0432\u044b\u0439 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u043d\u044b\u0439 \u043a\u043e\u0434:<\/p>\n<pre><code class=\"cs\">public override int GetHashCode() {   unchecked   {     var hashCode = MinimumRevisionsToKeep?.GetHashCode() ?? 0;     hashCode = (hashCode * 397) ^        (MinimumRevisionAgeToKeep?.GetHashCode() ?? 0);     hashCode = (hashCode * 397) ^        (MaximumRevisionsToDeleteUponDocumentUpdate?.GetHashCode() ?? 0);     hashCode = (hashCode * 397) ^        Disabled.GetHashCode();     hashCode = (hashCode * 397) ^        PurgeOnDelete.GetHashCode();     return hashCode;   } } <\/code><\/pre>\n<p><a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/78e57c9d293bbd1812031ea7e761dc5b981b5684\">\u041a\u043e\u043c\u043c\u0438\u0442 \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c<\/a>.<\/p>\n<p>\u0421\u043b\u043e\u0432\u0430 \u0440\u0443\u043a\u043e\u0432\u043e\u0434\u0438\u0442\u0435\u043b\u044f \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0438 \u041f\u0430\u0432\u043b\u0430 \u041f\u0435\u043a\u0440\u043e\u043b\u0430 \u043f\u0440\u043e \u044d\u0442\u0443 \u043e\u0448\u0438\u0431\u043a\u0443:<\/p>\n<blockquote>\n<p>Point 4 &#8212; gold \ud83d\ude42<\/p>\n<\/blockquote>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 5<\/h4>\n<pre><code class=\"cs\">public override void VisitMethod(MethodExpression expr) {   if (expr.Name.Value == \"id\" &amp;&amp; expr.Arguments.Count == 0)   {     if (expr.Arguments.Count != 1)   \/\/ &lt;=     {       throw new InvalidOperationException(\"....\");     }     _sb.Append(\"this\");   } } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3022\/\">V3022<\/a> Expression &#8216;expr.Arguments.Count != 1&#8217; is always true. JavascriptCodeQueryVisitor.cs 188<\/p>\n<p>\u041f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 <em>expr.Arguments.Count != 1<\/em> \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u0435\u0442\u0441\u044f \u0441\u0440\u0430\u0437\u0443 \u0436\u0435 \u043f\u043e\u0441\u043b\u0435 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 <em>expr.Arguments.Count == 0<\/em>. \u041e\u0447\u0435\u0432\u0438\u0434\u043d\u043e, \u0447\u0442\u043e \u0435\u0441\u043b\u0438 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 <em>expr.Arguments.Count<\/em> \u0440\u0430\u0432\u043d\u043e 0, \u0442\u043e \u043e\u043d\u043e \u0442\u0430\u043a\u0436\u0435 \u043d\u0435 \u0440\u0430\u0432\u043d\u043e 1. \u0415\u0441\u043b\u0438 \u043f\u043e\u0442\u043e\u043a \u0432\u044b\u043f\u043e\u043b\u043d\u0435\u043d\u0438\u044f \u0437\u0430\u0445\u043e\u0434\u0438\u0442 \u0432 \u0432\u0435\u0440\u0445\u043d\u0438\u0439 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440 <em>if<\/em>, \u0442\u043e \u0432\u0441\u0435\u0433\u0434\u0430 \u0431\u0443\u0434\u0435\u0442 \u0431\u0440\u043e\u0441\u0430\u0442\u044c\u0441\u044f \u0438\u0441\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u0435. \u041a\u0430\u043a \u0441\u043b\u0435\u0434\u0441\u0442\u0432\u0438\u0435, <em>_sb.Append(&#171;this&#187;)<\/em> \u043d\u0438\u043a\u043e\u0433\u0434\u0430 \u043d\u0435 \u0432\u044b\u043f\u043e\u043b\u043d\u0438\u0442\u0441\u044f.<\/p>\n<p>\u041f\u043e\u043b\u0443\u0447\u0430\u0435\u0442\u0441\u044f, \u0447\u0442\u043e \u043d\u0430 \u0442\u0435\u0441\u0442\u0430\u0445 \u043f\u0440\u043e\u0441\u0442\u043e &#171;\u0432\u0435\u0437\u043b\u043e&#187;: \u044d\u0442\u043e\u0442 \u043a\u043e\u0434 \u043d\u0435 \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u043b\u0441\u044f, \u0438\u043d\u0430\u0447\u0435 \u0431\u044b \u0432\u043e\u0437\u043d\u0438\u043a\u0430\u043b\u043e \u0438\u0441\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u0435. \u041f\u043e\u0434\u043e\u0431\u043d\u044b\u0435 \u043e\u0448\u0438\u0431\u043a\u0438 \u0441\u0442\u0440\u0430\u0448\u043d\u044b \u0442\u0435\u043c, \u0447\u0442\u043e \u043e\u043d\u0438 \u043d\u0435 \u0441\u0442\u0440\u0435\u043b\u044f\u044e\u0442 \u043d\u0430 \u0442\u0435\u0441\u0442\u0430\u0445, \u043d\u043e \u043c\u043e\u0433\u0443\u0442 \u0432\u044b\u0441\u0442\u0440\u0435\u043b\u0438\u0442\u044c \u0443 \u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u0435\u043b\u044f.<\/p>\n<p>\u0418\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c \u0441\u0442\u0430\u043b\u043e \u0443\u0434\u0430\u043b\u0435\u043d\u0438\u0435 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u0430 <em>if<\/em> \u0441 \u0432\u044b\u0431\u0440\u0430\u0441\u044b\u0432\u0430\u043d\u0438\u0435\u043c \u0438\u0441\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u044f.<\/p>\n<p><a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/f91d886ab9eb931d97a71ec66ac6453ff3099f03\">\u041a\u043e\u043c\u043c\u0438\u0442 \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c<\/a>.<\/p>\n<h3>\u0412\u044b\u0432\u043e\u0434\u044b<\/h3>\n<p>\u0412 \u043d\u0430\u0437\u0432\u0430\u043d\u0438\u0438 \u0441\u0442\u0430\u0442\u044c\u0438 \u043d\u0430\u043f\u0438\u0441\u0430\u043d\u043e, \u0447\u0442\u043e \u0432\u044b\u0438\u0433\u0440\u044b\u0432\u0430\u0435\u0442 \u043a\u0430\u0436\u0434\u044b\u0439. \u0412 \u0447\u0451\u043c \u0432\u044b\u0438\u0433\u0440\u0430\u043b RavenDB? \u0415\u0441\u0442\u0435\u0441\u0442\u0432\u0435\u043d\u043d\u043e, \u0447\u0442\u043e \u043a\u0430\u0436\u0434\u044b\u0439 \u043f\u0440\u043e\u0434\u0443\u043a\u0442 \u043a\u0430\u043a-\u0442\u043e \u0440\u0430\u0437\u0432\u0438\u0432\u0430\u0435\u0442\u0441\u044f \u0438 \u0441\u043e\u0432\u0435\u0440\u0448\u0435\u043d\u0441\u0442\u0432\u0443\u0435\u0442\u0441\u044f. \u0412 \u044d\u0442\u043e\u0442 \u0440\u0430\u0437 \u0443\u043b\u0443\u0447\u0448\u0435\u043d\u0438\u0435 \u043f\u0440\u043e\u0438\u0437\u043e\u0448\u043b\u043e \u043d\u0435 \u0447\u0435\u0440\u0435\u0437 \u043d\u043e\u0432\u044b\u0435 \u0444\u0438\u0447\u0438, \u0430 \u0447\u0435\u0440\u0435\u0437 \u043f\u043e\u0432\u044b\u0448\u0435\u043d\u0438\u0435 \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0430 \u0438\u0441\u0445\u043e\u0434\u043d\u043e\u0433\u043e \u043a\u043e\u0434\u0430. \u0412 \u0447\u0451\u043c \u0436\u0435 \u0432\u044b\u0438\u0433\u0440\u0430\u043b PVS-Studio? \u0412\u043e \u0432\u0440\u0435\u043c\u044f \u043f\u0440\u043e\u0441\u043c\u043e\u0442\u0440\u0430 \u043e\u0442\u0447\u0451\u0442\u0430 \u043e\u0431 \u043e\u0448\u0438\u0431\u043a\u0430\u0445 \u043c\u044b \u043d\u0430\u0448\u043b\u0438 \u043d\u0435\u0441\u043a\u043e\u043b\u044c\u043a\u043e \u043b\u043e\u0436\u043d\u044b\u0445 \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0439. \u041d\u0435\u043a\u043e\u0442\u043e\u0440\u044b\u0435 \u0438\u0437 \u043d\u0438\u0445 \u043c\u044b \u0443\u0436\u0435 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438, \u0430 \u0434\u043e \u043d\u0435\u043a\u043e\u0442\u043e\u0440\u044b\u0445 \u0435\u0449\u0451 \u043f\u0440\u0435\u0434\u0441\u0442\u043e\u0438\u0442 \u0434\u043e\u0431\u0440\u0430\u0442\u044c\u0441\u044f.<\/p>\n<p>\u0420\u0443\u043a\u043e\u0432\u043e\u0434\u0438\u0442\u0435\u043b\u044c \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0438 RavenDB \u041f\u0430\u0432\u0435\u043b \u041f\u0435\u043a\u0440\u043e\u043b:<\/p>\n<blockquote>\n<p>Going over the list made me realize that devs are only humans and no matter how many tests you have there will always be bugs. The tool is very helpful in finding obvious and non-obvious bugs.<\/p>\n<\/blockquote>\n<p>\u041d\u0435 \u043c\u043e\u0433\u0443 \u043d\u0435 \u0441\u043e\u0433\u043b\u0430\u0441\u0438\u0442\u044c\u0441\u044f. \u0420\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a \u043f\u0440\u043e\u0441\u0442\u043e \u043d\u0435 \u043c\u043e\u0436\u0435\u0442 \u043f\u043e\u043b\u043d\u043e\u0441\u0442\u044c\u044e \u043f\u043e\u043a\u0440\u044b\u0442\u044c \u043a\u043e\u0434 \u0442\u0435\u0441\u0442\u0430\u043c\u0438. \u0415\u0441\u043b\u0438 \u0432\u044b \u0437\u0430\u0431\u043e\u0442\u0438\u0442\u0435\u0441\u044c \u043e \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0435 \u043a\u043e\u0434\u0430 \u0438 \u043f\u0440\u043e\u0435\u043a\u0442\u0430, \u0442\u043e \u0441\u0442\u043e\u0438\u0442 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c \u0440\u0430\u0437\u043b\u0438\u0447\u043d\u044b\u0435 \u0438\u043d\u0441\u0442\u0440\u0443\u043c\u0435\u043d\u0442\u044b, \u043a\u043e\u0442\u043e\u0440\u044b\u0435 \u043f\u043e\u043c\u043e\u0433\u0443\u0442 \u0441\u0434\u0435\u043b\u0430\u0442\u044c \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0443 \u043f\u0440\u043e\u0449\u0435.<\/p>\n<p>\u0418\u0437\u0431\u0430\u0432\u043b\u044f\u0439\u0442\u0435\u0441\u044c \u043e\u0442 \u0431\u0430\u0433\u043e\u0432 \u0438 \u0434\u0435\u043b\u0430\u0439\u0442\u0435 \u0441\u0432\u043e\u0439 \u043a\u043e\u0434 \u0431\u0435\u0437\u043e\u043f\u0430\u0441\u043d\u0435\u0435 \u0441 \u043f\u043e\u043c\u043e\u0449\u044c\u044e <a href=\"https:\/\/pvs-studio.ru\/ru\/pvs-studio\/\">PVS-Studio<\/a>! \u0423\u0434\u043e\u0431\u043d\u043e \u0438 \u043f\u0440\u043e\u0441\u0442\u043e \u0445\u0440\u0430\u043d\u0438\u0442\u0435 \u0438 \u043e\u0431\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u0439\u0442\u0435 \u0434\u0430\u043d\u043d\u044b\u0435 \u0432\u043c\u0435\u0441\u0442\u0435 \u0441 <a href=\"https:\/\/ravendb.net\/\">RavenDB<\/a>!<\/p>\n<p>\u041f\u0440\u0438\u044f\u0442\u043d\u043e\u0433\u043e \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043d\u0438\u044f RavenDB \u0438 PVS-Studio!<\/p>\n<p>\u0415\u0441\u043b\u0438 \u0445\u043e\u0442\u0438\u0442\u0435 \u043f\u043e\u0434\u0435\u043b\u0438\u0442\u044c\u0441\u044f \u044d\u0442\u043e\u0439 \u0441\u0442\u0430\u0442\u044c\u0435\u0439 \u0441 \u0430\u043d\u0433\u043b\u043e\u044f\u0437\u044b\u0447\u043d\u043e\u0439 \u0430\u0443\u0434\u0438\u0442\u043e\u0440\u0438\u0435\u0439, \u0442\u043e \u043f\u0440\u043e\u0448\u0443 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c \u0441\u0441\u044b\u043b\u043a\u0443 \u043d\u0430 \u043f\u0435\u0440\u0435\u0432\u043e\u0434: Artem Rovenskii. <a href=\"https:\/\/pvs-studio.com\/en\/blog\/posts\/csharp\/1052\/\">RavenDB and PVS-Studio: win-win collaboration<\/a>.<\/p>\n<\/p>\n<\/div>\n<\/div>\n<\/div>\n<p><!----><!----><\/div>\n<p><!----><!----><br \/> \u0441\u0441\u044b\u043b\u043a\u0430 \u043d\u0430 \u043e\u0440\u0438\u0433\u0438\u043d\u0430\u043b \u0441\u0442\u0430\u0442\u044c\u0438 <a href=\"https:\/\/habr.com\/ru\/articles\/737224\/\"> https:\/\/habr.com\/ru\/articles\/737224\/<\/a><\/p>\n","protected":false},"excerpt":{"rendered":"<div><!--[--><!--]--><\/div>\n<div id=\"post-content-body\">\n<div>\n<div class=\"article-formatted-body article-formatted-body article-formatted-body_version-2\">\n<div xmlns=\"http:\/\/www.w3.org\/1999\/xhtml\">\n<p>\u041d\u0435\u0431\u043e\u043b\u044c\u0448\u0430\u044f \u0438\u0441\u0442\u043e\u0440\u0438\u044f \u043f\u0440\u043e \u0441\u043e\u0442\u0440\u0443\u0434\u043d\u0438\u0447\u0435\u0441\u0442\u0432\u043e PVS-Studio \u0438 RavenDB. PVS-Studio \u2014 \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0439 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 \u0434\u043b\u044f \u0443\u043b\u0443\u0447\u0448\u0435\u043d\u0438\u044f \u043a\u043e\u0434\u0430. RavenDB \u2014 Open Source \u0431\u0430\u0437\u0430 \u0434\u0430\u043d\u043d\u044b\u0445. \u041a\u0430\u043a \u043f\u043e\u0438\u0441\u043a \u043e\u0448\u0438\u0431\u043e\u043a \u0432 \u043e\u0434\u043d\u043e\u043c \u043f\u0440\u043e\u0435\u043a\u0442\u0435 \u043f\u0440\u0438\u0432\u043e\u0434\u0438\u0442 \u043a \u0443\u043b\u0443\u0447\u0448\u0435\u043d\u0438\u044e \u0441\u0440\u0430\u0437\u0443 \u0434\u0432\u0443\u0445? \u041e\u0442\u0432\u0435\u0442\u0438\u043c \u043d\u0430 \u044d\u0442\u043e\u0442 \u0432\u043e\u043f\u0440\u043e\u0441, \u0430 \u0442\u0430\u043a\u0436\u0435 \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0438\u043c \u043d\u0430 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f \u043d\u0430\u0439\u0434\u0435\u043d\u043d\u044b\u0445 \u043f\u0440\u043e\u0431\u043b\u0435\u043c \u0438 \u043a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0438 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u043e\u0432.<\/p>\n<figure class=\"full-width\"><\/figure>\n<h3>\u041d\u0430\u0447\u0430\u043b\u043e \u043a\u043e\u043b\u043b\u0430\u0431\u043e\u0440\u0430\u0446\u0438\u0438<\/h3>\n<p>\u041f\u0435\u0440\u0435\u0434 \u043d\u0430\u0447\u0430\u043b\u043e\u043c \u0438\u0441\u0442\u043e\u0440\u0438\u0438 \u043d\u0430\u043f\u043e\u043c\u043d\u044e \u043f\u0440\u043e <a href=\"https:\/\/ravendb.net\/\">RavenDB<\/a> \u0438 <a href=\"https:\/\/pvs-studio.ru\/ru\/pvs-studio\/\">PVS-Studio<\/a>. RavenDB \u2014 \u044d\u0442\u043e Open Source \u0434\u043e\u043a\u0443\u043c\u0435\u043d\u0442\u043d\u043e\u043e\u0440\u0438\u0435\u043d\u0442\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u0430\u044f <a href=\"https:\/\/en.wikipedia.org\/wiki\/ACID\">ACID<\/a> \u0431\u0430\u0437\u0430 \u0434\u0430\u043d\u043d\u044b\u0445, \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u044e\u0449\u0430\u044f \u043f\u043e\u0434\u0445\u043e\u0434 NoSQL. PVS-Studio \u2014 \u044d\u0442\u043e \u0438\u043d\u0441\u0442\u0440\u0443\u043c\u0435\u043d\u0442 \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u0433\u043e \u0430\u043d\u0430\u043b\u0438\u0437\u0430, \u043d\u0430\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u043d\u044b\u0439 \u043d\u0430 \u0443\u043b\u0443\u0447\u0448\u0435\u043d\u0438\u0435 \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0430, \u0437\u0430\u0449\u0438\u0449\u0451\u043d\u043d\u043e\u0441\u0442\u0438 (SAST) \u0438 \u0431\u0435\u0437\u043e\u043f\u0430\u0441\u043d\u043e\u0441\u0442\u0438 \u043a\u043e\u0434\u0430 \u0434\u043b\u044f C, C++, C# \u0438 Java.<\/p>\n<p>\u041a\u043e\u043c\u0430\u043d\u0434\u0430 RavenDB \u0440\u0435\u0448\u0438\u043b\u0430 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c PVS-Studio \u0434\u043b\u044f \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 \u043a\u043e\u0434\u0430 \u043f\u0440\u043e\u0435\u043a\u0442\u0430. \u041c\u044b \u0432\u044b\u0441\u0442\u0443\u043f\u0438\u043b\u0438 \u0441\u043e \u0432\u0441\u0442\u0440\u0435\u0447\u043d\u044b\u043c \u043f\u0440\u0435\u0434\u043b\u043e\u0436\u0435\u043d\u0438\u0435\u043c: \u0441\u043d\u0430\u0447\u0430\u043b\u0430 \u043f\u0440\u043e\u0432\u0435\u0441\u0442\u0438 \u0440\u0430\u0437\u043e\u0432\u044b\u0439 \u0430\u043d\u0430\u043b\u0438\u0437, \u0440\u0430\u0437\u043e\u0431\u0440\u0430\u0442\u044c \u043d\u0435\u0441\u043a\u043e\u043b\u044c\u043a\u043e \u043d\u0430\u0438\u0431\u043e\u043b\u0435\u0435 \u0438\u043d\u0442\u0435\u0440\u0435\u0441\u043d\u044b\u0445 \u043e\u0448\u0438\u0431\u043e\u043a \u0438 \u0440\u0430\u0441\u0441\u043a\u0430\u0437\u0430\u0442\u044c \u043e \u043d\u0438\u0445. \u0412 \u0438\u0442\u043e\u0433\u0435 \u0442\u0430\u043a \u0438 \u0440\u0435\u0448\u0438\u043b\u0438 \u0441\u0434\u0435\u043b\u0430\u0442\u044c.<\/p>\n<p>\u041c\u044b \u0432\u044b\u043a\u0430\u0447\u0430\u043b\u0438 <a href=\"https:\/\/github.com\/ravendb\/ravendb\/tree\/e1f32a6b234bcd9e9ea27b1c431dcd3156044aed\">\u0438\u0441\u0445\u043e\u0434\u043d\u0438\u043a\u0438<\/a> \u0438 \u043f\u0440\u0438\u043d\u044f\u043b\u0438\u0441\u044c \u0430\u043d\u0430\u043b\u0438\u0437\u0438\u0440\u043e\u0432\u0430\u0442\u044c. \u0421\u0442\u043e\u0438\u0442 \u0441\u043a\u0430\u0437\u0430\u0442\u044c, \u0447\u0442\u043e \u043a\u043e\u0434 \u043d\u0430\u043f\u0438\u0441\u0430\u043d \u0434\u043e\u0432\u043e\u043b\u044c\u043d\u043e \u0445\u043e\u0440\u043e\u0448\u043e \u0438 \u0447\u0438\u0442\u0430\u0435\u043c\u043e. \u0420\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u043e\u0432 \u0437\u0430 \u044d\u0442\u043e \u043c\u043e\u0436\u043d\u043e \u0442\u043e\u043b\u044c\u043a\u043e \u043f\u043e\u0445\u0432\u0430\u043b\u0438\u0442\u044c. \u041f\u043e\u0441\u043b\u0435 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 \u043c\u044b \u043e\u0442\u043f\u0440\u0430\u0432\u0438\u043b\u0438 \u043a\u043e\u043c\u0430\u043d\u0434\u0435 RavenDB 10 \u043d\u0430\u0438\u0431\u043e\u043b\u0435\u0435 \u043f\u043e\u0434\u043e\u0437\u0440\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u0445, \u043f\u043e \u043d\u0430\u0448\u0435\u043c\u0443 \u043c\u043d\u0435\u043d\u0438\u044e, \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u043e\u0432 \u043a\u043e\u0434\u0430.<\/p>\n<p>\u041e\u0442\u0432\u0435\u0442 \u043d\u0435 \u0437\u0430\u0441\u0442\u0430\u0432\u0438\u043b \u0441\u0435\u0431\u044f \u0434\u043e\u043b\u0433\u043e \u0436\u0434\u0430\u0442\u044c. Head of Development <a href=\"https:\/\/github.com\/ppekrol\">\u041f\u0430\u0432\u0435\u043b \u041f\u0435\u043a\u0440\u043e\u043b<\/a> \u043f\u0440\u0438\u0441\u043b\u0430\u043b \u043d\u0430\u043c \u0444\u0438\u0434\u0431\u044d\u043a \u043f\u043e \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u044f\u043c PVS-Studio \u0438 \u0441\u043e\u0437\u0434\u0430\u043b <a href=\"https:\/\/github.com\/ravendb\/ravendb\/pull\/16234\">pull request<\/a> c \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f\u043c\u0438 \u043e\u043f\u0438\u0441\u0430\u043d\u043d\u044b\u0445 \u043f\u0440\u043e\u0431\u043b\u0435\u043c.<\/p>\n<p>\u0420\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 RavenDB \u043e\u0447\u0435\u043d\u044c \u0431\u044b\u0441\u0442\u0440\u043e \u043e\u0442\u0440\u0435\u0430\u0433\u0438\u0440\u043e\u0432\u0430\u043b\u0438 \u043d\u0430 \u043f\u043e\u0434\u043e\u0437\u0440\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u0435 \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u044b \u043a\u043e\u0434\u0430 \u0438 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438 \u043d\u0430\u0439\u0434\u0435\u043d\u043d\u044b\u0435 \u043e\u0448\u0438\u0431\u043a\u0438. \u0414\u0430\u0432\u0430\u0439\u0442\u0435 \u043f\u0440\u043e\u0439\u0434\u0451\u043c\u0441\u044f \u043f\u043e \u0441\u0430\u043c\u044b\u043c \u0438\u043d\u0442\u0435\u0440\u0435\u0441\u043d\u044b\u043c \u0438\u0437 \u043d\u0438\u0445, \u0440\u0430\u0437\u0431\u0435\u0440\u0451\u043c \u043e\u0448\u0438\u0431\u043a\u0438 \u0438 \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0438\u043c, \u043a\u0430\u043a \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438 \u043a\u043e\u0434.<\/p>\n<h3>\u041e\u0448\u0438\u0431\u043a\u0438 \u0438 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f<\/h3>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 1<\/h4>\n<pre><code class=\"cs\">[Flags] public enum TypeAttributes {   ....   NotPublic = 0,   .... }  private static bool CheckIfAnonymousType(Type type) {   \/\/ hack: the only way to detect anonymous types right now   return type.IsDefined(typeof(CompilerGeneratedAttribute), false)       &amp;&amp; type.IsGenericType &amp;&amp; type.Name.Contains(\"AnonymousType\")       &amp;&amp; (type.Name.StartsWith(\"&lt;>\") || type.Name.StartsWith(\"VB$\"))       &amp;&amp; type.Attributes.HasFlag(TypeAttributes.NotPublic);        \/\/ &lt;= } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3180\/\">V3180<\/a> The &#8216;HasFlag&#8217; method always returns &#8216;true&#8217; because the value of &#8216;TypeAttributes.NotPublic&#8217; is &#8216;0&#8217;, and it is passed as the method&#8217;s argument. ExpressionStringBuilder.cs 2117<\/p>\n<p>\u041c\u0435\u0442\u043e\u0434 <em>HasFlag<\/em> \u0432\u0441\u0435\u0433\u0434\u0430 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 <em>true<\/em>, \u0435\u0441\u043b\u0438 \u0432 \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0435 \u0430\u0440\u0433\u0443\u043c\u0435\u043d\u0442\u0430 \u043f\u0440\u0438\u043d\u0438\u043c\u0430\u0435\u0442 \u0444\u043b\u0430\u0433 \u0441\u043e \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435\u043c 0 (\u0441\u043c. <a href=\"https:\/\/learn.microsoft.com\/en-us\/dotnet\/api\/system.enum.hasflag?view=net-8.0#remarks\">\u0434\u043e\u043a\u0443\u043c\u0435\u043d\u0442\u0430\u0446\u0438\u044e<\/a>). \u041f\u043e\u043b\u0443\u0447\u0430\u0435\u0442\u0441\u044f, \u0447\u0442\u043e \u0447\u0430\u0441\u0442\u044c \u0432\u044b\u0440\u0430\u0436\u0435\u043d\u0438\u044f \u0432\u0441\u0435\u0433\u0434\u0430 \u0440\u0430\u0432\u043d\u0430 <em>true<\/em> \u0438 \u043d\u0435 \u0438\u043c\u0435\u0435\u0442 \u0441\u043c\u044b\u0441\u043b\u0430.<\/p>\n<p>\u041a\u043e\u0434 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438, \u0443\u0434\u0430\u043b\u0438\u0432 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0443 \u0441 <em>HasFlag<\/em>.<\/p>\n<p><a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/d01ba0c60e51f8733ae5cdf1fb8a608b52056b1e\">\u041a\u043e\u043c\u043c\u0438\u0442 \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c<\/a>.<\/p>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 2<\/h4>\n<pre><code class=\"cs\">private JsValue Spatial_Distance(JsValue self, JsValue[] args) {   if (args.Length &lt; 4 &amp;&amp; args.Length > 5) \/\/ &lt;=     throw new ArgumentException(\"....\"); } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3022\/\">V3022<\/a> Expression &#8216;args.Length &lt; 4 &amp;&amp; args.Length > 5&#8217; is always false. Probably the &#8216;||&#8217; operator should be used here. ScriptRunner.cs 930<\/p>\n<p>\u0417\u043d\u0430\u0447\u0435\u043d\u0438\u0435 \u0432\u044b\u0440\u0430\u0436\u0435\u043d\u0438\u044f <em>args.Length<\/em> \u043d\u0435 \u043c\u043e\u0436\u0435\u0442 \u0431\u044b\u0442\u044c \u043e\u0434\u043d\u043e\u0432\u0440\u0435\u043c\u0435\u043d\u043d\u043e \u043c\u0435\u043d\u044c\u0448\u0435 4 \u0438 \u0431\u043e\u043b\u044c\u0448\u0435 5, \u0441\u043b\u0435\u0434\u043e\u0432\u0430\u0442\u0435\u043b\u044c\u043d\u043e, \u044d\u0442\u043e \u0443\u0441\u043b\u043e\u0432\u0438\u0435 \u0432\u0441\u0435\u0433\u0434\u0430 \u043b\u043e\u0436\u043d\u043e. \u0418\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435 \u043e\u0447\u0435\u0432\u0438\u0434\u043d\u043e \u2014 \u043f\u043e\u043c\u0435\u043d\u044f\u0442\u044c &#8216;&amp;&amp;&#8217; \u043d\u0430 &#8216;||&#8217;.<\/p>\n<p>\u041e\u0434\u043d\u0430\u043a\u043e \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 RavenDB \u0437\u0430\u043c\u0435\u0442\u0438\u043b\u0438 \u0435\u0449\u0451 \u043e\u0434\u043d\u0443 \u043e\u0448\u0438\u0431\u043a\u0443. \u041a\u0430\u043a \u043e\u043a\u0430\u0437\u0430\u043b\u043e\u0441\u044c, \u0432\u0435\u0440\u0445\u043d\u0435\u0439 \u0433\u0440\u0430\u043d\u0438\u0446\u0435\u0439 \u0434\u043b\u044f \u043a\u043e\u043b\u0438\u0447\u0435\u0441\u0442\u0432\u0430 \u0430\u0440\u0433\u0443\u043c\u0435\u043d\u0442\u043e\u0432 \u0434\u043e\u043b\u0436\u043d\u043e \u0431\u044b\u0442\u044c \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 6, \u0430 \u043d\u0435 5.<\/p>\n<p>\u0418\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u043d\u044b\u0439 \u043a\u043e\u0434 \u0432\u044b\u0433\u043b\u044f\u0434\u0438\u0442 \u0442\u0430\u043a:<\/p>\n<pre><code class=\"cs\">private static JsValue Spatial_Distance(JsValue self, JsValue[] args) {   if (args.Length is &lt; 4 or > 6)     throw new ArgumentException(\"....\"); } <\/code><\/pre>\n<p>\u041a\u043e\u043c\u043c\u0438\u0442\u044b \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f\u043c\u0438: <a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/bd487885bce7c3e1a1a441533c2cd106d7bce304\">\u0440\u0430\u0437<\/a>, <a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/0e5185dfbf367ee14ad4c647aa85f557b899f90d\">\u0434\u0432\u0430<\/a>.<\/p>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 3<\/h4>\n<pre><code class=\"cs\">public override DynamicJsonValue ToJson() {   var json = base.ToJson();   json[nameof(DataDirectory)] = DataDirectory;   json[nameof(JournalStoragePath)] = DataDirectory;      \/\/ &lt;=   json[nameof(SnapshotRestore)] = SnapshotRestore.ToJson();   return json; } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3127\/\">V3127<\/a> Two similar code fragments were found. Perhaps, this is a typo and &#8216;JournalStoragePath&#8217; variable should be used instead of &#8216;DataDirectory&#8217; RestoreResult.cs 24<\/p>\n<p>\u0418\u0441\u0445\u043e\u0434\u044f \u0438\u0437 \u0442\u043e\u0433\u043e, \u0447\u0442\u043e \u0432 <em>json[nameof(DataDirectory)]<\/em> \u0437\u0430\u043f\u0438\u0441\u044b\u0432\u0430\u0435\u0442\u0441\u044f \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 \u0441\u0432\u043e\u0439\u0441\u0442\u0432\u0430 <em>DataDirectory<\/em>, \u0432 <em>json[nameof(JournalStoragePath)]<\/em> \u0434\u043e\u043b\u0436\u043d\u043e \u0437\u0430\u043f\u0438\u0441\u044b\u0432\u0430\u0442\u044c\u0441\u044f \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 \u0441\u0432\u043e\u0439\u0441\u0442\u0432\u0430 <em>JournalStoragePath<\/em>. \u0418\u043c\u0435\u043d\u043d\u043e \u0442\u0430\u043a \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 \u0438 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438 \u043e\u0448\u0438\u0431\u043a\u0443:<\/p>\n<pre><code class=\"cs\">json[nameof(JournalStoragePath)] = JournalStoragePath; <\/code><\/pre>\n<p><a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/c6c14145b97b74db353e7f00480d75a168839634\">\u041a\u043e\u043c\u043c\u0438\u0442 \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c<\/a>.<\/p>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 4<\/h4>\n<pre><code class=\"cs\">public override int GetHashCode() {   unchecked   {     var hashCode = MinimumRevisionsToKeep?.GetHashCode() ?? 0;     hashCode = (hashCode * 397) ^        MinimumRevisionAgeToKeep?.GetHashCode() ?? 0; \/\/ &lt;=     hashCode = (hashCode * 397) ^        Disabled.GetHashCode();     hashCode = (hashCode * 397) ^       PurgeOnDelete.GetHashCode();     return hashCode;   } } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3123\/\">V3123<\/a> Perhaps the &#8216;??&#8217; operator works in a different way than it was expected. Its priority is lower than priority of other operators in its left part. RevisionsCollectionConfiguration.cs 40<\/p>\n<p>\u0417\u0434\u0435\u0441\u044c PVS-Studio \u0443\u043a\u0430\u0437\u044b\u0432\u0430\u0435\u0442 \u043d\u0430 \u0442\u043e, \u0447\u0442\u043e \u0432\u043e\u0437\u043c\u043e\u0436\u043d\u0430 \u043e\u0448\u0438\u0431\u043a\u0430 \u0441 \u043f\u0440\u0438\u043e\u0440\u0438\u0442\u0435\u0442\u043e\u043c \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u0430 &#8216;??&#8217;. \u0418 \u0434\u0435\u0439\u0441\u0442\u0432\u0438\u0442\u0435\u043b\u044c\u043d\u043e, \u043f\u0440\u0438\u043e\u0440\u0438\u0442\u0435\u0442 \u0443 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u0430 &#8216;^&#8217; \u0432\u044b\u0448\u0435, \u0447\u0435\u043c\u0443 \u0443 &#8216;??&#8217;. \u0415\u0441\u043b\u0438 <em>MinimumRevisionAgeToKeep<\/em> \u0431\u0443\u0434\u0435\u0442 \u0438\u043c\u0435\u0442\u044c \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 <em>null<\/em>, \u0442\u043e \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442 \u0432\u044b\u0447\u0438\u0441\u043b\u0435\u043d\u0438\u044f <em>hashCode<\/em> \u0432 \u0432\u044b\u0434\u0435\u043b\u0435\u043d\u043d\u043e\u0439 \u0441\u0442\u0440\u043e\u043a\u0435 \u0432\u0441\u0435\u0433\u0434\u0430 \u0431\u0443\u0434\u0435\u0442 \u0440\u0430\u0432\u0435\u043d 0. \u0414\u043b\u044f \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u044f \u0434\u043e\u0441\u0442\u0430\u0442\u043e\u0447\u043d\u043e \u0432\u0437\u044f\u0442\u044c \u0432\u044b\u0440\u0430\u0436\u0435\u043d\u0438\u0435 \u0441 &#8216;??&#8217; \u0432 \u0441\u043a\u043e\u0431\u043a\u0438.<\/p>\n<p>\u041a\u0441\u0442\u0430\u0442\u0438, \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438 \u0437\u0430\u043c\u0435\u0442\u0438\u043b\u0438 \u0435\u0449\u0451 \u043e\u0434\u043d\u0443 \u043e\u0448\u0438\u0431\u043a\u0443. \u041f\u0440\u0438 \u0432\u044b\u0447\u0438\u0441\u043b\u0435\u043d\u0438\u0438 <em>hashCode<\/em> \u0437\u0430\u0431\u044b\u043b\u0438 \u043f\u0440\u043e \u0441\u0432\u043e\u0439\u0441\u0442\u0432\u043e <em>MaximumRevisionsToDeleteUponDocumentUpdate<\/em>.<\/p>\n<p>\u0418\u0442\u043e\u0433\u043e\u0432\u044b\u0439 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u043d\u044b\u0439 \u043a\u043e\u0434:<\/p>\n<pre><code class=\"cs\">public override int GetHashCode() {   unchecked   {     var hashCode = MinimumRevisionsToKeep?.GetHashCode() ?? 0;     hashCode = (hashCode * 397) ^        (MinimumRevisionAgeToKeep?.GetHashCode() ?? 0);     hashCode = (hashCode * 397) ^        (MaximumRevisionsToDeleteUponDocumentUpdate?.GetHashCode() ?? 0);     hashCode = (hashCode * 397) ^        Disabled.GetHashCode();     hashCode = (hashCode * 397) ^        PurgeOnDelete.GetHashCode();     return hashCode;   } } <\/code><\/pre>\n<p><a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/78e57c9d293bbd1812031ea7e761dc5b981b5684\">\u041a\u043e\u043c\u043c\u0438\u0442 \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c<\/a>.<\/p>\n<p>\u0421\u043b\u043e\u0432\u0430 \u0440\u0443\u043a\u043e\u0432\u043e\u0434\u0438\u0442\u0435\u043b\u044f \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0438 \u041f\u0430\u0432\u043b\u0430 \u041f\u0435\u043a\u0440\u043e\u043b\u0430 \u043f\u0440\u043e \u044d\u0442\u0443 \u043e\u0448\u0438\u0431\u043a\u0443:<\/p>\n<blockquote>\n<p>Point 4 &#8212; gold \ud83d\ude42<\/p>\n<\/blockquote>\n<h4>\u041e\u0448\u0438\u0431\u043a\u0430 5<\/h4>\n<pre><code class=\"cs\">public override void VisitMethod(MethodExpression expr) {   if (expr.Name.Value == \"id\" &amp;&amp; expr.Arguments.Count == 0)   {     if (expr.Arguments.Count != 1)   \/\/ &lt;=     {       throw new InvalidOperationException(\"....\");     }     _sb.Append(\"this\");   } } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v3022\/\">V3022<\/a> Expression &#8216;expr.Arguments.Count != 1&#8217; is always true. JavascriptCodeQueryVisitor.cs 188<\/p>\n<p>\u041f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 <em>expr.Arguments.Count != 1<\/em> \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u0435\u0442\u0441\u044f \u0441\u0440\u0430\u0437\u0443 \u0436\u0435 \u043f\u043e\u0441\u043b\u0435 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 <em>expr.Arguments.Count == 0<\/em>. \u041e\u0447\u0435\u0432\u0438\u0434\u043d\u043e, \u0447\u0442\u043e \u0435\u0441\u043b\u0438 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 <em>expr.Arguments.Count<\/em> \u0440\u0430\u0432\u043d\u043e 0, \u0442\u043e \u043e\u043d\u043e \u0442\u0430\u043a\u0436\u0435 \u043d\u0435 \u0440\u0430\u0432\u043d\u043e 1. \u0415\u0441\u043b\u0438 \u043f\u043e\u0442\u043e\u043a \u0432\u044b\u043f\u043e\u043b\u043d\u0435\u043d\u0438\u044f \u0437\u0430\u0445\u043e\u0434\u0438\u0442 \u0432 \u0432\u0435\u0440\u0445\u043d\u0438\u0439 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440 <em>if<\/em>, \u0442\u043e \u0432\u0441\u0435\u0433\u0434\u0430 \u0431\u0443\u0434\u0435\u0442 \u0431\u0440\u043e\u0441\u0430\u0442\u044c\u0441\u044f \u0438\u0441\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u0435. \u041a\u0430\u043a \u0441\u043b\u0435\u0434\u0441\u0442\u0432\u0438\u0435, <em>_sb.Append(&#171;this&#187;)<\/em> \u043d\u0438\u043a\u043e\u0433\u0434\u0430 \u043d\u0435 \u0432\u044b\u043f\u043e\u043b\u043d\u0438\u0442\u0441\u044f.<\/p>\n<p>\u041f\u043e\u043b\u0443\u0447\u0430\u0435\u0442\u0441\u044f, \u0447\u0442\u043e \u043d\u0430 \u0442\u0435\u0441\u0442\u0430\u0445 \u043f\u0440\u043e\u0441\u0442\u043e &#171;\u0432\u0435\u0437\u043b\u043e&#187;: \u044d\u0442\u043e\u0442 \u043a\u043e\u0434 \u043d\u0435 \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u043b\u0441\u044f, \u0438\u043d\u0430\u0447\u0435 \u0431\u044b \u0432\u043e\u0437\u043d\u0438\u043a\u0430\u043b\u043e \u0438\u0441\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u0435. \u041f\u043e\u0434\u043e\u0431\u043d\u044b\u0435 \u043e\u0448\u0438\u0431\u043a\u0438 \u0441\u0442\u0440\u0430\u0448\u043d\u044b \u0442\u0435\u043c, \u0447\u0442\u043e \u043e\u043d\u0438 \u043d\u0435 \u0441\u0442\u0440\u0435\u043b\u044f\u044e\u0442 \u043d\u0430 \u0442\u0435\u0441\u0442\u0430\u0445, \u043d\u043e \u043c\u043e\u0433\u0443\u0442 \u0432\u044b\u0441\u0442\u0440\u0435\u043b\u0438\u0442\u044c \u0443 \u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u0435\u043b\u044f.<\/p>\n<p>\u0418\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c \u0441\u0442\u0430\u043b\u043e \u0443\u0434\u0430\u043b\u0435\u043d\u0438\u0435 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u0430 <em>if<\/em> \u0441 \u0432\u044b\u0431\u0440\u0430\u0441\u044b\u0432\u0430\u043d\u0438\u0435\u043c \u0438\u0441\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u044f.<\/p>\n<p><a href=\"https:\/\/github.com\/ravendb\/ravendb\/commit\/f91d886ab9eb931d97a71ec66ac6453ff3099f03\">\u041a\u043e\u043c\u043c\u0438\u0442 \u0441 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0435\u043c<\/a>.<\/p>\n<h3>\u0412\u044b\u0432\u043e\u0434\u044b<\/h3>\n<p>\u0412 \u043d\u0430\u0437\u0432\u0430\u043d\u0438\u0438 \u0441\u0442\u0430\u0442\u044c\u0438 \u043d\u0430\u043f\u0438\u0441\u0430\u043d\u043e, \u0447\u0442\u043e \u0432\u044b\u0438\u0433\u0440\u044b\u0432\u0430\u0435\u0442 \u043a\u0430\u0436\u0434\u044b\u0439. \u0412 \u0447\u0451\u043c \u0432\u044b\u0438\u0433\u0440\u0430\u043b RavenDB? \u0415\u0441\u0442\u0435\u0441\u0442\u0432\u0435\u043d\u043d\u043e, \u0447\u0442\u043e \u043a\u0430\u0436\u0434\u044b\u0439 \u043f\u0440\u043e\u0434\u0443\u043a\u0442 \u043a\u0430\u043a-\u0442\u043e \u0440\u0430\u0437\u0432\u0438\u0432\u0430\u0435\u0442\u0441\u044f \u0438 \u0441\u043e\u0432\u0435\u0440\u0448\u0435\u043d\u0441\u0442\u0432\u0443\u0435\u0442\u0441\u044f. \u0412 \u044d\u0442\u043e\u0442 \u0440\u0430\u0437 \u0443\u043b\u0443\u0447\u0448\u0435\u043d\u0438\u0435 \u043f\u0440\u043e\u0438\u0437\u043e\u0448\u043b\u043e \u043d\u0435 \u0447\u0435\u0440\u0435\u0437 \u043d\u043e\u0432\u044b\u0435 \u0444\u0438\u0447\u0438, \u0430 \u0447\u0435\u0440\u0435\u0437 \u043f\u043e\u0432\u044b\u0448\u0435\u043d\u0438\u0435 \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0430 \u0438\u0441\u0445\u043e\u0434\u043d\u043e\u0433\u043e \u043a\u043e\u0434\u0430. \u0412 \u0447\u0451\u043c \u0436\u0435 \u0432\u044b\u0438\u0433\u0440\u0430\u043b PVS-Studio? \u0412\u043e \u0432\u0440\u0435\u043c\u044f \u043f\u0440\u043e\u0441\u043c\u043e\u0442\u0440\u0430 \u043e\u0442\u0447\u0451\u0442\u0430 \u043e\u0431 \u043e\u0448\u0438\u0431\u043a\u0430\u0445 \u043c\u044b \u043d\u0430\u0448\u043b\u0438 \u043d\u0435\u0441\u043a\u043e\u043b\u044c\u043a\u043e \u043b\u043e\u0436\u043d\u044b\u0445 \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0439. \u041d\u0435\u043a\u043e\u0442\u043e\u0440\u044b\u0435 \u0438\u0437 \u043d\u0438\u0445 \u043c\u044b \u0443\u0436\u0435 \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438, \u0430 \u0434\u043e \u043d\u0435\u043a\u043e\u0442\u043e\u0440\u044b\u0445 \u0435\u0449\u0451 \u043f\u0440\u0435\u0434\u0441\u0442\u043e\u0438\u0442 \u0434\u043e\u0431\u0440\u0430\u0442\u044c\u0441\u044f.<\/p>\n<p>\u0420\u0443\u043a\u043e\u0432\u043e\u0434\u0438\u0442\u0435\u043b\u044c \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0438 RavenDB \u041f\u0430\u0432\u0435\u043b \u041f\u0435\u043a\u0440\u043e\u043b:<\/p>\n<blockquote>\n<p>Going over the list made me realize that devs are only humans and no matter how many tests you have there will always be bugs. The tool is very helpful in finding obvious and non-obvious bugs.<\/p>\n<\/blockquote>\n<p>\u041d\u0435 \u043c\u043e\u0433\u0443 \u043d\u0435 \u0441\u043e\u0433\u043b\u0430\u0441\u0438\u0442\u044c\u0441\u044f. \u0420\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a \u043f\u0440\u043e\u0441\u0442\u043e \u043d\u0435 \u043c\u043e\u0436\u0435\u0442 \u043f\u043e\u043b\u043d\u043e\u0441\u0442\u044c\u044e \u043f\u043e\u043a\u0440\u044b\u0442\u044c \u043a\u043e\u0434 \u0442\u0435\u0441\u0442\u0430\u043c\u0438. \u0415\u0441\u043b\u0438 \u0432\u044b \u0437\u0430\u0431\u043e\u0442\u0438\u0442\u0435\u0441\u044c \u043e \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0435 \u043a\u043e\u0434\u0430 \u0438 \u043f\u0440\u043e\u0435\u043a\u0442\u0430, \u0442\u043e \u0441\u0442\u043e\u0438\u0442 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c \u0440\u0430\u0437\u043b\u0438\u0447\u043d\u044b\u0435 \u0438\u043d\u0441\u0442\u0440\u0443\u043c\u0435\u043d\u0442\u044b, \u043a\u043e\u0442\u043e\u0440\u044b\u0435 \u043f\u043e\u043c\u043e\u0433\u0443\u0442 \u0441\u0434\u0435\u043b\u0430\u0442\u044c \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0443 \u043f\u0440\u043e\u0449\u0435.<\/p>\n<p>\u0418\u0437\u0431\u0430\u0432\u043b\u044f\u0439\u0442\u0435\u0441\u044c \u043e\u0442 \u0431\u0430\u0433\u043e\u0432 \u0438 \u0434\u0435\u043b\u0430\u0439\u0442\u0435 \u0441\u0432\u043e\u0439 \u043a\u043e\u0434 \u0431\u0435\u0437\u043e\u043f\u0430\u0441\u043d\u0435\u0435 \u0441 \u043f\u043e\u043c\u043e\u0449\u044c\u044e <a href=\"https:\/\/pvs-studio.ru\/ru\/pvs-studio\/\">PVS-Studio<\/a>! \u0423\u0434\u043e\u0431\u043d\u043e \u0438 \u043f\u0440\u043e\u0441\u0442\u043e \u0445\u0440\u0430\u043d\u0438\u0442\u0435 \u0438 \u043e\u0431\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u0439\u0442\u0435 \u0434\u0430\u043d\u043d\u044b\u0435 \u0432\u043c\u0435\u0441\u0442\u0435 \u0441 <a href=\"https:\/\/ravendb.net\/\">RavenDB<\/a>!<\/p>\n<p>\u041f\u0440\u0438\u044f\u0442\u043d\u043e\u0433\u043e \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043d\u0438\u044f RavenDB \u0438 PVS-Studio!<\/p>\n<p>\u0415\u0441\u043b\u0438 \u0445\u043e\u0442\u0438\u0442\u0435 \u043f\u043e\u0434\u0435\u043b\u0438\u0442\u044c\u0441\u044f \u044d\u0442\u043e\u0439 \u0441\u0442\u0430\u0442\u044c\u0435\u0439 \u0441 \u0430\u043d\u0433\u043b\u043e\u044f\u0437\u044b\u0447\u043d\u043e\u0439 \u0430\u0443\u0434\u0438\u0442\u043e\u0440\u0438\u0435\u0439, \u0442\u043e \u043f\u0440\u043e\u0448\u0443 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c \u0441\u0441\u044b\u043b\u043a\u0443 \u043d\u0430 \u043f\u0435\u0440\u0435\u0432\u043e\u0434: Artem Rovenskii. <a href=\"https:\/\/pvs-studio.com\/en\/blog\/posts\/csharp\/1052\/\">RavenDB and PVS-Studio: win-win collaboration<\/a>.<\/p>\n<\/p>\n<\/div>\n<\/div>\n<\/div>\n<p><!----><!----><\/div>\n<p><!----><!----><br \/> \u0441\u0441\u044b\u043b\u043a\u0430 \u043d\u0430 \u043e\u0440\u0438\u0433\u0438\u043d\u0430\u043b \u0441\u0442\u0430\u0442\u044c\u0438 <a href=\"https:\/\/habr.com\/ru\/articles\/737224\/\"> https:\/\/habr.com\/ru\/articles\/737224\/<\/a><br \/><\/br><\/br><\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[],"tags":[],"class_list":["post-404258","post","type-post","status-publish","format-standard","hentry"],"_links":{"self":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/posts\/404258","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Fcomments&post=404258"}],"version-history":[{"count":0,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/posts\/404258\/revisions"}],"wp:attachment":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=404258"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=404258"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=404258"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}