पीवीएस-स्टूडियो का उपयोग कर सीयूबीए प्लेटफॉर्म कोड विश्लेषण

जावा प्रोग्रामर के लिए, उच्च-गुणवत्ता कोड लिखने में आपकी मदद करने के लिए उपयोगी उपकरण हैं, उदाहरण के लिए, शक्तिशाली इंटेलीजे आईडीईए विकास वातावरण, फ्री स्पॉटबग्स, पीएमडी विश्लेषक और अन्य। यह सब पहले से ही सीयूबीए प्लेटफॉर्म परियोजना के विकास में उपयोग किया जाता है, और पाए गए कोड दोषों की इस समीक्षा में मैं आपको बताऊंगा कि आप पीवीएस-स्टूडियो स्टैटिक कोड विश्लेषक का उपयोग करके परियोजना की गुणवत्ता में और कैसे सुधार कर सकते हैं।

परियोजना और विश्लेषक के बारे में

CUBA प्लेटफार्म एक पूर्ण वेब इंटरफ़ेस के साथ एंटरप्राइज़ एप्लिकेशन को जल्दी से बनाने के लिए एक उच्च-स्तरीय जावा फ्रेमवर्क है। मंच विषम प्रौद्योगिकियों से डेवलपर को अमूर्त करता है ताकि आप व्यवसाय की समस्याओं को हल करने पर ध्यान केंद्रित कर सकें, साथ ही, उनके साथ सीधे काम करना असंभव बना सकें। स्रोत कोड GitHub पर रिपॉजिटरी से लिया गया है ।

पीवीएस-स्टूडियो सी, सी ++, सी # और जावा में लिखे गए कार्यक्रमों के स्रोत कोड में त्रुटियों और संभावित कमजोरियों का पता लगाने के लिए एक उपकरण है। यह विंडोज, लिनक्स और मैकओएस पर 64-बिट सिस्टम पर काम करता है। जावा प्रोग्रामर की सुविधा के लिए, हमने मावेन, ग्रैडल और इंटेलीज आईडीईए के लिए प्लगइन्स विकसित किए हैं। सीयूबीए प्लेटफॉर्म परियोजना को ग्रैडल प्लगइन का उपयोग करके आसानी से सत्यापित किया गया था।

शर्तों में त्रुटियां

चेतावनी १

V6007 अभिव्यक्ति ' StringUtils.isNotEmpty ("handleTabKey") हमेशा सच होता है। SourceCodeEditorLoader.java (60)

@Override public void loadComponent() { .... String handleTabKey = element.attributeValue("handleTabKey"); if (StringUtils.isNotEmpty("handleTabKey")) { resultComponent.setHandleTabKey(Boolean.parseBoolean(handleTabKey)); } .... } 

किसी तत्व से विशेषता मान प्राप्त करने के बाद, इस मान का सत्यापन नहीं किया जाता है। इसके बजाय, एक निरंतर स्ट्रिंग isNotEmpty फ़ंक्शन के लिए पास की जाती है, लेकिन हमें चर संभाल करना होगा ।

AbstractTableLoader.java फ़ाइल में एक और समान त्रुटि है:

• V6007 अभिव्यक्ति 'StringUtils.isNotEmpty ("संपादन योग्य") हमेशा सच होता है। AbstractTableLoader.java (596)

चेतावनी २

V6007 अभिव्यक्ति 'पिछले MenuItemFlatIndex > = 0' हमेशा सच होता है। क्यूबासाइडमेन्यूविजेट.जावा (328)

 protected MenuItemWidget findNextMenuItem(MenuItemWidget currentItem) { List<MenuTreeNode> menuTree = buildVisibleTree(this); List<MenuItemWidget> menuItemWidgets = menuTreeToList(menuTree); int menuItemFlatIndex = menuItemWidgets.indexOf(currentItem); int previousMenuItemFlatIndex = menuItemFlatIndex + 1; if (previousMenuItemFlatIndex >= 0) { return menuItemWidgets.get(previousMenuItemFlatIndex); } return null; } 

