{"id":464381,"date":"2025-06-25T03:00:15","date_gmt":"2025-06-25T03:00:15","guid":{"rendered":"http:\/\/savepearlharbor.com\/?p=464381"},"modified":"-0001-11-30T00:00:00","modified_gmt":"-0001-11-29T21:00:00","slug":"","status":"publish","type":"post","link":"https:\/\/savepearlharbor.com\/?p=464381","title":{"rendered":"\u041a\u0430\u0440\u0442\u044b, Java, 2 null’\u0430. XMage edition<\/span>"},"content":{"rendered":"
<\/div>\n
\n
\n
\n
\n

\u041a\u043e\u043c\u043f\u044c\u044e\u0442\u0435\u0440\u043d\u0430\u044f \u0438\u0433\u0440\u0430 \u043d\u0430 Java \u2014 \u0432\u0435\u0449\u044c \u0434\u043e\u0432\u043e\u043b\u044c\u043d\u043e \u0440\u0435\u0434\u043a\u0430\u044f, \u043d\u043e \u0432\u0441\u0435\u0433\u0434\u0430 \u0438\u043d\u0442\u0435\u0440\u0435\u0441\u043d\u0430\u044f. \u041f\u043e\u044d\u0442\u043e\u043c\u0443 \u043c\u044b \u043d\u0435 \u0443\u043f\u0443\u0441\u0442\u0438\u043b\u0438 \u0432\u043e\u0437\u043c\u043e\u0436\u043d\u043e\u0441\u0442\u044c \u043f\u0440\u043e\u0432\u0435\u0440\u0438\u0442\u044c \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u043c \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u043e\u043c \u043f\u0440\u043e\u0435\u043a\u0442 XMage \u0438 \u043f\u043e\u0434\u0435\u043b\u0438\u0442\u044c\u0441\u044f \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u0430\u043c\u0438. \u041f\u043e\u0441\u043c\u043e\u0442\u0440\u0438\u043c, \u0447\u0442\u043e \u043d\u0430\u0448\u0451\u043b PVS-Studio \u0432 \u0438\u0441\u0445\u043e\u0434\u043d\u043e\u043c \u043a\u043e\u0434\u0435 \u043f\u0440\u043e\u0435\u043a\u0442\u0430.<\/p>\n

<\/figure>\n

\u041e \u043f\u0440\u043e\u0435\u043a\u0442\u0435<\/h3>\n

XMage<\/a> \u2014 \u043f\u0440\u043e\u0435\u043a\u0442, \u043a\u043e\u0442\u043e\u0440\u044b\u0439 \u044f\u0432\u043b\u044f\u0435\u0442\u0441\u044f \u0431\u0435\u0441\u043f\u043b\u0430\u0442\u043d\u043e\u0439 open source \u0432\u0435\u0440\u0441\u0438\u0435\u0439 \u0438\u0433\u0440\u044b Magic: the Gathering<\/a>. \u0418\u0433\u0440\u0430 \u043f\u043e\u0437\u0432\u043e\u043b\u044f\u0435\u0442 \u0434\u0440\u0430\u0442\u044c\u0441\u044f \u0432 \u043a\u0430\u0440\u0442\u043e\u0447\u043d\u044b\u0445 \u0441\u0440\u0430\u0436\u0435\u043d\u0438\u044f\u0445 \u043a\u0430\u043a \u043b\u043e\u043a\u0430\u043b\u044c\u043d\u043e \u0441 \u0431\u043e\u0442\u0430\u043c\u0438, \u0442\u0430\u043a \u0438 \u043e\u043d\u043b\u0430\u0439\u043d \u0441 \u0434\u0440\u0443\u0437\u044c\u044f\u043c\u0438 \u0438\u043b\u0438 \u0441\u043b\u0443\u0447\u0430\u0439\u043d\u044b\u043c\u0438 \u0438\u0433\u0440\u043e\u043a\u0430\u043c\u0438.<\/p>\n

\u041f\u0440\u043e\u0435\u043a\u0442 \u043f\u043e\u043b\u043d\u043e\u0441\u0442\u044c\u044e \u043d\u0430\u043f\u0438\u0441\u0430\u043d \u043d\u0430 Java \u0438 \u0443\u0436\u0435 \u0431\u043e\u043b\u044c\u0448\u0435 10 \u043b\u0435\u0442 \u0430\u043a\u0442\u0438\u0432\u043d\u043e \u0440\u0430\u0437\u0432\u0438\u0432\u0430\u0435\u0442\u0441\u044f: \u0441\u0435\u0439\u0447\u0430\u0441 \u0432 \u0440\u0435\u043f\u043e\u0437\u0438\u0442\u043e\u0440\u0438\u0438 \u0431\u043e\u043b\u044c\u0448\u0435 47 \u0442\u044b\u0441\u044f\u0447 \u043a\u043e\u043c\u043c\u0438\u0442\u043e\u0432 \u0438 \u0435\u0433\u043e \u0440\u0430\u0437\u043c\u0435\u0440 \u2014 1.8 \u043c\u0438\u043b\u043b\u0438\u043e\u043d\u043e\u0432 \u0441\u0442\u0440\u043e\u043a. \u041f\u044f\u0442\u044c \u043b\u0435\u0442 \u043d\u0430\u0437\u0430\u0434 \u043c\u044b \u0443\u0436\u0435 \u043f\u0440\u043e\u0432\u0435\u0440\u044f\u043b\u0438 XMage \u0438 \u043d\u0430\u043f\u0438\u0441\u0430\u043b\u0438 \u0441\u0442\u0430\u0442\u044c\u044e<\/a>. \u0422\u043e\u0433\u0434\u0430 \u043f\u0440\u043e\u0435\u043a\u0442 \u0431\u044b\u043b \u043c\u0435\u043d\u044c\u0448\u0435 \u043d\u0430 ~800 \u0442\u044b\u0441\u044f\u0447 \u0441\u0442\u0440\u043e\u043a, \u0438 \u043c\u044b \u0440\u0435\u0448\u0438\u043b\u0438 \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0435\u0442\u044c, \u043a\u0430\u043a \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u044f \u043e\u0431\u0441\u0442\u043e\u0438\u0442 \u0441\u0435\u0439\u0447\u0430\u0441.<\/p>\n

\u0412\u0430\u0436\u043d\u043e \u0443\u043f\u043e\u043c\u044f\u043d\u0443\u0442\u044c, \u0447\u0442\u043e \u0432 \u043f\u0440\u043e\u0435\u043a\u0442\u0435 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u044e\u0442\u0441\u044f \u0440\u0430\u0437\u043b\u0438\u0447\u043d\u044b\u0435 \u0441\u0440\u0435\u0434\u0441\u0442\u0432\u0430<\/a> \u043a\u043e\u043d\u0442\u0440\u043e\u043b\u044f \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0430 \u043a\u043e\u0434\u0430. \u0418 \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0439 \u0430\u043d\u0430\u043b\u0438\u0437 \u0432 \u0442\u043e\u043c \u0447\u0438\u0441\u043b\u0435. \u0422\u0430\u043a \u0447\u0442\u043e \u043d\u0430\u043c \u0431\u044b\u043b\u043e \u0432\u0434\u0432\u043e\u0439\u043d\u0435 \u0438\u043d\u0442\u0435\u0440\u0435\u0441\u043d\u043e \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0435\u0442\u044c, \u0447\u0442\u043e \u043d\u0430\u0439\u0434\u0451\u0442 \u043d\u0430\u0448 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440. \u041f\u0440\u0438\u0441\u0442\u0443\u043f\u0438\u043c.<\/p>\n

Unit-\u0442\u0435\u0441\u0442\u044b \u0432\u0441\u0435\u043c\u0443 \u0433\u043e\u043b\u043e\u0432\u0430<\/h3>\n

\u0418\u0437\u0432\u0435\u0447\u043d\u044b\u0439 \u0432\u043e\u043f\u0440\u043e\u0441: \u0435\u0441\u043b\u0438 \u0442\u0435\u0441\u0442\u044b \u043a\u043e\u043d\u0442\u0440\u043e\u043b\u0438\u0440\u0443\u044e\u0442 \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u043e \u043d\u0430\u043f\u0438\u0441\u0430\u043d\u043d\u043e\u0433\u043e \u043a\u043e\u0434\u0430, \u0442\u043e \u0447\u0442\u043e \u043a\u043e\u043d\u0442\u0440\u043e\u043b\u0438\u0440\u0443\u0435\u0442 \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u043e \u043d\u0430\u043f\u0438\u0441\u0430\u043d\u043d\u044b\u0445 \u0442\u0435\u0441\u0442\u043e\u0432?<\/p>\n

\u0412 \u044d\u0442\u043e\u0439 \u0447\u0430\u0441\u0442\u0438 \u0441\u0442\u0430\u0442\u044c\u0438 \u043c\u044b \u0440\u0430\u0441\u0441\u043c\u043e\u0442\u0440\u0438\u043c \u043e\u0448\u0438\u0431\u043a\u0438, \u043d\u0430\u0439\u0434\u0435\u043d\u043d\u044b\u0435 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u043e\u043c \u0432 unit-\u0442\u0435\u0441\u0442\u0430\u0445.<\/p>\n

\u041f\u0440\u043e\u0432\u0435\u0440\u044f\u0435\u0442\u0441\u044f \u043d\u0435 \u0442\u043e<\/h4>\n
@Test public void test_NamesEquals() {   ....   \/\/ same names (empty names can't be same)   Assert.assertFalse(CardUtil.haveSameNames(         \"\",          \"\"));   Assert.assertFalse(CardUtil.haveSameNames(         EmptyNames.FACE_DOWN_CREATURE.getObjectName(),          \"\"));   Assert.assertFalse(CardUtil.haveSameNames(         EmptyNames.FACE_DOWN_CREATURE.getObjectName(),          EmptyNames.FACE_DOWN_CREATURE.getObjectName()));   Assert.assertFalse(CardUtil.haveSameNames(         EmptyNames.FACE_DOWN_TOKEN.getObjectNam(),          \"\"));   Assert.assertFalse(CardUtil.haveSameNames(         EmptyNames.FACE_DOWN_TOKEN.getObjectNam(),          EmptyNames.FACE_DOWN_CREATURE.getObjectName())); \/\/ <=   .... } <\/code><\/pre>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6072<\/a> Two similar code fragments were found. Perhaps, this is a typo and ‘FACE_DOWN_TOKEN’ variable should be used instead of ‘FACE_DOWN_CREATURE’. AliasesApiTest.java 30<\/p>\n
\u0410\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 \u0443\u043a\u0430\u0437\u044b\u0432\u0430\u0435\u0442, \u0447\u0442\u043e, \u0432\u043e\u0437\u043c\u043e\u0436\u043d\u043e, \u0432 \u043e\u0442\u043c\u0435\u0447\u0435\u043d\u043d\u043e\u0439 \u0441\u0442\u0440\u043e\u043a\u0435 \u0441\u0442\u043e\u0438\u0442 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c FACE_DOWN_TOKEN<\/code> \u0432\u043c\u0435\u0441\u0442\u043e FACE_DOWN_CREATURE<\/code>.<\/p>\n
\u0412 \u044d\u0442\u043e\u043c \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u0435 \u0442\u0435\u0441\u0442\u0430 \u043f\u0440\u043e\u0432\u0435\u0440\u044f\u0435\u0442\u0441\u044f \u043f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e\u0441\u0442\u044c \u0441\u0440\u0430\u0432\u043d\u0435\u043d\u0438\u044f «\u043f\u0443\u0441\u0442\u044b\u0445 \u0438\u043c\u0451\u043d». \u041f\u043e \u043b\u043e\u0433\u0438\u043a\u0435 \u0440\u0430\u0431\u043e\u0442\u044b, \u043f\u0443\u0441\u0442\u044b\u0435 \u0438\u043c\u0435\u043d\u0430 \u043f\u0440\u0438 \u0441\u0440\u0430\u0432\u043d\u0435\u043d\u0438\u0438 \u043d\u0430 \u0438\u0434\u0435\u043d\u0442\u0438\u0447\u043d\u043e\u0441\u0442\u044c \u0434\u043e\u043b\u0436\u043d\u044b \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0442\u044c \u043e\u0442\u0440\u0438\u0446\u0430\u0442\u0435\u043b\u044c\u043d\u044b\u0439 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442, \u0434\u0430\u0436\u0435 \u0435\u0441\u043b\u0438 \u043e\u043d\u0438 \u043f\u0440\u0435\u0434\u0441\u0442\u0430\u0432\u043b\u0435\u043d\u044b \u043e\u0434\u0438\u043d\u0430\u043a\u043e\u0432\u044b\u043c\u0438 \u043a\u043e\u043d\u0441\u0442\u0430\u043d\u0442\u0430\u043c\u0438. \u041e\u0431 \u044d\u0442\u043e\u043c \u0436\u0435 \u0433\u043e\u0432\u043e\u0440\u0438\u0442 \u043a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439 \u0432\u044b\u0448\u0435.<\/p>\n
\u0421\u0438\u0442\u0443\u0430\u0446\u0438\u044f \u0432\u044b\u0433\u043b\u044f\u0434\u0438\u0442 \u0442\u0430\u043a, \u0447\u0442\u043e \u043f\u0435\u0440\u0435\u0434 \u043d\u0430\u043c\u0438 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442 \u043d\u0435\u0443\u0434\u0430\u0447\u043d\u043e\u0439 \u043a\u043e\u043f\u0438\u043f\u0430\u0441\u0442\u044b. FACE_DOWN_CREATURE<\/code> \u043f\u0440\u043e\u0432\u0435\u0440\u044f\u0435\u0442\u0441\u044f \u043d\u0430 \u0438\u0434\u0435\u043d\u0442\u0438\u0447\u043d\u043e\u0441\u0442\u044c \u0441 \u0441\u043e\u0431\u043e\u0439 \u0436\u0435, \u0430 FACE_DOWN_TOKEN<\/code> \u2014 \u043d\u0435\u0442.<\/p>\n
\u042f \u0437\u0430\u043c\u0435\u043d\u0438\u043b \u043a\u043e\u043d\u0441\u0442\u0430\u043d\u0442\u0443, \u0438 \u0442\u0435\u0441\u0442 \u043f\u0440\u043e\u0448\u0451\u043b \u0443\u0434\u0430\u0447\u043d\u043e.<\/p>\n
\u041d\u0435\u0440\u0430\u0432\u043d\u044b\u0439 \u0431\u043e\u0439<\/h4>\n
@Test public void test_Simple_LongGame() {     ....     addCard(Zone.LIBRARY, playerA, \"Mountain\", 10);     addCard(Zone.LIBRARY, playerA, \"Forest\", 10);     addCard(Zone.LIBRARY, playerA, \"Lightning Bolt\", 20);     addCard(Zone.LIBRARY, playerA, \"Balduvian Bears\", 10);     \/\/     addCard(Zone.LIBRARY, playerB, \"Mountain\", 10);     addCard(Zone.LIBRARY, playerA, \"Forest\", 10);        \/\/ <=     addCard(Zone.LIBRARY, playerB, \"Lightning Bolt\", 20);     addCard(Zone.LIBRARY, playerB, \"Balduvian Bears\", 10);     .... } <\/code><\/pre>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6072<\/a> Two similar code fragments were found. Perhaps, this is a typo and ‘playerB’ variable should be used instead of ‘playerA’. SimulationPerformanceAITest.java 60<\/p>\n
\u0414\u0443\u043c\u0430\u044e, \u0443\u0432\u0438\u0434\u0435\u0432 \u0441\u0442\u0440\u0435\u043b\u043a\u0443 \u0432 \u043f\u0440\u0438\u0432\u0435\u0434\u0451\u043d\u043d\u043e\u043c \u0432\u044b\u0448\u0435 \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u0435 \u043a\u043e\u0434\u0430, \u0432\u044b \u043f\u043e\u043d\u044f\u043b\u0438, \u043f\u043e\u0447\u0435\u043c\u0443 \u043e\u043d \u0437\u0434\u0435\u0441\u044c \ud83d\ude42<\/p>\n
\u0412 \u044d\u0442\u043e\u043c \u0442\u0435\u0441\u0442\u0435 \u043f\u0440\u043e\u0432\u0435\u0440\u044f\u0435\u0442\u0441\u044f, \u0447\u0442\u043e \u0434\u0440\u0430\u043a\u0430 AI vs AI \u0441 \u0442\u043e\u0447\u043a\u0438 \u0437\u0440\u0435\u043d\u0438\u044f \u043f\u0435\u0440\u0444\u043e\u043c\u0430\u043d\u0441\u0430 \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u044b \u043f\u0440\u043e\u0438\u0441\u0445\u043e\u0434\u0438\u0442 \u0430\u0434\u0435\u043a\u0432\u0430\u0442\u043d\u043e. \u041e\u0434\u043d\u0430\u043a\u043e \u043d\u0435\u0447\u0435\u0441\u0442\u043d\u043e. \u0413\u043e\u0441\u043f\u043e\u0434\u0438\u043d\u0443 A \u0434\u043e\u0441\u0442\u0430\u043b\u043e\u0441\u044c \u0431\u043e\u043b\u044c\u0448\u0435 \u043b\u0435\u0441\u043d\u044b\u0445 \u043a\u0430\u0440\u0442, \u043d\u0435\u0436\u0435\u043b\u0438 \u0433\u043e\u0441\u043f\u043e\u0434\u0438\u043d\u0443 B. \u041e\u043f\u044f\u0442\u044c \u0436\u0435, \u0432\u044b\u0433\u043b\u044f\u0434\u0438\u0442 \u043a\u0430\u043a \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442 \u043d\u0435\u0443\u0434\u0430\u0447\u043d\u043e\u0439 \u043a\u043e\u043f\u0438\u043f\u0430\u0441\u0442\u044b. \u0422\u0430\u043a\u0436\u0435, \u043a\u0430\u043a \u0438 \u0432 \u043f\u0440\u0435\u0434\u044b\u0434\u0443\u0449\u0435\u043c \u0441\u043b\u0443\u0447\u0430\u0435, \u043f\u0440\u0438 \u0438\u0441\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u0438\u0438 \u044d\u0442\u043e\u0433\u043e \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u0430 \u043d\u0438\u0447\u0435\u0433\u043e \u043d\u0435 \u043f\u043e\u043b\u043e\u043c\u0430\u043b\u043e\u0441\u044c \u2014 \u0442\u0435\u0441\u0442 \u0437\u0435\u043b\u0451\u043d\u044b\u0439.<\/p>\n
\u041e\u0447\u0435\u043f\u044f\u0442\u043a\u0438 \u0438 \u0441\u0442\u0440\u0430\u043d\u043d\u044b\u0435 \u043c\u043e\u043c\u0435\u043d\u0442\u044b<\/h3>\n
\u041b\u043e\u0433\u0438\u0447\u0435\u0441\u043a\u0438\u0439 \u043f\u0435\u0440\u0435\u043f\u043e\u043b\u043e\u0445<\/h4>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6008<\/a> Null dereference of ‘permanent’. ThreeTreeScribe.java 78<\/p>\n
@Override public boolean checkTrigger(GameEvent event, Game game) {     ....     Permanent permanent = zEvent.getTarget();     return (    permanent != null              || !permanent.isControlledBy(getControllerId()))  \/\/ <=             && (permanent.getId().equals(getSourceId())                  || permanent.isCreature(game)); } <\/code><\/pre>\n
\u0412 \u044d\u0442\u043e\u043c \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u0435 \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u044b \u044f\u0432\u043d\u043e \u0435\u0441\u0442\u044c \u043e\u0448\u0438\u0431\u043e\u0447\u043d\u043e\u0435 \u0443\u0441\u043b\u043e\u0432\u0438\u0435, \u043a\u043e\u0442\u043e\u0440\u043e\u0435 \u0437\u0430\u0441\u0442\u0430\u0432\u043b\u044f\u0435\u0442 \u043d\u0435\u043c\u043d\u043e\u0433\u043e \u0443\u043b\u044b\u0431\u043d\u0443\u0442\u044c\u0441\u044f. \u0412\u043d\u0443\u0442\u0440\u0438 return<\/code> \u043c\u044b \u0440\u0430\u0437\u044b\u043c\u0435\u043d\u043e\u0432\u044b\u0432\u0430\u0435\u043c permanent<\/code> \u0432 \u0441\u043b\u0443\u0447\u0430\u0435, \u0435\u0441\u043b\u0438 \u043e\u043d \u0440\u0430\u0432\u0435\u043d null<\/code>.<\/p>\n
\u0421\u043b\u0435\u0434\u0443\u044e\u0449\u0430\u044f \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u044f \u0442\u043e\u0436\u0435 \u043f\u0440\u043e \u0434\u0438\u0430\u0433\u043d\u043e\u0441\u0442\u0438\u043a\u0443 V6008<\/a> \u0438 \u0442\u043e\u0436\u0435 \u043f\u0440\u043e \u0443\u0441\u043b\u043e\u0432\u0438\u044f. \u041d\u043e \u043d\u0430 \u043e\u0434\u0438\u043d \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442 \u043a\u043e\u0434\u0430 \u0441\u0440\u0430\u0437\u0443 \u0434\u0432\u0430 \u0441\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u043d\u0438\u044f:<\/p>\n
V6008<\/a> Potential null dereference of ‘player’. DelifsCone.java 104<\/p>\n
V6008<\/a> Potential null dereference of ‘permanent’. DelifsCone.java 104<\/p>\n
@Override public boolean apply(Game game, Ability source) {     Player player = game.getPlayer(source.getControllerId());     Permanent          permanent = game.getPermanent(....);     if (player != null || permanent != null) {         player.gainLife(permanent.getPower().getValue(), game, source);         return true;     }     return false; } <\/code><\/pre>\n
\u041f\u0440\u043e\u0432\u0435\u0440\u044f\u0442\u044c \u043d\u0430 null<\/code> \u043b\u0438\u0448\u044c \u043e\u0434\u0438\u043d \u043e\u0431\u044a\u0435\u043a\u0442, \u0430 \u0440\u0430\u0437\u044b\u043c\u0435\u043d\u043e\u0432\u044b\u0432\u0430\u0442\u044c \u0441\u0440\u0430\u0437\u0443 \u043e\u0431\u0430 \u0438\u0437 \u043d\u0438\u0445, \u043d\u0435\u043f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e. \u042f\u0432\u043d\u043e \u0432 \u0443\u0441\u043b\u043e\u0432\u0438\u0438 \u043f\u043e\u0434\u0440\u0430\u0437\u0443\u043c\u0435\u0432\u0430\u043b\u043e\u0441\u044c &&<\/code> \u0432\u043c\u0435\u0441\u0442\u043e ||<\/code>. \u0422\u0430\u043a\u0438\u0435 \u043e\u043f\u0435\u0447\u0430\u0442\u043a\u0438, \u043a\u0441\u0442\u0430\u0442\u0438, \u0432 \u0440\u0430\u0437\u043d\u044b\u0445 \u043f\u0440\u043e\u0435\u043a\u0442\u0430\u0445 \u043c\u044b \u0432\u0441\u0442\u0440\u0435\u0447\u0430\u0435\u043c \u0434\u043e\u0441\u0442\u0430\u0442\u043e\u0447\u043d\u043e \u0447\u0430\u0441\u0442\u043e.<\/p>\n
\u041d\u0443, \u043f\u043e\u0447\u0442\u0438…<\/h4>\n
static {     for (int count = 1; count <= 6; count++) {         FilterCard filter = new FilterCreatureCard(                 \"creature card that's exactly \" +                         CardUtil.numberToText(count) +                         \" color\" + (count > 0 ? \"s\" : \"\")         );         filter.add(new EvolvingDoorPredicate(count));         filterMap.put(count, filter);     } } <\/code><\/pre>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6007<\/a> Expression ‘count > 0’ is always true. EvolvingDoor.java 76<\/p>\n
\u0414\u0435\u0439\u0441\u0442\u0432\u0438\u0442\u0435\u043b\u044c\u043d\u043e, \u0441\u0442\u0440\u0430\u043d\u043d\u043e\u0435 \u0443\u0441\u043b\u043e\u0432\u0438\u0435. \u0415\u0441\u043b\u0438 \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0435\u0442\u044c \u043d\u0430 \u0444\u043e\u0440\u043c\u0438\u0440\u0443\u0435\u043c\u0443\u044e \u0441\u0442\u0440\u043e\u043a\u0443, \u0432\u0438\u0434\u043d\u043e, \u0447\u0442\u043e \u0432\u043d\u0435 \u0437\u0430\u0432\u0438\u0441\u0438\u043c\u043e\u0441\u0442\u0438 \u043e\u0442 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f \u0446\u0438\u043a\u043b\u0430 \u0431\u0443\u0434\u0435\u0442 \u0434\u043e\u0431\u0430\u0432\u043b\u044f\u0442\u044c\u0441\u044f \u043e\u043a\u043e\u043d\u0447\u0430\u043d\u0438\u0435 «s» \u043a \u0441\u043b\u043e\u0432\u0443 «color». \u0421 \u0437\u0430\u043c\u0435\u043d\u043e\u0439 0 \u043d\u0430 1 \u0432 \u0430\u043d\u0433\u043b\u0438\u0439\u0441\u043a\u043e\u0439 \u0433\u0440\u0430\u043c\u043c\u0430\u0442\u0438\u043a\u0435 \u0432\u0441\u0451 \u0431\u0443\u0434\u0435\u0442 \u0445\u043e\u0440\u043e\u0448\u043e \u0438 \u0446\u0438\u043a\u043b \u043e\u0442\u0440\u0430\u0431\u043e\u0442\u0430\u0435\u0442 \u043a\u0430\u043a \u043d\u0430\u0434\u043e.<\/p>\n
\u0412\u0441\u0451, \u0447\u0442\u043e \u043c\u044b \u0441\u0434\u0435\u043b\u0430\u043b\u0438 \u2013 \u043c\u044b \u0443\u0436\u0435 \u043c\u043e\u0433\u043b\u0438<\/h4>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6060<\/a> The ‘table’ reference was utilized before it was verified against null. TableController.java 725<\/p>\n
private void startGame(UUID choosingPlayerId) throws GameException {     try {         ....     } catch (Exception ex) {         logger.fatal(\"Error starting game table: \" + table.getId(), ex);         if (table != null) {             managerFactory.tableManager().removeTable(table.getId());         }         ....      } } <\/code><\/pre>\n
\u041f\u0440\u043e\u0432\u0435\u0440\u044f\u0442\u044c \u043d\u0430 null<\/code> \u0434\u0435\u043b\u043e \u0445\u043e\u0440\u043e\u0448\u0435\u0435. \u041d\u043e \u0432\u0441\u0435\u043c\u0443 \u0441\u0432\u043e\u0451 \u0432\u0440\u0435\u043c\u044f. \u0412\u043e\u0442 \u0438 \u0437\u0434\u0435\u0441\u044c \u0441\u043d\u0430\u0447\u0430\u043b\u0430 table<\/code> \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u044e\u0442, \u0430 \u0443\u0436\u0435 \u043f\u043e\u0442\u043e\u043c \u043f\u0440\u043e\u0432\u0435\u0440\u044f\u044e\u0442, \u043d\u0435 null<\/code> \u043b\u0438 \u043e\u043d.<\/p>\n
\u0412 \u0434\u0440\u0443\u0433\u043e\u043c \u0444\u0430\u0439\u043b\u0435 \u043f\u0440\u0430\u043a\u0442\u0438\u0447\u0435\u0441\u043a\u0438 \u0430\u043d\u0430\u043b\u043e\u0433\u0438\u0447\u043d\u043e\u0435 \u0441\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u043d\u0438\u0435:<\/p>\n
V6060<\/a> The ‘enchantment’ reference was utilized before it was verified against null. PublicEnemy.java 79<\/p>\n
@Override public boolean applies(Permanent permanent, Ability source, Game game) {     Permanent          enchantment = game.getPermanent(source.getSourceId());     Permanent          enchantedCreature = game.getPermanent(enchantment.getAttachedTo());     if (enchantment == null || enchantedCreature == null) {         return false;     }     if (permanent.isControlledBy(enchantedCreature.getControllerId())) {         return false;     }     return true; } <\/code><\/pre>\n
\u041e\u0431\u044a\u0435\u043a\u0442 enchantment<\/code> \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u044e\u0442, \u0447\u0442\u043e\u0431\u044b \u0441\u043e\u0437\u0434\u0430\u0442\u044c \u043e\u0431\u044a\u0435\u043a\u0442 enchantedCreature<\/code>, \u0430 \u0443\u0436\u0435 \u0442\u043e\u043b\u044c\u043a\u043e \u043f\u043e\u0442\u043e\u043c \u043f\u0440\u043e\u0432\u0435\u0440\u044f\u044e\u0442 \u043d\u0435 \u044f\u0432\u043b\u044f\u044e\u0442\u0441\u044f \u043b\u0438 \u043e\u043d\u0438 null<\/code>.<\/p>\n
\u041d\u0435\u0434\u043e\u043c\u0430\u043f\u0438\u043b\u0438<\/h4>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6012<\/a> The ‘?:’ operator, regardless of its conditional expression, always returns one and the same value ‘card’. SkyclaveApparition.java 108<\/p>\n
@Override public boolean apply(Game game, Ability source) {     ....      Set<UUID> owners = new HashSet<>();     int totalCMC = exile             .getCards(game)             .stream()             .filter(Objects::nonNull)             .map(card -> owners.add(card.getOwnerId()) ? card : card) \/\/ <=             .mapToInt(MageObject::getManaValue)             .sum();     ....     return true; } <\/code><\/pre>\n
\u0414\u043e\u0441\u0442\u0430\u0442\u043e\u0447\u043d\u043e \u0441\u0442\u0440\u0430\u043d\u043d\u044b\u0439 \u043a\u043e\u0434, \u0441\u043f\u043e\u0441\u043e\u0431\u043d\u044b\u0439 \u0437\u0430\u043f\u0443\u0442\u0430\u0442\u044c \u0442\u043e\u0433\u043e, \u043a\u0442\u043e \u043d\u0435 \u0432 \u043a\u043e\u043d\u0442\u0435\u043a\u0441\u0442\u0435 \u044d\u0442\u043e\u0433\u043e \u043a\u043b\u0430\u0441\u0441\u0430. \u0422\u043e \u043b\u0438 \u0441 \u043e\u0431\u044a\u0435\u043a\u0442\u043e\u043c card<\/code> \u0445\u043e\u0442\u0435\u043b\u0438 \u0447\u0442\u043e-\u0442\u043e \u0441\u0434\u0435\u043b\u0430\u0442\u044c \u0432 \u043e\u0434\u043d\u043e\u0439 \u0438\u0437 \u0432\u0435\u0442\u043e\u043a \u0442\u0435\u0440\u043d\u0430\u0440\u043d\u043e\u0433\u043e \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u0430, \u043d\u043e \u0437\u0430\u0431\u044b\u043b\u0438, \u0442\u043e \u043b\u0438 \u043f\u0440\u043e\u0441\u0442\u043e \u0432\u0441\u0435 \u043e\u0431\u044a\u0435\u043a\u0442\u044b stream’\u0430 \u0445\u043e\u0442\u044f\u0442 \u0434\u043e\u0431\u0430\u0432\u0438\u0442\u044c \u0432 Set<\/code> owners<\/code>. \u0415\u0441\u043b\u0438 \u0440\u0435\u0447\u044c \u043f\u0440\u043e \u0432\u0442\u043e\u0440\u043e\u0439 \u0441\u043b\u0443\u0447\u0430\u0439, \u0442\u043e \u0434\u043e\u0441\u0442\u0430\u0442\u043e\u0447\u043d\u043e \u0432\u043e\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c\u0441\u044f \u043c\u0435\u0442\u043e\u0434\u043e\u043c peek<\/code> \u0432\u043c\u0435\u0441\u0442\u043e map<\/code>.<\/p>\n
\n\u041d\u0430\u043f\u043e\u043c\u0438\u043d\u0430\u043d\u0438\u0435 \u043f\u0440\u043e \u043c\u0435\u0442\u043e\u0434\u044b peek \u0438 map<\/summary>\n
\n
peek<\/code> \u2014 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 \u043f\u043e\u0441\u0442\u0443\u043f\u0438\u0432\u0448\u0438\u0439 \u0435\u043c\u0443 \u043f\u043e\u0442\u043e\u043a \u0434\u0430\u043d\u043d\u044b\u0445, \u0434\u043e\u043f\u043e\u043b\u043d\u0438\u0442\u0435\u043b\u044c\u043d\u043e \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u044f \u0441 \u0435\u0433\u043e \u044d\u043b\u0435\u043c\u0435\u043d\u0442\u0430\u043c\u0438 \u043a\u0430\u043a\u0438\u0435-\u043b\u0438\u0431\u043e \u0434\u0435\u0439\u0441\u0442\u0432\u0438\u044f.<\/p>\n
map<\/code> \u2014 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 \u043f\u043e\u0442\u043e\u043a \u0434\u0430\u043d\u043d\u044b\u0445, \u0441\u043e\u0441\u0442\u043e\u044f\u0449\u0438\u0439 \u0438\u0437 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u043e\u0432 \u043f\u0440\u0438\u043c\u0435\u043d\u0435\u043d\u0438\u044f \u0437\u0430\u0434\u0430\u043d\u043d\u043e\u0439 \u0444\u0443\u043d\u043a\u0446\u0438\u0438 \u043a \u043f\u043e\u0441\u0442\u0443\u043f\u0438\u0432\u0448\u0435\u043c\u0443 \u0435\u043c\u0443 \u043f\u043e\u0442\u043e\u043a\u0443 \u0434\u0430\u043d\u043d\u044b\u0445.<\/p>\n<\/div>\n<\/details>\n
\u0418 \u0441\u043d\u043e\u0432\u0430 null<\/h4>\n
\u0412 \u0440\u0430\u043c\u043a\u0430\u0445 \u0441\u043b\u0435\u0434\u0443\u044e\u0449\u0435\u0433\u043e \u0441\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u043d\u0438\u044f \u043f\u0440\u0438\u0434\u0451\u0442\u0441\u044f \u043f\u0440\u0438\u0431\u0435\u0433\u043d\u0443\u0442\u044c \u043a \u043d\u0435\u0431\u043e\u043b\u044c\u0448\u043e\u043c\u0443 \u043c\u0435\u0436\u043f\u0440\u043e\u0446\u0435\u0434\u0443\u0440\u043d\u043e\u043c\u0443 \u0430\u043d\u0430\u043b\u0438\u0437\u0443 \u0441\u0432\u043e\u0438\u043c\u0438 \u0433\u043b\u0430\u0437\u043a\u0430\u043c\u0438.<\/p>\n
\u0423 \u043d\u0430\u0441 \u0435\u0441\u0442\u044c \u0432\u043e\u0442 \u0442\u0430\u043a\u043e\u0439 \u0443\u0447\u0430\u0441\u0442\u043e\u043a \u043a\u043e\u0434\u0430:<\/p>\n
@Override public boolean apply(Game game, Ability source) {     ....     Card card = getCard(player, toReveal, game);     ....     player.moveCards(card, Zone.BATTLEFIELD, source, game);     ....     Permanent creature = game.getPermanent(card.getId());  \/\/ <=     .... } <\/code><\/pre>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6008<\/a> Potential null dereference of ‘card’. AspiringChampion.java 84<\/p>\n
\u0417\u0430\u0433\u043b\u044f\u043d\u0435\u043c \u0432 \u043c\u0435\u0442\u043e\u0434 getCard<\/code>, \u0438\u0437 \u043a\u043e\u0442\u043e\u0440\u043e\u0433\u043e \u043c\u044b \u043f\u043e\u043b\u0443\u0447\u0430\u0435\u043c \u043d\u0430\u0448 \u043e\u0431\u044a\u0435\u043a\u0442 card<\/code>. \u0412 \u043d\u0451\u043c \u0435\u0441\u0442\u044c \u0432\u0430\u0440\u0438\u0430\u043d\u0442 \u0438\u0441\u043f\u043e\u043b\u043d\u0435\u043d\u0438\u044f, \u043f\u0440\u0438 \u043a\u043e\u0442\u043e\u0440\u043e\u043c \u043c\u043e\u0436\u0435\u0442 \u0432\u0435\u0440\u043d\u0443\u0442\u044c\u0441\u044f null<\/code>:<\/p>\n
private static Card getCard(Player player, Cards toReveal, Game game) {     for (Card card : player.getLibrary().getCards(game)) {         toReveal.add(card);         if (card.isCreature(game)) {             return card;         }     }     return null; } <\/code><\/pre>\n
\u0412 \u0440\u0435\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u044f\u0445 \u043c\u0435\u0442\u043e\u0434\u0430 moveCards<\/code>, \u043a\u043e\u0442\u043e\u0440\u044b\u0439 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u0435\u0442 \u043e\u0431\u044a\u0435\u043a\u0442 card<\/code>, \u044d\u0442\u043e \u0443\u0447\u0438\u0442\u044b\u0432\u0430\u0435\u0442\u0441\u044f:<\/p>\n
@Override public boolean moveCards(Card card, ....) {     Set<Card> cardList = new HashSet<>();     if (card != null) {         cardList.add(card);     }     return moveCards(cardList, toZone, source, game,                       tapped, faceDown, byOwner, appliedEffects); } <\/code><\/pre>\n
\u0410 \u0435\u0441\u043b\u0438 \u043c\u044b \u0432\u0435\u0440\u043d\u0451\u043c\u0441\u044f \u043a \u0438\u0437\u043d\u0430\u0447\u0430\u043b\u044c\u043d\u043e\u043c\u0443 \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u0443 \u043a\u043e\u0434\u0430, \u0442\u043e \u0443\u0432\u0438\u0434\u0438\u043c, \u0447\u0442\u043e \u0432 \u043e\u0442\u043c\u0435\u0447\u0435\u043d\u043d\u043e\u0439 \u0441\u0442\u0440\u043e\u0447\u043a\u0435 \u043a\u043e\u0434\u0430 \u043a \u043d\u0435\u043c\u0443 \u043e\u0431\u0440\u0430\u0449\u0430\u044e\u0442\u0441\u044f, \u043d\u0435 \u0443\u0447\u0438\u0442\u044b\u0432\u0430\u044f, \u0447\u0442\u043e \u043e\u043d \u043c\u043e\u0436\u0435\u0442 \u0431\u044b\u0442\u044c null<\/code>.<\/p>\n
\u0412\u0441\u043f\u043e\u043c\u0438\u043d\u0430\u0435\u043c \u043e\u0441\u043d\u043e\u0432\u044b Java \u0447\u0435\u0440\u0435\u0437 \u0441\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u043d\u0438\u044f \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u0430<\/h3>\n
\u0410\u043a\u043a\u0443\u0440\u0430\u0442\u043d\u0435\u0435 \u0441\u043e \u0441\u0440\u0430\u0432\u043d\u0435\u043d\u0438\u0435\u043c String \u043e\u0431\u044a\u0435\u043a\u0442\u043e\u0432<\/h4>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6013<\/a> Strings ‘referenceName’ and ‘»»‘ are compared by reference. Possibly an equality comparison was intended. UrzasHotTub.java 112<\/p>\n
private boolean sharesWordWithName(String str) {     if (referenceName == null || referenceName == \"\") {         return false;     }     . . . . } <\/code><\/pre>\n
\u041e\u0448\u0438\u0431\u043a\u0430, \u0441 \u043a\u043e\u0442\u043e\u0440\u043e\u0439 \u043c\u044b \u0447\u0430\u0441\u0442\u043e \u0432\u0441\u0442\u0440\u0435\u0447\u0430\u0435\u043c\u0441\u044f \u0432 \u0440\u0430\u0437\u043b\u0438\u0447\u043d\u044b\u0445 \u043f\u0440\u043e\u0435\u043a\u0442\u0430\u0445 (\u043e\u0441\u043e\u0431\u0435\u043d\u043d\u043e \u0435\u0441\u043b\u0438 \u0443\u0447\u0430\u0441\u0442\u043e\u043a \u043a\u043e\u0434\u0430 \u043e\u0447\u0435\u043d\u044c \u0432\u043e\u0437\u0440\u0430\u0441\u0442\u043d\u043e\u0439). \u0421\u0442\u0440\u043e\u043a\u0438 \u043d\u0435 \u044f\u0432\u043b\u044f\u044e\u0442\u0441\u044f \u043f\u0440\u0438\u043c\u0438\u0442\u0438\u0432\u0430\u043c\u0438, \u043f\u043e\u044d\u0442\u043e\u043c\u0443, \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u044f ==<\/code>, \u0441\u0440\u0430\u0432\u043d\u0438\u0432\u0430\u044e\u0442\u0441\u044f \u0441\u0441\u044b\u043b\u043a\u0438 \u043d\u0430 \u043e\u0431\u044a\u0435\u043a\u0442\u044b, \u0430 \u043d\u0435 \u0441\u0430\u043c\u043e \u0441\u043e\u0434\u0435\u0440\u0436\u0438\u043c\u043e\u0435 \u043e\u0431\u044a\u0435\u043a\u0442\u043e\u0432. \u0418 \u043d\u0435\u0441\u043c\u043e\u0442\u0440\u044f \u043d\u0430 \u0442\u043e, \u0447\u0442\u043e \u0432 Java \u0435\u0441\u0442\u044c String Pool, \u0438 \u0434\u0432\u0435 \u0440\u0430\u0437\u043d\u044b\u0435 \u0441\u0442\u0440\u043e\u043a\u043e\u0432\u044b\u0435 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u044b\u0435 \u043c\u043e\u0433\u0443\u0442 \u0441\u0441\u044b\u043b\u0430\u0442\u044c\u0441\u044f \u043d\u0430 \u043e\u0434\u0438\u043d \u043e\u0431\u044a\u0435\u043a\u0442, \u043d\u0435 \u0444\u0430\u043a\u0442, \u0447\u0442\u043e \u044d\u0442\u043e \u0434\u0435\u0439\u0441\u0442\u0432\u0438\u0442\u0435\u043b\u044c\u043d\u043e \u043f\u0440\u043e\u0438\u0437\u043e\u0439\u0434\u0451\u0442. \u042d\u0442\u043e \u043f\u043e\u0432\u0435\u0434\u0435\u043d\u0438\u0435 \u044f\u0432\u043b\u044f\u0435\u0442\u0441\u044f \u043d\u0435\u0434\u0435\u0442\u0435\u0440\u043c\u0438\u043d\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u044b\u043c, \u0438, \u043a\u0430\u043a \u0441\u043b\u0435\u0434\u0441\u0442\u0432\u0438\u0435, \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 \u0442\u043e\u0436\u0435. \u041d\u0443\u0436\u043d\u043e \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u0442\u044c \u043c\u0435\u0442\u043e\u0434 equals<\/code> (\u0430 \u043a\u043e\u043d\u043a\u0440\u0435\u0442\u043d\u043e \u0432 \u044d\u0442\u043e\u0439 \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u0438 \u2014 \u043c\u0435\u0442\u043e\u0434 isEmpty<\/code>).<\/p>\n
\u0410\u043a\u043a\u0443\u0440\u0430\u0442\u043d\u0435\u0435 \u0441 \u0434\u0435\u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u0435\u0439<\/h4>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6087<\/a> InvalidClassException may occur during deserialization. Non-serializable ancestor ‘RoomImpl’ is missing an accessible no-arg constructor. GamesRoomImpl.java 32<\/p>\n
public class GamesRoomImpl extends RoomImpl implements GamesRoom, Serializable {     .... } <\/code><\/pre>\n
\u041a\u043b\u0430\u0441\u0441 \u044f\u0432\u043b\u044f\u0435\u0442\u0441\u044f \u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0443\u0435\u043c\u044b\u043c, \u043d\u043e \u043f\u0440\u0438 \u044d\u0442\u043e\u043c \u0443 \u0435\u0433\u043e \u043f\u0435\u0440\u0432\u043e\u0433\u043e \u043d\u0435\u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0443\u0435\u043c\u043e\u0433\u043e \u043f\u0440\u0435\u0434\u043a\u0430 \u043d\u0435\u0442 public no-arg \u043a\u043e\u043d\u0441\u0442\u0440\u0443\u043a\u0442\u043e\u0440\u0430:<\/p>\n
public abstract class RoomImpl implements Room {     ....     public RoomImpl(ChatManager chatManager) {         roomId = UUID.randomUUID();         chatId = chatManager.createChatSession(\"Room \" + roomId);     }     .... } <\/code><\/pre>\n
\u041f\u043e\u0447\u0435\u043c\u0443 \u044d\u0442\u043e \u043d\u0435\u043f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e?<\/p>\n
\u0423 \u043d\u0430\u0441 \u0435\u0441\u0442\u044c \u0441\u0442\u0430\u0442\u044c\u044f, \u0432 \u043a\u043e\u0442\u043e\u0440\u043e\u0439 \u043c\u044b \u0447\u0443\u0442\u044c-\u0447\u0443\u0442\u044c \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0435\u043b\u0438 \u043d\u0430 \u0432\u043d\u0443\u0442\u0440\u0435\u043d\u043d\u043e\u0441\u0442\u0438 \u043d\u0430\u0442\u0438\u0432\u043d\u043e\u0439 \u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u0438 \u0438 \u0434\u0435\u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u0438 \u0432 Java (\u0442\u044b\u043a<\/a>). \u0421\u043e\u0447\u0442\u0443 \u043f\u0440\u0438\u0435\u043c\u043b\u0435\u043c\u044b\u043c \u0441\u043e\u0441\u043b\u0430\u0442\u044c\u0441\u044f \u043d\u0430 \u043e\u0434\u0438\u043d \u0435\u0451 \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442:<\/p>\n
\n
\u0414\u0435\u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0443\u0435\u043c\u044b\u0439 \u043e\u0431\u044a\u0435\u043a\u0442 \u0441\u043e\u0437\u0434\u0430\u0451\u0442\u0441\u044f \u043d\u0435 \u0447\u0435\u0440\u0435\u0437 \u0441\u0432\u043e\u0439 \u043a\u043e\u043d\u0441\u0442\u0440\u0443\u043a\u0442\u043e\u0440, \u0430 \u0447\u0435\u0440\u0435\u0437 \u043a\u043e\u043d\u0441\u0442\u0440\u0443\u043a\u0442\u043e\u0440 \u0441\u0432\u043e\u0435\u0433\u043e \u043f\u0435\u0440\u0432\u043e\u0433\u043e \u043d\u0435\u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0443\u0435\u043c\u043e\u0433\u043e \u043f\u0440\u0435\u0434\u043a\u0430. \u0418 \u044d\u0442\u043e \u0434\u043e\u0432\u043e\u043b\u044c\u043d\u043e \u043b\u043e\u0433\u0438\u0447\u043d\u043e, \u0432\u0435\u0434\u044c \u0434\u0435\u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0443\u0435\u043c\u044b\u0439 \u043e\u0431\u044a\u0435\u043a\u0442 \u0438 \u0435\u0433\u043e \u043f\u0440\u0435\u0434\u043a\u0438, \u043f\u043e\u0434\u0434\u0435\u0440\u0436\u0438\u0432\u0430\u044e\u0449\u0438\u0435 \u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u044e, \u0432\u043e\u0441\u0441\u0442\u0430\u043d\u0430\u0432\u043b\u0438\u0432\u0430\u044e\u0442\u0441\u044f \u0438\u0437 \u043f\u043e\u0442\u043e\u043a\u0430 \u0431\u0430\u0439\u0442\u043e\u0432. \u0414\u043b\u044f \u043d\u0438\u0445 \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u0442\u044c \u043a\u043e\u0434 \u0438\u0437 \u043a\u043e\u043d\u0441\u0442\u0440\u0443\u043a\u0442\u043e\u0440\u043e\u0432 \u0438 \u0438\u043d\u0438\u0446\u0438\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u043e\u0432 \u0431\u0435\u0441\u0441\u043c\u044b\u0441\u043b\u0435\u043d\u043d\u043e. \u041f\u0440\u0438 \u044d\u0442\u043e\u043c \u0443 \u043d\u0430\u0441 \u043e\u0441\u0442\u0430\u044e\u0442\u0441\u044f \u043d\u0435\u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0443\u0435\u043c\u044b\u0435 \u043f\u0440\u0435\u0434\u043a\u0438. \u0427\u0442\u043e\u0431\u044b \u0438\u0445 \u0438\u043d\u0438\u0446\u0438\u0430\u043b\u0438\u0437\u0438\u0440\u043e\u0432\u0430\u0442\u044c, \u043c\u044b \u0432\u044b\u0437\u044b\u0432\u0430\u0435\u043c \u043a\u043e\u043d\u0441\u0442\u0440\u0443\u043a\u0442\u043e\u0440 \u043f\u0435\u0440\u0432\u043e\u0433\u043e \u043d\u0435\u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0443\u0435\u043c\u043e\u0433\u043e \u0440\u043e\u0434\u0438\u0442\u0435\u043b\u044f \u0432\u043e\u0441\u0441\u0442\u0430\u043d\u0430\u0432\u043b\u0438\u0432\u0430\u0435\u043c\u043e\u0433\u043e \u043e\u0431\u044a\u0435\u043a\u0442\u0430.<\/p>\n<\/blockquote>\n
\u0412 \u0441\u043b\u0443\u0447\u0430\u0435, \u0435\u0441\u043b\u0438 \u043f\u0440\u0438 \u0434\u0435\u0441\u0435\u0440\u0438\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u0438 \u043d\u0443\u0436\u043d\u044b\u0439 \u043a\u043e\u043d\u0441\u0442\u0440\u0443\u043a\u0442\u043e\u0440 \u043d\u0435 \u0431\u0443\u0434\u0435\u0442 \u043d\u0430\u0439\u0434\u0435\u043d, \u043c\u044b \u0443\u0432\u0438\u0434\u0438\u043c \u0438\u0441\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u0435 InvalidClassException<\/code> c \u0441\u043e\u043e\u0431\u0449\u0435\u043d\u0438\u0435\u043c no valid constructor<\/code>.<\/p>\n
\u0422\u0430\u043a \u0447\u0442\u043e, \u043a\u0430\u043a \u0433\u043e\u0432\u043e\u0440\u0438\u0442\u0441\u044f: «\u041d\u0430\u0437\u0432\u0430\u043b\u0441\u044f Serializable<\/code> \u2014 \u0438\u043c\u0435\u0439 \u0440\u043e\u0434\u0438\u0442\u0435\u043b\u044f \u0441 public no-arg \u043a\u043e\u043d\u0441\u0442\u0440\u0443\u043a\u0442\u043e\u0440\u043e\u043c».<\/p>\n
\u0410\u043a\u043a\u0443\u0440\u0430\u0442\u043d\u0435\u0435 \u0441 \u0441\u0438\u043d\u0445\u0440\u043e\u043d\u0438\u0437\u0430\u0446\u0438\u0435\u0439<\/h4>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 PVS-Studio: V6102<\/a> Inconsistent synchronization of the ‘continuousRuleModifyingEffects’ field. Consider synchronizing the field on all usages. ContinuousEffects.java 43<\/p>\n
public class ContinuousEffects {     ....     private ContinuousEffectsList<....>        continuousRuleModifyingEffects = new ContinuousEffectsList<>();     ....     public synchronized void removeInactiveEffects(Game game) {         ....         continuousRuleModifyingEffects.removeInactiveEffects(game);         ....     }     ....     public synchronized void removeEndOfCombatEffects() {         ....         continuousRuleModifyingEffects.removeEndOfCombatEffects();         ....     }     ....     public boolean preventedByRuleModification(....) {         for (.... effect : continuousRuleModifyingEffects) {     \/\/ <=             if (!effect.checksEventType(event, game)) {                 continue;             }             ....         }         ....     }     ....     public synchronized void addEffect(....) {         ....         switch (effect.getEffectType()) {             ....             case CONTINUOUS_RULE_MODIFICATION:                 continuousRuleModifyingEffects.addEffect(....);                 break;             ....         }     }     .... } <\/code><\/pre>\n
\u0412 \u043f\u0440\u0438\u043c\u0435\u0440 \u044f \u043f\u0440\u0438\u0432\u0451\u043b \u043b\u0438\u0448\u044c \u043f\u0430\u0440\u0443 \u043c\u0435\u0441\u0442, \u043e\u0434\u043d\u0430\u043a\u043e \u0432 \u043a\u043e\u0434\u0435 \u0432 \u0432\u043e\u0441\u044c\u043c\u0438 \u043c\u0435\u0442\u043e\u0434\u0430\u0445 \u043f\u043e\u043b\u0435 continuousRuleModifyingEffects<\/code> \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u0435\u0442\u0441\u044f \u0432 \u0441\u0438\u043d\u0445\u0440\u043e\u043d\u0438\u0437\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u043e\u043c \u043a\u043e\u043d\u0442\u0435\u043a\u0441\u0442\u0435, \u0430 \u0432 \u0434\u0435\u0432\u044f\u0442\u043e\u043c \u043d\u0435\u0442. \u041a\u0430\u0436\u0434\u044b\u0439 \u0438\u0437 \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u043e\u0432 \u044f \u043f\u0440\u0438\u0432\u043e\u0434\u0438\u0442\u044c \u043d\u0435 \u0441\u0442\u0430\u043b \u2014 \u0432 \u0442\u0430\u043a\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u043a\u043e\u0434 \u0431\u044b \u0437\u0430\u043d\u0438\u043c\u0430\u043b \u0441\u043b\u0438\u0448\u043a\u043e\u043c \u043c\u043d\u043e\u0433\u043e \u043c\u0435\u0441\u0442\u0430.<\/p>\n
\u0420\u0430\u0437 \u043c\u0435\u0442\u043e\u0434\u044b, \u0447\u0435\u0440\u0435\u0437 \u043a\u043e\u0442\u043e\u0440\u044b\u0435 \u0441 \u043d\u0430\u0448\u0438\u043c \u043f\u043e\u043b\u0435\u043c \u0432\u0437\u0430\u0438\u043c\u043e\u0434\u0435\u0439\u0441\u0442\u0432\u0443\u044e\u0442, \u043f\u043e\u043c\u0435\u0447\u0435\u043d\u044b \u043a\u0430\u043a synchronized<\/code>, \u0442\u043e \u043f\u0440\u0435\u0434\u043f\u043e\u043b\u0430\u0433\u0430\u0435\u0442\u0441\u044f, \u0447\u0442\u043e \u0432\u0437\u0430\u0438\u043c\u043e\u0434\u0435\u0439\u0441\u0442\u0432\u0438\u0435 \u0441 \u043e\u0431\u044a\u0435\u043a\u0442\u043e\u043c \u043f\u0440\u043e\u0438\u0441\u0445\u043e\u0434\u0438\u0442 \u0432 \u043c\u043d\u043e\u0433\u043e\u043f\u043e\u0442\u043e\u0447\u043d\u043e\u0439 \u0441\u0440\u0435\u0434\u0435. \u041e\u0434\u043d\u0430\u043a\u043e \u0432\u0437\u0430\u0438\u043c\u043e\u0434\u0435\u0439\u0441\u0442\u0432\u0438\u0435 \u0441 \u043f\u043e\u043b\u0435\u043c continuousRuleModifyingEffects<\/code> \u0432 \u043c\u0435\u0442\u043e\u0434\u0435 preventedByRuleModification<\/code> \u043d\u0438\u043a\u0430\u043a \u043d\u0435 \u0441\u0438\u043d\u0445\u0440\u043e\u043d\u0438\u0437\u0438\u0440\u0443\u0435\u0442\u0441\u044f.<\/p>\n
\u0418\u0437\u043c\u0435\u043d\u0435\u043d\u0438\u0435 \u043a\u043e\u043b\u043b\u0435\u043a\u0446\u0438\u0438 \u043f\u0440\u043e\u0438\u0441\u0445\u043e\u0434\u0438\u0442 \u0432 \u043e\u0434\u043d\u043e\u043c \u043f\u043e\u0442\u043e\u043a\u0435 (\u0432 \u043e\u0434\u043d\u043e\u043c \u0438\u0437 \u0441\u0438\u043d\u0445\u0440\u043e\u043d\u0438\u0437\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u044b\u0445 \u043c\u0435\u0442\u043e\u0434\u043e\u0432), \u0430 \u0447\u0442\u0435\u043d\u0438\u0435 \u0432 \u0434\u0440\u0443\u0433\u043e\u043c (\u0432 \u043d\u0435\u0441\u0438\u043d\u0445\u0440\u043e\u043d\u0438\u0437\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u043e\u043c \u043c\u0435\u0442\u043e\u0434\u0435). \u042d\u0442\u043e \u043c\u043e\u0436\u0435\u0442 \u043f\u0440\u0438\u0432\u0435\u0441\u0442\u0438 \u043a \u0441\u043b\u0435\u0434\u0443\u044e\u0449\u0438\u043c \u043f\u0440\u043e\u0431\u043b\u0435\u043c\u0430\u043c:<\/p>\n