{"id":362378,"date":"2024-05-21T01:33:52","date_gmt":"2024-05-21T01:33:52","guid":{"rendered":"http:\/\/savepearlharbor.com\/?p=362378"},"modified":"-0001-11-30T00:00:00","modified_gmt":"-0001-11-29T21:00:00","slug":"","status":"publish","type":"post","link":"https:\/\/savepearlharbor.com\/?p=362378","title":{"rendered":"<span>\u041d\u0435\u043c\u043d\u043e\u0433\u043e \u043e \u0433\u0438\u0433\u0438\u0435\u043d\u0435 Java \u043a\u043e\u0434\u0430<\/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>\u0412\u043e \u0432\u0440\u0435\u043c\u044f \u043a\u043e\u0434-\u0440\u0435\u0432\u044c\u044e \u0438\u043b\u0438 \u0440\u0430\u0431\u043e\u0442\u044b \u043d\u0430\u0434 \u043d\u043e\u0432\u044b\u043c \u043f\u0440\u043e\u0435\u043a\u0442\u043e\u043c \u043c\u043e\u0436\u0435\u0442 \u0440\u0430\u0437\u0434\u0440\u0430\u0436\u0430\u0442\u044c \u043c\u043d\u043e\u0433\u043e\u0435: \u0441\u0442\u0438\u043b\u044c, \u043f\u043e\u0434\u0445\u043e\u0434\u044b, \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u043e. \u041d\u043e \u0431\u043e\u043b\u044c\u0448\u0435 \u0432\u0441\u0435\u0433\u043e \u0440\u0430\u0441\u0441\u0442\u0440\u0430\u0438\u0432\u0430\u0435\u0442 \u0431\u0430\u043d\u0430\u043b\u044c\u043d\u044b\u0439 \u043d\u0435\u0434\u043e\u0441\u0442\u0430\u0442\u043e\u043a \u0433\u0438\u0433\u0438\u0435\u043d\u044b \u043a\u043e\u0434\u0430. \u0412 \u0441\u0432\u044f\u0437\u0438 \u0441 \u0440\u0435\u0433\u0443\u043b\u044f\u0440\u043d\u043e\u0441\u0442\u044c\u044e \u043f\u0440\u043e\u0431\u043b\u0435\u043c\u044b \u0445\u043e\u0447\u0435\u0442\u0441\u044f \u043d\u0430 \u043d\u0435\u0451 \u043e\u0431\u0440\u0430\u0442\u0438\u0442\u044c \u0432\u043d\u0438\u043c\u0430\u043d\u0438\u0435 \u0438 \u043d\u0430\u043f\u043e\u043c\u043d\u0438\u0442\u044c, \u043a\u0430\u043a \u0433\u0438\u0433\u0438\u0435\u043d\u0443 \u043f\u043e\u0434\u0434\u0435\u0440\u0436\u0438\u0432\u0430\u0442\u044c.<\/p>\n<figure class=\"full-width\"><img loading=\"lazy\" decoding=\"async\" src=\"https:\/\/habrastorage.org\/r\/w1560\/getpro\/habr\/upload_files\/4e8\/106\/3f2\/4e81063f2b8d41cb227159f08bfe9600.png\" alt=\"1086_ngb_ru\/image1.png\" title=\"\" width=\"1600\" height=\"902\" data-src=\"https:\/\/habrastorage.org\/getpro\/habr\/upload_files\/4e8\/106\/3f2\/4e81063f2b8d41cb227159f08bfe9600.png\"\/><\/figure>\n<h3>\u0412\u043c\u0435\u0441\u0442\u043e \u0432\u0441\u0442\u0443\u043f\u043b\u0435\u043d\u0438\u044f<\/h3>\n<p>\u0420\u0430\u0437\u0443\u043c\u0435\u0435\u0442\u0441\u044f, \u0432\u0441\u0435 \u043c\u044b \u043b\u044e\u0434\u0438, \u0436\u0438\u0432\u0443\u0449\u0438\u0435 \u0432 \u043d\u0435\u0441\u043e\u0432\u0435\u0440\u0448\u0435\u043d\u043d\u043e\u043c \u043c\u0438\u0440\u0435. \u041f\u043e\u044d\u0442\u043e\u043c\u0443 \u043f\u043e\u0441\u043b\u0435\u0434\u043d\u0435\u0435, \u0447\u0442\u043e \u043c\u043d\u0435 \u0445\u043e\u0447\u0435\u0442\u0441\u044f, \u044d\u0442\u043e \u0436\u0443\u0440\u0438\u0442\u044c \u0438 \u0442\u0430\u043a \u0441\u0442\u0435\u0441\u043d\u0451\u043d\u043d\u044b\u0445 \u043e\u0431\u0441\u0442\u043e\u044f\u0442\u0435\u043b\u044c\u0441\u0442\u0432\u0430\u043c\u0438 \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0438\u0441\u0442\u043e\u0432. \u0422\u0435\u043c \u0431\u043e\u043b\u0435\u0435 \u043d\u0435 \u0445\u043e\u0447\u0435\u0442\u0441\u044f \u043f\u043e\u0447\u0451\u043c \u0437\u0440\u044f \u0440\u0443\u0433\u0430\u0442\u044c \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u043e\u0432 Open Source \u043f\u0440\u043e\u0435\u043a\u0442\u043e\u0432. \u041d\u043e \u0441\u0435\u0439\u0447\u0430\u0441 \u044f \u043f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u044e \u043f\u043e\u0431\u044b\u0442\u044c \u043f\u0435\u0440\u0444\u0435\u043a\u0446\u0438\u043e\u043d\u0438\u0441\u0442\u0430\u043c\u0438 \u0438 \u0432\u0441\u043f\u043e\u043c\u043d\u0438\u0442\u044c \u0432\u0441\u0435 \u0442\u0435 \u0441\u043b\u0443\u0447\u0430\u0438, \u043a\u043e\u0433\u0434\u0430 \u0441 \u043f\u043e\u0434\u043e\u0431\u043d\u044b\u043c \u043a\u043e\u0434\u043e\u043c \u043f\u0440\u0438\u0445\u043e\u0434\u0438\u043b\u043e\u0441\u044c \u0438\u043c\u0435\u0442\u044c \u0434\u0435\u043b\u043e \u0432\u0430\u043c.<\/p>\n<p>\u0414\u043b\u044f \u0432\u043e\u0441\u043f\u0440\u043e\u0438\u0437\u0432\u0435\u0434\u0435\u043d\u0438\u044f \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u0438 \u043f\u0440\u043e\u0441\u0442\u043e \u0432\u043e\u0437\u044c\u043c\u0451\u043c \u0441\u043b\u0443\u0447\u0430\u0439\u043d\u044b\u0439 \u043f\u0440\u043e\u0435\u043a\u0442 \u0441 GitHub, \u0432 \u0434\u0430\u043d\u043d\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u2014 <a href=\"https:\/\/github.com\/epam\/NGB\">NGB<\/a>, \u0432\u0435\u0431-\u043f\u0440\u0438\u043b\u043e\u0436\u0435\u043d\u0438\u0435 \u0434\u043b\u044f \u043f\u0440\u043e\u0441\u043c\u043e\u0442\u0440\u0430 \u0433\u0435\u043d\u043e\u043c\u043e\u0432. \u0410 \u0447\u0442\u043e\u0431\u044b \u0443\u043f\u0440\u043e\u0441\u0442\u0438\u0442\u044c \u043f\u043e\u0438\u0441\u043a \u043f\u043e\u0434\u043e\u0437\u0440\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u0445 \u043c\u0435\u0441\u0442 \u0432 \u043a\u043e\u0434\u0435, \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u0435\u043c \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0439 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 PVS-Studio.<\/p>\n<h3>\u041f\u043e\u0433\u0440\u0443\u0436\u0430\u0435\u043c\u0441\u044f<\/h3>\n<h4>\u042d\u0442\u043e\u043c\u0443 \u0437\u0434\u0435\u0441\u044c \u043d\u0435 \u043c\u0435\u0441\u0442\u043e<\/h4>\n<p>\u041d\u0443 \u0442\u0430\u043a \u0432\u043e\u0442, \u043d\u0440\u0430\u0432\u0438\u0442\u0441\u044f \u043b\u0438 \u0432\u0430\u043c \u0432\u0438\u0434\u0435\u0442\u044c \u043a\u043e\u0434, \u043a\u043e\u0442\u043e\u0440\u044b\u0439 \u043d\u0438\u0447\u0435\u0433\u043e \u043d\u0435 \u0434\u0435\u043b\u0430\u0435\u0442? \u041d\u0430\u0434\u0435\u044e\u0441\u044c, \u0447\u0442\u043e \u0434\u0430, \u0438\u043d\u0430\u0447\u0435 \u0442\u0430\u043a\u043e\u0435 \u0437\u0440\u0435\u043b\u0438\u0449\u0435 \u0438\u0441\u043f\u043e\u0440\u0442\u0438\u0442 \u0432\u0430\u043c \u043d\u0430\u0441\u0442\u0440\u043e\u0435\u043d\u0438\u0435:<\/p>\n<pre><code class=\"cpp\">public NggbIntervalTreeMap&lt;Gene> loadGenesIntervalMap(GeneFile geneFile,                                                       int startIndex,                                                       int endIndex,                                                       Chromosome chromosome) {     ....     List&lt;Future&lt;Boolean>> results;     try {         results = taskExecutorService.getExecutorService().invokeAll(callables);     } catch (InterruptedException | AssertionError e) {         throw new GeneReadingException(geneFile,                                         chromosome,                                         startIndex,                                         endIndex,                                         e);     }      results.stream().map(future -> {         try {             return future != null ? future.get() : null;         } catch (InterruptedException | ExecutionException e) {             log.error(getMessage(MessagesConstants.ERROR_GENE_BATCH_LOAD,                                  geneFile.getId(),                                   chromosome.getId(),                                   e));         }         return null;     });     ....     return genesRangeMap; } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6010\/\">V6010<\/a> The return value of function &#8216;map&#8217; is required to be utilized. GffManager.java(477)<\/p>\n<p>\u041c\u043e\u0436\u0435\u0442 \u043f\u043e\u043a\u0430\u0437\u0430\u0442\u044c\u0441\u044f, \u0431\u0443\u0434\u0442\u043e \u0437\u0434\u0435\u0441\u044c \u0432\u0441\u0451 \u043d\u043e\u0440\u043c\u0430\u043b\u044c\u043d\u043e. \u041f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u0430\u044f <em>results<\/em> \u043e\u0431\u044a\u044f\u0432\u043b\u044f\u0435\u0442\u0441\u044f, \u043d\u0430\u043f\u043e\u043b\u043d\u044f\u0435\u0442\u0441\u044f \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f\u043c\u0438 \u0438 \u043e\u0431\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u0435\u0442\u0441\u044f. \u041f\u043e\u043a\u0430 \u0432\u044b \u043d\u0435 \u0443\u0432\u0438\u0434\u0438\u0442\u0435, \u0447\u0442\u043e \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442 \u0432\u044b\u0437\u043e\u0432\u0430 <em>map()<\/em> \u043d\u0435 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u0435\u0442\u0441\u044f, \u0430 \u0438\u0437 \u043c\u0435\u0442\u043e\u0434\u0430 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442\u0441\u044f \u0441\u043e\u0432\u0441\u0435\u043c \u0434\u0440\u0443\u0433\u043e\u0435 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 (\u0435\u0433\u043e \u043e\u0431\u0440\u0430\u0431\u043e\u0442\u043a\u0443 \u044f \u043e\u043f\u0443\u0441\u0442\u0438\u043b \u0434\u043b\u044f \u0443\u0434\u043e\u0431\u0441\u0442\u0432\u0430 \u0447\u0442\u0435\u043d\u0438\u044f, \u043a\u0430\u044e\u0441\u044c). \u0412\u043b\u0438\u044f\u0435\u0442 \u043b\u0438 \u044d\u0442\u043e \u043d\u0435\u0433\u0430\u0442\u0438\u0432\u043d\u043e \u043d\u0430 \u0440\u0430\u0431\u043e\u0442\u0443 \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u044b \u043b\u0438\u0431\u043e \u043f\u0440\u043e\u0441\u0442\u043e \u043e\u0441\u0442\u0430\u043b\u043e\u0441\u044c \u043f\u043e\u0441\u043b\u0435 \u0440\u0435\u0444\u0430\u043a\u0442\u043e\u0440\u0438\u043d\u0433\u0430 \u2014 \u0433\u0430\u0434\u0430\u0442\u044c \u0432\u0430\u043c (\u043c\u044b \u0436\u0435 \u043f\u0440\u0435\u0434\u0441\u0442\u0430\u0432\u0438\u043b\u0438, \u0447\u0442\u043e \u044d\u0442\u043e \u043d\u0430\u0448 \u043d\u043e\u0432\u044b\u0439 \u043f\u0440\u043e\u0435\u043a\u0442).<\/p>\n<h4>\u042d\u0442\u043e \u043a\u0440\u0438\u0442\u0438\u0447\u043d\u0430\u044f \u043e\u0448\u0438\u0431\u043a\u0430?<\/h4>\n<p>\u0418 \u0432\u043d\u043e\u0432\u044c \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u044f, \u0433\u0434\u0435 \u044f\u0432\u043d\u043e \u0447\u0442\u043e-\u0442\u043e \u0438\u0434\u0451\u0442 \u043d\u0435 \u0442\u0430\u043a, \u0442\u043e\u043b\u044c\u043a\u043e \u0437\u0434\u0435\u0441\u044c \u043e\u043f\u0440\u0430\u0432\u0434\u0430\u043d\u0438\u0435 \u043f\u0440\u0438\u0434\u0443\u043c\u0430\u0442\u044c \u0435\u0449\u0451 \u0441\u043b\u043e\u0436\u043d\u0435\u0435:<\/p>\n<pre><code class=\"cpp\">private static OrganismType determineHeterozygousGenotypeGA4GH(     VariantGA4GH ga4GHEntity, List&lt;Allele> alleles, int[] genotypeArray) {      OrganismType organismType = null;     for (int i = 0; i &lt; genotypeArray.length; i++) {         if (alleles.get(i).isReference()) {             genotypeArray[i] = 0;             organismType = OrganismType.HETEROZYGOUS;         } else {             if (organismType == null) {                 organismType = OrganismType.HETERO_VAR;             }             genotypeArray[i] = ga4GHEntity.getAlternateBases()                                           .indexOf(alleles.get(i)) + 1; \/\/ &lt;=         }     }      return organismType; } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6066\/\">V6066<\/a> The type of object passed as argument is incompatible with the type of collection: String, Allele. VcfGa4ghReader.java(535)<\/p>\n<p>\u041c\u0435\u0442\u043e\u0434* ga4GHEntity.getAlternateBases()* \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 <em>List&lt;String><\/em>, \u0432 \u0442\u043e \u0432\u0440\u0435\u043c\u044f \u043a\u0430\u043a \u043d\u0430\u0439\u0442\u0438 \u043c\u044b \u043f\u044b\u0442\u0430\u0435\u043c\u0441\u044f \u043e\u0431\u044a\u0435\u043a\u0442 \u0442\u0438\u043f\u0430 <em>Allele<\/em>. \u0422\u0430\u043a \u043a\u0430\u043a \u043e\u0431\u044a\u0435\u043a\u0442 \u0434\u0440\u0443\u0433\u043e\u0433\u043e \u0442\u0438\u043f\u0430, \u043e\u0436\u0438\u0434\u0430\u0435\u043c\u043e, \u043d\u0430\u0439\u0434\u0435\u043d \u043d\u0435 \u0431\u0443\u0434\u0435\u0442, \u0432 \u043c\u0430\u0441\u0441\u0438\u0432 <em>genotypeArray<\/em> \u0431\u0443\u0434\u0443\u0442 \u0437\u0430\u043f\u0438\u0441\u0430\u043d\u044b \u043d\u0443\u043b\u0438 (\u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442 \u0441\u043b\u043e\u0436\u0435\u043d\u0438\u044f -1 \u0438 1). \u041f\u0440\u043e\u0439\u0434\u0451\u0442\u0435 \u043c\u0438\u043c\u043e, \u043f\u0440\u0438\u0442\u0432\u043e\u0440\u0438\u0432\u0448\u0438\u0441\u044c, \u0447\u0442\u043e \u043d\u0438\u0447\u0435\u0433\u043e \u043d\u0435 \u0437\u0430\u043c\u0435\u0442\u0438\u043b\u0438, \u0438\u043b\u0438 \u043f\u043e\u043f\u044b\u0442\u0430\u0435\u0442\u0435\u0441\u044c \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u0442\u044c? \u0414\u0430\u0432\u043b\u0435\u043d\u0438\u0435 \u0441\u043e\u0432\u0435\u0441\u0442\u0438 \u0443\u0441\u0438\u043b\u0438\u0432\u0430\u0435\u0442\u0441\u044f, \u0435\u0441\u043b\u0438 \u0443\u0447\u0435\u0441\u0442\u044c, \u0447\u0442\u043e \u044d\u0442\u0430 \u0438 \u043f\u0440\u043e\u0448\u043b\u0430\u044f \u043e\u0448\u0438\u0431\u043a\u0438 \u043b\u043e\u0432\u044f\u0442\u0441\u044f \u0434\u0430\u0436\u0435 \u0441\u0430\u043c\u0438\u043c IntelliJ IDEA.<\/p>\n<h4>\u041d\u0443 \u043a\u0430\u043a \u0442\u0430\u043a-\u0442\u043e?<\/h4>\n<p>\u041f\u0440\u043e\u0434\u043e\u043b\u0436\u0430\u0435\u043c \u0430\u043a\u0438 \u0434\u0435\u0442\u0435\u043a\u0442\u0438\u0432\u044b (\u043d\u0435\u0443\u0434\u0430\u0447\u043d\u043e) \u0432\u044b\u044f\u0441\u043d\u044f\u0442\u044c, \u0447\u0442\u043e \u0436\u0435 \u0445\u043e\u0442\u0435\u043b\u0438 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438. \u0412\u043e\u0442 \u0434\u0432\u0430 \u0437\u0430\u0431\u0430\u0432\u043d\u044b\u0445 \u043c\u0435\u0441\u0442\u0430. \u041d\u043e\u043c\u0435\u0440 1:<\/p>\n<pre><code class=\"cpp\">private VariantSet variationMetadata(final String path) {     final int index = path.indexOf('-');     VariantSet variantSet;     final String variantSetId;     if (index >= 0) {         variantSetId = path.substring(0, index - 1);     } else {         variantSetId = path;     }     ....     return variantSet; } <\/code><\/pre>\n<p>\u0418 \u043d\u043e\u043c\u0435\u0440 2:<\/p>\n<pre><code class=\"cpp\">private TIntv getIntv(final String s) {     ....     int end = s.indexOf('\\t', beg);     while (end >= 0 || end == -1) {         if (col == mSc) {             ....         } else {             if ((mPreset &amp; TYPE_FLAG) == 0) {                 if (col == mEc) {                     intv.end = Integer.parseInt(end != -1                                                  ? s.substring(beg, end)                                                  : s.substring(beg));                 }             } else if ((mPreset &amp; TYPE_FLAG) == 1) {                 if (col == 6) {                     String cigar = s.substring(beg, end);                     ....                 }             } else if ((mPreset &amp; TYPE_FLAG) == 2) {                 String alt;                 alt = end >= 0 ? s.substring(beg, end)                                 : s.substring(beg);                 ....             }         }         if (end == -1) {             break;         }         beg = end + 1;         end = s.indexOf('\\t', beg);     }     return intv; } <\/code><\/pre>\n<p>\u041e\u0437\u0430\u0433\u043b\u0430\u0432\u0438\u043b \u044d\u0442\u043e\u0442 \u0440\u0430\u0437\u0434\u0435\u043b \u0444\u0440\u0430\u0437\u043e\u0439, \u043a\u043e\u0442\u043e\u0440\u0443\u044e \u044f \u0431\u044b \u0441\u043a\u0430\u0437\u0430\u043b \u043d\u0430 \u043a\u043e\u0434-\u0440\u0435\u0432\u044c\u044e, \u0443\u0432\u0438\u0434\u0435\u0432 \u0442\u0430\u043a\u043e\u0435. \u041e\u0448\u0438\u0431\u043a\u0443, \u043a\u0441\u0442\u0430\u0442\u0438, \u044f \u043d\u0435 \u043f\u0440\u0438\u0432\u0451\u043b \u0441\u0440\u0430\u0437\u0443 \u0441\u043f\u0435\u0446\u0438\u0430\u043b\u044c\u043d\u043e \u2014 \u043f\u043e\u043f\u0440\u043e\u0431\u0443\u0439\u0442\u0435 \u043d\u0430\u0439\u0442\u0438 \u0441\u0430\u043c\u0438 (\u044d\u0442\u043e \u043d\u0435 \u0442\u0440\u0443\u0434\u043d\u043e, \u044f \u0441\u043e\u043a\u0440\u0430\u0442\u0438\u043b \u043a\u043e\u0434 \ud83d\ude42 ). \u041f\u043e\u043a\u0430 \u0436\u0435 \u043e\u0441\u0442\u0430\u0432\u043b\u044e \u0441\u0432\u043e\u0438 \u043c\u044b\u0441\u043b\u0438 \u043d\u0430 \u044d\u0442\u043e\u0442 \u0441\u0447\u0451\u0442.<\/p>\n<p>\u041f\u043e\u0441\u043b\u0435 \u0442\u043e\u0433\u043e \u043a\u0430\u043a \u044f \u043f\u043e\u043a\u043e\u043f\u0430\u043b\u0441\u044f \u0432 \u043f\u0440\u043e\u0435\u043a\u0442\u0435, \u0443 \u043c\u0435\u043d\u044f \u043d\u0435 \u043f\u043e\u044f\u0432\u0438\u043b\u043e\u0441\u044c \u0443\u0432\u0435\u0440\u0435\u043d\u043d\u043e\u0441\u0442\u0438, \u0447\u0442\u043e \u043f\u0440\u0438\u043b\u043e\u0436\u0435\u043d\u0438\u0435 \u043b\u0435\u0433\u043a\u043e \u0443\u0440\u043e\u043d\u0438\u0442\u044c. \u041d\u043e \u0432\u0441\u0451 \u0436\u0435 \u0441\u0430\u043c \u0444\u0430\u043a\u0442 \u0434\u043e\u043f\u0443\u0449\u0435\u043d\u0438\u044f \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f \u043e\u0442\u0440\u0438\u0446\u0430\u0442\u0435\u043b\u044c\u043d\u043e\u0433\u043e \u0438\u043d\u0434\u0435\u043a\u0441\u0430 (\u043d\u0430\u043f\u0440\u044f\u043c\u0443\u044e \u0432\u043e \u0432\u0442\u043e\u0440\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u0438 \u0432\u0441\u043b\u0435\u0434\u0441\u0442\u0432\u0438\u0435 \u0432\u044b\u0447\u0438\u0442\u0430\u043d\u0438\u044f \u2014 \u0432 \u043f\u0435\u0440\u0432\u043e\u043c) \u0434\u043b\u044f \u0438\u0437\u0432\u043b\u0435\u0447\u0435\u043d\u0438\u044f \u043f\u043e\u0434\u0441\u0442\u0440\u043e\u043a\u0438 \u0432\u044b\u0437\u044b\u0432\u0430\u0435\u0442 \u0444\u0440\u0443\u0441\u0442\u0440\u0430\u0446\u0438\u044e. \u041e\u0441\u043e\u0431\u0435\u043d\u043d\u043e \u0432 \u043f\u0435\u0440\u0432\u043e\u043c \u043f\u0440\u0438\u043c\u0435\u0440\u0435, \u0433\u0434\u0435 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 \u0438\u043d\u0434\u0435\u043a\u0441\u0430 \u0441 \u0438\u0437\u0432\u043b\u0435\u0447\u0435\u043d\u0438\u0435\u043c \u043f\u043e\u0434\u0441\u0442\u0440\u043e\u043a\u0438 \u0431\u0443\u043a\u0432\u0430\u043b\u044c\u043d\u043e \u0440\u044f\u0434\u043e\u043c. \u0410 \u0432\u043e \u0432\u0442\u043e\u0440\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u0435\u0441\u0442\u044c \u043e\u0449\u0443\u0449\u0435\u043d\u0438\u0435, \u0447\u0442\u043e \u0431\u044b\u043b \u043f\u0440\u043e\u043f\u0443\u0449\u0435\u043d \u0442\u0435\u0440\u043d\u0430\u0440\u043d\u044b\u0439 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440, \u0438\u0431\u043e \u0432 \u0434\u0432\u0443\u0445 \u0434\u0440\u0443\u0433\u0438\u0445 \u043c\u0435\u0441\u0442\u0430\u0445 \u043e\u043d \u0438\u043c\u0435\u0435\u0442\u0441\u044f.<\/p>\n<p>\u0410 \u0432\u043e\u0442 \u0438 \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u044f \u043d\u0430 \u044d\u0442\u0438 \u0434\u0432\u0430 \u0443\u0447\u0430\u0441\u0442\u043a\u0430 \u043a\u043e\u0434\u0430:<\/p>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6009\/\">V6009<\/a> The &#8216;substring&#8217; function could receive the &#8216;-1&#8217; value while non-negative value is expected. Inspect argument: 2. VcfGa4ghReader.java(212)<\/p>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6009\/\">V6009<\/a> The &#8216;substring&#8217; function could receive the &#8216;-1&#8217; value while non-negative value is expected. Inspect argument: 2. TabixReader.java(430)<\/p>\n<h4>\u042d\u0442\u043e \u0442\u0430\u043a \u0438 \u0437\u0430\u0434\u0443\u043c\u0430\u043d\u043e, \u0438\u043b\u0438 \u0442\u0430\u043a \u0438 \u0437\u0430\u0434\u0443\u043c\u0430\u043d\u043e?<\/h4>\n<p>\u041d\u0443 \u0438\u043b\u0438 \u0432\u043e\u0442 \u0432\u0430\u043c \u043f\u043e\u043d\u0430\u0434\u043e\u0431\u0438\u043b\u043e\u0441\u044c \u043f\u043e\u0434\u043f\u0440\u0430\u0432\u0438\u0442\u044c \u0438\u043d\u0444\u0440\u0430\u0441\u0442\u0440\u0443\u043a\u0442\u0443\u0440\u0443 DAO, \u0437\u0430\u0445\u043e\u0434\u0438\u0442\u0435 \u0432\u044b \u0432 \u043d\u0443\u0436\u043d\u044b\u0439 \u043c\u043e\u0434\u0443\u043b\u044c, \u043a\u0430\u043a \u0432\u0438\u0434\u0438\u0442\u0435 \u044d\u0442\u043e\u0442 \u043c\u0435\u0442\u043e\u0434:<\/p>\n<pre><code class=\"cpp\">private String getGroupByField(final List&lt;VcfFile> files, final String groupBy){     final VcfIndexSortField sortField = VcfIndexSortField.getByName(groupBy);     if (sortField == null) {         ....         if (   infoItem.getType() == VCFHeaderLineType.Integer             || infoItem.getType() == VCFHeaderLineType.Float) {             return infoItem.getName().toLowerCase();         } else {             return infoItem.getName().toLowerCase();         }     } else {         if (   sortField.getType() == SortField.Type.INT             || sortField.getType() == SortField.Type.FLOAT) {             return sortField.getField().fieldName;         } else {             return sortField.getField().fieldName;         }     } } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6004\/\">V6004<\/a> The &#8216;then&#8217; statement is equivalent to the &#8216;else&#8217; statement. FeatureIndexDao.java(852), FeatureIndexDao.java(854)<\/p>\n<p>\u041d\u0435\u043f\u043e\u043d\u0438\u043c\u0430\u043d\u0438\u0435, \u0433\u0440\u0443\u0441\u0442\u044c, \u0440\u0430\u0437\u043e\u0447\u0430\u0440\u043e\u0432\u0430\u043d\u0438\u0435. \u0421\u0443\u0434\u044f \u043f\u043e \u0431\u043b\u0435\u0439\u043c\u0443, \u0442\u0430\u043a\u0430\u044f \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u044f \u0432\u043e\u0437\u043d\u0438\u043a\u043b\u0430 \u043f\u0440\u0438 \u043f\u0440\u0430\u0432\u043a\u0435 \u043e\u0434\u043d\u043e\u0439 \u0447\u0430\u0441\u0442\u0438 \u043a\u043e\u0434\u0430 \u0440\u0430\u0437\u043d\u044b\u043c\u0438 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0430\u043c\u0438. \u041d\u043e \u0434\u0430\u0436\u0435 \u043a\u043e\u0433\u0434\u0430 \u043c\u044b \u043d\u0430\u0448\u043b\u0438 \u043d\u0430 \u043a\u043e\u0433\u043e \u0441\u0432\u0430\u043b\u0438\u0442\u044c \u0432\u0438\u043d\u0443, \u043e\u0448\u0435\u043b\u043e\u043c\u043b\u0435\u043d\u0438\u0435 \u043d\u0435 \u043f\u0440\u043e\u043f\u0430\u0434\u0430\u0435\u0442: \u0437\u0430\u0447\u0435\u043c \u043a\u043e\u043c\u0443-\u0442\u043e \u043f\u043e\u043d\u0430\u0434\u043e\u0431\u0438\u043b\u043e\u0441\u044c \u0434\u0432\u0430 \u0443\u0441\u043b\u043e\u0432\u0438\u044f \u043f\u043e\u0434\u0440\u044f\u0434 \u0441 \u043e\u0434\u0438\u043d\u0430\u043a\u043e\u0432\u044b\u043c\u0438 \u0432\u0435\u0442\u0432\u044f\u043c\u0438? \u0418\u043b\u0438 \u044d\u0442\u043e \u044f \u0447\u0442\u043e-\u0442\u043e \u043d\u0435 \u043f\u043e\u043d\u044f\u043b?<\/p>\n<figure class=\"full-width\"><img loading=\"lazy\" decoding=\"async\" src=\"https:\/\/habrastorage.org\/r\/w1560\/getpro\/habr\/upload_files\/b07\/b9b\/030\/b07b9b03000414d73c2cd00ba0c85868.png\" alt=\"1086_ngb_ru\/image2.png\" title=\"\" width=\"1600\" height=\"902\" data-src=\"https:\/\/habrastorage.org\/getpro\/habr\/upload_files\/b07\/b9b\/030\/b07b9b03000414d73c2cd00ba0c85868.png\"\/><\/figure>\n<p>\u0415\u0441\u0442\u044c \u0431\u043e\u043b\u044c\u0448\u0430\u044f \u0432\u0435\u0440\u043e\u044f\u0442\u043d\u043e\u0441\u0442\u044c, \u0447\u0442\u043e \u043d\u0435\u043f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e\u0433\u043e \u043f\u043e\u0432\u0435\u0434\u0435\u043d\u0438\u044f \u0442\u0443\u0442 \u043d\u0435\u0442. \u041d\u043e \u0430 \u0447\u0442\u043e, \u0435\u0441\u043b\u0438 \u0435\u0441\u0442\u044c? \u041e\u0441\u0442\u0430\u0451\u0442\u0441\u044f \u0442\u043e\u043b\u044c\u043a\u043e \u0442\u044b\u043a\u0430\u0442\u044c \u044e\u043d\u0438\u0442-\u0442\u0435\u0441\u0442\u044b&#8230;<\/p>\n<h4>\u041a\u0430\u043a \u0442\u0435\u0441\u0442\u0438\u0440\u043e\u0432\u0430\u0442\u044c \u044e\u043d\u0438\u0442-\u0442\u0435\u0441\u0442\u044b<\/h4>\n<p>\u041a\u0441\u0442\u0430\u0442\u0438, \u043d\u0430\u0441\u0447\u0451\u0442 \u044e\u043d\u0438\u0442-\u0442\u0435\u0441\u0442\u043e\u0432. \u0418 \u043a \u043d\u0438\u043c \u0434\u043e\u0441\u0442\u0430\u0442\u043e\u0447\u043d\u043e \u043b\u0435\u0433\u043a\u043e \u043f\u043e\u0434\u043e\u0440\u0432\u0430\u0442\u044c \u0434\u043e\u0432\u0435\u0440\u0438\u0435. \u041d\u0430\u043f\u0440\u0438\u043c\u0435\u0440, \u0434\u043e\u043f\u0443\u0441\u0442\u0438\u0442\u044c \u043e\u0448\u0438\u0431\u043a\u0443 \u0432 \u0438\u0445 \u0441\u0435\u0442\u0430\u043f\u0435:<\/p>\n<pre><code class=\"cpp\">public void addFeatureIndexedFileRegistration(Long refId,                                               String path,                                               String indexPath,                                               String name,                                               Long fileId,                                               Long fileBioId,                                                BiologicalDataItemFormat format,                                               boolean doIndex,                                               String prettyName) {     String absolutePath = getNormalizeAndAbsolutePath(path);     onRequest().havingMethodEqualTo(HTTP_POST)                .havingPathEqualTo(....)                .havingBodyEqualTo(TestDataProvider                                      .getRegistrationJsonWithPrettyName(                                          refId,                                           absolutePath,                                           indexPath,                                          name,                                           doIndex,                                           prettyName))                .respond()                .withBody(....)                .withStatus(HTTP_STATUS_OK); } <\/code><\/pre>\n<p>\u0427\u0442\u043e\u0431\u044b \u0440\u0430\u0441\u043a\u0443\u0441\u0438\u0442\u044c \u043f\u043e\u0434\u0432\u043e\u0445, \u043d\u0430\u0434\u043e \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0435\u0442\u044c \u043d\u0430 \u0441\u0438\u0433\u043d\u0430\u0442\u0443\u0440\u0443 \u043c\u0435\u0442\u043e\u0434\u0430: TestDataProvider.<em>getRegistrationJsonWithPrettyName(&#8230;.)<\/em>:<\/p>\n<pre><code class=\"cpp\">public static String getRegistrationJsonWithPrettyName(     Long referenceID,     String path,     String name,     String indexPath,     boolean doIndex,     String prettyName) {     .... } <\/code><\/pre>\n<p>\u0417\u0430\u043c\u0435\u0442\u0438\u043b\u0438? \u0415\u0441\u043b\u0438 \u0434\u0430, \u0442\u043e \u043c\u043e\u0436\u0435\u0442\u0435 \u043e\u0442\u0431\u043b\u0430\u0433\u043e\u0434\u0430\u0440\u0438\u0442\u044c \u0437\u0430 \u0442\u043e, \u0447\u0442\u043e \u044f \u043d\u0435\u043c\u043d\u043e\u0433\u043e \u043e\u0442\u0444\u043e\u0440\u043c\u0430\u0442\u0438\u0440\u043e\u0432\u0430\u043b \u043a\u043e\u0434. \u0412\u043e\u0442 \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435:<\/p>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6029\/\">V6029<\/a> Possible incorrect order of arguments passed to method: &#8216;indexPath&#8217;, &#8216;name&#8217;. TestHttpServer.java(139), TestHttpServer.java(140)<\/p>\n<p>\u0421\u0443\u0434\u044f \u043f\u043e \u0432\u0441\u0435\u043c\u0443, \u043f\u0440\u0438 \u0432\u044b\u0437\u043e\u0432\u0435 \u043c\u0435\u0442\u043e\u0434\u0430 \u043f\u0435\u0440\u0435\u043f\u0443\u0442\u0430\u043d \u043f\u043e\u0440\u044f\u0434\u043e\u043a \u0430\u0440\u0433\u0443\u043c\u0435\u043d\u0442\u043e\u0432: \u043f\u043e\u0440\u044f\u0434\u043e\u043a \u043f\u0435\u0440\u0435\u0434\u0430\u0447\u0438 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u044b\u0445 <em>indexPath<\/em> \u0438 <em>name<\/em> \u043d\u0435 \u0441\u043e\u043e\u0442\u0432\u0435\u0442\u0441\u0442\u0432\u0443\u0435\u0442 \u043f\u043e\u0440\u044f\u0434\u043a\u0443 \u043f\u0430\u0440\u0430\u043c\u0435\u0442\u0440\u043e\u0432 \u0441 \u0430\u043d\u0430\u043b\u043e\u0433\u0438\u0447\u043d\u044b\u043c\u0438 \u0438\u043c\u0435\u043d\u0430\u043c\u0438 \u0432 \u043c\u0435\u0442\u043e\u0434\u0435. \u041a\u043e\u043c\u043f\u0435\u043d\u0441\u0438\u0440\u043e\u0432\u0430\u043d \u043b\u0438 \u044d\u0442\u043e\u0442 \u0431\u0430\u0433 \u0434\u0440\u0443\u0433\u0438\u043c, \u0438\u043b\u0438 \u043e\u043d \u0433\u0434\u0435-\u0442\u043e \u043d\u0435\u0433\u0430\u0442\u0438\u0432\u043d\u043e \u0432\u043b\u0438\u044f\u0435\u0442 \u043d\u0430 \u0442\u0435\u0441\u0442\u044b, \u2014 \u043e\u0441\u0442\u0430\u0451\u0442\u0441\u044f \u0442\u043e\u043b\u044c\u043a\u043e \u0434\u043e\u0433\u0430\u0434\u044b\u0432\u0430\u0442\u044c\u0441\u044f. \u0418 \u0440\u0430\u0434\u043e\u0432\u0430\u0442\u044c\u0441\u044f, \u0447\u0442\u043e \u043f\u0440\u0430\u0432\u0438\u0442\u044c \u044d\u0442\u043e \u0432\u0441\u0451 \u0436\u0435 \u043d\u0435 \u043d\u0430\u043c.<\/p>\n<h3>\u041a\u0430\u043a \u0441\u043b\u0435\u0434\u0438\u0442\u044c \u0437\u0430 \u043a\u043e\u0434\u043e\u043c<\/h3>\n<p>\u0418 \u0432\u043e\u0442 \u0442\u0430\u043a\u043e\u0435 \u043f\u043e\u043f\u0430\u0434\u0430\u0435\u0442\u0441\u044f \u043f\u043e\u0441\u0442\u043e\u044f\u043d\u043d\u043e. \u0425\u043e\u0442\u044f \u0432\u0440\u043e\u0434\u0435 \u0432\u0441\u0451 \u043f\u043e\u043d\u044f\u0442\u043d\u043e: \u0445\u043e\u0440\u043e\u0448\u0438\u0439 \u043a\u043e\u0434 \u043f\u0438\u0441\u0430\u0442\u044c, \u043f\u043b\u043e\u0445\u043e\u0439 \u2014 \u043d\u0435 \u043f\u0438\u0441\u0430\u0442\u044c, \u043e\u0448\u0438\u0431\u0430\u0442\u044c\u0441\u044f \u043d\u0435 \u043d\u0430\u0434\u043e. \u0412 \u0441\u043b\u0435\u0434\u0443\u044e\u0449\u0438\u0439 \u0440\u0430\u0437 \u041a\u0430\u043f\u0438\u0442\u0430\u043d \u041e\u0447\u0435\u0432\u0438\u0434\u043d\u043e\u0441\u0442\u044c \u0432\u0435\u0440\u043d\u0451\u0442\u0441\u044f \u043a \u0432\u0430\u043c \u0441 \u043d\u043e\u0432\u044b\u043c \u0443\u0434\u0438\u0432\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u043c \u043e\u0442\u043a\u0440\u044b\u0442\u0438\u0435\u043c.<\/p>\n<p>\u041d\u043e \u0435\u0441\u043b\u0438 \u0441\u0435\u0440\u044c\u0451\u0437\u043d\u043e, \u0442\u043e \u0432\u0441\u0435 \u0432\u0441\u0451 \u0437\u043d\u0430\u044e\u0442. \u041a\u0430\u043a \u044f \u0438 \u0433\u043e\u0432\u043e\u0440\u0438\u043b, \u043c\u043e\u0451 \u0434\u0435\u043b\u043e \u043b\u0438\u0448\u044c \u043d\u0430\u043f\u043e\u043c\u043d\u0438\u0442\u044c. \u041f\u043e\u043c\u0438\u043c\u043e \u0431\u0430\u043d\u0430\u043b\u044c\u043d\u043e\u0439 \u0433\u043e\u043b\u043e\u0432\u044b \u043d\u0430 \u043f\u043b\u0435\u0447\u0430\u0445 \u043d\u0435 \u043d\u0430\u0434\u043e \u0438\u0433\u043d\u043e\u0440\u0438\u0440\u043e\u0432\u0430\u0442\u044c \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u044f IDE. \u0414\u0430\u043b\u0435\u0435 \u043c\u043e\u0436\u043d\u043e \u043f\u043e\u0434\u0443\u043c\u0430\u0442\u044c \u043e \u043b\u0438\u043d\u0442\u0435\u0440\u0430\u0445 \u0434\u043b\u044f \u043f\u0440\u043e\u0441\u0442\u044b\u0445 \u0441\u043b\u0443\u0447\u0430\u0435\u0432 \u0438 \u043e \u0441\u043f\u0435\u0446\u0438\u0430\u043b\u0438\u0437\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u044b\u0445 \u0438\u043d\u0441\u0442\u0440\u0443\u043c\u0435\u043d\u0442\u0430\u0445 \u0434\u043b\u044f \u043d\u0430\u0445\u043e\u0436\u0434\u0435\u043d\u0438\u044f \u0431\u043e\u043b\u044c\u0448\u0435\u0433\u043e \u043a\u043e\u043b\u0438\u0447\u0435\u0441\u0442\u0432\u0430 \u043f\u043e\u0434\u043e\u0437\u0440\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u0445 \u043c\u0435\u0441\u0442. \u0422\u0430\u043a, \u0432 \u044d\u0442\u043e\u0439 \u0441\u0442\u0430\u0442\u044c\u0435 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043b\u0441\u044f \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0439 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 PVS-Studio. \u041d\u043e \u0438 \u0430\u0432\u0442\u043e\u043c\u0430\u0442\u0438\u043a\u0430, \u043a \u0441\u043e\u0436\u0430\u043b\u0435\u043d\u0438\u044e, \u043d\u0435 \u043f\u0430\u043d\u0430\u0446\u0435\u044f \u2014 \u043f\u0440\u043e\u0441\u0442\u043e\u0435 \u0447\u0435\u043b\u043e\u0432\u0435\u0447\u0435\u0441\u043a\u043e\u0435 \u043a\u043e\u0434-\u0440\u0435\u0432\u044c\u044e \u043d\u0438\u043a\u0442\u043e \u043d\u0435 \u043e\u0442\u043c\u0435\u043d\u044f\u043b. \u0422\u0443\u0442 \u0447\u0435\u043b\u043e\u0432\u0435\u043a\u0430 \u043c\u0430\u0448\u0438\u043d\u0430 \u043f\u043e\u043a\u0430 \u043d\u0435 \u0437\u0430\u043c\u0435\u043d\u0438\u0442.<\/p>\n<h3>\u041f\u043e\u0441\u043b\u0435\u0441\u043b\u043e\u0432\u0438\u0435<\/h3>\n<p>\u0415\u0441\u043b\u0438 \u0447\u0442\u043e, \u043a \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0430\u043c NGB \u0443 \u043c\u0435\u043d\u044f \u0431\u043e\u043b\u044c\u0448\u0438\u0445 \u043f\u0440\u0435\u0442\u0435\u043d\u0437\u0438\u0439 \u043f\u043e \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0443 \u043a\u043e\u0434\u0430 \u043d\u0435\u0442, \u044f \u043b\u0438\u0448\u044c \u043f\u043e\u0434\u0451\u0440\u0433\u0430\u043b \u043f\u0440\u0438\u043c\u0435\u0440\u044b \u043a\u043e\u0434\u0430, \u043a\u043e\u0442\u043e\u0440\u044b\u043c\u0438 \u043b\u0435\u0433\u043a\u043e \u0441\u0431\u0438\u0442\u044c \u0441 \u0442\u043e\u043b\u043a\u0443. \u0418 \u043d\u0430 \u044d\u0442\u043e\u043c \u0441\u043e \u0441\u0432\u043e\u0438\u043c \u043d\u0435\u0433\u043e\u0434\u043e\u0432\u0430\u043d\u0438\u0435\u043c \u044f \u0437\u0430\u043a\u0430\u043d\u0447\u0438\u0432\u0430\u044e, \u0430 \u0432\u044b \u0441\u0432\u043e\u0438\u043c \u043c\u043e\u0436\u0435\u0442\u0435 \u043f\u043e\u0434\u0435\u043b\u0438\u0442\u044c\u0441\u044f \u0432 \u043a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u044f\u0445.<\/p>\n<p>\u0415\u0441\u043b\u0438 \u0445\u043e\u0447\u0435\u0442\u0441\u044f \u0441\u0430\u043c\u0438\u043c \u043f\u0440\u043e\u0432\u0435\u0440\u0438\u0442\u044c \u043f\u0440\u043e\u0435\u043a\u0442 \u0441 \u043f\u043e\u043c\u043e\u0449\u044c\u044e PVS-Studio, \u0442\u043e \u043f\u043e\u043f\u0440\u043e\u0431\u043e\u0432\u0430\u0442\u044c \u0435\u0433\u043e \u043c\u043e\u0436\u043d\u043e, \u043f\u0435\u0440\u0435\u0439\u0434\u044f \u043f\u043e <a href=\"https:\/\/pvs-studio.ru\/ru\/pvs-studio\/try-free\/?utm_source=website&amp;utm_medium=habr&amp;utm_campaign=article&amp;utm_content=1086\">\u044d\u0442\u043e\u0439<\/a> \u0441\u0441\u044b\u043b\u043a\u0435.<\/p>\n<p>\u0410 \u0447\u0442\u043e\u0431\u044b \u0441\u043b\u0435\u0434\u0438\u0442\u044c \u0437\u0430 \u0432\u044b\u0445\u043e\u0434\u043e\u043c \u043d\u043e\u0432\u044b\u0445 \u0441\u0442\u0430\u0442\u0435\u0439 \u043d\u0430 \u0442\u0435\u043c\u0443 \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0430 \u043a\u043e\u0434\u0430, \u043c\u043e\u0436\u0435\u0442\u0435 \u043f\u043e\u0434\u043f\u0438\u0441\u0430\u0442\u044c\u0441\u044f \u043d\u0430:<\/p>\n<ul>\n<li>\n<p><a href=\"https:\/\/twitter.com\/Code_Analysis\">\u0422\u0432\u0438\u0442\u0442\u0435\u0440<\/a> PVS-Studio;<\/p>\n<\/li>\n<li>\n<p>\u041d\u0430\u0448 \u0435\u0436\u0435\u043c\u0435\u0441\u044f\u0447\u043d\u044b\u0439 <a href=\"https:\/\/pvs-studio.ru\/ru\/subscribe\/\">\u0434\u0430\u0439\u0434\u0436\u0435\u0441\u0442<\/a> \u0441\u0442\u0430\u0442\u0435\u0439;<\/p>\n<\/li>\n<li>\n<p>\u041c\u043e\u0439 \u043b\u0438\u0447\u043d\u044b\u0439 <a href=\"https:\/\/twitter.com\/kvolokhovskii\">\u0431\u043b\u043e\u0433<\/a>.<\/p>\n<\/li>\n<\/ul>\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: Konstantin Volohovsky. <a href=\"https:\/\/pvs-studio.com\/en\/blog\/posts\/java\/1086\/\">Few words about Java code hygiene<\/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\/779300\/\"> https:\/\/habr.com\/ru\/articles\/779300\/<\/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>\u0412\u043e \u0432\u0440\u0435\u043c\u044f \u043a\u043e\u0434-\u0440\u0435\u0432\u044c\u044e \u0438\u043b\u0438 \u0440\u0430\u0431\u043e\u0442\u044b \u043d\u0430\u0434 \u043d\u043e\u0432\u044b\u043c \u043f\u0440\u043e\u0435\u043a\u0442\u043e\u043c \u043c\u043e\u0436\u0435\u0442 \u0440\u0430\u0437\u0434\u0440\u0430\u0436\u0430\u0442\u044c \u043c\u043d\u043e\u0433\u043e\u0435: \u0441\u0442\u0438\u043b\u044c, \u043f\u043e\u0434\u0445\u043e\u0434\u044b, \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u043e. \u041d\u043e \u0431\u043e\u043b\u044c\u0448\u0435 \u0432\u0441\u0435\u0433\u043e \u0440\u0430\u0441\u0441\u0442\u0440\u0430\u0438\u0432\u0430\u0435\u0442 \u0431\u0430\u043d\u0430\u043b\u044c\u043d\u044b\u0439 \u043d\u0435\u0434\u043e\u0441\u0442\u0430\u0442\u043e\u043a \u0433\u0438\u0433\u0438\u0435\u043d\u044b \u043a\u043e\u0434\u0430. \u0412 \u0441\u0432\u044f\u0437\u0438 \u0441 \u0440\u0435\u0433\u0443\u043b\u044f\u0440\u043d\u043e\u0441\u0442\u044c\u044e \u043f\u0440\u043e\u0431\u043b\u0435\u043c\u044b \u0445\u043e\u0447\u0435\u0442\u0441\u044f \u043d\u0430 \u043d\u0435\u0451 \u043e\u0431\u0440\u0430\u0442\u0438\u0442\u044c \u0432\u043d\u0438\u043c\u0430\u043d\u0438\u0435 \u0438 \u043d\u0430\u043f\u043e\u043c\u043d\u0438\u0442\u044c, \u043a\u0430\u043a \u0433\u0438\u0433\u0438\u0435\u043d\u0443 \u043f\u043e\u0434\u0434\u0435\u0440\u0436\u0438\u0432\u0430\u0442\u044c.<\/p>\n<figure class=\"full-width\"><\/figure>\n<h3>\u0412\u043c\u0435\u0441\u0442\u043e \u0432\u0441\u0442\u0443\u043f\u043b\u0435\u043d\u0438\u044f<\/h3>\n<p>\u0420\u0430\u0437\u0443\u043c\u0435\u0435\u0442\u0441\u044f, \u0432\u0441\u0435 \u043c\u044b \u043b\u044e\u0434\u0438, \u0436\u0438\u0432\u0443\u0449\u0438\u0435 \u0432 \u043d\u0435\u0441\u043e\u0432\u0435\u0440\u0448\u0435\u043d\u043d\u043e\u043c \u043c\u0438\u0440\u0435. \u041f\u043e\u044d\u0442\u043e\u043c\u0443 \u043f\u043e\u0441\u043b\u0435\u0434\u043d\u0435\u0435, \u0447\u0442\u043e \u043c\u043d\u0435 \u0445\u043e\u0447\u0435\u0442\u0441\u044f, \u044d\u0442\u043e \u0436\u0443\u0440\u0438\u0442\u044c \u0438 \u0442\u0430\u043a \u0441\u0442\u0435\u0441\u043d\u0451\u043d\u043d\u044b\u0445 \u043e\u0431\u0441\u0442\u043e\u044f\u0442\u0435\u043b\u044c\u0441\u0442\u0432\u0430\u043c\u0438 \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0438\u0441\u0442\u043e\u0432. \u0422\u0435\u043c \u0431\u043e\u043b\u0435\u0435 \u043d\u0435 \u0445\u043e\u0447\u0435\u0442\u0441\u044f \u043f\u043e\u0447\u0451\u043c \u0437\u0440\u044f \u0440\u0443\u0433\u0430\u0442\u044c \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u043e\u0432 Open Source \u043f\u0440\u043e\u0435\u043a\u0442\u043e\u0432. \u041d\u043e \u0441\u0435\u0439\u0447\u0430\u0441 \u044f \u043f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u044e \u043f\u043e\u0431\u044b\u0442\u044c \u043f\u0435\u0440\u0444\u0435\u043a\u0446\u0438\u043e\u043d\u0438\u0441\u0442\u0430\u043c\u0438 \u0438 \u0432\u0441\u043f\u043e\u043c\u043d\u0438\u0442\u044c \u0432\u0441\u0435 \u0442\u0435 \u0441\u043b\u0443\u0447\u0430\u0438, \u043a\u043e\u0433\u0434\u0430 \u0441 \u043f\u043e\u0434\u043e\u0431\u043d\u044b\u043c \u043a\u043e\u0434\u043e\u043c \u043f\u0440\u0438\u0445\u043e\u0434\u0438\u043b\u043e\u0441\u044c \u0438\u043c\u0435\u0442\u044c \u0434\u0435\u043b\u043e \u0432\u0430\u043c.<\/p>\n<p>\u0414\u043b\u044f \u0432\u043e\u0441\u043f\u0440\u043e\u0438\u0437\u0432\u0435\u0434\u0435\u043d\u0438\u044f \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u0438 \u043f\u0440\u043e\u0441\u0442\u043e \u0432\u043e\u0437\u044c\u043c\u0451\u043c \u0441\u043b\u0443\u0447\u0430\u0439\u043d\u044b\u0439 \u043f\u0440\u043e\u0435\u043a\u0442 \u0441 GitHub, \u0432 \u0434\u0430\u043d\u043d\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u2014 <a href=\"https:\/\/github.com\/epam\/NGB\">NGB<\/a>, \u0432\u0435\u0431-\u043f\u0440\u0438\u043b\u043e\u0436\u0435\u043d\u0438\u0435 \u0434\u043b\u044f \u043f\u0440\u043e\u0441\u043c\u043e\u0442\u0440\u0430 \u0433\u0435\u043d\u043e\u043c\u043e\u0432. \u0410 \u0447\u0442\u043e\u0431\u044b \u0443\u043f\u0440\u043e\u0441\u0442\u0438\u0442\u044c \u043f\u043e\u0438\u0441\u043a \u043f\u043e\u0434\u043e\u0437\u0440\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u0445 \u043c\u0435\u0441\u0442 \u0432 \u043a\u043e\u0434\u0435, \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u0435\u043c \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0439 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 PVS-Studio.<\/p>\n<h3>\u041f\u043e\u0433\u0440\u0443\u0436\u0430\u0435\u043c\u0441\u044f<\/h3>\n<h4>\u042d\u0442\u043e\u043c\u0443 \u0437\u0434\u0435\u0441\u044c \u043d\u0435 \u043c\u0435\u0441\u0442\u043e<\/h4>\n<p>\u041d\u0443 \u0442\u0430\u043a \u0432\u043e\u0442, \u043d\u0440\u0430\u0432\u0438\u0442\u0441\u044f \u043b\u0438 \u0432\u0430\u043c \u0432\u0438\u0434\u0435\u0442\u044c \u043a\u043e\u0434, \u043a\u043e\u0442\u043e\u0440\u044b\u0439 \u043d\u0438\u0447\u0435\u0433\u043e \u043d\u0435 \u0434\u0435\u043b\u0430\u0435\u0442? \u041d\u0430\u0434\u0435\u044e\u0441\u044c, \u0447\u0442\u043e \u0434\u0430, \u0438\u043d\u0430\u0447\u0435 \u0442\u0430\u043a\u043e\u0435 \u0437\u0440\u0435\u043b\u0438\u0449\u0435 \u0438\u0441\u043f\u043e\u0440\u0442\u0438\u0442 \u0432\u0430\u043c \u043d\u0430\u0441\u0442\u0440\u043e\u0435\u043d\u0438\u0435:<\/p>\n<pre><code class=\"cpp\">public NggbIntervalTreeMap&lt;Gene> loadGenesIntervalMap(GeneFile geneFile,                                                       int startIndex,                                                       int endIndex,                                                       Chromosome chromosome) {     ....     List&lt;Future&lt;Boolean>> results;     try {         results = taskExecutorService.getExecutorService().invokeAll(callables);     } catch (InterruptedException | AssertionError e) {         throw new GeneReadingException(geneFile,                                         chromosome,                                         startIndex,                                         endIndex,                                         e);     }      results.stream().map(future -> {         try {             return future != null ? future.get() : null;         } catch (InterruptedException | ExecutionException e) {             log.error(getMessage(MessagesConstants.ERROR_GENE_BATCH_LOAD,                                  geneFile.getId(),                                   chromosome.getId(),                                   e));         }         return null;     });     ....     return genesRangeMap; } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6010\/\">V6010<\/a> The return value of function &#8216;map&#8217; is required to be utilized. GffManager.java(477)<\/p>\n<p>\u041c\u043e\u0436\u0435\u0442 \u043f\u043e\u043a\u0430\u0437\u0430\u0442\u044c\u0441\u044f, \u0431\u0443\u0434\u0442\u043e \u0437\u0434\u0435\u0441\u044c \u0432\u0441\u0451 \u043d\u043e\u0440\u043c\u0430\u043b\u044c\u043d\u043e. \u041f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u0430\u044f <em>results<\/em> \u043e\u0431\u044a\u044f\u0432\u043b\u044f\u0435\u0442\u0441\u044f, \u043d\u0430\u043f\u043e\u043b\u043d\u044f\u0435\u0442\u0441\u044f \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f\u043c\u0438 \u0438 \u043e\u0431\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u0435\u0442\u0441\u044f. \u041f\u043e\u043a\u0430 \u0432\u044b \u043d\u0435 \u0443\u0432\u0438\u0434\u0438\u0442\u0435, \u0447\u0442\u043e \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442 \u0432\u044b\u0437\u043e\u0432\u0430 <em>map()<\/em> \u043d\u0435 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u0435\u0442\u0441\u044f, \u0430 \u0438\u0437 \u043c\u0435\u0442\u043e\u0434\u0430 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442\u0441\u044f \u0441\u043e\u0432\u0441\u0435\u043c \u0434\u0440\u0443\u0433\u043e\u0435 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 (\u0435\u0433\u043e \u043e\u0431\u0440\u0430\u0431\u043e\u0442\u043a\u0443 \u044f \u043e\u043f\u0443\u0441\u0442\u0438\u043b \u0434\u043b\u044f \u0443\u0434\u043e\u0431\u0441\u0442\u0432\u0430 \u0447\u0442\u0435\u043d\u0438\u044f, \u043a\u0430\u044e\u0441\u044c). \u0412\u043b\u0438\u044f\u0435\u0442 \u043b\u0438 \u044d\u0442\u043e \u043d\u0435\u0433\u0430\u0442\u0438\u0432\u043d\u043e \u043d\u0430 \u0440\u0430\u0431\u043e\u0442\u0443 \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u044b \u043b\u0438\u0431\u043e \u043f\u0440\u043e\u0441\u0442\u043e \u043e\u0441\u0442\u0430\u043b\u043e\u0441\u044c \u043f\u043e\u0441\u043b\u0435 \u0440\u0435\u0444\u0430\u043a\u0442\u043e\u0440\u0438\u043d\u0433\u0430 \u2014 \u0433\u0430\u0434\u0430\u0442\u044c \u0432\u0430\u043c (\u043c\u044b \u0436\u0435 \u043f\u0440\u0435\u0434\u0441\u0442\u0430\u0432\u0438\u043b\u0438, \u0447\u0442\u043e \u044d\u0442\u043e \u043d\u0430\u0448 \u043d\u043e\u0432\u044b\u0439 \u043f\u0440\u043e\u0435\u043a\u0442).<\/p>\n<h4>\u042d\u0442\u043e \u043a\u0440\u0438\u0442\u0438\u0447\u043d\u0430\u044f \u043e\u0448\u0438\u0431\u043a\u0430?<\/h4>\n<p>\u0418 \u0432\u043d\u043e\u0432\u044c \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u044f, \u0433\u0434\u0435 \u044f\u0432\u043d\u043e \u0447\u0442\u043e-\u0442\u043e \u0438\u0434\u0451\u0442 \u043d\u0435 \u0442\u0430\u043a, \u0442\u043e\u043b\u044c\u043a\u043e \u0437\u0434\u0435\u0441\u044c \u043e\u043f\u0440\u0430\u0432\u0434\u0430\u043d\u0438\u0435 \u043f\u0440\u0438\u0434\u0443\u043c\u0430\u0442\u044c \u0435\u0449\u0451 \u0441\u043b\u043e\u0436\u043d\u0435\u0435:<\/p>\n<pre><code class=\"cpp\">private static OrganismType determineHeterozygousGenotypeGA4GH(     VariantGA4GH ga4GHEntity, List&lt;Allele> alleles, int[] genotypeArray) {      OrganismType organismType = null;     for (int i = 0; i &lt; genotypeArray.length; i++) {         if (alleles.get(i).isReference()) {             genotypeArray[i] = 0;             organismType = OrganismType.HETEROZYGOUS;         } else {             if (organismType == null) {                 organismType = OrganismType.HETERO_VAR;             }             genotypeArray[i] = ga4GHEntity.getAlternateBases()                                           .indexOf(alleles.get(i)) + 1; \/\/ &lt;=         }     }      return organismType; } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6066\/\">V6066<\/a> The type of object passed as argument is incompatible with the type of collection: String, Allele. VcfGa4ghReader.java(535)<\/p>\n<p>\u041c\u0435\u0442\u043e\u0434* ga4GHEntity.getAlternateBases()* \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u0442 <em>List&lt;String><\/em>, \u0432 \u0442\u043e \u0432\u0440\u0435\u043c\u044f \u043a\u0430\u043a \u043d\u0430\u0439\u0442\u0438 \u043c\u044b \u043f\u044b\u0442\u0430\u0435\u043c\u0441\u044f \u043e\u0431\u044a\u0435\u043a\u0442 \u0442\u0438\u043f\u0430 <em>Allele<\/em>. \u0422\u0430\u043a \u043a\u0430\u043a \u043e\u0431\u044a\u0435\u043a\u0442 \u0434\u0440\u0443\u0433\u043e\u0433\u043e \u0442\u0438\u043f\u0430, \u043e\u0436\u0438\u0434\u0430\u0435\u043c\u043e, \u043d\u0430\u0439\u0434\u0435\u043d \u043d\u0435 \u0431\u0443\u0434\u0435\u0442, \u0432 \u043c\u0430\u0441\u0441\u0438\u0432 <em>genotypeArray<\/em> \u0431\u0443\u0434\u0443\u0442 \u0437\u0430\u043f\u0438\u0441\u0430\u043d\u044b \u043d\u0443\u043b\u0438 (\u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442 \u0441\u043b\u043e\u0436\u0435\u043d\u0438\u044f -1 \u0438 1). \u041f\u0440\u043e\u0439\u0434\u0451\u0442\u0435 \u043c\u0438\u043c\u043e, \u043f\u0440\u0438\u0442\u0432\u043e\u0440\u0438\u0432\u0448\u0438\u0441\u044c, \u0447\u0442\u043e \u043d\u0438\u0447\u0435\u0433\u043e \u043d\u0435 \u0437\u0430\u043c\u0435\u0442\u0438\u043b\u0438, \u0438\u043b\u0438 \u043f\u043e\u043f\u044b\u0442\u0430\u0435\u0442\u0435\u0441\u044c \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u0442\u044c? \u0414\u0430\u0432\u043b\u0435\u043d\u0438\u0435 \u0441\u043e\u0432\u0435\u0441\u0442\u0438 \u0443\u0441\u0438\u043b\u0438\u0432\u0430\u0435\u0442\u0441\u044f, \u0435\u0441\u043b\u0438 \u0443\u0447\u0435\u0441\u0442\u044c, \u0447\u0442\u043e \u044d\u0442\u0430 \u0438 \u043f\u0440\u043e\u0448\u043b\u0430\u044f \u043e\u0448\u0438\u0431\u043a\u0438 \u043b\u043e\u0432\u044f\u0442\u0441\u044f \u0434\u0430\u0436\u0435 \u0441\u0430\u043c\u0438\u043c IntelliJ IDEA.<\/p>\n<h4>\u041d\u0443 \u043a\u0430\u043a \u0442\u0430\u043a-\u0442\u043e?<\/h4>\n<p>\u041f\u0440\u043e\u0434\u043e\u043b\u0436\u0430\u0435\u043c \u0430\u043a\u0438 \u0434\u0435\u0442\u0435\u043a\u0442\u0438\u0432\u044b (\u043d\u0435\u0443\u0434\u0430\u0447\u043d\u043e) \u0432\u044b\u044f\u0441\u043d\u044f\u0442\u044c, \u0447\u0442\u043e \u0436\u0435 \u0445\u043e\u0442\u0435\u043b\u0438 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0438. \u0412\u043e\u0442 \u0434\u0432\u0430 \u0437\u0430\u0431\u0430\u0432\u043d\u044b\u0445 \u043c\u0435\u0441\u0442\u0430. \u041d\u043e\u043c\u0435\u0440 1:<\/p>\n<pre><code class=\"cpp\">private VariantSet variationMetadata(final String path) {     final int index = path.indexOf('-');     VariantSet variantSet;     final String variantSetId;     if (index >= 0) {         variantSetId = path.substring(0, index - 1);     } else {         variantSetId = path;     }     ....     return variantSet; } <\/code><\/pre>\n<p>\u0418 \u043d\u043e\u043c\u0435\u0440 2:<\/p>\n<pre><code class=\"cpp\">private TIntv getIntv(final String s) {     ....     int end = s.indexOf('\\t', beg);     while (end >= 0 || end == -1) {         if (col == mSc) {             ....         } else {             if ((mPreset &amp; TYPE_FLAG) == 0) {                 if (col == mEc) {                     intv.end = Integer.parseInt(end != -1                                                  ? s.substring(beg, end)                                                  : s.substring(beg));                 }             } else if ((mPreset &amp; TYPE_FLAG) == 1) {                 if (col == 6) {                     String cigar = s.substring(beg, end);                     ....                 }             } else if ((mPreset &amp; TYPE_FLAG) == 2) {                 String alt;                 alt = end >= 0 ? s.substring(beg, end)                                 : s.substring(beg);                 ....             }         }         if (end == -1) {             break;         }         beg = end + 1;         end = s.indexOf('\\t', beg);     }     return intv; } <\/code><\/pre>\n<p>\u041e\u0437\u0430\u0433\u043b\u0430\u0432\u0438\u043b \u044d\u0442\u043e\u0442 \u0440\u0430\u0437\u0434\u0435\u043b \u0444\u0440\u0430\u0437\u043e\u0439, \u043a\u043e\u0442\u043e\u0440\u0443\u044e \u044f \u0431\u044b \u0441\u043a\u0430\u0437\u0430\u043b \u043d\u0430 \u043a\u043e\u0434-\u0440\u0435\u0432\u044c\u044e, \u0443\u0432\u0438\u0434\u0435\u0432 \u0442\u0430\u043a\u043e\u0435. \u041e\u0448\u0438\u0431\u043a\u0443, \u043a\u0441\u0442\u0430\u0442\u0438, \u044f \u043d\u0435 \u043f\u0440\u0438\u0432\u0451\u043b \u0441\u0440\u0430\u0437\u0443 \u0441\u043f\u0435\u0446\u0438\u0430\u043b\u044c\u043d\u043e \u2014 \u043f\u043e\u043f\u0440\u043e\u0431\u0443\u0439\u0442\u0435 \u043d\u0430\u0439\u0442\u0438 \u0441\u0430\u043c\u0438 (\u044d\u0442\u043e \u043d\u0435 \u0442\u0440\u0443\u0434\u043d\u043e, \u044f \u0441\u043e\u043a\u0440\u0430\u0442\u0438\u043b \u043a\u043e\u0434 \ud83d\ude42 ). \u041f\u043e\u043a\u0430 \u0436\u0435 \u043e\u0441\u0442\u0430\u0432\u043b\u044e \u0441\u0432\u043e\u0438 \u043c\u044b\u0441\u043b\u0438 \u043d\u0430 \u044d\u0442\u043e\u0442 \u0441\u0447\u0451\u0442.<\/p>\n<p>\u041f\u043e\u0441\u043b\u0435 \u0442\u043e\u0433\u043e \u043a\u0430\u043a \u044f \u043f\u043e\u043a\u043e\u043f\u0430\u043b\u0441\u044f \u0432 \u043f\u0440\u043e\u0435\u043a\u0442\u0435, \u0443 \u043c\u0435\u043d\u044f \u043d\u0435 \u043f\u043e\u044f\u0432\u0438\u043b\u043e\u0441\u044c \u0443\u0432\u0435\u0440\u0435\u043d\u043d\u043e\u0441\u0442\u0438, \u0447\u0442\u043e \u043f\u0440\u0438\u043b\u043e\u0436\u0435\u043d\u0438\u0435 \u043b\u0435\u0433\u043a\u043e \u0443\u0440\u043e\u043d\u0438\u0442\u044c. \u041d\u043e \u0432\u0441\u0451 \u0436\u0435 \u0441\u0430\u043c \u0444\u0430\u043a\u0442 \u0434\u043e\u043f\u0443\u0449\u0435\u043d\u0438\u044f \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f \u043e\u0442\u0440\u0438\u0446\u0430\u0442\u0435\u043b\u044c\u043d\u043e\u0433\u043e \u0438\u043d\u0434\u0435\u043a\u0441\u0430 (\u043d\u0430\u043f\u0440\u044f\u043c\u0443\u044e \u0432\u043e \u0432\u0442\u043e\u0440\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u0438 \u0432\u0441\u043b\u0435\u0434\u0441\u0442\u0432\u0438\u0435 \u0432\u044b\u0447\u0438\u0442\u0430\u043d\u0438\u044f \u2014 \u0432 \u043f\u0435\u0440\u0432\u043e\u043c) \u0434\u043b\u044f \u0438\u0437\u0432\u043b\u0435\u0447\u0435\u043d\u0438\u044f \u043f\u043e\u0434\u0441\u0442\u0440\u043e\u043a\u0438 \u0432\u044b\u0437\u044b\u0432\u0430\u0435\u0442 \u0444\u0440\u0443\u0441\u0442\u0440\u0430\u0446\u0438\u044e. \u041e\u0441\u043e\u0431\u0435\u043d\u043d\u043e \u0432 \u043f\u0435\u0440\u0432\u043e\u043c \u043f\u0440\u0438\u043c\u0435\u0440\u0435, \u0433\u0434\u0435 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 \u0438\u043d\u0434\u0435\u043a\u0441\u0430 \u0441 \u0438\u0437\u0432\u043b\u0435\u0447\u0435\u043d\u0438\u0435\u043c \u043f\u043e\u0434\u0441\u0442\u0440\u043e\u043a\u0438 \u0431\u0443\u043a\u0432\u0430\u043b\u044c\u043d\u043e \u0440\u044f\u0434\u043e\u043c. \u0410 \u0432\u043e \u0432\u0442\u043e\u0440\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u0435\u0441\u0442\u044c \u043e\u0449\u0443\u0449\u0435\u043d\u0438\u0435, \u0447\u0442\u043e \u0431\u044b\u043b \u043f\u0440\u043e\u043f\u0443\u0449\u0435\u043d \u0442\u0435\u0440\u043d\u0430\u0440\u043d\u044b\u0439 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440, \u0438\u0431\u043e \u0432 \u0434\u0432\u0443\u0445 \u0434\u0440\u0443\u0433\u0438\u0445 \u043c\u0435\u0441\u0442\u0430\u0445 \u043e\u043d \u0438\u043c\u0435\u0435\u0442\u0441\u044f.<\/p>\n<p>\u0410 \u0432\u043e\u0442 \u0438 \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u044f \u043d\u0430 \u044d\u0442\u0438 \u0434\u0432\u0430 \u0443\u0447\u0430\u0441\u0442\u043a\u0430 \u043a\u043e\u0434\u0430:<\/p>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6009\/\">V6009<\/a> The &#8216;substring&#8217; function could receive the &#8216;-1&#8217; value while non-negative value is expected. Inspect argument: 2. VcfGa4ghReader.java(212)<\/p>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6009\/\">V6009<\/a> The &#8216;substring&#8217; function could receive the &#8216;-1&#8217; value while non-negative value is expected. Inspect argument: 2. TabixReader.java(430)<\/p>\n<h4>\u042d\u0442\u043e \u0442\u0430\u043a \u0438 \u0437\u0430\u0434\u0443\u043c\u0430\u043d\u043e, \u0438\u043b\u0438 \u0442\u0430\u043a \u0438 \u0437\u0430\u0434\u0443\u043c\u0430\u043d\u043e?<\/h4>\n<p>\u041d\u0443 \u0438\u043b\u0438 \u0432\u043e\u0442 \u0432\u0430\u043c \u043f\u043e\u043d\u0430\u0434\u043e\u0431\u0438\u043b\u043e\u0441\u044c \u043f\u043e\u0434\u043f\u0440\u0430\u0432\u0438\u0442\u044c \u0438\u043d\u0444\u0440\u0430\u0441\u0442\u0440\u0443\u043a\u0442\u0443\u0440\u0443 DAO, \u0437\u0430\u0445\u043e\u0434\u0438\u0442\u0435 \u0432\u044b \u0432 \u043d\u0443\u0436\u043d\u044b\u0439 \u043c\u043e\u0434\u0443\u043b\u044c, \u043a\u0430\u043a \u0432\u0438\u0434\u0438\u0442\u0435 \u044d\u0442\u043e\u0442 \u043c\u0435\u0442\u043e\u0434:<\/p>\n<pre><code class=\"cpp\">private String getGroupByField(final List&lt;VcfFile> files, final String groupBy){     final VcfIndexSortField sortField = VcfIndexSortField.getByName(groupBy);     if (sortField == null) {         ....         if (   infoItem.getType() == VCFHeaderLineType.Integer             || infoItem.getType() == VCFHeaderLineType.Float) {             return infoItem.getName().toLowerCase();         } else {             return infoItem.getName().toLowerCase();         }     } else {         if (   sortField.getType() == SortField.Type.INT             || sortField.getType() == SortField.Type.FLOAT) {             return sortField.getField().fieldName;         } else {             return sortField.getField().fieldName;         }     } } <\/code><\/pre>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6004\/\">V6004<\/a> The &#8216;then&#8217; statement is equivalent to the &#8216;else&#8217; statement. FeatureIndexDao.java(852), FeatureIndexDao.java(854)<\/p>\n<p>\u041d\u0435\u043f\u043e\u043d\u0438\u043c\u0430\u043d\u0438\u0435, \u0433\u0440\u0443\u0441\u0442\u044c, \u0440\u0430\u0437\u043e\u0447\u0430\u0440\u043e\u0432\u0430\u043d\u0438\u0435. \u0421\u0443\u0434\u044f \u043f\u043e \u0431\u043b\u0435\u0439\u043c\u0443, \u0442\u0430\u043a\u0430\u044f \u0441\u0438\u0442\u0443\u0430\u0446\u0438\u044f \u0432\u043e\u0437\u043d\u0438\u043a\u043b\u0430 \u043f\u0440\u0438 \u043f\u0440\u0430\u0432\u043a\u0435 \u043e\u0434\u043d\u043e\u0439 \u0447\u0430\u0441\u0442\u0438 \u043a\u043e\u0434\u0430 \u0440\u0430\u0437\u043d\u044b\u043c\u0438 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0430\u043c\u0438. \u041d\u043e \u0434\u0430\u0436\u0435 \u043a\u043e\u0433\u0434\u0430 \u043c\u044b \u043d\u0430\u0448\u043b\u0438 \u043d\u0430 \u043a\u043e\u0433\u043e \u0441\u0432\u0430\u043b\u0438\u0442\u044c \u0432\u0438\u043d\u0443, \u043e\u0448\u0435\u043b\u043e\u043c\u043b\u0435\u043d\u0438\u0435 \u043d\u0435 \u043f\u0440\u043e\u043f\u0430\u0434\u0430\u0435\u0442: \u0437\u0430\u0447\u0435\u043c \u043a\u043e\u043c\u0443-\u0442\u043e \u043f\u043e\u043d\u0430\u0434\u043e\u0431\u0438\u043b\u043e\u0441\u044c \u0434\u0432\u0430 \u0443\u0441\u043b\u043e\u0432\u0438\u044f \u043f\u043e\u0434\u0440\u044f\u0434 \u0441 \u043e\u0434\u0438\u043d\u0430\u043a\u043e\u0432\u044b\u043c\u0438 \u0432\u0435\u0442\u0432\u044f\u043c\u0438? \u0418\u043b\u0438 \u044d\u0442\u043e \u044f \u0447\u0442\u043e-\u0442\u043e \u043d\u0435 \u043f\u043e\u043d\u044f\u043b?<\/p>\n<figure class=\"full-width\"><\/figure>\n<p>\u0415\u0441\u0442\u044c \u0431\u043e\u043b\u044c\u0448\u0430\u044f \u0432\u0435\u0440\u043e\u044f\u0442\u043d\u043e\u0441\u0442\u044c, \u0447\u0442\u043e \u043d\u0435\u043f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e\u0433\u043e \u043f\u043e\u0432\u0435\u0434\u0435\u043d\u0438\u044f \u0442\u0443\u0442 \u043d\u0435\u0442. \u041d\u043e \u0430 \u0447\u0442\u043e, \u0435\u0441\u043b\u0438 \u0435\u0441\u0442\u044c? \u041e\u0441\u0442\u0430\u0451\u0442\u0441\u044f \u0442\u043e\u043b\u044c\u043a\u043e \u0442\u044b\u043a\u0430\u0442\u044c \u044e\u043d\u0438\u0442-\u0442\u0435\u0441\u0442\u044b&#8230;<\/p>\n<h4>\u041a\u0430\u043a \u0442\u0435\u0441\u0442\u0438\u0440\u043e\u0432\u0430\u0442\u044c \u044e\u043d\u0438\u0442-\u0442\u0435\u0441\u0442\u044b<\/h4>\n<p>\u041a\u0441\u0442\u0430\u0442\u0438, \u043d\u0430\u0441\u0447\u0451\u0442 \u044e\u043d\u0438\u0442-\u0442\u0435\u0441\u0442\u043e\u0432. \u0418 \u043a \u043d\u0438\u043c \u0434\u043e\u0441\u0442\u0430\u0442\u043e\u0447\u043d\u043e \u043b\u0435\u0433\u043a\u043e \u043f\u043e\u0434\u043e\u0440\u0432\u0430\u0442\u044c \u0434\u043e\u0432\u0435\u0440\u0438\u0435. \u041d\u0430\u043f\u0440\u0438\u043c\u0435\u0440, \u0434\u043e\u043f\u0443\u0441\u0442\u0438\u0442\u044c \u043e\u0448\u0438\u0431\u043a\u0443 \u0432 \u0438\u0445 \u0441\u0435\u0442\u0430\u043f\u0435:<\/p>\n<pre><code class=\"cpp\">public void addFeatureIndexedFileRegistration(Long refId,                                               String path,                                               String indexPath,                                               String name,                                               Long fileId,                                               Long fileBioId,                                                BiologicalDataItemFormat format,                                               boolean doIndex,                                               String prettyName) {     String absolutePath = getNormalizeAndAbsolutePath(path);     onRequest().havingMethodEqualTo(HTTP_POST)                .havingPathEqualTo(....)                .havingBodyEqualTo(TestDataProvider                                      .getRegistrationJsonWithPrettyName(                                          refId,                                           absolutePath,                                           indexPath,                                          name,                                           doIndex,                                           prettyName))                .respond()                .withBody(....)                .withStatus(HTTP_STATUS_OK); } <\/code><\/pre>\n<p>\u0427\u0442\u043e\u0431\u044b \u0440\u0430\u0441\u043a\u0443\u0441\u0438\u0442\u044c \u043f\u043e\u0434\u0432\u043e\u0445, \u043d\u0430\u0434\u043e \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0435\u0442\u044c \u043d\u0430 \u0441\u0438\u0433\u043d\u0430\u0442\u0443\u0440\u0443 \u043c\u0435\u0442\u043e\u0434\u0430: TestDataProvider.<em>getRegistrationJsonWithPrettyName(&#8230;.)<\/em>:<\/p>\n<pre><code class=\"cpp\">public static String getRegistrationJsonWithPrettyName(     Long referenceID,     String path,     String name,     String indexPath,     boolean doIndex,     String prettyName) {     .... } <\/code><\/pre>\n<p>\u0417\u0430\u043c\u0435\u0442\u0438\u043b\u0438? \u0415\u0441\u043b\u0438 \u0434\u0430, \u0442\u043e \u043c\u043e\u0436\u0435\u0442\u0435 \u043e\u0442\u0431\u043b\u0430\u0433\u043e\u0434\u0430\u0440\u0438\u0442\u044c \u0437\u0430 \u0442\u043e, \u0447\u0442\u043e \u044f \u043d\u0435\u043c\u043d\u043e\u0433\u043e \u043e\u0442\u0444\u043e\u0440\u043c\u0430\u0442\u0438\u0440\u043e\u0432\u0430\u043b \u043a\u043e\u0434. \u0412\u043e\u0442 \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435:<\/p>\n<p><a href=\"https:\/\/pvs-studio.ru\/ru\/docs\/warnings\/v6029\/\">V6029<\/a> Possible incorrect order of arguments passed to method: &#8216;indexPath&#8217;, &#8216;name&#8217;. TestHttpServer.java(139), TestHttpServer.java(140)<\/p>\n<p>\u0421\u0443\u0434\u044f \u043f\u043e \u0432\u0441\u0435\u043c\u0443, \u043f\u0440\u0438 \u0432\u044b\u0437\u043e\u0432\u0435<\/p>\n<\/div>\n<\/div>\n<\/div>\n<\/div>\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-362378","post","type-post","status-publish","format-standard","hentry"],"_links":{"self":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/posts\/362378","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=362378"}],"version-history":[{"count":0,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/posts\/362378\/revisions"}],"wp:attachment":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=362378"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=362378"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=362378"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}