यदि सूची में आइटम नहीं मिला है, तो indexOf फ़ंक्शन -1 वापस आ सकता है। फिर एक को सूचकांक में जोड़ा जाता है, इस प्रकार उस स्थिति को छिपाया जाता है जब वांछित तत्व गायब होता है। एक अन्य संभावित समस्या यह तथ्य हो सकती है कि चर पिछले MenuItemFlatIndex हमेशा शून्य से अधिक या बराबर होगा। यदि, उदाहरण के लिए, menuItemWidgets सूची रिक्त है, तो यह सरणी की सीमा से परे जाना संभव हो जाता है।

चेतावनी ३

V6009 'डिलीट' फ़ंक्शन '-1' मान प्राप्त कर सकता है जबकि गैर-नकारात्मक मान अपेक्षित है। निरीक्षण तर्क: 1. AbstractCollectionDatasource.java (556)

 protected DataLoadContextQuery createDataQuery(....) { .... StringBuilder orderBy = new StringBuilder(); .... if (orderBy.length() > 0) { orderBy.delete(orderBy.length() - 2, orderBy.length()); orderBy.insert(0, " order by "); } .... } 

वर्ण बफ़र आदेश में, अंतिम 2 वर्ण हटा दिए जाते हैं यदि उनकी कुल संख्या शून्य से अधिक है, अर्थात। एक स्ट्रिंग में एक वर्ण या अधिक होता है। लेकिन पात्रों को हटाने के लिए शुरुआती स्थिति 2 वर्णों की भरपाई के साथ निर्धारित की गई थी। इस प्रकार, यदि ऑर्डरबाय में अचानक 1 वर्ण होता है, तो हटाने का प्रयास एक StringIndexOutOfBoundsException को फेंक देगा ।

चेतावनी ४

V6013 ऑब्जेक्ट 'मास्टरकॉलक्शन' और 'एंटिटीज़' की तुलना संदर्भ से की जाती है। संभवतः एक समानता तुलना का इरादा था। कलेक्शनप्रॉपर्टी कॉन्टेनरइम्प्ल.जावा (81)

 @Override public void setItems(@Nullable Collection<E> entities) { super.setItems(entities); Entity masterItem = master.getItemOrNull(); if (masterItem != null) { MetaProperty masterProperty = getMasterProperty(); Collection masterCollection = masterItem.getValue(masterProperty.getName()); if (masterCollection != entities) { updateMasterCollection(masterProperty, masterCollection, entities); } } } 

UpdateMasterCollection फ़ंक्शन में , संस्थाओं के मूल्यों को MasterCollection में कॉपी किया जाता है । इससे पहले, संग्रहों की तुलना संदर्भ से की जाती थी, लेकिन शायद उन्हें मूल्य से तुलना करने की योजना बनाई गई थी।

चेतावनी ५

V6013 ऑब्जेक्ट्स की 'वैल्यू' और 'ओल्डवैल्यू' की तुलना संदर्भ से की जाती है। संभवतः एक समानता तुलना का इरादा था। WebOptionsList.java (278)

 protected boolean isCollectionValuesChanged(Collection<I> value, Collection<I> oldValue) { return value != oldValue; } 

यह उदाहरण पिछले एक के समान है। यहाँ संग्रह की तुलना के लिए है। शायद संदर्भ से तुलना भी वह तरीका नहीं है जिसकी अपेक्षा की गई थी।

अतिरिक्त स्थिति

चेतावनी १

V6007 एक्सप्रेशन 'mask.charAt (i + ऑफसेट)! = प्लेसहोल्डर' हमेशा सच होता है। DatePickerDocument.java (238)

 private String calculateFormattedString(int offset, String text) .... { .... if ((mask.charAt(i + offset) == placeHolder)) { // <= .... } else if ((mask.charAt(i + offset) != placeHolder) && // <= (Character.isDigit(text.charAt(i)))) { .... } .... } 

दूसरी तुलना में, पहले के विपरीत एक अभिव्यक्ति की जाँच की जाती है। कोड को सरल बनाने के लिए दूसरी जांच को हटाया जा सकता है।

V6007 अभिव्यक्ति 'कनेक्टर == नल' हमेशा गलत होता है। HTML5Support.java (169)

 private boolean validate(NativeEvent event) { .... while (connector == null) { widget = widget.getParent(); connector = Util.findConnectorFor(widget); } if (this.connector == connector) { return true; } else if (connector == null) { // <= return false; } else if (connector.getWidget() instanceof VDDHasDropHandler) { return false; } return true; } 

लूप पूरा होने के बाद, कनेक्टर चर का मूल्य शून्य नहीं होगा, इसलिए, अनावश्यक जाँच को हटाया जा सकता है।

एक और संदेहास्पद जगह जिसे डेवलपर्स को ध्यान देना चाहिए:

• V6007 अभिव्यक्ति 'StringUtils.isBlank (strValue)' हमेशा सच होता है। परम.जावा (818)

परीक्षणों में अनुपलब्ध कोड

V6019 अनुपलब्ध कोड का पता चला। यह संभव है कि कोई त्रुटि मौजूद हो। TransactionTest.java (283)

 private void throwException() { throw new RuntimeException(TEST_EXCEPTION_MSG); } @Test public void testSuspendRollback() { Transaction tx = cont.persistence().createTransaction(); try { .... Transaction tx1 = cont.persistence().createTransaction(); try { EntityManager em1 = cont.persistence().getEntityManager(); assertTrue(em != em1); Server server1 = em1.find(Server.class, server.getId()); assertNull(server1); throwException(); // <= tx1.commit(); // <= } catch (Exception e) { // } finally { tx1.end(); } tx.commit(); } finally { tx.end(); } } 

थ्रो एक्ससेप्शन फ़ंक्शन एक अपवाद को फेंकता है जो कॉल को tx1.commit फ़ंक्शन पर रोकता है । शायद इन लाइनों को आपस में जोड़ा जाए।

अन्य परीक्षणों में कुछ और समान स्थान:

• V6019 अनुपलब्ध कोड का पता चला। यह संभव है कि कोई त्रुटि मौजूद हो। TransactionTest.java (218)
• V6019 अनुपलब्ध कोड का पता चला। यह संभव है कि कोई त्रुटि मौजूद हो। TransTestest.java (163)
• V6019 अनुपलब्ध कोड का पता चला। यह संभव है कि कोई त्रुटि मौजूद हो। TransactionTest.java (203)
• V6019 अनुपलब्ध कोड का पता चला। यह संभव है कि कोई त्रुटि मौजूद हो। TransactionTest.java (137)
• V6019 अनुपलब्ध कोड का पता चला। यह संभव है कि कोई त्रुटि मौजूद हो। UpdateDetachedTest.java (153)
• V6019 अनुपलब्ध कोड का पता चला। यह संभव है कि कोई त्रुटि मौजूद हो। EclipseLinkDetachedTest.java (132)
• V6019 अनुपलब्ध कोड का पता चला। यह संभव है कि कोई त्रुटि मौजूद हो। दृढ़ता Tj.java (223)

संदिग्ध कार्य तर्क

चेतावनी १

V6023 पैरामीटर 'नमक' हमेशा इस्तेमाल होने से पहले विधि शरीर में फिर से लिखा जाता है। BCryptEnc एन्क्रिप्शनModule.java (47)

 @Override public String getHash(String content, String salt) { salt = BCrypt.gensalt(); return BCrypt.hashpw(content, salt); } 

क्रिप्टोग्राफी में, नमक डेटा का एक स्ट्रिंग है जो पासवर्ड के साथ हैश फ़ंक्शन में जाता है। इसका उपयोग मुख्य रूप से शब्दकोश खोजों और इंद्रधनुष तालिकाओं का उपयोग करने वाले हमलों के साथ-साथ समान पासवर्ड को छिपाने के लिए किया जाता है। और पढ़ें: नमक (क्रिप्टोग्राफी) ।

इस फंक्शन में, फंक्शन में प्रवेश करने पर लाइन को तुरंत रगड़ा जाता है। शायद पारित मूल्य की अनदेखी एक संभावित भेद्यता है।

चेतावनी २

माना समारोह के लिए, विश्लेषक एक बार में दो चेतावनी देता है:

• V6023 पैरामीटर 'ऑफसेटविदथ' हमेशा उपयोग किए जाने से पहले विधि निकाय में फिर से लिखा जाता है। क्यूबासुग्मेंटेशनफिल्डवेट.जावा (433)
• V6023 पैरामीटर 'ऑफसेटहाइट' हमेशा उपयोग किए जाने से पहले विधि निकाय में फिर से लिखा जाता है। क्यूबासुग्मेंटेशनफिल्डवेट.जावा (433)

 @Override public void setPosition(int offsetWidth, int offsetHeight) { offsetHeight = getOffsetHeight(); .... if (offsetHeight + getPopupTop() > ....)) { .... } .... offsetWidth = containerFirstChild.getOffsetWidth(); if (offsetWidth + getPopupLeft() > ....)) { .... } else { left = getPopupLeft(); } setPopupPosition(left, top); } 

मनोरंजक कोड। फ़ंक्शन केवल दो चर लेता है: ऑफ़सेटविंड और ऑफसेटहाइट , दोनों उपयोग से पहले ओवरराइट किए गए हैं।

चेतावनी ३

V6022 पैरामीटर 'शॉर्टकट' का उपयोग कंस्ट्रक्टर बॉडी के अंदर नहीं किया जाता है। DeclarativeTrackingAction.java (47)

 public DeclarativeTrackingAction(String id, String caption, String description, String icon, String enable, String visible, String methodName, @Nullable String shortcut, ActionsHolder holder) { super(id); this.caption = caption; this.description = description; this.icon = icon; setEnabled(enable == null || Boolean.parseBoolean(enable)); setVisible(visible == null || Boolean.parseBoolean(visible)); this.methodName = methodName; checkActionsHolder(holder); } 

शॉर्टकट पैरामीटर का मान फ़ंक्शन में उपयोग नहीं किया जाता है। फ़ंक्शन इंटरफ़ेस पुराना हो सकता है या यह चेतावनी कोई त्रुटि नहीं है।

कुछ और समान स्थान:

• V6022 पैरामीटर 'टाइप' का उपयोग कंस्ट्रक्टर बॉडी के अंदर नहीं किया जाता है। QueryNode.java (36)
• V6022 पैरामीटर 'text2' का उपयोग कंस्ट्रक्टर बॉडी के अंदर नहीं किया जाता है। MarkerAddition.java (22)
• V6022 पैरामीटर 'चयन' का उपयोग कंस्ट्रक्टर बॉडी के अंदर नहीं किया जाता है। AceEditor.java (114)
• V6022 पैरामीटर 'विकल्प' का उपयोग कंस्ट्रक्टर बॉडी के अंदर नहीं किया जाता है। EntitySerialization.java (379)

एक ही कोड के साथ विभिन्न कार्य

चेतावनी १

V6032 यह विचित्र है कि विधि 'firstItemId' का शरीर किसी अन्य विधि 'lastItemId' के शरीर के समतुल्य है। कंटेनरटेबल इट्स.जावा (213), कंटेनरटेबल इट्स.जावा (219)

 @Override public Object firstItemId() { List<E> items = container.getItems(); return items.isEmpty() ? null : items.get(0).getId(); } @Override public Object lastItemId() { List<E> items = container.getItems(); return items.isEmpty() ? null : items.get(0).getId(); } 

फ़र्स्ट इटइमआईड और लास्ट इटिमिड का कार्यान्वयन समान है। सबसे अधिक संभावना है, उत्तरार्द्ध में एक तत्व प्राप्त करना आवश्यक था, जिसमें सूचकांक 0 के साथ नहीं था, लेकिन अंतिम तत्व के सूचकांक की गणना करने के लिए।

चेतावनी २

V6032 यह अजीब है कि विधि का शरीर पूरी तरह से किसी अन्य विधि के शरीर के बराबर है। SearchComboBoxPainter.java (495), SearchComboBoxPainter.java (501)

 private void paintBackgroundDisabledAndEditable(Graphics2D g) { rect = decodeRect1(); g.setPaint(color53); g.fill(rect); } private void paintBackgroundEnabledAndEditable(Graphics2D g) { rect = decodeRect1(); g.setPaint(color53); g.fill(rect); } 

संदिग्ध समान कार्यान्वयन के साथ दो और कार्य। मैं यह सुझाव देने के लिए उद्यम करूंगा कि उनमें से एक में color53 की तुलना में एक अलग रंग का उपयोग करना आवश्यक था।

अशक्त संदर्भ

चेतावनी १

V6060 नल के खिलाफ सत्यापित होने से पहले 'descriptionPopup' संदर्भ का उपयोग किया गया था। SuggestPopup.java (252), SuggestPopup.java (251)

 protected void updateDescriptionPopupPosition() { int x = getAbsoluteLeft() + WIDTH; int y = getAbsoluteTop(); descriptionPopup.setPopupPosition(x, y); if (descriptionPopup!=null) { descriptionPopup.setPopupPosition(x, y); } } 

केवल दो पंक्तियों में, लेखक एक बहुत ही संदिग्ध कोड लिखने में कामयाब रहा। सबसे पहले, setPopupPosition विधि का वर्णन Popup ऑब्जेक्ट पर किया जाता है, और फिर ऑब्जेक्ट की तुलना नल से की जाती है । सबसे अधिक संभावना है, सेटपॉपपपशन फ़ंक्शन का पहला कॉल निरर्थक और खतरनाक है। यह असफल रीफैक्टरिंग के परिणामों की तरह दिखता है।

चेतावनी २

V6060 नल के खिलाफ सत्यापित होने से पहले 'tableModel' संदर्भ का उपयोग किया गया था। DesktopAbstractTable.java (1580), DesktopAbstractTable.java (1564)

 protected Column addRuntimeGeneratedColumn(String columnId) { // store old cell editors / renderers TableCellEditor[] cellEditors = new TableCellEditor[tableModel.getColumnCount() + 1]; // <= TableCellRenderer[] cellRenderers = new TableCellRenderer[tableModel.getColumnCount() + 1]; // <= for (int i = 0; i < tableModel.getColumnCount(); i++) { // <= Column tableModelColumn = tableModel.getColumn(i); if (tableModel.isGeneratedColumn(tableModelColumn)) { // <= TableColumn tableColumn = getColumn(tableModelColumn); cellEditors[i] = tableColumn.getCellEditor(); cellRenderers[i] = tableColumn.getCellRenderer(); } } Column col = new Column(columnId, columnId); col.setEditable(false); columns.put(col.getId(), col); if (tableModel != null) { // <= tableModel.addColumn(col); } .... } 

इस फ़ंक्शन में एक समान स्थिति मौजूद है। TableModel ऑब्जेक्ट में कई कॉल के बाद, यह देखने के लिए एक चेक बनाया जाता है कि यह शून्य है या नहीं।

एक और उदाहरण:

• V6060 नल के खिलाफ सत्यापित होने से पहले 'tableModel' संदर्भ का उपयोग किया गया था। DesktopAbstractTable.java (596), DesktopAbstractTable.java (579)

शायद एक तार्किक गलती

V6026 यह मान पहले से ही 'SortAscending' चर को सौंपा गया है। क्यूबास्क्रोलटेबलवेगेट.जावा (488)

 @Override protected void sortColumn() { .... if (sortAscending) { if (sortClickCounter < 2) { // special case for initial revert sorting instead of reset sort order if (sortClickCounter == 0) { client.updateVariable(paintableId, "sortascending", false, false); } else { reloadDataFromServer = false; sortClickCounter = 0; sortColumn = null; sortAscending = true; // <= client.updateVariable(paintableId, "resetsortorder", "", true); } } else { client.updateVariable(paintableId, "sortascending", false, false); } } else { if (sortClickCounter < 2) { // special case for initial revert sorting instead of reset sort order if (sortClickCounter == 0) { client.updateVariable(paintableId, "sortascending", true, false); } else { reloadDataFromServer = false; sortClickCounter = 0; sortColumn = null; sortAscending = true; client.updateVariable(paintableId, "resetsortorder", "", true); } } else { reloadDataFromServer = false; sortClickCounter = 0; sortColumn = null; sortAscending = true; client.updateVariable(paintableId, "resetsortorder", "", true); } } .... } 

पहली स्थिति में, सॉर्ट करने योग्य चर पहले से ही सत्य है , लेकिन इसे अभी भी समान मान असाइन किया गया है। शायद यह एक गलती है और झूठे को असाइन करना चाहता था।

एक अन्य फ़ाइल से एक समान उदाहरण:

• V6026 यह मान पहले से ही 'SortAscending' चर को सौंपा गया है। क्यूबाट्रीटेबलवेट.जावा (444)

अजीब समारोह रिटर्न मान

चेतावनी १

V6037 लूप के भीतर बिना शर्त 'वापसी'। QueryCacheManager.java (128)

 public <T> T getSingleResultFromCache(QueryKey queryKey, List<View> views) { .... for (Object id : queryResult.getResult()) { return (T) em.find(metaClass.getJavaClass(), id, views.toArray(....)); } .... } 

विश्लेषक ने लूप के लिए पहली पुनरावृत्ति पर रिटर्न स्टेटमेंट के लिए बिना शर्त कॉल का पता लगाया। शायद यह एक गलती है, या आपको यदि कथन का उपयोग करने के लिए कोड को फिर से लिखना होगा।

चेतावनी २

V6014 यह अजीब है कि यह विधि हमेशा एक और एक ही मान लौटाती है। DefaultExceptionHandler.java (40)

 @Override public boolean handle(ErrorEvent event, App app) { Throwable t = event.getThrowable(); if (t instanceof SocketException || ExceptionUtils.getRootCause(t) instanceof SocketException) { return true; } if (ExceptionUtils.getThrowableList(t).stream() .anyMatch(o -> o.getClass().getName().equals("...."))) { return true; } if (StringUtils.contains(ExceptionUtils.getMessage(t), "....")) { return true; } AppUI ui = AppUI.getCurrent(); if (ui == null) { return true; } if (t != null) { if (app.getConnection().getSession() != null) { showDialog(app, t); } else { showNotification(app, t); } } return true; } 

सभी मामलों में यह फ़ंक्शन सही है । लेकिन यहाँ बहुत ही अंतिम पंक्ति में झूठी वापसी की भीख माँगता हूँ। शायद कोई गलती हो।

समान कोड के साथ संदिग्ध कार्यों की पूरी सूची:

• V6014 यह अजीब है कि यह विधि हमेशा एक और एक ही मान लौटाती है। ErrorNodesFinder.java (31)
• V6014 यह अजीब है कि यह विधि हमेशा एक और एक ही मान लौटाती है। FileDownloadController.java (69)
• V6014 यह अजीब है कि यह विधि हमेशा एक और एक ही मान लौटाती है। IdVarSelector.java (73)
• V6014 यह अजीब है कि यह विधि हमेशा एक और एक ही मान लौटाती है। IdVarSelector.java (48)
• V6014 यह अजीब है कि यह विधि हमेशा एक और एक ही मान लौटाती है। IdVarSelector.java (67)
• V6014 यह अजीब है कि यह विधि हमेशा एक और एक ही मान लौटाती है। IdVarSelector.java (46)
• V6014 यह अजीब है कि यह विधि हमेशा एक और एक ही मान लौटाती है। JoinVariableNode.java (57)

चेतावनी ३

V6007 अभिव्यक्ति 'जरूरतरैलोद' हमेशा झूठी होती है। WebAbstractTable.java (2702)

 protected boolean handleSpecificVariables(Map<String, Object> variables) { boolean needReload = false; if (isUsePresentations() && presentations != null) { Presentations p = getPresentations(); if (p.getCurrent() != null && p.isAutoSave(p.getCurrent()) && needUpdatePresentation(variables)) { Element e = p.getSettings(p.getCurrent()); saveSettings(e); p.setSettings(p.getCurrent(), e); } } return needReload; } 

फ़ंक्शन की आवश्यकता है रिटर्नरेलैड चर, जिसका मान हमेशा गलत होता है । सबसे अधिक संभावना है, एक स्थिति में वे चर के मूल्य को बदलने के लिए कोड जोड़ना भूल गए।

चेतावनी ४

V6062 'इनफोकस' पद्धति के अंदर संभावित अनंत पुनरावृत्ति। GwtAceEditor.java (189), GwtAceEditor.java (190)

 public final native void focus() /*-{ this.focus(); }-*/; public final boolean isFocused() { return this.isFocused(); } 

विश्लेषक ने एक फ़ंक्शन का पता लगाया जिसे पुनरावृत्ति को रोकने के लिए बिना किसी शर्त के पुनरावर्ती कहा जाता है। इस फ़ाइल में कई कार्य हैं जो मूल कीवर्ड के साथ टैग किए गए हैं और इसमें टिप्पणी कोड है। सबसे अधिक संभावना है, फ़ाइल वर्तमान में अधिलेखित की जा रही है और जल्द ही डेवलपर्स isFocused फ़ंक्शन पर ध्यान देंगे।

विविध चेतावनियाँ

चेतावनी १

V6002 स्विच स्टेटमेंट 'ऑपरेशन' एनम के सभी मूल्यों को शामिल नहीं करता है: ADD। DesktopAbstractTable.java (665)

 /** * Operation which caused the datasource change. */ enum Operation { REFRESH, CLEAR, ADD, REMOVE, UPDATE } @Override public void setDatasource(final CollectionDatasource datasource) { .... collectionChangeListener = e -> { switch (e.getOperation()) { case CLEAR: case REFRESH: fieldDatasources.clear(); break; case UPDATE: case REMOVE: for (Object entity : e.getItems()) { fieldDatasources.remove(entity); } break; } }; .... } 

स्विच विवरण ADD के मान को संबोधित नहीं करता है। यह एकमात्र अप्रमाणित है, इसलिए यह जाँचने योग्य है कि यह कोई गलती है या नहीं।

चेतावनी २

V6021 चर 'स्रोत' का उपयोग नहीं किया जाता है। डिफाल्ट हॉरिज़ॉन्टल लाईटड्रॉपहैंडलर.जावा (177)

 @Override protected void handleHTML5Drop(DragAndDropEvent event) { LayoutBoundTransferable transferable = (LayoutBoundTransferable) event .getTransferable(); HorizontalLayoutTargetDetails details = (HorizontalLayoutTargetDetails) event .getTargetDetails(); AbstractOrderedLayout layout = (AbstractOrderedLayout) details .getTarget(); Component source = event.getTransferable().getSourceComponent(); // <= int idx = (details).getOverIndex(); HorizontalDropLocation loc = (details).getDropLocation(); if (loc == HorizontalDropLocation.CENTER || loc == HorizontalDropLocation.RIGHT) { idx++; } Component comp = resolveComponentFromHTML5Drop(event); if (idx >= 0) { layout.addComponent(comp, idx); } else { layout.addComponent(comp); } if (dropAlignment != null) { layout.setComponentAlignment(comp, dropAlignment); } } 

स्रोत चर घोषित किया गया है और कोड में उपयोग नहीं किया गया है। शायद, उसी प्रकार के एक अन्य COMP चर की तरह, वे लेआउट में स्रोत जोड़ना भूल गए।

अप्रयुक्त चर के साथ अधिक कार्य:

• V6021 चर 'स्रोत' का उपयोग नहीं किया जाता है। DefaultHor क्षैतिजलूटआउटड्रॉपलर .java (175)
• V6021 मान को 'r' वेरिएबल में असाइन किया गया है, लेकिन इसका उपयोग नहीं किया गया है। ExcelExporter.java (262)
• V6021 चर 'ओवर' का उपयोग नहीं किया जाता है। DefaultCssLoutoutDropHandler.java (49)
• V6021 चर 'हस्तांतरणीय' का उपयोग नहीं किया जाता है। डिफॉल्टहोरलॉयआउटड्रॉपहैंडलर.जावा (171)
• V6021 चर 'हस्तांतरणीय' का उपयोग नहीं किया जाता है। DefaultHor क्षैतिजलूटआउटड्रॉपलर .java (169)
• V6021 परिवर्तनीय 'बीनलोकेटर' का उपयोग नहीं किया जाता है। ScreenEventMixin.java (28)

चेतावनी ३

V6054 वर्गों की उनके नाम से तुलना नहीं की जानी चाहिए। MessageTools.java (283)

 public boolean hasPropertyCaption(MetaProperty property) { Class<?> declaringClass = property.getDeclaringClass(); if (declaringClass == null) return false; String caption = getPropertyCaption(property); int i = caption.indexOf('.'); if (i > 0 && declaringClass.getSimpleName().equals(caption.substring(0, i))) return false; else return true; } 

विश्लेषक ने एक स्थिति का पता लगाया जब वर्गों की तुलना नाम से की जाती है। इस तरह की तुलना गलत है, क्योंकि, विनिर्देश के अनुसार, जेवीएम कक्षाओं का पैकेज के अंदर एक अनूठा नाम है। यह गलत कोड की गलत तुलना और निष्पादन का कारण बन सकता है जिसे योजना बनाई गई थी।

CUBA प्लेटफ़ॉर्म डेवलपर फीडबैक

बेशक, किसी भी बड़ी परियोजना में कीड़े हैं। यही कारण है कि हम अपने प्रोजेक्ट की जांच के लिए पीवीएस-स्टूडियो टीम के प्रस्ताव पर सहर्ष सहमत हुए। CUBA रिपॉजिटरी में Apache 2 लाइसेंस के तहत कुछ तृतीय-पक्ष OSS पुस्तकालयों के कांटे शामिल हैं, और ऐसा लगता है कि हमें इस कोड पर अधिक ध्यान देने की आवश्यकता है, विश्लेषक को इन स्रोतों में काफी कुछ समस्याएं मिलीं। अब हम स्पॉटबग्स को मुख्य विश्लेषक के रूप में उपयोग करते हैं, और यह पीवीएस-स्टूडियो द्वारा पाई गई कुछ महत्वपूर्ण समस्याओं का पता नहीं लगाता है। यह समय है कि आप खुद जाकर अतिरिक्त जांच लिखें। किए गए काम के लिए पीवीएस-स्टूडियो टीम को बहुत धन्यवाद।

डेवलपर्स ने यह भी नोट किया कि V6013 और V6054 की चेतावनी गलत है। कोड को जानबूझकर लिखा गया था। स्टेटिक एनालाइज़र को संदिग्ध कोड अंशों का पता लगाने के लिए डिज़ाइन किया गया है और एक त्रुटि खोजने की संभावना सभी निरीक्षणों के लिए अलग है। फिर भी, स्रोत कोड फ़ाइलों को संशोधित किए बिना विश्लेषक चेतावनियों के बड़े पैमाने पर दमन के लिए सुविधाजनक तंत्र का उपयोग करके ऐसी चेतावनियों के साथ काम करना आसान है।

एक और पीवीएस-स्टूडियो टीम वाक्यांश को नजरअंदाज नहीं कर सकती है "यह समय है कि वह खुद जाकर अतिरिक्त चेक लिखे" और इस तस्वीर को न छोड़ें :)

निष्कर्ष

पीवीएस-स्टूडियो कोड गुणवत्ता उपकरणों में सुधार करने के लिए आपकी मौजूदा परियोजना के लिए एक बढ़िया अतिरिक्त होगा। खासकर अगर दर्जनों, सैकड़ों और हजारों कर्मचारी हैं। पीवीएस-स्टूडियो न केवल त्रुटियों को खोजने के लिए, बल्कि उन्हें ठीक करने के लिए भी बनाया गया है। इसके अलावा, हम स्वचालित कोड संपादन के बारे में बात नहीं कर रहे हैं, लेकिन विश्वसनीय कोड गुणवत्ता नियंत्रण के बारे में। एक बड़ी कंपनी में, ऐसी स्थिति की कल्पना करना असंभव है जहां बिल्कुल सभी डेवलपर्स स्वतंत्र रूप से विभिन्न उपकरणों के साथ अपने कोड को सत्यापित करेंगे, इसलिए पीवीएस-स्टूडियो जैसे उपकरण ऐसी कंपनियों के लिए अधिक उपयुक्त हैं, जहां कोड का गुणवत्ता नियंत्रण केवल एक सामान्य प्रोग्रामर के लिए नहीं, बल्कि विकास के सभी चरणों में प्रदान किया जाता है।

यदि आप इस लेख को अंग्रेजी बोलने वाले दर्शकों के साथ साझा करना चाहते हैं, तो कृपया अनुवाद के लिंक का उपयोग करें: Svyatoslav Razmyslov। पीवीएस-स्टूडियो के साथ सीयूबीए प्लेटफार्म के कोड का विश्लेषण