cppdepend
report summary
application nameCoco2DX (v1.0.0.0)report build date07/10/2015 12:34:08analysis duration21:25cppdepend
version
6.0.0.8890baseline for comparison
Not Defined. To define a Baseline for Comparison, please read this online documentation.
Get started.Quick tips.Back to CppDepend.
The present HTML report is a summary of data gathered by the analysis.
It is recommended to use the CppDepend interactive UI capabilities
to make the most of CppDepend by mastering all aspects of your code.
It is recommended to use the CppDepend interactive UI capabilities
to make the most of CppDepend by mastering all aspects of your code.
Diagrams
Application Metrics
Note: Further Application Statistics are available.
|
Rules summary
31520This section lists all Rules violated, and Rules or Queries with Error- Number of Rules or Queries with Error (syntax error, exception thrown, time-out): 0
- Number of Rules violated: 55
Summary of Rules violated
Rules can be checked live atdevelopment-time, from within Visual
Studio. Online documentation.
CppDepend rules report too many flawson existing code base? Use the option
Recent Violations Only!
Some Critical Rules are violated. Critical Rulescan be used to break the build process if
violated. Online documentation.
Application Statistics
| Stat | # Occurences | Avg | StdDev | Max |
|---|---|---|---|---|
| Public properties on classes | 2,411 Classes | 0.18 | 5.83 | 215 public properties on __Globals |
| Public methods on classes | 2,411 classes | 12.62 | 104.47 | 5096 public methods on __Globals |
| Arguments on public methods on classes | 30,425 methods | 0.85 | 0.93 | 16 arguments on cocos2d.Mat4.Mat4(float,float,float,float,float,float,float,float,float,float,float,float,float,float,float,float) |
| Cyclomatic complexity on non abstract Methods | 35,027 Methods | 0.61 | 2.17 | CC = 130 for RootTests.RootTests() |
Projects Metrics
If you wish to define thresholds on Projects' Code Metrics, consider writing some Rules.
Clicking column header arrows sorts values.
Clicking column header title text redirect to the online Code Metric definition.
Clicking column header arrows sorts values.
Clicking column header title text redirect to the online Code Metric definition.
| Projects | # lines of code | # Types | # Abstract Types | # lines of comment | % Comment | Afferent Coupling | Efferent Coupling | Relational Cohesion | Instability | Abstractness | Distance |
|---|---|---|---|---|---|---|---|---|---|---|---|
| libcocos2d v1.0.0.0 | 81055 | 1356 | 51 | 42447 | 34 | 1337 | 264 | 6.57 | 0.16 | 0.04 | 0.56 |
| libSpine v1.0.0.0 | 3119 | 65 | 0 | 330 | 9 | 6 | 57 | 2.49 | 0.9 | 0 | 0.07 |
| libbox2d v1.0.0.0 | 6977 | 133 | 6 | 2123 | 23 | 63 | 12 | 5.1 | 0.16 | 0.05 | 0.56 |
| cpp-tests v1.0.0.0 | 37025 | 1505 | 15 | 5975 | 13 | 0 | 647 | 2.56 | 1 | 0.01 | 0.01 |
| libluacocos2d v1.0.0.0 | 112781 | 52 | 0 | 3288 | 2 | 4 | 642 | 2.19 | 0.99 | 0 | 0 |
| lua-tests v1.0.0.0 | 443 | 5 | 0 | 77 | 14 | 0 | 62 | 0.86 | 1 | 0 | 0 |
| cpp-empty-test v1.0.0.0 | 54 | 4 | 0 | 65 | 54 | 0 | 26 | 1.33 | 1 | 0 | 0 |
| lua-empty-test v1.0.0.0 | 25 | 2 | 0 | 28 | 52 | 0 | 10 | 0.75 | 1 | 0 | 0 |
| External v1.0.0.0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 |
Types Metrics : Code Quality
For a particular Code Metric defined for types, values in red represent the 15% highest values.
If you wish to define thresholds on types' Code Metrics, consider writing some Rule.
Clicking column header arrows sorts values.
Clicking column header title text redirect to the online Code Metric definition.
If you wish to define thresholds on types' Code Metrics, consider writing some Rule.
Clicking column header arrows sorts values.
Clicking column header title text redirect to the online Code Metric definition.
If the code base analyzed has too many types, CppDepend doesn't list Types Metrics to avoid a too big report. The section Types Metrics can be activated by unchecking the option:
CppDepend Project Properties > Report > Avoid too big report for large code base > Hide section Types Metrics if...
It is recommended to use the CppDepend interactive UI capabilities to browse large applications.
CppDepend Project Properties > Report > Avoid too big report for large code base > Hide section Types Metrics if...
It is recommended to use the CppDepend interactive UI capabilities to browse large applications.
Types Metrics : Code Members and Inheritance
| Type Name | # Instance Methods | Nb Static Methods | # Fields | # Children Classes | Depth Of Inheritance Tree | Type Namespace |
|---|
Types Metrics : Lack Of Cohesion Of Methods and Association Between Classes
| Type Name | Lack Of Cohesion Of Methods | Lack Of Cohesion Of Methods HS | Association Between Classes | Type Namespace |
|---|
Namespaces Metrics
If you wish to define thresholds on namespaces' Code Metrics, consider writing some Rules.
Clicking column header arrows sorts values.
Clicking column header title text redirect to the online Code Metric definition.
Clicking column header arrows sorts values.
Clicking column header title text redirect to the online Code Metric definition.
| Namespaces | # lines of code | # Types | # lines of comment | % Comment | Afferent Coupling | Efferent Coupling |
|---|---|---|---|---|---|---|
| libcocos2d::GlobalNamespace | 1993 | 25 | 0 | 0 | 20 | 10 |
| libcocos2d::cocos2d | 44902 | 840 | 0 | 0 | 29 | 24 |
| libcocos2d::cocos2d.extension | 3401 | 52 | 0 | 0 | 3 | 9 |
| libcocos2d::cocos2d.StringUtils | 60 | 1 | 0 | 0 | 7 | 4 |
| libcocos2d::cocos2d.tweenfunc | 244 | 2 | 0 | 0 | 3 | 1 |
| libcocos2d::cocos2d.GL | 62 | 2 | 0 | 0 | 9 | 5 |
| libcocos2d::cocos2d.experimental | 1565 | 13 | 0 | 0 | 2 | 10 |
| libcocos2d::cocos2d.ui | 6990 | 82 | 0 | 0 | 7 | 8 |
| libcocos2d::cocos2d.allocator | 165 | 8 | 0 | 0 | 0 | 3 |
| libcocos2d::cocos2d.utils | 75 | 1 | 0 | 0 | 5 | 5 |
| libcocos2d::cocos2d.network | 817 | 19 | 0 | 0 | 2 | 9 |
| libcocos2d::cocos2d.DrawPrimitives | 226 | 1 | 0 | 0 | 5 | 5 |
| libcocos2d::cocos2d .anonymous_namespace{ccphysicsbody.cpp} | 0 | 1 | 0 | 0 | 1 | 1 |
| libcocos2d::cocos2d .anonymous_namespace{ccphysicsworld.cpp} | 0 | 3 | 0 | 0 | 1 | 2 |
| libcocos2d::cocos2d .anonymous_namespace{ccglview.cpp} | 21 | 1 | 0 | 0 | 1 | 3 |
| libcocos2d::cocos2d .anonymous_namespace{ccimage.cpp} | 94 | 20 | 0 | 0 | 1 | 4 |
| libcocos2d::cocos2d .anonymous_namespace{ccglstatecache.cpp} | 0 | 1 | 0 | 0 | 1 | 1 |
| libcocos2d::cocos2d .anonymous_namespace{cctexture2d.cpp} | 0 | 1 | 0 | 0 | 0 | 2 |
| libcocos2d::rapidjson | 992 | 29 | 0 | 0 | 7 | 3 |
| libcocos2d::rapidjson.internal | 43 | 2 | 0 | 0 | 1 | 2 |
| libcocos2d::llvm | 73 | 1 | 0 | 0 | 1 | 3 |
| libcocos2d::flatbuffers | 1726 | 30 | 0 | 0 | 6 | 6 |
| libcocos2d::flatbuffers.cpp | 259 | 1 | 0 | 0 | 1 | 3 |
| libcocos2d::flatbuffers.go | 327 | 1 | 0 | 0 | 1 | 3 |
| libcocos2d::p2t | 769 | 12 | 0 | 0 | 1 | 3 |
| libcocos2d::tinyxml2 | 1180 | 29 | 0 | 0 | 3 | 3 |
| libcocos2d::CocosDenshion | 181 | 3 | 0 | 0 | 6 | 4 |
| libcocos2d::anonymous_namespace{cceventd ispatcher.cpp} | 3 | 1 | 0 | 0 | 1 | 1 |
| libcocos2d::cocosbuilder | 2030 | 45 | 0 | 0 | 2 | 9 |
| libcocos2d::cocostudio | 11756 | 106 | 0 | 0 | 6 | 16 |
| libcocos2d::cocostudio.timeline | 1101 | 23 | 0 | 0 | 4 | 9 |
| libSpine::GlobalNamespace | 2749 | 60 | 0 | 0 | 3 | 5 |
| libSpine::spine | 370 | 5 | 0 | 0 | 2 | 9 |
| libbox2d::GlobalNamespace | 6977 | 133 | 0 | 0 | 1 | 3 |
| cpp-tests::GlobalNamespace | 36823 | 1497 | 0 | 0 | 0 | 30 |
| cpp-tests::anonymous_namespace{allocator test.cpp} | 2 | 2 | 0 | 0 | 1 | 2 |
| cpp-tests::cocos2d | 96 | 2 | 0 | 0 | 1 | 6 |
| cpp-tests::anonymous_namespace{newaudioe nginetest.cpp} | 102 | 3 | 0 | 0 | 1 | 5 |
| cpp-tests::anonymous_namespace{physicste st.cpp} | 2 | 1 | 0 | 0 | 1 | 2 |
| libluacocos2d::GlobalNamespace | 111389 | 37 | 0 | 0 | 4 | 24 |
| libluacocos2d::cocos2d | 1292 | 13 | 0 | 0 | 4 | 7 |
| libluacocos2d::anonymous_namespace{cclua stack.cpp} | 60 | 1 | 0 | 0 | 1 | 3 |
| libluacocos2d::cocostudio | 40 | 1 | 0 | 0 | 1 | 7 |
| lua-tests::GlobalNamespace | 333 | 2 | 0 | 0 | 0 | 9 |
| lua-tests::cocos2d | 110 | 3 | 0 | 0 | 1 | 7 |
| cpp-empty-test::GlobalNamespace | 54 | 4 | 0 | 0 | 0 | 5 |
| lua-empty-test::GlobalNamespace | 25 | 2 | 0 | 0 | 0 | 6 |
| Project Rules | 151540 |
|
| Code Quality | 3100 |
|
 | Critical Rule warning: Types too big - critical |
• Rule Description:
This rule matches types with more than 500 lines of code.
Types where NbLinesOfCode > 500 are extremely complex to develop and maintain. See the definition of the NbLinesOfCode metric here http://www.cppdepend.com/Metrics.aspx#NbLinesOfCode
Maybe you are facing the God Class phenomenon: A God Class is a class that controls way too many other classes in the system and has grown beyond all logic to become The Class That Does Everything.
• How to Fix Violations: Types with many lines of code should be split in a group of smaller types. To refactor a God Class you'll need patience, and you might even need to recreate everything from scratch. Here are a few advices: • Think before pulling out methods: on what data does this method operate? What responsibility does it have? • Try to maintain the interface of the god class at first and delegate calls to the new extracted classes. In the end the god class should be a pure facade without own logic. Then you can keep it for convenience or throw it away and start to use the new classes only. • Unit Tests can help: write tests for each method before extracting it to ensure you don't break functionality.
• How to Fix Violations: Types with many lines of code should be split in a group of smaller types. To refactor a God Class you'll need patience, and you might even need to recreate everything from scratch. Here are a few advices: • Think before pulling out methods: on what data does this method operate? What responsibility does it have? • Try to maintain the interface of the god class at first and delegate calls to the new extracted classes. In the end the god class should be a pure facade without own logic. Then you can keep it for convenience or throw it away and start to use the new classes only. • Unit Tests can help: write tests for each method before extracting it to ensure you don't break functionality.
20 types matched
| types | # lines of code (LOC) | # Methods | # Fields | Full Name |
|---|---|---|---|---|
| DataReaderHelper | 1 238 | 46 | 14 | cocostudio.DataReaderHelper |
| Bundle3D | 1 133 | 48 | 10 | cocos2d.Bundle3D |
| ScrollView | 936 | 101 | 35 | cocos2d.ui.ScrollView |
| Layout | 929 | 93 | 50 | cocos2d.ui.Layout |
| Image | 886 | 46 | 12 | cocos2d.Image |
| Node | 884 | 233 | 73 | cocos2d.Node |
| Label | 728 | 96 | 55 | cocos2d.Label |
| Parser | 701 | 30 | 18 | flatbuffers.Parser |
| Scale9Sprite | 653 | 84 | 33 | cocos2d.ui.Scale9Sprite |
| b2World | 619 | 50 | 18 | b2World |
| FlatBuffersSerialize | 606 | 30 | 6 | cocostudio.FlatBuffersSerialize |
| PUParticleSystem3D | 597 | 78 | 32 | cocos2d.PUParticleSystem3D |
| LuaEngine | 591 | 53 | 2 | cocos2d.LuaEngine |
| CSLoader | 576 | 45 | 8 | cocos2d.CSLoader |
| Widget | 565 | 135 | 38 | cocos2d.ui.Widget |
| Texture2D | 557 | 76 | 12 | cocos2d.Texture2D |
| Director | 546 | 84 | 44 | cocos2d.Director |
| NodeLoader | 546 | 65 | 1 | cocosbuilder.NodeLoader |
| ParticleSystem | 538 | 118 | 43 | cocos2d.ParticleSystem |
| EventDispatcher | 511 | 35 | 11 | cocos2d.EventDispatcher |
Statistics
| Stat | # lines of code (LOC) | # Methods | # Fields |
|---|---|---|---|
| Sum: | 14 340 | 1 546 | 515 |
| Average: | 717 | 77.3 | 25.75 |
| Minimum: | 511 | 30 | 1 |
| Maximum: | 1 238 | 233 | 73 |
| Standard deviation: | 206.41 | 45.87 | 19.52 |
| Variance: | 42 603 | 2 104 | 380.89 |
| Critical Rule warning: Methods too complex - critical |
• Rule Description:
This rule matches methods where CyclomaticComplexity > 30 or ILCyclomaticComplexity > 60 or ILNestingDepth > 6. Such method is typically hard to understand and maintain.
Maybe you are facing the God Method phenomenon. A "God Method" is a method that does way too many processes in the system and has grown beyond all logic to become The Method That Does Everything. When need for new processes increases suddenly some programmers realize: why should I create a new method for each processe if I can only add an if.
See the definition of the CyclomaticComplexity metric here: http://www.cppdepend.com/Metrics.aspx#CC
See the definition of the NestingDepth metric here http://www.cppdepend.com/Metrics.aspx#NestingDepth
• How to Fix Violations: A large and complex method should be split in smaller methods, or even one or several classes can be created for that. During this process it is important to question the scope of each variable local to the method. This can be an indication if such local variable will become an instance field of the newly created class(es). Large switch…case structures might be refactored through the help of a set of types that implement a common interface, the interface polymorphism playing the role of the switch cases tests. Unit Tests can help: write tests for each method before extracting it to ensure you don't break functionality.
• How to Fix Violations: A large and complex method should be split in smaller methods, or even one or several classes can be created for that. During this process it is important to question the scope of each variable local to the method. This can be an indication if such local variable will become an instance field of the newly created class(es). Large switch…case structures might be refactored through the help of a set of types that implement a common interface, the interface polymorphism playing the role of the switch cases tests. Unit Tests can help: write tests for each method before extracting it to ensure you don't break functionality.
46 methods matched
| methods | Cyclomatic Complexity (CC) | NestingDepth | Full Name |
|---|---|---|---|
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 141 | 9 | cocos2d.PURendererTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 123 | 7 | cocos2d.PUEmitterTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| _spSkeletonJson_readAnimation(spSkeletonJson*,Json*,spSkeletonData*) | 80 | 11 | __Globals._spSkeletonJson_readAnimation(spSkeletonJson*,Json* ,spSkeletonData*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 66 | 7 | cocostudio.WidgetReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| parseProperties(cocos2d::Node*,cocos2d::Node*,cocosbuilder::CCBReader* ) | 66 | 6 | cocosbuilder.NodeLoader.parseProperties(cocos2d::Node*,cocos2d::Node* ,cocosbuilder::CCBReader*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 64 | 7 | cocostudio.NodeReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| Next() | 60 | 7 | flatbuffers.Parser.Next() |
| spSkeletonJson_readSkeletonData(spSkeletonJson*,constchar*) | 56 | 9 | __Globals.spSkeletonJson_readSkeletonData(spSkeletonJson*,constchar*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 55 | 22 | cocos2d.PUMaterialPassTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| unz64local_GetCurrentFileInfoInternal(unzFile,unz_file_info64* ,unz_file_info64_internal*,char*,uLong,void*,uLong,char*,uLong) | 55 | 13 | cocos2d.__Globals.unz64local_GetCurrentFileInfoInternal(unzFile ,unz_file_info64*,unz_file_info64_internal*,char*,uLong,void*,uLong ,char*,uLong) |
| GenStruct(constflatbuffers::LanguageParameters& ,constflatbuffers::Parser&,flatbuffers::StructDef&,std::string*) | 52 | 8 | flatbuffers.__Globals.GenStruct(constflatbuffers::LanguageParameters& ,constflatbuffers::Parser&,flatbuffers::StructDef&,std::string*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 48 | 13 | cocos2d.PUTechniqueTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| startElement(void*,constchar*,constchar**) | 48 | 10 | cocos2d.TMXMapInfo.startElement(void*,constchar*,constchar**) |
| SolveTOI(constb2TimeStep&) | 48 | 6 | b2World.SolveTOI(constb2TimeStep&) |
| parse(PUConcreteNodeList&,constPUScriptTokenList&) | 47 | 6 | cocos2d.PUScriptParser.parse(PUConcreteNodeList& ,constPUScriptTokenList&) |
| serialize(void*) | 42 | 16 | cocostudio.ComRender.serialize(void*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 42 | 8 | cocos2d.PUDynamicAttributeTranslator.translate (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 41 | 6 | cocostudio.CheckBoxReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 41 | 6 | cocostudio.SliderReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| dispatchTouchEvent(cocos2d::EventTouch*) | 36 | 7 | cocos2d.EventDispatcher.dispatchTouchEvent(cocos2d::EventTouch*) |
| GenTable(constflatbuffers::Parser&,flatbuffers::StructDef& ,constflatbuffers::GeneratorOptions&,std::string*) | 36 | 7 | flatbuffers.cpp.__Globals.GenTable(constflatbuffers::Parser& ,flatbuffers::StructDef&,constflatbuffers::GeneratorOptions& ,std::string*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 36 | 6 | cocostudio.ButtonReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| readDataTask() | 35 | 6 | cocos2d.experimental.AudioCache.readDataTask() |
| buildJson(rapidjson::Document&,cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*) | 32 | 12 | cocostudio.TriggerMng.buildJson(rapidjson::Document& ,cocostudio::CocoLoader*,cocostudio::stExpCocoNode*) |
| decodeFrame(tinyxml2::XMLElement*,tinyxml2::XMLElement* ,cocostudio::BoneData*,DataInfo*) | 32 | 10 | cocostudio.DataReaderHelper.decodeFrame(tinyxml2::XMLElement* ,tinyxml2::XMLElement*,cocostudio::BoneData*,DataInfo*) |
| rotateBufferThread(int) | 32 | 6 | cocos2d.experimental.AudioPlayer.rotateBufferThread(int) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 31 | 7 | cocos2d.PUAffectorTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| readNodeGraph(cocos2d::Node*) | 31 | 7 | cocosbuilder.CCBReader.readNodeGraph(cocos2d::Node*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 29 | 8 | cocos2d.PUObserverTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| loadAnimationDataBinary(conststd::string&,cocos2d::Animation3DData*) | 27 | 6 | cocos2d.Bundle3D.loadAnimationDataBinary(conststd::string& ,cocos2d::Animation3DData*) |
| spAtlas_create(constchar*,int,constchar*,void*) | 27 | 6 | __Globals.spAtlas_create(constchar*,int,constchar*,void*) |
| onSocketCallback(structlibwebsocket_context*,structlibwebsocket*,int ,void*,void*,ssize_t) | 26 | 10 | cocos2d.network.WebSocket.onSocketCallback(structlibwebsocket_context* ,structlibwebsocket*,int,void*,void*,ssize_t) |
| main(int,constchar**) | 26 | 6 | __Globals.main(int,constchar**) |
| addDataFromBinaryCache(constchar*,DataInfo*) | 25 | 7 | cocostudio.DataReaderHelper.addDataFromBinaryCache(constchar* ,DataInfo*) |
| decodeFrame(cocostudio::CocoLoader*,cocostudio::stExpCocoNode* ,DataInfo*) | 25 | 6 | cocostudio.DataReaderHelper.decodeFrame(cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*,DataInfo*) |
| lua_cocos2dx_RenderTexture_saveToFile(lua_State*) | 25 | 6 | __Globals.lua_cocos2dx_RenderTexture_saveToFile(lua_State*) |
| easeValue(float) | 24 | 6 | cocostudio.ActionFrameEasing.easeValue(float) |
| handle(cocos2d::PUParticleSystem3D*,cocos2d::PUParticle3D*,float) | 23 | 8 | cocos2d.PUDoEnableComponentEventHandler.handle (cocos2d::PUParticleSystem3D*,cocos2d::PUParticle3D*,float) |
| findNextFocusedWidget(cocos2d::ui::Widget::FocusDirection ,cocos2d::ui::Widget*) | 23 | 7 | cocos2d.ui.Layout.findNextFocusedWidget (cocos2d::ui::Widget::FocusDirection,cocos2d::ui::Widget*) |
| initWithDictionary(ValueMap&,conststd::string&) | 23 | 6 | cocos2d.ParticleSystem.initWithDictionary(ValueMap&,conststd::string&) |
| parseNodesRecursivelyBinary(bool&,bool) | 22 | 6 | cocos2d.Bundle3D.parseNodesRecursivelyBinary(bool&,bool) |
| generateTestFunctions() | 21 | 29 | TouchEventDispatchingPerfTest.generateTestFunctions() |
| decodeBoneDisplay(cocostudio::CocoLoader*,cocostudio::stExpCocoNode* ,DataInfo*) | 21 | 8 | cocostudio.DataReaderHelper.decodeBoneDisplay(cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*,DataInfo*) |
| addDataFromJsonCache(conststd::string&,DataInfo*) | 21 | 6 | cocostudio.DataReaderHelper.addDataFromJsonCache(conststd::string& ,DataInfo*) |
| DrawDebugData() | 21 | 6 | b2World.DrawDebugData() |
| updateCamera(float) | 21 | 6 | Camera3DTestDemo.updateCamera(float) |
Statistics
| Stat | Cyclomatic Complexity (CC) | NestingDepth |
|---|---|---|
| Sum: | 1 906 | 388 |
| Average: | 41.43 | 8.43 |
| Minimum: | 21 | 6 |
| Maximum: | 141 | 29 |
| Standard deviation: | 24.33 | 4.34 |
| Variance: | 591.94 | 18.85 |
| Critical Rule warning: Methods with too many parameters - critical |
• Rule Description:
This rule matches methods with more than 8 parameters. Such method is painful to call and might degrade performance. See the definition of the NbParameters metric here: http://www.cppdepend.com/Metrics.aspx#NbParameters
• How to Fix Violations: More properties/fields can be added to the declaring type to handle numerous states. An alternative is to provide a class or a structure dedicated to handle arguments passing.
• How to Fix Violations: More properties/fields can be added to the declaring type to handle numerous states. An alternative is to provide a class or a structure dedicated to handle arguments passing.
11 methods matched
| methods | # Parameters | Full Name |
|---|---|---|
| Mat4(float,float,float,float,float,float,float,float,float,float,float ,float,float,float,float,float) | 16 | cocos2d.Mat4.Mat4(float,float,float,float,float,float,float,float ,float,float,float,float,float,float,float,float) |
| set(float,float,float,float,float,float,float,float,float,float,float ,float,float,float,float,float) | 16 | cocos2d.Mat4.set(float,float,float,float,float,float,float,float,float ,float,float,float,float,float,float,float) |
| slerp(float,float,float,float,float,float,float,float,float,float* ,float*,float*,float*) | 13 | cocos2d.Quaternion.slerp(float,float,float,float,float,float,float ,float,float,float*,float*,float*,float*) |
| init(float,GLuint,cocos2d::GLProgramState*,cocos2d::BlendFunc,GLuint ,GLuint,GLenum,GLenum,ssize_t,constcocos2d::Mat4&,uint32_t) | 11 | cocos2d.MeshCommand.init(float,GLuint,cocos2d::GLProgramState* ,cocos2d::BlendFunc,GLuint,GLuint,GLenum,GLenum,ssize_t ,constcocos2d::Mat4&,uint32_t) |
| createLookAt(float,float,float,float,float,float,float,float,float ,cocos2d::Mat4*) | 10 | cocos2d.Mat4.createLookAt(float,float,float,float,float,float,float ,float,float,cocos2d::Mat4*) |
| init(float,GLuint,cocos2d::GLProgramState*,cocos2d::BlendFunc,GLuint ,GLuint,GLenum,GLenum,ssize_t,constcocos2d::Mat4&) | 10 | cocos2d.MeshCommand.init(float,GLuint,cocos2d::GLProgramState* ,cocos2d::BlendFunc,GLuint,GLuint,GLenum,GLenum,ssize_t ,constcocos2d::Mat4&) |
| distaa3(double*,double*,double*,int,int,int,int,int,int) | 9 | __Globals.distaa3(double*,double*,double*,int,int,int,int,int,int) |
| unz64local_GetCurrentFileInfoInternal(unzFile,unz_file_info64* ,unz_file_info64_internal*,char*,uLong,void*,uLong,char*,uLong) | 9 | cocos2d.__Globals.unz64local_GetCurrentFileInfoInternal(unzFile ,unz_file_info64*,unz_file_info64_internal*,char*,uLong,void*,uLong ,char*,uLong) |
| ccVertexLineIntersect(float,float,float,float,float,float,float,float ,float*) | 9 | cocos2d.__Globals.ccVertexLineIntersect(float,float,float,float,float ,float,float,float,float*) |
| TerrainData(constchar*,constchar*,constcocos2d::Terrain::DetailMap& ,constcocos2d::Terrain::DetailMap&,constcocos2d::Terrain::DetailMap& ,constcocos2d::Terrain::DetailMap&,constcocos2d::Size&,float,float) | 9 | cocos2d.Terrain+TerrainData.TerrainData(constchar*,constchar* ,constcocos2d::Terrain::DetailMap&,constcocos2d::Terrain::DetailMap& ,constcocos2d::Terrain::DetailMap&,constcocos2d::Terrain::DetailMap& ,constcocos2d::Size&,float,float) |
| EventAssetsManagerEx(conststd::string& ,cocos2d::extension::AssetsManagerEx* ,constcocos2d::extension::EventAssetsManagerEx::EventCode&,float,float ,conststd::string&,conststd::string&,int,int) | 9 | cocos2d.extension.EventAssetsManagerEx.EventAssetsManagerEx (conststd::string&,cocos2d::extension::AssetsManagerEx* ,constcocos2d::extension::EventAssetsManagerEx::EventCode&,float,float ,conststd::string&,conststd::string&,int,int) |
Statistics
| Stat | # Parameters |
|---|---|
| Sum: | 121 |
| Average: | 11 |
| Minimum: | 9 |
| Maximum: | 16 |
| Standard deviation: | 2.63 |
| Variance: | 6.91 |
 | Rule warning: Quick summary of methods to refactor |
• Rule Description:
Methods matched by this rule somehow violate one or several basic quality principles, whether it is too large (too many lines of code), too complex (too many if, switch case, loops…) has too many variables, too many parameters or has too many overloads.
• How to Fix Violations: To refactor such method and increase code quality and maintainability, certainly you'll have to split the method into several smaller methods or even create one or several classes to implement the logic. During this process it is important to question the scope of each variable local to the method. This can be an indication if such local variable will become an instance field of the newly created class(es). Large switch…case structures might be refactored through the help of a set of types that implement a common interface, the interface polymorphism playing the role of the switch cases tests. Unit Tests can help: write tests for each method before extracting it to ensure you don't break functionality.
• How to Fix Violations: To refactor such method and increase code quality and maintainability, certainly you'll have to split the method into several smaller methods or even create one or several classes to implement the logic. During this process it is important to question the scope of each variable local to the method. This can be an indication if such local variable will become an instance field of the newly created class(es). Large switch…case structures might be refactored through the help of a set of types that implement a common interface, the interface polymorphism playing the role of the switch cases tests. Unit Tests can help: write tests for each method before extracting it to ensure you don't break functionality.
1,907 methods matched
| methods | # lines of code (LOC) | MaxNestedLoop | Cyclomatic Complexity (CC) | NestingDepth | # Parameters | # Variables | # Overloads | Full Name |
|---|---|---|---|---|---|---|---|---|
| ConvertUTF8toUTF16(constUTF8**,constUTF8*,UTF16**,UTF16* ,ConversionFlags) | 63 | 1 | 17 | 5 | 5 | 5 | 1 | __Globals.ConvertUTF8toUTF16(constUTF8**,constUTF8*,UTF16**,UTF16* ,ConversionFlags) |
| ConvertUTF8toUTF32(constUTF8**,constUTF8*,UTF32**,UTF32* ,ConversionFlags) | 51 | 1 | 14 | 4 | 5 | 5 | 1 | __Globals.ConvertUTF8toUTF32(constUTF8**,constUTF8*,UTF32**,UTF32* ,ConversionFlags) |
| ConvertUTF16toUTF8(constUTF16**,constUTF16*,UTF8**,UTF8* ,ConversionFlags) | 62 | 1 | 17 | 3 | 5 | 9 | 1 | __Globals.ConvertUTF16toUTF8(constUTF16**,constUTF16*,UTF8**,UTF8* ,ConversionFlags) |
| ConvertUTF32toUTF8(constUTF32**,constUTF32*,UTF8**,UTF8* ,ConversionFlags) | 48 | 1 | 13 | 3 | 5 | 7 | 1 | __Globals.ConvertUTF32toUTF8(constUTF32**,constUTF32*,UTF8**,UTF8* ,ConversionFlags) |
| ConvertUTF16toUTF32(constUTF16**,constUTF16*,UTF32**,UTF32* ,ConversionFlags) | 34 | 1 | 9 | 3 | 5 | 6 | 1 | __Globals.ConvertUTF16toUTF32(constUTF16**,constUTF16*,UTF32**,UTF32* ,ConversionFlags) |
| ConvertUTF32toUTF16(constUTF32**,constUTF32*,UTF16**,UTF16* ,ConversionFlags) | 35 | 1 | 9 | 5 | 5 | 4 | 1 | __Globals.ConvertUTF32toUTF16(constUTF32**,constUTF32*,UTF16**,UTF16* ,ConversionFlags) |
| isLegalUTF8(constUTF8*,int) | 40 | 0 | 21 | 3 | 2 | 2 | 1 | __Globals.isLegalUTF8(constUTF8*,int) |
| distaa3(double*,double*,double*,int,int,int,int,int,int) | 20 | 0 | 5 | 2 | 9 | 8 | 1 | __Globals.distaa3(double*,double*,double*,int,int,int,int,int,int) |
| edtaa3(double*,double*,double*,int,int,short*,short*,double*) | 290 | 2 | 42 | 5 | 8 | 20 | 1 | __Globals.edtaa3(double*,double*,double*,int,int,short*,short*,double* ) |
| main(int,constchar**) | 79 | 2 | 26 | 6 | 2 | 20 | 1 | __Globals.main(int,constchar**) |
| XXH32_endian_align(constvoid*,int,U32,XXH_endianess,XXH_alignment) | 45 | 1 | 4 | 4 | 5 | 8 | 1 | __Globals.XXH32_endian_align(constvoid*,int,U32,XXH_endianess ,XXH_alignment) |
| XXH32_update_endian(void*,constvoid*,int,XXH_endianess) | 61 | 0 | 5 | 3 | 4 | 9 | 1 | __Globals.XXH32_update_endian(void*,constvoid*,int,XXH_endianess) |
| atitc_decode_block(uint8_t**,uint32_t*,unsignedint,bool,uint64_t ,ATITCDecodeFlag) | 67 | 2 | 9 | 4 | 6 | 19 | 1 | __Globals.atitc_decode_block(uint8_t**,uint32_t*,unsignedint,bool ,uint64_t,ATITCDecodeFlag) |
| etc1_decode_block(constetc1_byte*,etc1_byte*) | 27 | 0 | 2 | 2 | 2 | 16 | 1 | __Globals.etc1_decode_block(constetc1_byte*,etc1_byte*) |
| etc1_encode_image(constetc1_byte*,etc1_uint32,etc1_uint32,etc1_uint32 ,etc1_uint32,etc1_byte*) | 34 | 4 | 9 | 6 | 6 | 17 | 1 | __Globals.etc1_encode_image(constetc1_byte*,etc1_uint32,etc1_uint32 ,etc1_uint32,etc1_uint32,etc1_byte*) |
| etc1_decode_image(constetc1_byte*,etc1_byte*,etc1_uint32,etc1_uint32 ,etc1_uint32,etc1_uint32) | 29 | 4 | 9 | 6 | 6 | 15 | 1 | __Globals.etc1_decode_image(constetc1_byte*,etc1_byte*,etc1_uint32 ,etc1_uint32,etc1_uint32,etc1_uint32) |
| decode_subblock(etc1_byte*,int,int,int,constint*,etc1_uint32,bool,bool ) | 22 | 1 | 5 | 3 | 8 | 9 | 1 | __Globals.decode_subblock(etc1_byte*,int,int,int,constint*,etc1_uint32 ,bool,bool) |
| etc_average_colors_subblock(constetc1_byte*,etc1_uint32,etc1_byte* ,bool,bool) | 32 | 2 | 10 | 5 | 5 | 15 | 1 | __Globals.etc_average_colors_subblock(constetc1_byte*,etc1_uint32 ,etc1_byte*,bool,bool) |
| chooseModifier(constetc1_byte*,constetc1_byte*,etc1_uint32*,int ,constint*) | 26 | 1 | 7 | 3 | 5 | 15 | 1 | __Globals.chooseModifier(constetc1_byte*,constetc1_byte*,etc1_uint32* ,int,constint*) |
| etc_encode_subblock_helper(constetc1_byte*,etc1_uint32,etc_compressed* ,bool,bool,constetc1_byte*,constint*) | 22 | 2 | 10 | 5 | 7 | 11 | 1 | __Globals.etc_encode_subblock_helper(constetc1_byte*,etc1_uint32 ,etc_compressed*,bool,bool,constetc1_byte*,constint*) |
| etc_encodeBaseColors(etc1_byte*,constetc1_byte*,etc_compressed*) | 40 | 0 | 3 | 3 | 3 | 22 | 1 | __Globals.etc_encodeBaseColors(etc1_byte*,constetc1_byte* ,etc_compressed*) |
| PVRDecompress(AMTC_BLOCK_STRUCT*,constbool,constint,constint,constint ,unsignedchar*) | 59 | 4 | 10 | 6 | 6 | 25 | 1 | __Globals.PVRDecompress(AMTC_BLOCK_STRUCT*,constbool,constint,constint ,constint,unsignedchar*) |
| UnpackModulations(constAMTC_BLOCK_STRUCT*,constint,int(*)[16],int(* )[16],int,int) | 28 | 2 | 11 | 5 | 6 | 4 | 1 | __Globals.UnpackModulations(constAMTC_BLOCK_STRUCT*,constint,int(* )[16],int(*)[16],int,int) |
| InterpolateColours(constint*,constint*,constint*,constint*,constint ,constint,constint,int*) | 40 | 2 | 11 | 3 | 8 | 10 | 1 | __Globals.InterpolateColours(constint*,constint*,constint*,constint* ,constint,constint,constint,int*) |
| GetModulationValue(int,int,constint,constint(*)[16],constint(*)[16] ,int*,int*) | 24 | 0 | 7 | 3 | 7 | 3 | 1 | __Globals.GetModulationValue(int,int,constint,constint(*)[16],constint (*)[16],int*,int*) |
| TwiddleUV(U32,U32,U32,U32) | 33 | 1 | 6 | 3 | 4 | 6 | 1 | __Globals.TwiddleUV(U32,U32,U32,U32) |
| s3tc_decode_block(uint8_t**,uint32_t*,unsignedint,bool,uint64_t ,S3TCDecodeFlag) | 62 | 2 | 9 | 4 | 6 | 18 | 1 | __Globals.s3tc_decode_block(uint8_t**,uint32_t*,unsignedint,bool ,uint64_t,S3TCDecodeFlag) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 34 | 2 | 12 | 5 | 2 | 14 | 1 | ArmatureNodeReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 25 | 0 | 4 | 3 | 2 | 9 | 1 | ArmatureNodeReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| DlgProc(HWND,UINT,WPARAM,LPARAM) | 35 | 2 | 10 | 10 | 4 | 13 | 1 | CWin32InputBox.DlgProc(HWND,UINT,WPARAM,LPARAM) |
| InitDialog() | 28 | 1 | 4 | 3 | 0 | 9 | 1 | CWin32InputBox.InitDialog() |
| InputBoxEx(WIN32INPUTBOX_PARAM*) | 41 | 0 | 14 | 2 | 1 | 7 | 1 | CWin32InputBox.InputBoxEx(WIN32INPUTBOX_PARAM*) |
| InputBox(LPCSTR,LPCSTR,std::string*,DWORD,bool,HWND) | 8 | 0 | 1 | 1 | 6 | 1 | 1 | CWin32InputBox.InputBox(LPCSTR,LPCSTR,std::string*,DWORD,bool,HWND) |
| ccpLineIntersect(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,float*,float*) | 1 | 0 | 1 | 1 | 6 | 0 | 1 | cocos2d.__Globals.ccpLineIntersect(constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,constcocos2d::Vec2&,float* ,float*) |
| CCAffineTransformMake(float,float,float,float,float,float) | 1 | 0 | 1 | 1 | 6 | 0 | 1 | cocos2d.__Globals.CCAffineTransformMake(float,float,float,float,float ,float) |
| unzGetCurrentFileInfo64(unzFile,unz_file_info64*,char*,uLong,void* ,uLong,char*,uLong) | 1 | 0 | 1 | 1 | 8 | 0 | 1 | cocos2d.__Globals.unzGetCurrentFileInfo64(unzFile,unz_file_info64* ,char*,uLong,void*,uLong,char*,uLong) |
| unzGetCurrentFileInfo(unzFile,unz_file_info*,char*,uLong,void*,uLong ,char*,uLong) | 19 | 0 | 2 | 2 | 8 | 2 | 1 | cocos2d.__Globals.unzGetCurrentFileInfo(unzFile,unz_file_info*,char* ,uLong,void*,uLong,char*,uLong) |
| unzOpenCurrentFile3(unzFile,int*,int*,int,constchar*) | 73 | 0 | 17 | 4 | 5 | 6 | 1 | cocos2d.__Globals.unzOpenCurrentFile3(unzFile,int*,int*,int,constchar* ) |
| unzReadCurrentFile(unzFile,voidp,unsignedint) | 78 | 2 | 24 | 4 | 3 | 12 | 1 | cocos2d.__Globals.unzReadCurrentFile(unzFile,voidp,unsignedint) |
| unz64local_getLong64(constzlib_filefunc64_32_def*,voidpf,ZPOS64_T*) | 31 | 0 | 9 | 1 | 3 | 3 | 1 | cocos2d.__Globals.unz64local_getLong64(constzlib_filefunc64_32_def* ,voidpf,ZPOS64_T*) |
| unz64local_SearchCentralDir(constzlib_filefunc64_32_def*,voidpf) | 35 | 2 | 12 | 3 | 2 | 8 | 1 | cocos2d.__Globals.unz64local_SearchCentralDir (constzlib_filefunc64_32_def*,voidpf) |
| unz64local_SearchCentralDir64(constzlib_filefunc64_32_def*,voidpf) | 59 | 2 | 23 | 3 | 2 | 10 | 1 | cocos2d.__Globals.unz64local_SearchCentralDir64 (constzlib_filefunc64_32_def*,voidpf) |
| unzOpenInternal(constvoid*,zlib_filefunc64_32_def*,int) | 93 | 0 | 31 | 4 | 3 | 10 | 1 | cocos2d.__Globals.unzOpenInternal(constvoid*,zlib_filefunc64_32_def* ,int) |
| unz64local_GetCurrentFileInfoInternal(unzFile,unz_file_info64* ,unz_file_info64_internal*,char*,uLong,void*,uLong,char*,uLong) | 132 | 1 | 55 | 13 | 9 | 14 | 1 | cocos2d.__Globals.unz64local_GetCurrentFileInfoInternal(unzFile ,unz_file_info64*,unz_file_info64_internal*,char*,uLong,void*,uLong ,char*,uLong) |
| unz64local_CheckCurrentFileCoherencyHeader(unz64_s*,uInt*,ZPOS64_T* ,uInt*) | 49 | 0 | 20 | 2 | 4 | 6 | 1 | cocos2d.__Globals.unz64local_CheckCurrentFileCoherencyHeader(unz64_s* ,uInt*,ZPOS64_T*,uInt*) |
| getChildMap(std::map<int,std::vector<int>>&,cocos2d::SkinData* ,constrapidjson::Value&) | 28 | 1 | 9 | 3 | 3 | 10 | 1 | cocos2d.__Globals.getChildMap(std::map<int,std::vector<int>>& ,cocos2d::SkinData*,constrapidjson::Value&) |
| _getEigenVectors(cocos2d::Mat4*,cocos2d::Vec3*,cocos2d::Mat4) | 70 | 7 | 18 | 9 | 3 | 21 | 1 | cocos2d.__Globals._getEigenVectors(cocos2d::Mat4*,cocos2d::Vec3* ,cocos2d::Mat4) |
| updateVertex(std::map<vertex_index,ssize_t>&,std::vector<float>& ,std::vector<float>&,std::vector<float>&,conststd::vector<float>& ,conststd::vector<float>&,conststd::vector<float>& ,constcocos2d::vertex_index&) | 17 | 0 | 4 | 2 | 8 | 2 | 1 | cocos2d.__Globals.updateVertex(std::map<vertex_index,ssize_t>& ,std::vector<float>&,std::vector<float>&,std::vector<float>& ,conststd::vector<float>&,conststd::vector<float>&,conststd::vector <float>&,constcocos2d::vertex_index&) |
| exportFaceGroupToShape(std::map<vertex_index,ssize_t>& ,ObjLoader::shapes_t&,conststd::vector<float>&,conststd::vector<float >&,conststd::vector<float>&,conststd::vector<std::vector<vertex_index> >&,constObjLoader::material_t&,conststd::string&) | 27 | 2 | 4 | 3 | 8 | 15 | 1 | cocos2d.__Globals.exportFaceGroupToShape(std::map<vertex_index,ssize_t >&,ObjLoader::shapes_t&,conststd::vector<float>&,conststd::vector <float>&,conststd::vector<float>&,conststd::vector<std::vector <vertex_index>>&,constObjLoader::material_t&,conststd::string&) |
| LoadMtl(std::map<std::string,ObjLoader::material_t>&,constchar* ,constchar*) | 126 | 1 | 46 | 3 | 3 | 28 | 1 | cocos2d.__Globals.LoadMtl(std::map<std::string,ObjLoader::material_t>& ,constchar*,constchar*) |
| getImageData(cocos2d::Image*,Texture2D::PixelFormat&) | 41 | 1 | 9 | 4 | 2 | 16 | 1 | cocos2d.__Globals.getImageData(cocos2d::Image*,Texture2D::PixelFormat& ) |
| _base64Decode(constunsignedchar*,unsignedint,unsignedchar* ,unsignedint*) | 44 | 1 | 13 | 3 | 4 | 9 | 1 | cocos2d.__Globals._base64Decode(constunsignedchar*,unsignedint ,unsignedchar*,unsignedint*) |
| printFileUtils(int) | 16 | 0 | 1 | 2 | 1 | 16 | 1 | cocos2d.__Globals.printFileUtils(int) |
| RectFromString(conststd::string&) | 29 | 1 | 3 | 4 | 1 | 14 | 1 | cocos2d.__Globals.RectFromString(conststd::string&) |
| tgaLoadBuffer(unsignedchar*,long) | 35 | 0 | 9 | 4 | 2 | 4 | 1 | cocos2d.__Globals.tgaLoadBuffer(unsignedchar*,long) |
| tgaLoadRLEImageData(unsignedchar*,unsignedlong,tImageTGA*) | 30 | 1 | 6 | 4 | 3 | 10 | 1 | cocos2d.__Globals.tgaLoadRLEImageData(unsignedchar*,unsignedlong ,tImageTGA*) |
| ccDrawCircle(constcocos2d::Vec2&,float,float,unsignedint,bool,float ,float) | 1 | 0 | 1 | 1 | 7 | 0 | 2 | cocos2d.__Globals.ccDrawCircle(constcocos2d::Vec2&,float,float ,unsignedint,bool,float,float) |
| ccDrawSolidCircle(constcocos2d::Vec2&,float,float,unsignedint,float ,float) | 1 | 0 | 1 | 1 | 6 | 0 | 2 | cocos2d.__Globals.ccDrawSolidCircle(constcocos2d::Vec2&,float,float ,unsignedint,float,float) |
| kmMat4OrthographicProjection(cocos2d::Mat4*,float,float,float,float ,float,float) | 2 | 0 | 1 | 1 | 7 | 0 | 1 | cocos2d.__Globals.kmMat4OrthographicProjection(cocos2d::Mat4*,float ,float,float,float,float,float) |
| visitArray(constValueVector&) | 18 | 0 | 3 | 3 | 1 | 10 | 1 | cocos2d.__Globals.visitArray(constValueVector&) |
| ccdictionary_to_valuemap(cocos2d::__Dictionary*) | 32 | 0 | 10 | 4 | 1 | 12 | 1 | cocos2d.__Globals.ccdictionary_to_valuemap(cocos2d::__Dictionary*) |
| ccarray_to_valuevector(cocos2d::__Array*) | 29 | 0 | 9 | 3 | 1 | 10 | 1 | cocos2d.__Globals.ccarray_to_valuevector(cocos2d::__Array*) |
| __CCAffineTransformMake(float,float,float,float,float,float) | 8 | 0 | 1 | 1 | 6 | 1 | 1 | cocos2d.__Globals.__CCAffineTransformMake(float,float,float,float ,float,float) |
| RectApplyAffineTransform(constcocos2d::Rect& ,constcocos2d::AffineTransform&) | 13 | 0 | 1 | 1 | 2 | 12 | 1 | cocos2d.__Globals.RectApplyAffineTransform(constcocos2d::Rect& ,constcocos2d::AffineTransform&) |
| RectApplyTransform(constcocos2d::Rect&,constcocos2d::Mat4&) | 17 | 0 | 1 | 1 | 2 | 12 | 1 | cocos2d.__Globals.RectApplyTransform(constcocos2d::Rect& ,constcocos2d::Mat4&) |
| ccVertexLineToPolygon(cocos2d::Vec2*,float,cocos2d::Vec2*,unsignedint ,unsignedint) | 45 | 1 | 12 | 3 | 5 | 18 | 1 | cocos2d.__Globals.ccVertexLineToPolygon(cocos2d::Vec2*,float ,cocos2d::Vec2*,unsignedint,unsignedint) |
| ccVertexLineIntersect(float,float,float,float,float,float,float,float ,float*) | 22 | 0 | 3 | 1 | 9 | 4 | 1 | cocos2d.__Globals.ccVertexLineIntersect(float,float,float,float,float ,float,float,float,float*) |
| isOneDimensionSegmentOverlap(float,float,float,float,float*,float*) | 23 | 0 | 11 | 3 | 6 | 4 | 1 | cocos2d.__Globals.isOneDimensionSegmentOverlap(float,float,float,float ,float*,float*) |
| getData(conststd::string&,bool) | 35 | 0 | 6 | 3 | 2 | 8 | 2 | cocos2d.__Globals.getData(conststd::string&,bool) |
| glew_dynamic_binding() | 46 | 0 | 4 | 5 | 0 | 1 | 1 | cocos2d.__Globals.glew_dynamic_binding() |
| getData(conststd::string&,bool) | 39 | 0 | 8 | 4 | 2 | 11 | 2 | cocos2d.__Globals.getData(conststd::string&,bool) |
| ccCardinalSplineAt(cocos2d::Vec2&,cocos2d::Vec2&,cocos2d::Vec2& ,cocos2d::Vec2&,float,float) | 10 | 0 | 1 | 1 | 6 | 9 | 1 | cocos2d.__Globals.ccCardinalSplineAt(cocos2d::Vec2&,cocos2d::Vec2& ,cocos2d::Vec2&,cocos2d::Vec2&,float,float) |
| makeDistanceMap(unsignedchar*,long,long) | 45 | 2 | 11 | 3 | 3 | 13 | 1 | cocos2d.__Globals.makeDistanceMap(unsignedchar*,long,long) |
| listenOnTCP(int) | 53 | 0 | 13 | 4 | 1 | 14 | 1 | cocos2d.Console.listenOnTCP(int) |
| loop() | 49 | 1 | 15 | 6 | 0 | 26 | 1 | cocos2d.Console.loop() |
| parseCommand(int) | 50 | 1 | 11 | 4 | 1 | 16 | 1 | cocos2d.Console.parseCommand(int) |
| commandResolution(int,conststd::string&) | 16 | 0 | 2 | 3 | 2 | 12 | 1 | cocos2d.Console.commandResolution(int,conststd::string&) |
| commandTouch(int,conststd::string&) | 70 | 1 | 18 | 7 | 2 | 24 | 1 | cocos2d.Console.commandTouch(int,conststd::string&) |
| commandUpload(int) | 41 | 2 | 13 | 5 | 1 | 20 | 1 | cocos2d.Console.commandUpload(int) |
| isLineIntersect(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,float*,float*) | 10 | 0 | 5 | 2 | 6 | 1 | 1 | cocos2d.Vec2.isLineIntersect(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,float*,float*) |
| isSegmentOverlap(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,cocos2d::Vec2*,cocos2d::Vec2* ) | 3 | 0 | 2 | 2 | 6 | 0 | 1 | cocos2d.Vec2.isSegmentOverlap(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,cocos2d::Vec2*,cocos2d::Vec2* ) |
| Mat4(float,float,float,float,float,float,float,float,float,float,float ,float,float,float,float,float) | 1 | 0 | 1 | 1 | 16 | 0 | 4 | cocos2d.Mat4.Mat4(float,float,float,float,float,float,float,float ,float,float,float,float,float,float,float,float) |
| createLookAt(float,float,float,float,float,float,float,float,float ,cocos2d::Mat4*) | 30 | 0 | 1 | 1 | 10 | 6 | 2 | cocos2d.Mat4.createLookAt(float,float,float,float,float,float,float ,float,float,cocos2d::Mat4*) |
| createOrthographicOffCenter(float,float,float,float,float,float ,cocos2d::Mat4*) | 15 | 0 | 1 | 1 | 7 | 0 | 1 | cocos2d.Mat4.createOrthographicOffCenter(float,float,float,float,float ,float,cocos2d::Mat4*) |
| createRotation(constcocos2d::Quaternion&,cocos2d::Mat4*) | 29 | 0 | 1 | 1 | 2 | 12 | 2 | cocos2d.Mat4.createRotation(constcocos2d::Quaternion&,cocos2d::Mat4*) |
| createRotation(constcocos2d::Vec3&,float,cocos2d::Mat4*) | 40 | 0 | 3 | 3 | 3 | 16 | 2 | cocos2d.Mat4.createRotation(constcocos2d::Vec3&,float,cocos2d::Mat4*) |
| decompose(cocos2d::Vec3*,cocos2d::Quaternion*,cocos2d::Vec3*) | 63 | 0 | 10 | 3 | 3 | 13 | 1 | cocos2d.Mat4.decompose(cocos2d::Vec3*,cocos2d::Quaternion* ,cocos2d::Vec3*) |
| determinant() | 13 | 0 | 1 | 1 | 0 | 12 | 1 | cocos2d.Mat4.determinant() |
| inverse() | 34 | 0 | 2 | 1 | 0 | 14 | 1 | cocos2d.Mat4.inverse() |
| set(float,float,float,float,float,float,float,float,float,float,float ,float,float,float,float,float) | 16 | 0 | 1 | 1 | 16 | 0 | 3 | cocos2d.Mat4.set(float,float,float,float,float,float,float,float,float ,float,float,float,float,float,float,float) |
| squad(constcocos2d::Quaternion&,constcocos2d::Quaternion& ,constcocos2d::Quaternion&,constcocos2d::Quaternion&,float ,cocos2d::Quaternion*) | 6 | 0 | 1 | 1 | 6 | 2 | 1 | cocos2d.Quaternion.squad(constcocos2d::Quaternion& ,constcocos2d::Quaternion&,constcocos2d::Quaternion& ,constcocos2d::Quaternion&,float,cocos2d::Quaternion*) |
| slerp(float,float,float,float,float,float,float,float,float,float* ,float*,float*,float*) | 63 | 0 | 6 | 2 | 13 | 18 | 2 | cocos2d.Quaternion.slerp(float,float,float,float,float,float,float ,float,float,float*,float*,float*,float*) |
| unionWithRect(constcocos2d::Rect&) | 21 | 0 | 5 | 2 | 1 | 12 | 1 | cocos2d.Rect.unionWithRect(constcocos2d::Rect&) |
| initWithData(constvoid*,ssize_t,Texture2D::PixelFormat,int,int ,constcocos2d::Size&) | 5 | 0 | 1 | 1 | 6 | 1 | 1 | cocos2d.Texture2D.initWithData(constvoid*,ssize_t ,Texture2D::PixelFormat,int,int,constcocos2d::Size&) |
| initWithMipmaps(MipmapInfo*,int,Texture2D::PixelFormat,int,int) | 67 | 1 | 18 | 14 | 5 | 9 | 1 | cocos2d.Texture2D.initWithMipmaps(MipmapInfo*,int ,Texture2D::PixelFormat,int,int) |
| initWithImage(cocos2d::Image*,cocos2d::Texture2D::PixelFormat) | 34 | 0 | 9 | 5 | 2 | 11 | 2 | cocos2d.Texture2D.initWithImage(cocos2d::Image* ,cocos2d::Texture2D::PixelFormat) |
| initWithString(constchar*,conststd::string&,float,constcocos2d::Size& ,cocos2d::TextHAlignment,cocos2d::TextVAlignment) | 10 | 0 | 1 | 1 | 6 | 1 | 2 | cocos2d.Texture2D.initWithString(constchar*,conststd::string&,float ,constcocos2d::Size&,cocos2d::TextHAlignment,cocos2d::TextVAlignment) |
| initWithString(constchar*,constcocos2d::FontDefinition&) | 37 | 0 | 13 | 2 | 2 | 12 | 2 | cocos2d.Texture2D.initWithString(constchar* ,constcocos2d::FontDefinition&) |
| convertDataToFormat(constunsignedchar*,ssize_t ,cocos2d::Texture2D::PixelFormat,cocos2d::Texture2D::PixelFormat ,unsignedchar**,ssize_t*) | 18 | 0 | 7 | 2 | 6 | 0 | 1 | cocos2d.Texture2D.convertDataToFormat(constunsignedchar*,ssize_t ,cocos2d::Texture2D::PixelFormat,cocos2d::Texture2D::PixelFormat ,unsignedchar**,ssize_t*) |
| convertI8ToFormat(constunsignedchar*,ssize_t ,cocos2d::Texture2D::PixelFormat,unsignedchar**,ssize_t*) | 38 | 0 | 9 | 3 | 5 | 0 | 1 | cocos2d.Texture2D.convertI8ToFormat(constunsignedchar*,ssize_t ,cocos2d::Texture2D::PixelFormat,unsignedchar**,ssize_t*) |
Statistics
| Stat | # lines of code (LOC) | MaxNestedLoop | Cyclomatic Complexity (CC) | NestingDepth | # Parameters | # Variables | # Overloads |
|---|---|---|---|---|---|---|---|
| Sum: | 79 434 | 856 | 15 963 | 6 340 | 3 235 | 20 777 | 3 176 |
| Average: | 41.65 | 0.45 | 8.37 | 3.32 | 1.7 | 10.9 | 1.67 |
| Minimum: | 0 | 0 | 1 | 0 | 0 | 0 | 1 |
| Maximum: | 290 | 9 | 141 | 29 | 16 | 87 | 16 |
| Standard deviation: | 31.07 | 0.78 | 9.36 | 2.06 | 1.93 | 9.26 | 2.26 |
| Variance: | 965.3 | 0.61 | 87.52 | 4.23 | 3.72 | 85.69 | 5.09 |
| Rule warning: Methods too big |
• Rule Description:
This rule matches methods where NbLinesOfCode > 30 or (commented per default) NbILInstructions > 200. Such method can be hard to understand and maintain.
However rules like Methods too complex or Methods with too many variables might be more relevant to detect painful to maintain methods, because complexity is more related to numbers of if, switch case, loops… than to just number of lines.
See the definition of the NbLinesOfCode metric here http://www.cppdepend.com/Metrics.aspx#NbLinesOfCode
• How to Fix Violations: Usually too big methods should be split in smaller methods. But long methods with no branch conditions, that typically initialize some data, are not necessarily a problem to maintain nor to test, and might not need refactoring.
• How to Fix Violations: Usually too big methods should be split in smaller methods. But long methods with no branch conditions, that typically initialize some data, are not necessarily a problem to maintain nor to test, and might not need refactoring.
1,304 methods matched
| methods | # lines of code (LOC) | Full Name |
|---|---|---|
| edtaa3(double*,double*,double*,int,int,short*,short*,double*) | 290 | __Globals.edtaa3(double*,double*,double*,int,int,short*,short*,double* ) |
| spSkeletonJson_readSkeletonData(spSkeletonJson*,constchar*) | 273 | __Globals.spSkeletonJson_readSkeletonData(spSkeletonJson*,constchar*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 269 | cocos2d.PUEmitterTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| register_all_cocos2dx(lua_State*) | 256 | __Globals.register_all_cocos2dx(lua_State*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 255 | cocos2d.PURendererTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| _spSkeletonJson_readAnimation(spSkeletonJson*,Json*,spSkeletonData*) | 241 | __Globals._spSkeletonJson_readAnimation(spSkeletonJson*,Json* ,spSkeletonData*) |
| parse(PUConcreteNodeList&,constPUScriptTokenList&) | 239 | cocos2d.PUScriptParser.parse(PUConcreteNodeList& ,constPUScriptTokenList&) |
| Collide(b2Manifold*,constb2EdgeShape*,constb2Transform& ,constb2PolygonShape*,constb2Transform&) | 230 | b2EPCollider.Collide(b2Manifold*,constb2EdgeShape*,constb2Transform& ,constb2PolygonShape*,constb2Transform&) |
| startElement(void*,constchar*,constchar**) | 223 | cocos2d.TMXMapInfo.startElement(void*,constchar*,constchar**) |
| GenStruct(constflatbuffers::LanguageParameters& ,constflatbuffers::Parser&,flatbuffers::StructDef&,std::string*) | 216 | flatbuffers.__Globals.GenStruct(constflatbuffers::LanguageParameters& ,constflatbuffers::Parser&,flatbuffers::StructDef&,std::string*) |
| onEnter() | 211 | TemplateVectorTest.onEnter() |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 194 | cocostudio.CheckBoxReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 194 | cocostudio.SliderReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 193 | cocostudio.ButtonReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 184 | cocostudio.WidgetReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| parseProperties(cocos2d::Node*,cocos2d::Node*,cocosbuilder::CCBReader* ) | 183 | cocosbuilder.NodeLoader.parseProperties(cocos2d::Node*,cocos2d::Node* ,cocosbuilder::CCBReader*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 178 | cocostudio.NodeReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| generateTestFunctions() | 173 | TouchEventDispatchingPerfTest.generateTestFunctions() |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 170 | cocostudio.ButtonReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| Next() | 165 | flatbuffers.Parser.Next() |
| tolua_opengl_open(lua_State*) | 161 | __Globals.tolua_opengl_open(lua_State*) |
| createSlicedSprites() | 160 | cocos2d.ui.Scale9Sprite.createSlicedSprites() |
| serialize(void*) | 159 | cocostudio.ComRender.serialize(void*) |
| lua_cocos2dx_experimental_SpritePolygon_create(lua_State*) | 158 | __Globals.lua_cocos2dx_experimental_SpritePolygon_create(lua_State*) |
| caculateFinalPositionWithRelativeWidget(cocos2d::ui::LayoutProtocol*) | 157 | cocos2d.ui.RelativeLayoutManager .caculateFinalPositionWithRelativeWidget(cocos2d::ui::LayoutProtocol*) |
| SolveTOI(constb2TimeStep&) | 152 | b2World.SolveTOI(constb2TimeStep&) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 151 | cocostudio.ListViewReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 146 | cocostudio.ScrollViewReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| tolua_cocos2d_Mat4_decompose(lua_State*) | 144 | __Globals.tolua_cocos2d_Mat4_decompose(lua_State*) |
| scrollChildrenBoth(float,float) | 143 | cocos2d.ui.ScrollView.scrollChildrenBoth(float,float) |
| onEnter() | 142 | PhysicsDemoJoints.onEnter() |
| lua_cocos2dx_Texture2D_initWithString(lua_State*) | 142 | __Globals.lua_cocos2dx_Texture2D_initWithString(lua_State*) |
| updateIndicesLOD() | 141 | cocos2d.Terrain+Chunk.updateIndicesLOD() |
| lua_register_cocos2dx_Node(lua_State*) | 140 | __Globals.lua_register_cocos2dx_Node(lua_State*) |
| Car() | 139 | Car.Car() |
| onEnter() | 134 | TemplateMapTest.onEnter() |
| unz64local_GetCurrentFileInfoInternal(unzFile,unz_file_info64* ,unz_file_info64_internal*,char*,uLong,void*,uLong,char*,uLong) | 132 | cocos2d.__Globals.unz64local_GetCurrentFileInfoInternal(unzFile ,unz_file_info64*,unz_file_info64_internal*,char*,uLong,void*,uLong ,char*,uLong) |
| GenTable(constflatbuffers::Parser&,flatbuffers::StructDef& ,constflatbuffers::GeneratorOptions&,std::string*) | 131 | flatbuffers.cpp.__Globals.GenTable(constflatbuffers::Parser& ,flatbuffers::StructDef&,constflatbuffers::GeneratorOptions& ,std::string*) |
| RootTests() | 130 | RootTests.RootTests() |
| SpriteTests() | 129 | SpriteTests.SpriteTests() |
| CharacterCollision() | 127 | CharacterCollision.CharacterCollision() |
| LoadMtl(std::map<std::string,ObjLoader::material_t>&,constchar* ,constchar*) | 126 | cocos2d.__Globals.LoadMtl(std::map<std::string,ObjLoader::material_t>& ,constchar*,constchar*) |
| initWithDictionary(ValueMap&,conststd::string&) | 126 | cocos2d.ParticleSystem.initWithDictionary(ValueMap&,conststd::string&) |
| Texture2DTests() | 125 | Texture2DTests.Texture2DTests() |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 124 | cocostudio.CheckBoxReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 124 | cocostudio.SliderReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 123 | cocostudio.LayoutReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 123 | cocostudio.PageViewReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| lua_cocos2dx_RenderTexture_saveToFile(lua_State*) | 123 | __Globals.lua_cocos2dx_RenderTexture_saveToFile(lua_State*) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 120 | cocos2d.PUForceFieldAffectorTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| buildJson(rapidjson::Document&,cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*) | 120 | cocostudio.TriggerMng.buildJson(rapidjson::Document& ,cocostudio::CocoLoader*,cocostudio::stExpCocoNode*) |
| readDataTask() | 119 | cocos2d.experimental.AudioCache.readDataTask() |
| SolveVelocityConstraints() | 119 | b2ContactSolver.SolveVelocityConstraints() |
| Step(Settings*) | 119 | Test.Step(Settings*) |
| lua_cocos2dx_LabelTTF_create(lua_State*) | 119 | __Globals.lua_cocos2dx_LabelTTF_create(lua_State*) |
| Dominos() | 118 | Dominos.Dominos() |
| LoadObj(shapes_t&,constchar*,constchar*) | 115 | cocos2d.ObjLoader.LoadObj(shapes_t&,constchar*,constchar*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 114 | cocostudio.TextReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| register_all_cocos2dx_physics_manual(lua_State*) | 114 | __Globals.register_all_cocos2dx_physics_manual(lua_State*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 113 | cocos2d.PUMaterialPassTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| lua_cocos2dx_cocosbuilder_CCBReader_constructor(lua_State*) | 113 | __Globals.lua_cocos2dx_cocosbuilder_CCBReader_constructor(lua_State*) |
| openLexer(conststd::string&,conststd::string&,PUScriptTokenList&) | 111 | cocos2d.PUScriptLexer.openLexer(conststd::string&,conststd::string& ,PUScriptTokenList&) |
| tweenTo(float,cocos2d::tweenfunc::TweenType,float*) | 111 | cocos2d.tweenfunc.__Globals.tweenTo(float ,cocos2d::tweenfunc::TweenType,float*) |
| onSocketCallback(structlibwebsocket_context*,structlibwebsocket*,int ,void*,void*,ssize_t) | 111 | cocos2d.network.WebSocket.onSocketCallback(structlibwebsocket_context* ,structlibwebsocket*,int,void*,void*,ssize_t) |
| ParseNumber<parseFlags,Stream,Handler>(Stream&,Handler&) | 111 | rapidjson.GenericReader<Encoding,Allocator>.ParseNumber<parseFlags ,Stream,Handler>(Stream&,Handler&) |
| generateTestFunctions() | 111 | TemplateVectorPerfTest.generateTestFunctions() |
| test(float) | 111 | NodeNameTest.test(float) |
| lua_cocos2dx_ui_CheckBox_create(lua_State*) | 111 | __Globals.lua_cocos2dx_ui_CheckBox_create(lua_State*) |
| lua_cocos2dx_extension_ControlSlider_create(lua_State*) | 110 | __Globals.lua_cocos2dx_extension_ControlSlider_create(lua_State*) |
| initWithQuantityOfNodes(unsignedint) | 109 | PerformanceEventDispatcherScene.initWithQuantityOfNodes(unsignedint) |
| drawSkeleton(constcocos2d::Mat4&,uint32_t) | 108 | spine.SkeletonRenderer.drawSkeleton(constcocos2d::Mat4&,uint32_t) |
| Cantilever() | 106 | Cantilever.Cantilever() |
| DumpShell() | 106 | DumpShell.DumpShell() |
| decodeFrame(tinyxml2::XMLElement*,tinyxml2::XMLElement* ,cocostudio::BoneData*,DataInfo*) | 105 | cocostudio.DataReaderHelper.decodeFrame(tinyxml2::XMLElement* ,tinyxml2::XMLElement*,cocostudio::BoneData*,DataInfo*) |
| spAtlas_create(constchar*,int,constchar*,void*) | 105 | __Globals.spAtlas_create(constchar*,int,constchar*,void*) |
| rotateBufferThread(int) | 104 | cocos2d.experimental.AudioPlayer.rotateBufferThread(int) |
| getEasingAction(cocos2d::ActionInterval*) | 104 | cocostudio.ActionFrame.getEasingAction(cocos2d::ActionInterval*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 104 | cocostudio.ListViewReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| lua_register_cocos2dx_ParticleSystem(lua_State*) | 104 | __Globals.lua_register_cocos2dx_ParticleSystem(lua_State*) |
| render(cocos2d::Renderer*,constcocos2d::Mat4& ,cocos2d::ParticleSystem3D*) | 103 | cocos2d.PUParticle3DQuadRender.render(cocos2d::Renderer* ,constcocos2d::Mat4&,cocos2d::ParticleSystem3D*) |
| lua_cocos2dx_ui_EditBox_create(lua_State*) | 103 | __Globals.lua_cocos2dx_ui_EditBox_create(lua_State*) |
| lua_cocos2dx_LabelBMFont_create(lua_State*) | 103 | __Globals.lua_cocos2dx_LabelBMFont_create(lua_State*) |
| initWithPVRv3Data(constunsignedchar*,ssize_t) | 102 | cocos2d.Image.initWithPVRv3Data(constunsignedchar*,ssize_t) |
| initWithBinary(cocostudio::CocoLoader*,cocostudio::stExpCocoNode* ,cocos2d::Ref*) | 102 | cocostudio.ActionNode.initWithBinary(cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*,cocos2d::Ref*) |
| Solve(b2Profile*,constb2TimeStep&,constb2Vec2&,bool) | 102 | b2Island.Solve(b2Profile*,constb2TimeStep&,constb2Vec2&,bool) |
| ActionsTests() | 101 | ActionsTests.ActionsTests() |
| createOptionsWithFlatBuffersForNode(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 100 | cocostudio.Node3DReader.createOptionsWithFlatBuffersForNode (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| initWithQuantityOfNodes(unsignedint) | 100 | PerformanceContainerScene.initWithQuantityOfNodes(unsignedint) |
| readNodeGraph(cocos2d::Node*) | 99 | cocosbuilder.CCBReader.readNodeGraph(cocos2d::Node*) |
| Web() | 99 | Web.Web() |
| ParticleTests() | 98 | ParticleTests.ParticleTests() |
| lua_cocos2dx_RenderTexture_beginWithClear(lua_State*) | 98 | __Globals.lua_cocos2dx_RenderTexture_beginWithClear(lua_State*) |
| dispatchTouchEvent(cocos2d::EventTouch*) | 97 | cocos2d.EventDispatcher.dispatchTouchEvent(cocos2d::EventTouch*) |
| createStringSprites(cocos2d::Label*) | 97 | cocos2d.LabelTextFormatter.createStringSprites(cocos2d::Label*) |
| groupBatchDownload(constDownloadUnits&) | 97 | cocos2d.extension.Downloader.groupBatchDownload(constDownloadUnits&) |
| generateTestFunctions() | 97 | ArrayPerfTest.generateTestFunctions() |
| lua_cocos2dx_ui_Scale9Sprite_init(lua_State*) | 97 | __Globals.lua_cocos2dx_ui_Scale9Sprite_init(lua_State*) |
| b2TimeOfImpact(b2TOIOutput*,constb2TOIInput*) | 96 | __Globals.b2TimeOfImpact(b2TOIOutput*,constb2TOIInput*) |
| parseNodesRecursivelyBinary(bool&,bool) | 94 | cocos2d.Bundle3D.parseNodesRecursivelyBinary(bool&,bool) |
| applicationDidFinishLaunching() | 94 | AppDelegate.applicationDidFinishLaunching() |
Statistics
| Stat | # lines of code (LOC) |
|---|---|
| Sum: | 69 561 |
| Average: | 53.34 |
| Minimum: | 31 |
| Maximum: | 290 |
| Standard deviation: | 30.68 |
| Variance: | 941.23 |
| Rule warning: Methods too complex |
• Rule Description:
This rule matches methods where CyclomaticComplexity > 20 or ILCyclomaticComplexity > 40 or ILNestingDepth > 4. Such method is typically hard to understand and maintain.
See the definition of the CyclomaticComplexity metric here: http://www.cppdepend.com/Metrics.aspx#CC
See the definition of the NestingDepth metric here: http://www.cppdepend.com/Metrics.aspx#NestingDepth
• How to Fix Violations: A large and complex method should be split in smaller methods, or even one or several classes can be created for that. During this process it is important to question the scope of each variable local to the method. This can be an indication if such local variable will become an instance field of the newly created class(es). Large switch…case structures might be refactored through the help of a set of types that implement a common interface, the interface polymorphism playing the role of the switch cases tests. Unit Tests can help: write tests for each method before extracting it to ensure you don't break functionality.
• How to Fix Violations: A large and complex method should be split in smaller methods, or even one or several classes can be created for that. During this process it is important to question the scope of each variable local to the method. This can be an indication if such local variable will become an instance field of the newly created class(es). Large switch…case structures might be refactored through the help of a set of types that implement a common interface, the interface polymorphism playing the role of the switch cases tests. Unit Tests can help: write tests for each method before extracting it to ensure you don't break functionality.
269 methods matched
| methods | Cyclomatic Complexity (CC) | MaxNestedLoop | NestingDepth | Full Name |
|---|---|---|---|---|
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 141 | 1 | 9 | cocos2d.PURendererTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 123 | 1 | 7 | cocos2d.PUEmitterTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| _spSkeletonJson_readAnimation(spSkeletonJson*,Json*,spSkeletonData*) | 80 | 9 | 11 | __Globals._spSkeletonJson_readAnimation(spSkeletonJson*,Json* ,spSkeletonData*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 66 | 2 | 7 | cocostudio.WidgetReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 66 | 2 | 5 | cocostudio.ButtonReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| parseProperties(cocos2d::Node*,cocos2d::Node*,cocosbuilder::CCBReader* ) | 66 | 1 | 6 | cocosbuilder.NodeLoader.parseProperties(cocos2d::Node*,cocos2d::Node* ,cocosbuilder::CCBReader*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 64 | 2 | 7 | cocostudio.NodeReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| Next() | 60 | 2 | 7 | flatbuffers.Parser.Next() |
| spSkeletonJson_readSkeletonData(spSkeletonJson*,constchar*) | 56 | 5 | 9 | __Globals.spSkeletonJson_readSkeletonData(spSkeletonJson*,constchar*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 55 | 2 | 22 | cocos2d.PUMaterialPassTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| unz64local_GetCurrentFileInfoInternal(unzFile,unz_file_info64* ,unz_file_info64_internal*,char*,uLong,void*,uLong,char*,uLong) | 55 | 1 | 13 | cocos2d.__Globals.unz64local_GetCurrentFileInfoInternal(unzFile ,unz_file_info64*,unz_file_info64_internal*,char*,uLong,void*,uLong ,char*,uLong) |
| GenStruct(constflatbuffers::LanguageParameters& ,constflatbuffers::Parser&,flatbuffers::StructDef&,std::string*) | 52 | 2 | 8 | flatbuffers.__Globals.GenStruct(constflatbuffers::LanguageParameters& ,constflatbuffers::Parser&,flatbuffers::StructDef&,std::string*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 52 | 2 | 5 | cocostudio.ScrollViewReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 52 | 2 | 5 | cocostudio.ListViewReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| SolveTOI(constb2TimeStep&) | 48 | 3 | 6 | b2World.SolveTOI(constb2TimeStep&) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 48 | 1 | 13 | cocos2d.PUTechniqueTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| startElement(void*,constchar*,constchar**) | 48 | 1 | 10 | cocos2d.TMXMapInfo.startElement(void*,constchar*,constchar**) |
| caculateFinalPositionWithRelativeWidget(cocos2d::ui::LayoutProtocol*) | 48 | 0 | 4 | cocos2d.ui.RelativeLayoutManager .caculateFinalPositionWithRelativeWidget(cocos2d::ui::LayoutProtocol*) |
| parse(PUConcreteNodeList&,constPUScriptTokenList&) | 47 | 2 | 6 | cocos2d.PUScriptParser.parse(PUConcreteNodeList& ,constPUScriptTokenList&) |
| LoadMtl(std::map<std::string,ObjLoader::material_t>&,constchar* ,constchar*) | 46 | 1 | 3 | cocos2d.__Globals.LoadMtl(std::map<std::string,ObjLoader::material_t>& ,constchar*,constchar*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 45 | 2 | 5 | cocostudio.TextReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 45 | 0 | 5 | cocos2d.PUForceFieldAffectorTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| wstrerror(int) | 44 | 0 | 2 | __Globals.wstrerror(int) |
| serialize(void*) | 42 | 2 | 16 | cocostudio.ComRender.serialize(void*) |
| edtaa3(double*,double*,double*,int,int,short*,short*,double*) | 42 | 2 | 5 | __Globals.edtaa3(double*,double*,double*,int,int,short*,short*,double* ) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 42 | 2 | 5 | cocostudio.LayoutReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 42 | 2 | 5 | cocostudio.PageViewReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 42 | 1 | 8 | cocos2d.PUDynamicAttributeTranslator.translate (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| ParseNumber<parseFlags,Stream,Handler>(Stream&,Handler&) | 42 | 1 | 5 | rapidjson.GenericReader<Encoding,Allocator>.ParseNumber<parseFlags ,Stream,Handler>(Stream&,Handler&) |
| scrollChildrenBoth(float,float) | 42 | 0 | 4 | cocos2d.ui.ScrollView.scrollChildrenBoth(float,float) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 41 | 0 | 6 | cocostudio.CheckBoxReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 41 | 0 | 6 | cocostudio.SliderReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| Collide(b2Manifold*,constb2EdgeShape*,constb2Transform& ,constb2PolygonShape*,constb2Transform&) | 38 | 1 | 4 | b2EPCollider.Collide(b2Manifold*,constb2EdgeShape*,constb2Transform& ,constb2PolygonShape*,constb2Transform&) |
| Solve(constb2TimeStep&) | 37 | 3 | 5 | b2World.Solve(constb2TimeStep&) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 37 | 2 | 5 | cocostudio.CheckBoxReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 37 | 0 | 4 | cocos2d.PUDoPlacementParticleEventHandlerTranslator .translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| tweenTo(float,cocos2d::tweenfunc::TweenType,float*) | 37 | 0 | 4 | cocos2d.tweenfunc.__Globals.tweenTo(float ,cocos2d::tweenfunc::TweenType,float*) |
| GenTable(constflatbuffers::Parser&,flatbuffers::StructDef& ,constflatbuffers::GeneratorOptions&,std::string*) | 36 | 2 | 7 | flatbuffers.cpp.__Globals.GenTable(constflatbuffers::Parser& ,flatbuffers::StructDef&,constflatbuffers::GeneratorOptions& ,std::string*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 36 | 2 | 5 | cocostudio.SliderReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| dispatchTouchEvent(cocos2d::EventTouch*) | 36 | 1 | 7 | cocos2d.EventDispatcher.dispatchTouchEvent(cocos2d::EventTouch*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 36 | 0 | 6 | cocostudio.ButtonReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| readDataTask() | 35 | 1 | 6 | cocos2d.experimental.AudioCache.readDataTask() |
| openLexer(conststd::string&,conststd::string&,PUScriptTokenList&) | 35 | 1 | 5 | cocos2d.PUScriptLexer.openLexer(conststd::string&,conststd::string& ,PUScriptTokenList&) |
| lua_cocos2dx_experimental_SpritePolygon_create(lua_State*) | 35 | 0 | 4 | __Globals.lua_cocos2dx_experimental_SpritePolygon_create(lua_State*) |
| refreshLayout() | 35 | 0 | 3 | cocos2d.ui.LayoutComponent.refreshLayout() |
| getEasingAction(cocos2d::ActionInterval*) | 35 | 0 | 3 | cocostudio.ActionFrame.getEasingAction(cocos2d::ActionInterval*) |
| LoadObj(shapes_t&,constchar*,constchar*) | 34 | 2 | 4 | cocos2d.ObjLoader.LoadObj(shapes_t&,constchar*,constchar*) |
| Parse(constchar*,constchar**,constchar*) | 33 | 2 | 5 | flatbuffers.Parser.Parse(constchar*,constchar**,constchar*) |
| buildJson(rapidjson::Document&,cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*) | 32 | 6 | 12 | cocostudio.TriggerMng.buildJson(rapidjson::Document& ,cocostudio::CocoLoader*,cocostudio::stExpCocoNode*) |
| rotateBufferThread(int) | 32 | 2 | 6 | cocos2d.experimental.AudioPlayer.rotateBufferThread(int) |
| decodeFrame(tinyxml2::XMLElement*,tinyxml2::XMLElement* ,cocostudio::BoneData*,DataInfo*) | 32 | 0 | 10 | cocostudio.DataReaderHelper.decodeFrame(tinyxml2::XMLElement* ,tinyxml2::XMLElement*,cocostudio::BoneData*,DataInfo*) |
| readNodeGraph(cocos2d::Node*) | 31 | 3 | 7 | cocosbuilder.CCBReader.readNodeGraph(cocos2d::Node*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 31 | 2 | 7 | cocos2d.PUAffectorTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| checkCustomScrollDestination(float*,float*) | 31 | 0 | 5 | cocos2d.ui.ScrollView.checkCustomScrollDestination(float*,float*) |
| unzOpenInternal(constvoid*,zlib_filefunc64_32_def*,int) | 31 | 0 | 4 | cocos2d.__Globals.unzOpenInternal(constvoid*,zlib_filefunc64_32_def* ,int) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 31 | 0 | 4 | cocos2d.PURandomiserTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 29 | 1 | 8 | cocos2d.PUObserverTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 29 | 0 | 4 | cocos2d.PUCircleEmitterTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| lua_cocos2dx_Texture2D_initWithString(lua_State*) | 29 | 0 | 4 | __Globals.lua_cocos2dx_Texture2D_initWithString(lua_State*) |
| loadDefaultGLProgram(cocos2d::GLProgram*,int) | 29 | 0 | 3 | cocos2d.GLProgramCache.loadDefaultGLProgram(cocos2d::GLProgram*,int) |
| updateIndicesLOD() | 28 | 2 | 4 | cocos2d.Terrain+Chunk.updateIndicesLOD() |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 28 | 0 | 5 | cocos2d.PUTextureAnimatorTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| tolua_cocos2d_Layer_registerScriptTouchHandler(lua_State*) | 28 | 0 | 4 | __Globals.tolua_cocos2d_Layer_registerScriptTouchHandler(lua_State*) |
| handleEvent(ScriptHandlerMgr::HandlerType,void*) | 28 | 0 | 3 | cocos2d.LuaEngine.handleEvent(ScriptHandlerMgr::HandlerType,void*) |
| loadAnimationDataBinary(conststd::string&,cocos2d::Animation3DData*) | 27 | 3 | 6 | cocos2d.Bundle3D.loadAnimationDataBinary(conststd::string& ,cocos2d::Animation3DData*) |
| spAtlas_create(constchar*,int,constchar*,void*) | 27 | 1 | 6 | __Globals.spAtlas_create(constchar*,int,constchar*,void*) |
| setPropsFromBinary(cocos2d::ui::Widget*,cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*) | 27 | 1 | 4 | cocostudio.LayoutReader.setPropsFromBinary(cocos2d::ui::Widget* ,cocostudio::CocoLoader*,cocostudio::stExpCocoNode*) |
| deallocate(void*,size_t) | 27 | 0 | 4 | cocos2d.allocator.AllocatorStrategyGlobalSmallBlock.deallocate(void* ,size_t) |
| getTranslator(conststd::string&) | 27 | 0 | 2 | cocos2d.PUAffectorManager.getTranslator(conststd::string&) |
| createAffector(conststd::string&) | 27 | 0 | 2 | cocos2d.PUAffectorManager.createAffector(conststd::string&) |
| main(int,constchar**) | 26 | 2 | 6 | __Globals.main(int,constchar**) |
| onSocketCallback(structlibwebsocket_context*,structlibwebsocket*,int ,void*,void*,ssize_t) | 26 | 1 | 10 | cocos2d.network.WebSocket.onSocketCallback(structlibwebsocket_context* ,structlibwebsocket*,int,void*,void*,ssize_t) |
| createStringSprites(cocos2d::Label*) | 26 | 1 | 5 | cocos2d.LabelTextFormatter.createStringSprites(cocos2d::Label*) |
| parse_string(Json*,constchar*) | 26 | 1 | 5 | __Globals.parse_string(Json*,constchar*) |
| lua_cocos2dx_LabelTTF_create(lua_State*) | 26 | 0 | 4 | __Globals.lua_cocos2dx_LabelTTF_create(lua_State*) |
| addDataFromBinaryCache(constchar*,DataInfo*) | 25 | 2 | 7 | cocostudio.DataReaderHelper.addDataFromBinaryCache(constchar* ,DataInfo*) |
| decodeFrame(cocostudio::CocoLoader*,cocostudio::stExpCocoNode* ,DataInfo*) | 25 | 2 | 6 | cocostudio.DataReaderHelper.decodeFrame(cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*,DataInfo*) |
| createOptionsWithFlatBuffersForNode(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 25 | 2 | 5 | cocostudio.Node3DReader.createOptionsWithFlatBuffersForNode (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| initWithPVRv3Data(constunsignedchar*,ssize_t) | 25 | 1 | 5 | cocos2d.Image.initWithPVRv3Data(constunsignedchar*,ssize_t) |
| lua_cocos2dx_RenderTexture_saveToFile(lua_State*) | 25 | 0 | 6 | __Globals.lua_cocos2dx_RenderTexture_saveToFile(lua_State*) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 25 | 0 | 4 | cocos2d.PULineAffectorTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 25 | 0 | 4 | cocos2d.PULineEmitterTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| spAnimationState_apply(spAnimationState*,structspSkeleton*) | 24 | 2 | 5 | __Globals.spAnimationState_apply(spAnimationState*,structspSkeleton*) |
| unzReadCurrentFile(unzFile,voidp,unsignedint) | 24 | 2 | 4 | cocos2d.__Globals.unzReadCurrentFile(unzFile,voidp,unsignedint) |
| easeValue(float) | 24 | 0 | 6 | cocostudio.ActionFrameEasing.easeValue(float) |
| lua_cocos2dx_ui_CheckBox_create(lua_State*) | 24 | 0 | 4 | __Globals.lua_cocos2dx_ui_CheckBox_create(lua_State*) |
| caculateFinalPositionWithRelativeAlign() | 24 | 0 | 2 | cocos2d.ui.RelativeLayoutManager .caculateFinalPositionWithRelativeAlign() |
| unz64local_SearchCentralDir64(constzlib_filefunc64_32_def*,voidpf) | 23 | 2 | 3 | cocos2d.__Globals.unz64local_SearchCentralDir64 (constzlib_filefunc64_32_def*,voidpf) |
| handle(cocos2d::PUParticleSystem3D*,cocos2d::PUParticle3D*,float) | 23 | 0 | 8 | cocos2d.PUDoEnableComponentEventHandler.handle (cocos2d::PUParticleSystem3D*,cocos2d::PUParticle3D*,float) |
| findNextFocusedWidget(cocos2d::ui::Widget::FocusDirection ,cocos2d::ui::Widget*) | 23 | 0 | 7 | cocos2d.ui.Layout.findNextFocusedWidget (cocos2d::ui::Widget::FocusDirection,cocos2d::ui::Widget*) |
| initWithDictionary(ValueMap&,conststd::string&) | 23 | 0 | 6 | cocos2d.ParticleSystem.initWithDictionary(ValueMap&,conststd::string&) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 23 | 0 | 5 | cocos2d.PUBaseColliderTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| onChangeLayoutComponent(cocos2d::Ref*) | 23 | 0 | 3 | UILayoutComponentTest_Editor.onChangeLayoutComponent(cocos2d::Ref*) |
| tolua_cocos2d_Mat4_decompose(lua_State*) | 23 | 0 | 3 | __Globals.tolua_cocos2d_Mat4_decompose(lua_State*) |
| createEffect(int,float) | 23 | 0 | 2 | __Globals.createEffect(int,float) |
| parseNodesRecursivelyBinary(bool&,bool) | 22 | 3 | 6 | cocos2d.Bundle3D.parseNodesRecursivelyBinary(bool&,bool) |
| initWithPngData(constunsignedchar*,ssize_t) | 22 | 1 | 4 | cocos2d.Image.initWithPngData(constunsignedchar*,ssize_t) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 22 | 0 | 5 | cocos2d.PUOnPositionObserverTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| translateChildProperty(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) | 22 | 0 | 4 | cocos2d.PUBoxColliderTranslator.translateChildProperty (cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) |
| marchSquare(int,int) | 22 | 0 | 4 | cocos2d.MarchingSquare.marchSquare(int,int) |
Statistics
| Stat | Cyclomatic Complexity (CC) | MaxNestedLoop | NestingDepth |
|---|---|---|---|
| Sum: | 5 896 | 265 | 1 775 |
| Average: | 21.92 | 0.99 | 6.6 |
| Minimum: | 2 | 0 | 2 |
| Maximum: | 141 | 9 | 29 |
| Standard deviation: | 16.65 | 1.25 | 2.86 |
| Variance: | 277.16 | 1.57 | 8.2 |
| Rule warning: Methods potentially poorly commented |
• Rule Description:
This rule matches methods with less than 20% of comment lines and that have at least 20 lines of code. Such method might need to be more commented.
See the definitions of the Comments metric here: http://www.cppdepend.com/Metrics.aspx#PercentageComment http://www.cppdepend.com/Metrics.aspx#NbLinesOfComment
• How to Fix Violations: Typically add more comment. But code commenting is subject to controversy. While poorly written and designed code would needs a lot of comment to be understood, clean code doesn't need that much comment, especially if variables and methods are properly named and convey enough information. Unit-Test code can also play the role of code commenting. However, even when writing clean and well-tested code, one will have to write hacks at a point, usually to circumvent some API limitations or bugs. A hack is a non-trivial piece of code, that doesn't make sense at first glance, and that took time and web research to be found. In such situation comments must absolutely be used to express the intention, the need for the hacks and the source where the solution has been found.
• How to Fix Violations: Typically add more comment. But code commenting is subject to controversy. While poorly written and designed code would needs a lot of comment to be understood, clean code doesn't need that much comment, especially if variables and methods are properly named and convey enough information. Unit-Test code can also play the role of code commenting. However, even when writing clean and well-tested code, one will have to write hacks at a point, usually to circumvent some API limitations or bugs. A hack is a non-trivial piece of code, that doesn't make sense at first glance, and that took time and web research to be found. In such situation comments must absolutely be used to express the intention, the need for the hacks and the source where the solution has been found.
4,445 methods matched
| methods | Percentage Comment | # lines of code (LOC) | # lines of comment | Full Name |
|---|---|---|---|---|
| XXH32_intermediateDigest_endian(void*,XXH_endianess) | 0 | 23 | 0 | __Globals.XXH32_intermediateDigest_endian(void*,XXH_endianess) |
| decode_subblock(etc1_byte*,int,int,int,constint*,etc1_uint32,bool,bool ) | 0 | 22 | 0 | __Globals.decode_subblock(etc1_byte*,int,int,int,constint*,etc1_uint32 ,bool,bool) |
| etc_average_colors_subblock(constetc1_byte*,etc1_uint32,etc1_byte* ,bool,bool) | 0 | 32 | 0 | __Globals.etc_average_colors_subblock(constetc1_byte*,etc1_uint32 ,etc1_byte*,bool,bool) |
| chooseModifier(constetc1_byte*,constetc1_byte*,etc1_uint32*,int ,constint*) | 0 | 26 | 0 | __Globals.chooseModifier(constetc1_byte*,constetc1_byte*,etc1_uint32* ,int,constint*) |
| etc_encode_subblock_helper(constetc1_byte*,etc1_uint32,etc_compressed* ,bool,bool,constetc1_byte*,constint*) | 0 | 22 | 0 | __Globals.etc_encode_subblock_helper(constetc1_byte*,etc1_uint32 ,etc_compressed*,bool,bool,constetc1_byte*,constint*) |
| etc_encode_block_helper(constetc1_byte*,etc1_uint32,constetc1_byte* ,etc_compressed*,bool) | 0 | 26 | 0 | __Globals.etc_encode_block_helper(constetc1_byte*,etc1_uint32 ,constetc1_byte*,etc_compressed*,bool) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 0 | 34 | 0 | ArmatureNodeReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 0 | 25 | 0 | ArmatureNodeReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| unz64local_getLong64(constzlib_filefunc64_32_def*,voidpf,ZPOS64_T*) | 0 | 31 | 0 | cocos2d.__Globals.unz64local_getLong64(constzlib_filefunc64_32_def* ,voidpf,ZPOS64_T*) |
| _getEigenVectors(cocos2d::Mat4*,cocos2d::Vec3*,cocos2d::Mat4) | 0 | 70 | 0 | cocos2d.__Globals._getEigenVectors(cocos2d::Mat4*,cocos2d::Vec3* ,cocos2d::Mat4) |
| _base64Decode(constunsignedchar*,unsignedint,unsignedchar* ,unsignedint*) | 0 | 44 | 0 | cocos2d.__Globals._base64Decode(constunsignedchar*,unsignedint ,unsignedchar*,unsignedint*) |
| _base64Encode(constunsignedchar*,unsignedint,char*) | 0 | 29 | 0 | cocos2d.__Globals._base64Encode(constunsignedchar*,unsignedint,char*) |
| visit(constcocos2d::Value&,int) | 0 | 24 | 0 | cocos2d.__Globals.visit(constcocos2d::Value&,int) |
| ccdictionary_to_valuemap(cocos2d::__Dictionary*) | 0 | 32 | 0 | cocos2d.__Globals.ccdictionary_to_valuemap(cocos2d::__Dictionary*) |
| ccarray_to_valuevector(cocos2d::__Array*) | 0 | 29 | 0 | cocos2d.__Globals.ccarray_to_valuevector(cocos2d::__Array*) |
| parseCommand(int) | 0 | 50 | 0 | cocos2d.Console.parseCommand(int) |
| commandProjection(int,conststd::string&) | 0 | 27 | 0 | cocos2d.Console.commandProjection(int,conststd::string&) |
| commandTouch(int,conststd::string&) | 0 | 70 | 0 | cocos2d.Console.commandTouch(int,conststd::string&) |
| processParentFlags(constcocos2d::Mat4&,uint32_t) | 0 | 21 | 0 | cocos2d.Node.processParentFlags(constcocos2d::Mat4&,uint32_t) |
| startWithTarget(cocos2d::Node*) | 0 | 21 | 0 | cocos2d.SkewTo.startWithTarget(cocos2d::Node*) |
| update(float) | 0 | 53 | 0 | cocos2d.Animate3D.update(float) |
| init() | 0 | 24 | 0 | cocos2d.EventListenerMouse.init() |
| update(float) | 0 | 21 | 0 | cocos2d.Lens3D.update(float) |
| alignText() | 0 | 37 | 0 | cocos2d.Label.alignText() |
| updateShaderProgram() | 0 | 21 | 0 | cocos2d.Label.updateShaderProgram() |
| createShadowSpriteForSystemFont() | 0 | 23 | 0 | cocos2d.Label.createShadowSpriteForSystemFont() |
| reset() | 0 | 23 | 0 | cocos2d.Label.reset() |
| setTextureCoords(cocos2d::Rect) | 0 | 41 | 0 | cocos2d.Sprite.setTextureCoords(cocos2d::Rect) |
| saveImageToPNG(conststd::string&,bool) | 0 | 77 | 0 | cocos2d.Image.saveImageToPNG(conststd::string&,bool) |
| calculateVertexPoints() | 0 | 50 | 0 | cocos2d.Grid3D.calculateVertexPoints() |
| calculateVertexPoints() | 0 | 58 | 0 | cocos2d.TiledGrid3D.calculateVertexPoints() |
| loadDefaultGLProgram(cocos2d::GLProgram*,int) | 0 | 92 | 0 | cocos2d.GLProgramCache.loadDefaultGLProgram(cocos2d::GLProgram*,int) |
| PhysicsBody() | 0 | 21 | 0 | cocos2d.PhysicsBody.PhysicsBody() |
| onEvent(cocos2d::EventCustom*) | 0 | 30 | 0 | cocos2d.EventListenerPhysicsContact.onEvent(cocos2d::EventCustom*) |
| update(float,bool) | 0 | 26 | 0 | cocos2d.PhysicsWorld.update(float,bool) |
| drawShape(cocos2d::PhysicsShape&) | 0 | 30 | 0 | cocos2d.PhysicsDebugDraw.drawShape(cocos2d::PhysicsShape&) |
| drawJoint(cocos2d::PhysicsJoint&) | 0 | 41 | 0 | cocos2d.PhysicsDebugDraw.drawJoint(cocos2d::PhysicsJoint&) |
| getCurrentLanguage() | 0 | 62 | 0 | cocos2d.Application.getCurrentLanguage() |
| parseVersion1(constValueMap&) | 0 | 25 | 0 | cocos2d.AnimationCache.parseVersion1(constValueMap&) |
| getVertexZForPos(constcocos2d::Vec2&) | 0 | 23 | 0 | cocos2d.TMXLayer.getVertexZForPos(constcocos2d::Vec2&) |
| updateAtlasValueAt(constcocos2d::Vec2&,constcocos2d::Color3B&,int) | 0 | 43 | 0 | cocos2d.TileMapAtlas.updateAtlasValueAt(constcocos2d::Vec2& ,constcocos2d::Color3B&,int) |
| onDraw(constcocos2d::Mat4&,uint32_t) | 0 | 42 | 0 | cocos2d.Skybox.onDraw(constcocos2d::Mat4&,uint32_t) |
| render(cocos2d::Renderer*,constcocos2d::Mat4& ,cocos2d::ParticleSystem3D*) | 0 | 35 | 0 | cocos2d.Particle3DModelRender.render(cocos2d::Renderer* ,constcocos2d::Mat4&,cocos2d::ParticleSystem3D*) |
| copyAttributesTo(cocos2d::PUParticleSystem3D*) | 0 | 49 | 0 | cocos2d.PUParticleSystem3D.copyAttributesTo (cocos2d::PUParticleSystem3D*) |
| parseChunk(PUConcreteNodeList&,constPUScriptTokenList&) | 0 | 33 | 0 | cocos2d.PUScriptParser.parseChunk(PUConcreteNodeList& ,constPUScriptTokenList&) |
| createAffector(conststd::string&) | 0 | 53 | 0 | cocos2d.PUAffectorManager.createAffector(conststd::string&) |
| initRender(conststd::string&) | 0 | 21 | 0 | cocos2d.PUParticle3DEntityRender.initRender(conststd::string&) |
| getOriginOffset(int&,int&) | 0 | 37 | 0 | cocos2d.PUParticle3DQuadRender.getOriginOffset(int&,int&) |
| render(cocos2d::Renderer*,constcocos2d::Mat4& ,cocos2d::ParticleSystem3D*) | 0 | 41 | 0 | cocos2d.PUParticle3DModelRender.render(cocos2d::Renderer* ,constcocos2d::Mat4&,cocos2d::ParticleSystem3D*) |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 0 | 113 | 1 | cocos2d.PUMaterialPassTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| createObserver(conststd::string&) | 0 | 23 | 0 | cocos2d.PUObserverManager.createObserver(conststd::string&) |
| loadAnimationDataJson(conststd::string&,cocos2d::Animation3DData*) | 0 | 49 | 0 | cocos2d.Bundle3D.loadAnimationDataJson(conststd::string& ,cocos2d::Animation3DData*) |
| getFontAtlasTTF(constTTFConfig&) | 0 | 24 | 0 | cocos2d.FontAtlasCache.getFontAtlasTTF(constTTFConfig&) |
| getGlyphBitmap(unsignedshort,long&,long&,cocos2d::Rect&,int&) | 0 | 69 | 0 | cocos2d.FontFreeType.getGlyphBitmap(unsignedshort,long&,long& ,cocos2d::Rect&,int&) |
| getGlyphBitmapWithOutline(unsignedshort,FT_BBox&) | 0 | 30 | 0 | cocos2d.FontFreeType.getGlyphBitmapWithOutline(unsignedshort,FT_BBox&) |
| CSLoader() | 0 | 28 | 0 | cocos2d.CSLoader.CSLoader() |
| init() | 0 | 25 | 0 | cocos2d.CSLoader.init() |
| initNode(cocos2d::Node*,constrapidjson::Value&) | 0 | 51 | 0 | cocos2d.CSLoader.initNode(cocos2d::Node*,constrapidjson::Value&) |
| endElement(void*,constchar*) | 0 | 40 | 0 | cocos2d.DictMaker.endElement(void*,constchar*) |
| createFontAtlas() | 0 | 25 | 0 | cocos2d.FontCharMap.createFontAtlas() |
| DrawShape(cpShape*,cocos2d::DrawNode*) | 0 | 24 | 0 | cocos2d.extension.__Globals.DrawShape(cpShape*,cocos2d::DrawNode*) |
| __indexFromOffset(cocos2d::Vec2) | 0 | 23 | 0 | cocos2d.extension.TableView.__indexFromOffset(cocos2d::Vec2) |
| rotateBufferThread(int) | 0 | 104 | 1 | cocos2d.experimental.AudioPlayer.rotateBufferThread(int) |
| play2d(cocos2d::experimental::AudioCache*) | 0 | 33 | 0 | cocos2d.experimental.AudioPlayer.play2d (cocos2d::experimental::AudioCache*) |
| play2d(conststd::string&,bool,float) | 0 | 50 | 0 | cocos2d.experimental.AudioEngineImpl.play2d(conststd::string&,bool ,float) |
| update(float) | 0 | 38 | 0 | cocos2d.experimental.AudioEngineImpl.update(float) |
| cleanupSlicedSprites() | 0 | 35 | 0 | cocos2d.ui.Scale9Sprite.cleanupSlicedSprites() |
| copyProperties(cocos2d::ui::Widget*) | 0 | 37 | 0 | cocos2d.ui.Widget.copyProperties(cocos2d::ui::Widget*) |
| getClippingRect() | 0 | 48 | 0 | cocos2d.ui.Layout.getClippingRect() |
| onPressStateChangedToPressed() | 0 | 25 | 0 | cocos2d.ui.Button.onPressStateChangedToPressed() |
| normalTextureScaleChangedWithSize() | 0 | 21 | 0 | cocos2d.ui.Button.normalTextureScaleChangedWithSize() |
| pressedTextureScaleChangedWithSize() | 0 | 21 | 0 | cocos2d.ui.Button.pressedTextureScaleChangedWithSize() |
| copySpecialProperties(cocos2d::ui::Widget*) | 0 | 23 | 0 | cocos2d.ui.Button.copySpecialProperties(cocos2d::ui::Widget*) |
| autoScrollChildren(float) | 0 | 28 | 0 | cocos2d.ui.ScrollView.autoScrollChildren(float) |
| checkBounceBoundary() | 0 | 24 | 0 | cocos2d.ui.ScrollView.checkBounceBoundary() |
| checkNeedBounce() | 0 | 46 | 0 | cocos2d.ui.ScrollView.checkNeedBounce() |
| scrollChildrenVertical(float,float) | 0 | 26 | 0 | cocos2d.ui.ScrollView.scrollChildrenVertical(float,float) |
| scrollChildrenHorizontal(float,float) | 0 | 26 | 0 | cocos2d.ui.ScrollView.scrollChildrenHorizontal(float,float) |
| endRecordSlidAction() | 0 | 32 | 0 | cocos2d.ui.ScrollView.endRecordSlidAction() |
| interceptTouchEvent(Widget::TouchEventType,cocos2d::ui::Widget* ,cocos2d::Touch*) | 0 | 21 | 0 | cocos2d.ui.ScrollView.interceptTouchEvent(Widget::TouchEventType ,cocos2d::ui::Widget*,cocos2d::Touch*) |
| updateInnerContainerSize() | 0 | 21 | 0 | cocos2d.ui.ListView.updateInnerContainerSize() |
| barRendererScaleChangedWithSize() | 0 | 22 | 0 | cocos2d.ui.Slider.barRendererScaleChangedWithSize() |
| progressBarRendererScaleChangedWithSize() | 0 | 24 | 0 | cocos2d.ui.Slider.progressBarRendererScaleChangedWithSize() |
| autoScroll(float) | 0 | 28 | 0 | cocos2d.ui.PageView.autoScroll(float) |
| handleReleaseLogic(cocos2d::Touch*) | 0 | 23 | 0 | cocos2d.ui.PageView.handleReleaseLogic(cocos2d::Touch*) |
| interceptTouchEvent(cocos2d::ui::Widget::TouchEventType ,cocos2d::ui::Widget*,cocos2d::Touch*) | 0 | 21 | 0 | cocos2d.ui.PageView.interceptTouchEvent (cocos2d::ui::Widget::TouchEventType,cocos2d::ui::Widget* ,cocos2d::Touch*) |
| handleTextRenderer(conststd::string&,conststd::string&,float ,constcocos2d::Color3B&,GLubyte) | 0 | 31 | 0 | cocos2d.ui.RichText.handleTextRenderer(conststd::string& ,conststd::string&,float,constcocos2d::Color3B&,GLubyte) |
| formarRenderers() | 0 | 52 | 0 | cocos2d.ui.RichText.formarRenderers() |
| openKeyboard() | 0 | 34 | 0 | cocos2d.ui.EditBoxImplWin.openKeyboard() |
| doLayout(cocos2d::ui::LayoutProtocol*) | 0 | 30 | 0 | cocos2d.ui.LinearVerticalLayoutManager.doLayout (cocos2d::ui::LayoutProtocol*) |
| doLayout(cocos2d::ui::LayoutProtocol*) | 0 | 30 | 0 | cocos2d.ui.LinearHorizontalLayoutManager.doLayout (cocos2d::ui::LayoutProtocol*) |
| caculateFinalPositionWithRelativeWidget(cocos2d::ui::LayoutProtocol*) | 0 | 157 | 0 | cocos2d.ui.RelativeLayoutManager .caculateFinalPositionWithRelativeWidget(cocos2d::ui::LayoutProtocol*) |
| onMessage(cocos2d::network::WebSocket*,constWebSocket::Data&) | 0 | 73 | 0 | cocos2d.network.SIOClientImpl.onMessage(cocos2d::network::WebSocket* ,constWebSocket::Data&) |
| handshakeResponse(cocos2d::network::HttpClient* ,cocos2d::network::HttpResponse*) | 0 | 38 | 0 | cocos2d.network.SIOClientImpl.handshakeResponse (cocos2d::network::HttpClient*,cocos2d::network::HttpResponse*) |
| tiffReadProc(thandle_t,void*,tmsize_t) | 0 | 25 | 0 | cocos2d.anonymous_namespace{ccimage.cpp}.__Globals.tiffReadProc (thandle_t,void*,tmsize_t) |
| testFormatForPvr3TCSupport(cocos2d::(anonymousnamespace )::PVR3TexturePixelFormat) | 0 | 23 | 0 | cocos2d.anonymous_namespace{ccimage.cpp}.__Globals .testFormatForPvr3TCSupport(cocos2d::(anonymousnamespace )::PVR3TexturePixelFormat) |
| Parse<parseFlags,Stream,Handler>(Stream&,Handler&) | 0 | 22 | 0 | rapidjson.GenericReader<Encoding,Allocator>.Parse<parseFlags,Stream ,Handler>(Stream&,Handler&) |
| ParseValue<parseFlags,Stream,Handler>(Stream&,Handler&) | 0 | 21 | 0 | rapidjson.GenericReader<Encoding,Allocator>.ParseValue<parseFlags ,Stream,Handler>(Stream&,Handler&) |
| Accept<Handler>(Handler&) | 0 | 39 | 0 | rapidjson.GenericValue<Encoding,Allocator>.Accept<Handler>(Handler&) |
| ParseAnyValue(flatbuffers::Value&,flatbuffers::FieldDef*) | 0 | 26 | 0 | flatbuffers.Parser.ParseAnyValue(flatbuffers::Value& ,flatbuffers::FieldDef*) |
Statistics
| Stat | Percentage Comment | # lines of code (LOC) | # lines of comment |
|---|---|---|---|
| Sum: | 7 697 | 143 675 | 4 060 |
| Average: | 1.73 | 32.32 | 0.91 |
| Minimum: | 0 | 21 | 0 |
| Maximum: | 19 | 290 | 41 |
| Standard deviation: | 4.25 | 20.15 | 2.75 |
| Variance: | 18.09 | 405.95 | 7.55 |
| Rule warning: Methods with too many parameters |
• Rule Description:
This rule matches methods with more than 5 parameters. Such method might be painful to call and might degrade performance. See the definition of the NbParameters metric here: http://www.cppdepend.com/Metrics.aspx#NbParameters
• How to Fix Violations: More properties/fields can be added to the declaring type to handle numerous states. An alternative is to provide a class or a structure dedicated to handle arguments passing.
• How to Fix Violations: More properties/fields can be added to the declaring type to handle numerous states. An alternative is to provide a class or a structure dedicated to handle arguments passing.
161 methods matched
| methods | # Parameters | Full Name |
|---|---|---|
| Mat4(float,float,float,float,float,float,float,float,float,float,float ,float,float,float,float,float) | 16 | cocos2d.Mat4.Mat4(float,float,float,float,float,float,float,float ,float,float,float,float,float,float,float,float) |
| set(float,float,float,float,float,float,float,float,float,float,float ,float,float,float,float,float) | 16 | cocos2d.Mat4.set(float,float,float,float,float,float,float,float,float ,float,float,float,float,float,float,float) |
| slerp(float,float,float,float,float,float,float,float,float,float* ,float*,float*,float*) | 13 | cocos2d.Quaternion.slerp(float,float,float,float,float,float,float ,float,float,float*,float*,float*,float*) |
| init(float,GLuint,cocos2d::GLProgramState*,cocos2d::BlendFunc,GLuint ,GLuint,GLenum,GLenum,ssize_t,constcocos2d::Mat4&,uint32_t) | 11 | cocos2d.MeshCommand.init(float,GLuint,cocos2d::GLProgramState* ,cocos2d::BlendFunc,GLuint,GLuint,GLenum,GLenum,ssize_t ,constcocos2d::Mat4&,uint32_t) |
| createLookAt(float,float,float,float,float,float,float,float,float ,cocos2d::Mat4*) | 10 | cocos2d.Mat4.createLookAt(float,float,float,float,float,float,float ,float,float,cocos2d::Mat4*) |
| init(float,GLuint,cocos2d::GLProgramState*,cocos2d::BlendFunc,GLuint ,GLuint,GLenum,GLenum,ssize_t,constcocos2d::Mat4&) | 10 | cocos2d.MeshCommand.init(float,GLuint,cocos2d::GLProgramState* ,cocos2d::BlendFunc,GLuint,GLuint,GLenum,GLenum,ssize_t ,constcocos2d::Mat4&) |
| distaa3(double*,double*,double*,int,int,int,int,int,int) | 9 | __Globals.distaa3(double*,double*,double*,int,int,int,int,int,int) |
| unz64local_GetCurrentFileInfoInternal(unzFile,unz_file_info64* ,unz_file_info64_internal*,char*,uLong,void*,uLong,char*,uLong) | 9 | cocos2d.__Globals.unz64local_GetCurrentFileInfoInternal(unzFile ,unz_file_info64*,unz_file_info64_internal*,char*,uLong,void*,uLong ,char*,uLong) |
| ccVertexLineIntersect(float,float,float,float,float,float,float,float ,float*) | 9 | cocos2d.__Globals.ccVertexLineIntersect(float,float,float,float,float ,float,float,float,float*) |
| TerrainData(constchar*,constchar*,constcocos2d::Terrain::DetailMap& ,constcocos2d::Terrain::DetailMap&,constcocos2d::Terrain::DetailMap& ,constcocos2d::Terrain::DetailMap&,constcocos2d::Size&,float,float) | 9 | cocos2d.Terrain+TerrainData.TerrainData(constchar*,constchar* ,constcocos2d::Terrain::DetailMap&,constcocos2d::Terrain::DetailMap& ,constcocos2d::Terrain::DetailMap&,constcocos2d::Terrain::DetailMap& ,constcocos2d::Size&,float,float) |
| EventAssetsManagerEx(conststd::string& ,cocos2d::extension::AssetsManagerEx* ,constcocos2d::extension::EventAssetsManagerEx::EventCode&,float,float ,conststd::string&,conststd::string&,int,int) | 9 | cocos2d.extension.EventAssetsManagerEx.EventAssetsManagerEx (conststd::string&,cocos2d::extension::AssetsManagerEx* ,constcocos2d::extension::EventAssetsManagerEx::EventCode&,float,float ,conststd::string&,conststd::string&,int,int) |
| edtaa3(double*,double*,double*,int,int,short*,short*,double*) | 8 | __Globals.edtaa3(double*,double*,double*,int,int,short*,short*,double* ) |
| decode_subblock(etc1_byte*,int,int,int,constint*,etc1_uint32,bool,bool ) | 8 | __Globals.decode_subblock(etc1_byte*,int,int,int,constint*,etc1_uint32 ,bool,bool) |
| InterpolateColours(constint*,constint*,constint*,constint*,constint ,constint,constint,int*) | 8 | __Globals.InterpolateColours(constint*,constint*,constint*,constint* ,constint,constint,constint,int*) |
| unzGetCurrentFileInfo64(unzFile,unz_file_info64*,char*,uLong,void* ,uLong,char*,uLong) | 8 | cocos2d.__Globals.unzGetCurrentFileInfo64(unzFile,unz_file_info64* ,char*,uLong,void*,uLong,char*,uLong) |
| unzGetCurrentFileInfo(unzFile,unz_file_info*,char*,uLong,void*,uLong ,char*,uLong) | 8 | cocos2d.__Globals.unzGetCurrentFileInfo(unzFile,unz_file_info*,char* ,uLong,void*,uLong,char*,uLong) |
| updateVertex(std::map<vertex_index,ssize_t>&,std::vector<float>& ,std::vector<float>&,std::vector<float>&,conststd::vector<float>& ,conststd::vector<float>&,conststd::vector<float>& ,constcocos2d::vertex_index&) | 8 | cocos2d.__Globals.updateVertex(std::map<vertex_index,ssize_t>& ,std::vector<float>&,std::vector<float>&,std::vector<float>& ,conststd::vector<float>&,conststd::vector<float>&,conststd::vector <float>&,constcocos2d::vertex_index&) |
| exportFaceGroupToShape(std::map<vertex_index,ssize_t>& ,ObjLoader::shapes_t&,conststd::vector<float>&,conststd::vector<float >&,conststd::vector<float>&,conststd::vector<std::vector<vertex_index> >&,constObjLoader::material_t&,conststd::string&) | 8 | cocos2d.__Globals.exportFaceGroupToShape(std::map<vertex_index,ssize_t >&,ObjLoader::shapes_t&,conststd::vector<float>&,conststd::vector <float>&,conststd::vector<float>&,conststd::vector<std::vector <vertex_index>>&,constObjLoader::material_t&,conststd::string&) |
| init(float,GLuint,cocos2d::GLProgramState*,constcocos2d::BlendFunc& ,cocos2d::V3F_C4B_T2F_Quad*,ssize_t,constcocos2d::Mat4&,uint32_t) | 8 | cocos2d.QuadCommand.init(float,GLuint,cocos2d::GLProgramState* ,constcocos2d::BlendFunc&,cocos2d::V3F_C4B_T2F_Quad*,ssize_t ,constcocos2d::Mat4&,uint32_t) |
| drawCircle(constcocos2d::Vec2&,float,float,unsignedint,bool,float ,float,constcocos2d::Color4F&) | 8 | cocos2d.DrawNode.drawCircle(constcocos2d::Vec2&,float,float ,unsignedint,bool,float,float,constcocos2d::Color4F&) |
| TerrainData(constchar*,constchar*,constcocos2d::Terrain::DetailMap& ,constcocos2d::Terrain::DetailMap&,constcocos2d::Terrain::DetailMap& ,constcocos2d::Size&,float,float) | 8 | cocos2d.Terrain+TerrainData.TerrainData(constchar*,constchar* ,constcocos2d::Terrain::DetailMap&,constcocos2d::Terrain::DetailMap& ,constcocos2d::Terrain::DetailMap&,constcocos2d::Size&,float,float) |
| initialise(cocos2d::PUForceField::ForceFieldType,constcocos2d::Vec3& ,unsignedint,unsignedshort,double,double,double,constcocos2d::Vec3&) | 8 | cocos2d.PUForceField.initialise(cocos2d::PUForceField::ForceFieldType ,constcocos2d::Vec3&,unsignedint,unsignedshort,double,double,double ,constcocos2d::Vec3&) |
| spAnimation_mix(constspAnimation*,structspSkeleton*,float,float,int ,spEvent**,int*,float) | 8 | __Globals.spAnimation_mix(constspAnimation*,structspSkeleton*,float ,float,int,spEvent**,int*,float) |
| etc_encode_subblock_helper(constetc1_byte*,etc1_uint32,etc_compressed* ,bool,bool,constetc1_byte*,constint*) | 7 | __Globals.etc_encode_subblock_helper(constetc1_byte*,etc1_uint32 ,etc_compressed*,bool,bool,constetc1_byte*,constint*) |
| GetModulationValue(int,int,constint,constint(*)[16],constint(*)[16] ,int*,int*) | 7 | __Globals.GetModulationValue(int,int,constint,constint(*)[16],constint (*)[16],int*,int*) |
| ccDrawCircle(constcocos2d::Vec2&,float,float,unsignedint,bool,float ,float) | 7 | cocos2d.__Globals.ccDrawCircle(constcocos2d::Vec2&,float,float ,unsignedint,bool,float,float) |
| kmMat4OrthographicProjection(cocos2d::Mat4*,float,float,float,float ,float,float) | 7 | cocos2d.__Globals.kmMat4OrthographicProjection(cocos2d::Mat4*,float ,float,float,float,float,float) |
| createOrthographicOffCenter(float,float,float,float,float,float ,cocos2d::Mat4*) | 7 | cocos2d.Mat4.createOrthographicOffCenter(float,float,float,float,float ,float,cocos2d::Mat4*) |
| initWithCallback(cocos2d::Scheduler*,constccSchedulerFunc&,void* ,conststd::string&,float,unsignedint,float) | 7 | cocos2d.TimerTargetCallback.initWithCallback(cocos2d::Scheduler* ,constccSchedulerFunc&,void*,conststd::string&,float,unsignedint,float ) |
| schedule(constccSchedulerFunc&,void*,float,unsignedint,float,bool ,conststd::string&) | 7 | cocos2d.Scheduler.schedule(constccSchedulerFunc&,void*,float ,unsignedint,float,bool,conststd::string&) |
| create(float,float,float,float,float,float,float) | 7 | cocos2d.OrbitCamera.create(float,float,float,float,float,float,float) |
| initWithDuration(float,float,float,float,float,float,float) | 7 | cocos2d.OrbitCamera.initWithDuration(float,float,float,float,float ,float,float) |
| init(float,GLuint,cocos2d::GLProgramState*,constcocos2d::BlendFunc& ,cocos2d::V3F_C4B_T2F_Quad*,ssize_t,constcocos2d::Mat4&) | 7 | cocos2d.QuadCommand.init(float,GLuint,cocos2d::GLProgramState* ,constcocos2d::BlendFunc&,cocos2d::V3F_C4B_T2F_Quad*,ssize_t ,constcocos2d::Mat4&) |
| drawSolidCircle(constcocos2d::Vec2&,float,float,unsignedint,float ,float,constcocos2d::Color4F&) | 7 | cocos2d.DrawNode.drawSolidCircle(constcocos2d::Vec2&,float,float ,unsignedint,float,float,constcocos2d::Color4F&) |
| create(conststd::string&,conststd::string&,int,int,char,cocos2d::Ref* ,SEL_MenuHandler) | 7 | cocos2d.MenuItemAtlasFont.create(conststd::string&,conststd::string& ,int,int,char,cocos2d::Ref*,SEL_MenuHandler) |
| initWithString(conststd::string&,conststd::string&,int,int,char ,cocos2d::Ref*,SEL_MenuHandler) | 7 | cocos2d.MenuItemAtlasFont.initWithString(conststd::string& ,conststd::string&,int,int,char,cocos2d::Ref*,SEL_MenuHandler) |
| beginWithClear(float,float,float,float,float,int,GLbitfield) | 7 | cocos2d.RenderTexture.beginWithClear(float,float,float,float,float,int ,GLbitfield) |
| init(float,GLuint,cocos2d::GLProgramState*,cocos2d::BlendFunc ,cocos2d::Primitive*,constcocos2d::Mat4&,uint32_t) | 7 | cocos2d.PrimitiveCommand.init(float,GLuint,cocos2d::GLProgramState* ,cocos2d::BlendFunc,cocos2d::Primitive*,constcocos2d::Mat4&,uint32_t) |
| init(float,GLuint,cocos2d::GLProgramState*,cocos2d::BlendFunc ,constcocos2d::TrianglesCommand::Triangles&,constcocos2d::Mat4& ,uint32_t) | 7 | cocos2d.TrianglesCommand.init(float,GLuint,cocos2d::GLProgramState* ,cocos2d::BlendFunc,constcocos2d::TrianglesCommand::Triangles& ,constcocos2d::Mat4&,uint32_t) |
| initialise(cocos2d::PUForceField::ForceFieldType,unsignedint ,unsignedshort,double,double,double,constcocos2d::Vec3&) | 7 | cocos2d.PUForceField.initialise(cocos2d::PUForceField::ForceFieldType ,unsignedint,unsignedshort,double,double,double,constcocos2d::Vec3&) |
| PUBillboardChain(conststd::string&,conststd::string&,size_t,size_t ,bool,bool,bool) | 7 | cocos2d.PUBillboardChain.PUBillboardChain(conststd::string& ,conststd::string&,size_t,size_t,bool,bool,bool) |
| processGetTask(cocos2d::network::HttpRequest*,write_callback,void* ,long*,write_callback,void*,char*) | 7 | cocos2d.network.__Globals.processGetTask (cocos2d::network::HttpRequest*,write_callback,void*,long* ,write_callback,void*,char*) |
| processPostTask(cocos2d::network::HttpRequest*,write_callback,void* ,long*,write_callback,void*,char*) | 7 | cocos2d.network.__Globals.processPostTask (cocos2d::network::HttpRequest*,write_callback,void*,long* ,write_callback,void*,char*) |
| processPutTask(cocos2d::network::HttpRequest*,write_callback,void* ,long*,write_callback,void*,char*) | 7 | cocos2d.network.__Globals.processPutTask (cocos2d::network::HttpRequest*,write_callback,void*,long* ,write_callback,void*,char*) |
| processDeleteTask(cocos2d::network::HttpRequest*,write_callback,void* ,long*,write_callback,void*,char*) | 7 | cocos2d.network.__Globals.processDeleteTask (cocos2d::network::HttpRequest*,write_callback,void*,long* ,write_callback,void*,char*) |
| drawCircle(constcocos2d::Vec2&,float,float,unsignedint,bool,float ,float) | 7 | cocos2d.DrawPrimitives.__Globals.drawCircle(constcocos2d::Vec2&,float ,float,unsignedint,bool,float,float) |
| GenFieldOffset(constflatbuffers::FieldDef&,constflatbuffers::Table* ,bool,int,flatbuffers::StructDef*,constflatbuffers::GeneratorOptions& ,std::string*) | 7 | flatbuffers.__Globals.GenFieldOffset(constflatbuffers::FieldDef& ,constflatbuffers::Table*,bool,int,flatbuffers::StructDef* ,constflatbuffers::GeneratorOptions&,std::string*) |
| spAnimation_apply(constspAnimation*,structspSkeleton*,float,float,int ,spEvent**,int*) | 7 | __Globals.spAnimation_apply(constspAnimation*,structspSkeleton*,float ,float,int,spEvent**,int*) |
| spTimeline_apply(constspTimeline*,structspSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals.spTimeline_apply(constspTimeline*,structspSkeleton*,float ,float,spEvent**,int*,float) |
| spColorTimeline_setFrame(spColorTimeline*,int,float,float,float,float ,float) | 7 | __Globals.spColorTimeline_setFrame(spColorTimeline*,int,float,float ,float,float,float) |
| _spRotateTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spRotateTimeline_apply(constspTimeline*,spSkeleton*,float ,float,spEvent**,int*,float) |
| _spTranslateTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spTranslateTimeline_apply(constspTimeline*,spSkeleton* ,float,float,spEvent**,int*,float) |
| _spScaleTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spScaleTimeline_apply(constspTimeline*,spSkeleton*,float ,float,spEvent**,int*,float) |
| _spColorTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spColorTimeline_apply(constspTimeline*,spSkeleton*,float ,float,spEvent**,int*,float) |
| _spAttachmentTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spAttachmentTimeline_apply(constspTimeline*,spSkeleton* ,float,float,spEvent**,int*,float) |
| _spEventTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spEventTimeline_apply(constspTimeline*,spSkeleton*,float ,float,spEvent**,int*,float) |
| _spDrawOrderTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spDrawOrderTimeline_apply(constspTimeline*,spSkeleton* ,float,float,spEvent**,int*,float) |
| _spFFDTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spFFDTimeline_apply(constspTimeline*,spSkeleton*,float ,float,spEvent**,int*,float) |
| _spIkConstraintTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spIkConstraintTimeline_apply(constspTimeline*,spSkeleton* ,float,float,spEvent**,int*,float) |
| _spFlipTimeline_apply(constspTimeline*,spSkeleton*,float,float ,spEvent**,int*,float) | 7 | __Globals._spFlipTimeline_apply(constspTimeline*,spSkeleton*,float ,float,spEvent**,int*,float) |
| add(constcocos2d::Texture2D*,constfloat*,constfloat*,int,constint*,int ,cocos2d::Color4B*) | 7 | spine.PolygonBatch.add(constcocos2d::Texture2D*,constfloat* ,constfloat*,int,constint*,int,cocos2d::Color4B*) |
| Initialize(b2Body*,b2Body*,constb2Vec2&,constb2Vec2&,constb2Vec2& ,constb2Vec2&,float32) | 7 | b2PulleyJointDef.Initialize(b2Body*,b2Body*,constb2Vec2&,constb2Vec2& ,constb2Vec2&,constb2Vec2&,float32) |
| socket_sendto(p_socket,constchar*,size_t,size_t*,SA*,socklen_t ,p_timeout) | 7 | __Globals.socket_sendto(p_socket,constchar*,size_t,size_t*,SA* ,socklen_t,p_timeout) |
| socket_recvfrom(p_socket,char*,size_t,size_t*,SA*,socklen_t*,p_timeout ) | 7 | __Globals.socket_recvfrom(p_socket,char*,size_t,size_t*,SA*,socklen_t* ,p_timeout) |
| atitc_decode_block(uint8_t**,uint32_t*,unsignedint,bool,uint64_t ,ATITCDecodeFlag) | 6 | __Globals.atitc_decode_block(uint8_t**,uint32_t*,unsignedint,bool ,uint64_t,ATITCDecodeFlag) |
| etc1_encode_image(constetc1_byte*,etc1_uint32,etc1_uint32,etc1_uint32 ,etc1_uint32,etc1_byte*) | 6 | __Globals.etc1_encode_image(constetc1_byte*,etc1_uint32,etc1_uint32 ,etc1_uint32,etc1_uint32,etc1_byte*) |
| etc1_decode_image(constetc1_byte*,etc1_byte*,etc1_uint32,etc1_uint32 ,etc1_uint32,etc1_uint32) | 6 | __Globals.etc1_decode_image(constetc1_byte*,etc1_byte*,etc1_uint32 ,etc1_uint32,etc1_uint32,etc1_uint32) |
| PVRDecompress(AMTC_BLOCK_STRUCT*,constbool,constint,constint,constint ,unsignedchar*) | 6 | __Globals.PVRDecompress(AMTC_BLOCK_STRUCT*,constbool,constint,constint ,constint,unsignedchar*) |
| UnpackModulations(constAMTC_BLOCK_STRUCT*,constint,int(*)[16],int(* )[16],int,int) | 6 | __Globals.UnpackModulations(constAMTC_BLOCK_STRUCT*,constint,int(* )[16],int(*)[16],int,int) |
| s3tc_decode_block(uint8_t**,uint32_t*,unsignedint,bool,uint64_t ,S3TCDecodeFlag) | 6 | __Globals.s3tc_decode_block(uint8_t**,uint32_t*,unsignedint,bool ,uint64_t,S3TCDecodeFlag) |
| InputBox(LPCSTR,LPCSTR,std::string*,DWORD,bool,HWND) | 6 | CWin32InputBox.InputBox(LPCSTR,LPCSTR,std::string*,DWORD,bool,HWND) |
| ccpLineIntersect(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,float*,float*) | 6 | cocos2d.__Globals.ccpLineIntersect(constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,constcocos2d::Vec2&,float* ,float*) |
| CCAffineTransformMake(float,float,float,float,float,float) | 6 | cocos2d.__Globals.CCAffineTransformMake(float,float,float,float,float ,float) |
| ccDrawSolidCircle(constcocos2d::Vec2&,float,float,unsignedint,float ,float) | 6 | cocos2d.__Globals.ccDrawSolidCircle(constcocos2d::Vec2&,float,float ,unsignedint,float,float) |
| __CCAffineTransformMake(float,float,float,float,float,float) | 6 | cocos2d.__Globals.__CCAffineTransformMake(float,float,float,float ,float,float) |
| isOneDimensionSegmentOverlap(float,float,float,float,float*,float*) | 6 | cocos2d.__Globals.isOneDimensionSegmentOverlap(float,float,float,float ,float*,float*) |
| ccCardinalSplineAt(cocos2d::Vec2&,cocos2d::Vec2&,cocos2d::Vec2& ,cocos2d::Vec2&,float,float) | 6 | cocos2d.__Globals.ccCardinalSplineAt(cocos2d::Vec2&,cocos2d::Vec2& ,cocos2d::Vec2&,cocos2d::Vec2&,float,float) |
| isLineIntersect(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,float*,float*) | 6 | cocos2d.Vec2.isLineIntersect(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,float*,float*) |
| isSegmentOverlap(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,cocos2d::Vec2*,cocos2d::Vec2* ) | 6 | cocos2d.Vec2.isSegmentOverlap(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,cocos2d::Vec2*,cocos2d::Vec2* ) |
| squad(constcocos2d::Quaternion&,constcocos2d::Quaternion& ,constcocos2d::Quaternion&,constcocos2d::Quaternion&,float ,cocos2d::Quaternion*) | 6 | cocos2d.Quaternion.squad(constcocos2d::Quaternion& ,constcocos2d::Quaternion&,constcocos2d::Quaternion& ,constcocos2d::Quaternion&,float,cocos2d::Quaternion*) |
| initWithData(constvoid*,ssize_t,Texture2D::PixelFormat,int,int ,constcocos2d::Size&) | 6 | cocos2d.Texture2D.initWithData(constvoid*,ssize_t ,Texture2D::PixelFormat,int,int,constcocos2d::Size&) |
| initWithString(constchar*,conststd::string&,float,constcocos2d::Size& ,cocos2d::TextHAlignment,cocos2d::TextVAlignment) | 6 | cocos2d.Texture2D.initWithString(constchar*,conststd::string&,float ,constcocos2d::Size&,cocos2d::TextHAlignment,cocos2d::TextVAlignment) |
| convertDataToFormat(constunsignedchar*,ssize_t ,cocos2d::Texture2D::PixelFormat,cocos2d::Texture2D::PixelFormat ,unsignedchar**,ssize_t*) | 6 | cocos2d.Texture2D.convertDataToFormat(constunsignedchar*,ssize_t ,cocos2d::Texture2D::PixelFormat,cocos2d::Texture2D::PixelFormat ,unsignedchar**,ssize_t*) |
| PixelFormatInfo(GLenum,GLenum,GLenum,int,bool,bool) | 6 | cocos2d.Texture2D+PixelFormatInfo.PixelFormatInfo(GLenum,GLenum,GLenum ,int,bool,bool) |
| initWithSelector(cocos2d::Scheduler*,SEL_SCHEDULE,cocos2d::Ref*,float ,unsignedint,float) | 6 | cocos2d.TimerTargetSelector.initWithSelector(cocos2d::Scheduler* ,SEL_SCHEDULE,cocos2d::Ref*,float,unsignedint,float) |
| schedule(SEL_SCHEDULE,cocos2d::Ref*,float,unsignedint,float,bool) | 6 | cocos2d.Scheduler.schedule(SEL_SCHEDULE,cocos2d::Ref*,float ,unsignedint,float,bool) |
| scheduleSelector(SEL_SCHEDULE,cocos2d::Ref*,float,unsignedint,float ,bool) | 6 | cocos2d.Scheduler.scheduleSelector(SEL_SCHEDULE,cocos2d::Ref*,float ,unsignedint,float,bool) |
| transformVec4(constfloat*,float,float,float,float,float*) | 6 | cocos2d.MathUtil.transformVec4(constfloat*,float,float,float,float ,float*) |
| create(float,constcocos2d::Size&,constcocos2d::Vec2&,float,unsignedint ,float) | 6 | cocos2d.Ripple3D.create(float,constcocos2d::Size&,constcocos2d::Vec2& ,float,unsignedint,float) |
| initWithDuration(float,constcocos2d::Size&,constcocos2d::Vec2&,float ,unsignedint,float) | 6 | cocos2d.Ripple3D.initWithDuration(float,constcocos2d::Size& ,constcocos2d::Vec2&,float,unsignedint,float) |
| create(float,constcocos2d::Size&,unsignedint,float,bool,bool) | 6 | cocos2d.Waves.create(float,constcocos2d::Size&,unsignedint,float,bool ,bool) |
| initWithDuration(float,constcocos2d::Size&,unsignedint,float,bool,bool ) | 6 | cocos2d.Waves.initWithDuration(float,constcocos2d::Size&,unsignedint ,float,bool,bool) |
| setVertexAttribPointer(conststd::string&,GLint,GLenum,GLboolean ,GLsizei,GLvoid*) | 6 | cocos2d.GLProgramState.setVertexAttribPointer(conststd::string&,GLint ,GLenum,GLboolean,GLsizei,GLvoid*) |
| drawCircle(constcocos2d::Vec2&,float,float,unsignedint,bool ,constcocos2d::Color4F&) | 6 | cocos2d.DrawNode.drawCircle(constcocos2d::Vec2&,float,float ,unsignedint,bool,constcocos2d::Color4F&) |
| drawCubicBezier(constcocos2d::Vec2&,constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,unsignedint ,constcocos2d::Color4F&) | 6 | cocos2d.DrawNode.drawCubicBezier(constcocos2d::Vec2& ,constcocos2d::Vec2&,constcocos2d::Vec2&,constcocos2d::Vec2& ,unsignedint,constcocos2d::Color4F&) |
| create(conststd::string&,conststd::string&,float,constcocos2d::Size& ,cocos2d::TextHAlignment,cocos2d::TextVAlignment) | 6 | cocos2d.LabelTTF.create(conststd::string&,conststd::string&,float ,constcocos2d::Size&,cocos2d::TextHAlignment,cocos2d::TextVAlignment) |
| initWithString(conststd::string&,conststd::string&,float ,constcocos2d::Size&,cocos2d::TextHAlignment,cocos2d::TextVAlignment) | 6 | cocos2d.LabelTTF.initWithString(conststd::string&,conststd::string& ,float,constcocos2d::Size&,cocos2d::TextHAlignment ,cocos2d::TextVAlignment) |
| init(float,cocos2d::GLProgram*,cocos2d::BlendFunc ,cocos2d::TextureAtlas*,constcocos2d::Mat4&,uint32_t) | 6 | cocos2d.BatchCommand.init(float,cocos2d::GLProgram*,cocos2d::BlendFunc ,cocos2d::TextureAtlas*,constcocos2d::Mat4&,uint32_t) |
| _ttfConfig(constchar*,int,constcocos2d::GlyphCollection&,constchar* ,bool,int) | 6 | cocos2d._ttfConfig._ttfConfig(constchar*,int ,constcocos2d::GlyphCollection&,constchar*,bool,int) |
| createWithSystemFont(conststd::string&,conststd::string&,float ,constcocos2d::Size&,cocos2d::TextHAlignment,cocos2d::TextVAlignment) | 6 | cocos2d.Label.createWithSystemFont(conststd::string&,conststd::string& ,float,constcocos2d::Size&,cocos2d::TextHAlignment ,cocos2d::TextVAlignment) |
Statistics
| Stat | # Parameters |
|---|---|
| Sum: | 1 086 |
| Average: | 6.75 |
| Minimum: | 6 |
| Maximum: | 16 |
| Standard deviation: | 1.48 |
| Variance: | 2.18 |
| Rule warning: Methods with too many local variables |
• Rule Description:
This rule matches methods with more than 15 variables.
Methods where NbVariables > 8 are hard to understand and maintain. Methods where NbVariables > 15 are extremely complex and must be refactored.
See the definition of the Nbvariables metric here: http://www.cppdepend.com/Metrics.aspx#Nbvariables
• How to Fix Violations: To refactor such method and increase code quality and maintainability, certainly you'll have to split the method into several smaller methods or even create one or several classes to implement the logic. During this process it is important to question the scope of each variable local to the method. This can be an indication if such local variable will become an instance field of the newly created class(es).
• How to Fix Violations: To refactor such method and increase code quality and maintainability, certainly you'll have to split the method into several smaller methods or even create one or several classes to implement the logic. During this process it is important to question the scope of each variable local to the method. This can be an indication if such local variable will become an instance field of the newly created class(es).
388 methods matched
| methods | # Variables | Full Name |
|---|---|---|
| startElement(void*,constchar*,constchar**) | 87 | cocos2d.TMXMapInfo.startElement(void*,constchar*,constchar**) |
| onEnter() | 70 | TemplateMapTest.onEnter() |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 64 | cocos2d.PUEmitterTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| generateTestFunctions() | 63 | TouchEventDispatchingPerfTest.generateTestFunctions() |
| _spSkeletonJson_readAnimation(spSkeletonJson*,Json*,spSkeletonData*) | 62 | __Globals._spSkeletonJson_readAnimation(spSkeletonJson*,Json* ,spSkeletonData*) |
| onEnter() | 61 | TemplateVectorTest.onEnter() |
| buildJson(rapidjson::Document&,cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*) | 60 | cocostudio.TriggerMng.buildJson(rapidjson::Document& ,cocostudio::CocoLoader*,cocostudio::stExpCocoNode*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 60 | cocostudio.ButtonReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 60 | cocostudio.CheckBoxReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 60 | cocostudio.SliderReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 57 | cocostudio.ButtonReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 56 | cocostudio.WidgetReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 54 | cocostudio.NodeReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| generateTestFunctions() | 54 | TemplateVectorPerfTest.generateTestFunctions() |
| spSkeletonJson_readSkeletonData(spSkeletonJson*,constchar*) | 53 | __Globals.spSkeletonJson_readSkeletonData(spSkeletonJson*,constchar*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 52 | cocostudio.ListViewReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| SolveVelocityConstraints() | 52 | b2ContactSolver.SolveVelocityConstraints() |
| createOptionsWithFlatBuffersForNode(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 50 | cocostudio.Node3DReader.createOptionsWithFlatBuffersForNode (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| generateTestFunctions() | 50 | TemplateMapStringKeyPerfTest.generateTestFunctions() |
| translate(cocos2d::PUScriptCompiler*,cocos2d::PUAbstractNode*) | 48 | cocos2d.PURendererTranslator.translate(cocos2d::PUScriptCompiler* ,cocos2d::PUAbstractNode*) |
| InitializeVelocityConstraints() | 46 | b2ContactSolver.InitializeVelocityConstraints() |
| Cantilever() | 46 | Cantilever.Cantilever() |
| serialize(void*) | 45 | cocostudio.ComRender.serialize(void*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 45 | cocostudio.CheckBoxReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 45 | cocostudio.SliderReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| generateTestFunctions() | 45 | ArrayPerfTest.generateTestFunctions() |
| generateTestFunctions() | 44 | TemplateMapIntKeyPerfTest.generateTestFunctions() |
| CharacterCollision() | 44 | CharacterCollision.CharacterCollision() |
| createSlicedSprites() | 43 | cocos2d.ui.Scale9Sprite.createSlicedSprites() |
| parseProperties(cocos2d::Node*,cocos2d::Node*,cocosbuilder::CCBReader* ) | 43 | cocosbuilder.NodeLoader.parseProperties(cocos2d::Node*,cocos2d::Node* ,cocosbuilder::CCBReader*) |
| generateTestFunctions() | 43 | DictionaryStringKeyPerfTest.generateTestFunctions() |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 42 | cocostudio.ListViewReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| SolveTOI(constb2TimeStep&) | 42 | b2World.SolveTOI(constb2TimeStep&) |
| SolvePositionConstraints(constb2SolverData&) | 42 | b2PrismaticJoint.SolvePositionConstraints(constb2SolverData&) |
| Dominos() | 42 | Dominos.Dominos() |
| initWithBinary(cocostudio::CocoLoader*,cocostudio::stExpCocoNode* ,cocos2d::Ref*) | 41 | cocostudio.ActionNode.initWithBinary(cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*,cocos2d::Ref*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 41 | cocostudio.ScrollViewReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| onEnter() | 41 | PhysicsDemoJoints.onEnter() |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 39 | cocostudio.ScrollViewReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| setLightUniforms() | 38 | cocos2d.MeshCommand.setLightUniforms() |
| Solve(b2Profile*,constb2TimeStep&,constb2Vec2&,bool) | 38 | b2Island.Solve(b2Profile*,constb2TimeStep&,constb2Vec2&,bool) |
| generateTestFunctions() | 37 | DictionaryIntKeyPerfTest.generateTestFunctions() |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 36 | cocostudio.LayoutReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 36 | cocostudio.PageViewReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| Car() | 36 | Car.Car() |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 35 | cocostudio.LayoutReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 35 | cocostudio.PageViewReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| b2CollidePolygons(b2Manifold*,constb2PolygonShape*,constb2Transform& ,constb2PolygonShape*,constb2Transform&) | 35 | __Globals.b2CollidePolygons(b2Manifold*,constb2PolygonShape* ,constb2Transform&,constb2PolygonShape*,constb2Transform&) |
| LoadObj(shapes_t&,constchar*,constchar*) | 34 | cocos2d.ObjLoader.LoadObj(shapes_t&,constchar*,constchar*) |
| drawSkeleton(constcocos2d::Mat4&,uint32_t) | 34 | spine.SkeletonRenderer.drawSkeleton(constcocos2d::Mat4&,uint32_t) |
| SolvePositionConstraints(constb2SolverData&) | 34 | b2GearJoint.SolvePositionConstraints(constb2SolverData&) |
| initWithQuantityOfNodes(unsignedint) | 34 | PerformanceEventDispatcherScene.initWithQuantityOfNodes(unsignedint) |
| lua_cocos2dx_Texture2D_initWithString(lua_State*) | 34 | __Globals.lua_cocos2dx_Texture2D_initWithString(lua_State*) |
| init(constcocos2d::Animation3DData&) | 33 | cocos2d.Animation3D.init(constcocos2d::Animation3DData&) |
| initWithDictionary(constrapidjson::Value&,cocos2d::Ref*) | 33 | cocostudio.ActionNode.initWithDictionary(constrapidjson::Value& ,cocos2d::Ref*) |
| copyAttributesTo(cocos2d::PUParticleSystem3D*) | 32 | cocos2d.PUParticleSystem3D.copyAttributesTo (cocos2d::PUParticleSystem3D*) |
| ~PUParticleSystem3D() | 32 | cocos2d.PUParticleSystem3D.~PUParticleSystem3D() |
| readNodeGraph(cocos2d::Node*) | 32 | cocosbuilder.CCBReader.readNodeGraph(cocos2d::Node*) |
| createOptionsWithFlatBuffers(consttinyxml2::XMLElement* ,flatbuffers::FlatBufferBuilder*) | 32 | cocostudio.TextReader.createOptionsWithFlatBuffers (consttinyxml2::XMLElement*,flatbuffers::FlatBufferBuilder*) |
| lua_cocos2dx_experimental_SpritePolygon_create(lua_State*) | 32 | __Globals.lua_cocos2dx_experimental_SpritePolygon_create(lua_State*) |
| handle(cocos2d::PUParticleSystem3D*,cocos2d::PUParticle3D*,float) | 31 | cocos2d.PUDoEnableComponentEventHandler.handle (cocos2d::PUParticleSystem3D*,cocos2d::PUParticle3D*,float) |
| b2TimeOfImpact(b2TOIOutput*,constb2TOIInput*) | 31 | __Globals.b2TimeOfImpact(b2TOIOutput*,constb2TOIInput*) |
| Collide(b2Manifold*,constb2EdgeShape*,constb2Transform& ,constb2PolygonShape*,constb2Transform&) | 31 | b2EPCollider.Collide(b2Manifold*,constb2EdgeShape*,constb2Transform& ,constb2PolygonShape*,constb2Transform&) |
| SolvePositionConstraints() | 31 | b2ContactSolver.SolvePositionConstraints() |
| SolveTOIPositionConstraints(int32,int32) | 31 | b2ContactSolver.SolveTOIPositionConstraints(int32,int32) |
| resetTest() | 31 | PhysicsContactTest.resetTest() |
| Gears() | 31 | Gears.Gears() |
| drawPolygon(constcocos2d::Vec2*,int,constcocos2d::Color4F&,float ,constcocos2d::Color4F&) | 30 | cocos2d.DrawNode.drawPolygon(constcocos2d::Vec2*,int ,constcocos2d::Color4F&,float,constcocos2d::Color4F&) |
| addSpriteFramesWithDictionary(ValueMap&,cocos2d::Texture2D*) | 30 | cocos2d.SpriteFrameCache.addSpriteFramesWithDictionary(ValueMap& ,cocos2d::Texture2D*) |
| caculateFinalPositionWithRelativeWidget(cocos2d::ui::LayoutProtocol*) | 30 | cocos2d.ui.RelativeLayoutManager .caculateFinalPositionWithRelativeWidget(cocos2d::ui::LayoutProtocol*) |
| setPropsForButtonFromJsonDictionary(cocos2d::ui::Widget* ,constrapidjson::Value&) | 30 | cocostudio.WidgetPropertiesReader0250 .setPropsForButtonFromJsonDictionary(cocos2d::ui::Widget* ,constrapidjson::Value&) |
| setPropsFromJsonDictionary(cocos2d::ui::Widget*,constrapidjson::Value& ) | 30 | cocostudio.LayoutReader.setPropsFromJsonDictionary (cocos2d::ui::Widget*,constrapidjson::Value&) |
| groupBatchDownload(constDownloadUnits&) | 29 | cocos2d.extension.Downloader.groupBatchDownload(constDownloadUnits&) |
| SolveVelocityConstraints(constb2SolverData&) | 29 | b2PrismaticJoint.SolveVelocityConstraints(constb2SolverData&) |
| AllocatorTest() | 29 | AllocatorTest.AllocatorTest() |
| init() | 29 | Bug1174Layer.init() |
| CompoundShapes() | 29 | CompoundShapes.CompoundShapes() |
| lua_cocos2dx_cocosdenshion_SimpleAudioEngine_playEffect(lua_State*) | 29 | __Globals.lua_cocos2dx_cocosdenshion_SimpleAudioEngine_playEffect (lua_State*) |
| LoadMtl(std::map<std::string,ObjLoader::material_t>&,constchar* ,constchar*) | 28 | cocos2d.__Globals.LoadMtl(std::map<std::string,ObjLoader::material_t>& ,constchar*,constchar*) |
| calculateVertexPoints() | 28 | cocos2d.Grid3D.calculateVertexPoints() |
| render(cocos2d::Renderer*,constcocos2d::Mat4& ,cocos2d::ParticleSystem3D*) | 28 | cocos2d.PUParticle3DQuadRender.render(cocos2d::Renderer* ,constcocos2d::Mat4&,cocos2d::ParticleSystem3D*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 28 | cocostudio.TextReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
| Step(Settings*) | 28 | Test.Step(Settings*) |
| lua_cocos2dx_extension_ControlSlider_create(lua_State*) | 28 | __Globals.lua_cocos2dx_extension_ControlSlider_create(lua_State*) |
| onDraw(constcocos2d::Mat4&,bool) | 27 | cocos2d.Label.onDraw(constcocos2d::Mat4&,bool) |
| createStringSprites(cocos2d::Label*) | 27 | cocos2d.LabelTextFormatter.createStringSprites(cocos2d::Label*) |
| updateTiles(constcocos2d::Rect&) | 27 | cocos2d.experimental.TMXLayer.updateTiles(constcocos2d::Rect&) |
| scrollChildrenBoth(float,float) | 27 | cocos2d.ui.ScrollView.scrollChildrenBoth(float,float) |
| GenStruct(constflatbuffers::LanguageParameters& ,constflatbuffers::Parser&,flatbuffers::StructDef&,std::string*) | 27 | flatbuffers.__Globals.GenStruct(constflatbuffers::LanguageParameters& ,constflatbuffers::Parser&,flatbuffers::StructDef&,std::string*) |
| createNodeWithSceneFile(conststd::string& ,cocostudio::SceneReader::AttachComponentType) | 27 | cocostudio.SceneReader.createNodeWithSceneFile(conststd::string& ,cocostudio::SceneReader::AttachComponentType) |
| createNodeWithFlatBuffers(constflatbuffers::Table*) | 27 | cocostudio.GameMapReader.createNodeWithFlatBuffers (constflatbuffers::Table*) |
| setPropsFromBinary(cocos2d::ui::Widget*,cocostudio::CocoLoader* ,cocostudio::stExpCocoNode*) | 27 | cocostudio.SliderReader.setPropsFromBinary(cocos2d::ui::Widget* ,cocostudio::CocoLoader*,cocostudio::stExpCocoNode*) |
| b2CollideEdgeAndCircle(b2Manifold*,constb2EdgeShape*,constb2Transform& ,constb2CircleShape*,constb2Transform&) | 27 | __Globals.b2CollideEdgeAndCircle(b2Manifold*,constb2EdgeShape* ,constb2Transform&,constb2CircleShape*,constb2Transform&) |
| InitVelocityConstraints(constb2SolverData&) | 27 | b2PrismaticJoint.InitVelocityConstraints(constb2SolverData&) |
| lua_cocos2dx_cocosbuilder_CCBReader_constructor(lua_State*) | 27 | __Globals.lua_cocos2dx_cocosbuilder_CCBReader_constructor(lua_State*) |
| loop() | 26 | cocos2d.Console.loop() |
| prepared() | 26 | cocos2d.PUParticleSystem3D.prepared() |
| parseBinaryConfigFile(unsignedchar*,unsignedlong,conststd::string&) | 26 | cocos2d.BMFontConfiguration.parseBinaryConfigFile(unsignedchar* ,unsignedlong,conststd::string&) |
| decodeFrame(tinyxml2::XMLElement*,tinyxml2::XMLElement* ,cocostudio::BoneData*,DataInfo*) | 26 | cocostudio.DataReaderHelper.decodeFrame(tinyxml2::XMLElement* ,tinyxml2::XMLElement*,cocostudio::BoneData*,DataInfo*) |
| setPropsWithFlatBuffers(cocos2d::Node*,constflatbuffers::Table*) | 26 | cocostudio.WidgetReader.setPropsWithFlatBuffers(cocos2d::Node* ,constflatbuffers::Table*) |
Statistics
| Stat | # Variables |
|---|---|
| Sum: | 9 447 |
| Average: | 24.35 |
| Minimum: | 16 |
| Maximum: | 87 |
| Standard deviation: | 10.67 |
| Variance: | 113.85 |
| Rule warning: Methods with too many overloads |
• Rule Description:
Method overloading is the ability to create multiple methods of the same name with different implementations, and various set of parameters.
This rule matches sets of method with more than 6 overloads.
Such method set might be a problem to maintain and provokes higher coupling than necessary.
See the definition of the NbOverloads metric here http://www.cppdepend.com/Metrics.aspx#NbOverloads
• How to Fix Violations: Typically the too many overloads phenomenon appears when an algorithm takes a various set of in-parameters. Each overload is presented as a facility to provide a various set of in-parameters. The too many overloads phenomenon can also be a consequence of the usage of the visitor design pattern http://en.wikipedia.org/wiki/Visitor_pattern since a method named Visit() must be provided for each sub type. In such situation there is no need for fix.
• How to Fix Violations: Typically the too many overloads phenomenon appears when an algorithm takes a various set of in-parameters. Each overload is presented as a facility to provide a various set of in-parameters. The too many overloads phenomenon can also be a consequence of the usage of the visitor design pattern http://en.wikipedia.org/wiki/Visitor_pattern since a method named Visit() must be provided for each sub type. In such situation there is no need for fix.
59 methods matched
| methods | # Overloads | Full Name |
|---|---|---|
| Value() | 16 | cocos2d.Value.Value() |
| Value(unsignedchar) | 16 | cocos2d.Value.Value(unsignedchar) |
| Value(int) | 16 | cocos2d.Value.Value(int) |
| Value(float) | 16 | cocos2d.Value.Value(float) |
| Value(double) | 16 | cocos2d.Value.Value(double) |
| Value(bool) | 16 | cocos2d.Value.Value(bool) |
| Value(constchar*) | 16 | cocos2d.Value.Value(constchar*) |
| Value(conststd::string&) | 16 | cocos2d.Value.Value(conststd::string&) |
| Value(constValueVector&) | 16 | cocos2d.Value.Value(constValueVector&) |
| Value(ValueVector&&) | 16 | cocos2d.Value.Value(ValueVector&&) |
| Value(constValueMap&) | 16 | cocos2d.Value.Value(constValueMap&) |
| Value(ValueMap&&) | 16 | cocos2d.Value.Value(ValueMap&&) |
| Value(constValueMapIntKey&) | 16 | cocos2d.Value.Value(constValueMapIntKey&) |
| Value(ValueMapIntKey&&) | 16 | cocos2d.Value.Value(ValueMapIntKey&&) |
| Value(constcocos2d::Value&) | 16 | cocos2d.Value.Value(constcocos2d::Value&) |
| Value(cocos2d::Value&&) | 16 | cocos2d.Value.Value(cocos2d::Value&&) |
| GenericValue<Encoding,Allocator>() | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>() |
| GenericValue<Encoding,Allocator>(constGenericValue<Encoding,Allocator >&) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(constGenericValue<Encoding,Allocator>&) |
| GenericValue<Encoding,Allocator>(rapidjson::Type) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(rapidjson::Type) |
| GenericValue<Encoding,Allocator>(bool) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(bool) |
| GenericValue<Encoding,Allocator>(int) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(int) |
| GenericValue<Encoding,Allocator>(unsignedint) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(unsignedint) |
| GenericValue<Encoding,Allocator>(int64_t) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(int64_t) |
| GenericValue<Encoding,Allocator>(uint64_t) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(uint64_t) |
| GenericValue<Encoding,Allocator>(double) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(double) |
| GenericValue<Encoding,Allocator>(constCh*,SizeType) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(constCh*,SizeType) |
| GenericValue<Encoding,Allocator>(constCh*) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(constCh*) |
| GenericValue<Encoding,Allocator>(constCh*,SizeType,Allocator&) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(constCh*,SizeType,Allocator&) |
| GenericValue<Encoding,Allocator>(constCh*,Allocator&) | 13 | rapidjson.GenericValue<Encoding,Allocator>.GenericValue<Encoding ,Allocator>(constCh*,Allocator&) |
| visit(constcocos2d::__Bool*) | 8 | cocos2d.DataVisitor.visit(constcocos2d::__Bool*) |
| visit(constcocos2d::__Integer*) | 8 | cocos2d.DataVisitor.visit(constcocos2d::__Integer*) |
| visit(constcocos2d::__Float*) | 8 | cocos2d.DataVisitor.visit(constcocos2d::__Float*) |
| visit(constcocos2d::__Double*) | 8 | cocos2d.DataVisitor.visit(constcocos2d::__Double*) |
| visit(constcocos2d::__String*) | 8 | cocos2d.DataVisitor.visit(constcocos2d::__String*) |
| visit(constcocos2d::__Array*) | 8 | cocos2d.DataVisitor.visit(constcocos2d::__Array*) |
| visit(constcocos2d::__Dictionary*) | 8 | cocos2d.DataVisitor.visit(constcocos2d::__Dictionary*) |
| visit(constcocos2d::__Set*) | 8 | cocos2d.DataVisitor.visit(constcocos2d::__Set*) |
| visit(constcocos2d::__Bool*) | 8 | cocos2d.PrettyPrinter.visit(constcocos2d::__Bool*) |
| visit(constcocos2d::__Integer*) | 8 | cocos2d.PrettyPrinter.visit(constcocos2d::__Integer*) |
| visit(constcocos2d::__Float*) | 8 | cocos2d.PrettyPrinter.visit(constcocos2d::__Float*) |
| visit(constcocos2d::__Double*) | 8 | cocos2d.PrettyPrinter.visit(constcocos2d::__Double*) |
| visit(constcocos2d::__String*) | 8 | cocos2d.PrettyPrinter.visit(constcocos2d::__String*) |
| visit(constcocos2d::__Array*) | 8 | cocos2d.PrettyPrinter.visit(constcocos2d::__Array*) |
| visit(constcocos2d::__Dictionary*) | 8 | cocos2d.PrettyPrinter.visit(constcocos2d::__Dictionary*) |
| visit(constcocos2d::__Set*) | 8 | cocos2d.PrettyPrinter.visit(constcocos2d::__Set*) |
| create() | 7 | cocos2d.MenuItemImage.create() |
| create(conststd::string&,conststd::string&) | 7 | cocos2d.MenuItemImage.create(conststd::string&,conststd::string&) |
| create(conststd::string&,conststd::string&,conststd::string&) | 7 | cocos2d.MenuItemImage.create(conststd::string&,conststd::string& ,conststd::string&) |
| create(conststd::string&,conststd::string&,cocos2d::Ref* ,SEL_MenuHandler) | 7 | cocos2d.MenuItemImage.create(conststd::string&,conststd::string& ,cocos2d::Ref*,SEL_MenuHandler) |
| create(conststd::string&,conststd::string&,constccMenuCallback&) | 7 | cocos2d.MenuItemImage.create(conststd::string&,conststd::string& ,constccMenuCallback&) |
| create(conststd::string&,conststd::string&,conststd::string& ,cocos2d::Ref*,SEL_MenuHandler) | 7 | cocos2d.MenuItemImage.create(conststd::string&,conststd::string& ,conststd::string&,cocos2d::Ref*,SEL_MenuHandler) |
| create(conststd::string&,conststd::string&,conststd::string& ,constccMenuCallback&) | 7 | cocos2d.MenuItemImage.create(conststd::string&,conststd::string& ,conststd::string&,constccMenuCallback&) |
| b2Mul(constb2Mat22&,constb2Vec2&) | 7 | __Globals.b2Mul(constb2Mat22&,constb2Vec2&) |
| b2Mul(constb2Mat22&,constb2Mat22&) | 7 | __Globals.b2Mul(constb2Mat22&,constb2Mat22&) |
| b2Mul(constb2Mat33&,constb2Vec3&) | 7 | __Globals.b2Mul(constb2Mat33&,constb2Vec3&) |
| b2Mul(constb2Rot&,constb2Rot&) | 7 | __Globals.b2Mul(constb2Rot&,constb2Rot&) |
| b2Mul(constb2Rot&,constb2Vec2&) | 7 | __Globals.b2Mul(constb2Rot&,constb2Vec2&) |
| b2Mul(constb2Transform&,constb2Vec2&) | 7 | __Globals.b2Mul(constb2Transform&,constb2Vec2&) |
| b2Mul(constb2Transform&,constb2Transform&) | 7 | __Globals.b2Mul(constb2Transform&,constb2Transform&) |
Statistics
| Stat | # Overloads |
|---|---|
| Sum: | 651 |
| Average: | 11.03 |
| Minimum: | 7 |
| Maximum: | 16 |
| Standard deviation: | 3.73 |
| Variance: | 13.9 |
| Rule warning: Types with too many methods |
• Rule Description:
This rule matches types with more than 20 methods. Such type might be hard to understand and maintain.
Notice that methods like constructors or property and event accessors are not taken account.
Having many methods for a type might be a symptom of too many responsibilities implemented.
Maybe you are facing the God Class phenomenon: A God Class is a class that controls way too many other classes in the system and has grown beyond all logic to become The Class That Does Everything.
• How to Fix Violations: To refactor such type and increase code quality and maintainability, certainly you'll have to split the type into several smaller types that together, implement the same logic. To refactor a God Class you'll need patience, and you might even need to recreate everything from scratch. Here are a few advices: • Think before pulling out methods: What responsibility does it have? Can you isolate some subsets of methods that operate on the same subsets of fields? • Try to maintain the interface of the god class at first and delegate calls to the new extracted classes. In the end the god class should be a pure facade without own logic. Then you can keep it for convenience or throw it away and start to use the new classes only. • Unit Tests can help: write tests for each method before extractin
• How to Fix Violations: To refactor such type and increase code quality and maintainability, certainly you'll have to split the type into several smaller types that together, implement the same logic. To refactor a God Class you'll need patience, and you might even need to recreate everything from scratch. Here are a few advices: • Think before pulling out methods: What responsibility does it have? Can you isolate some subsets of methods that operate on the same subsets of fields? • Try to maintain the interface of the god class at first and delegate calls to the new extracted classes. In the end the god class should be a pure facade without own logic. Then you can keep it for convenience or throw it away and start to use the new classes only. • Unit Tests can help: write tests for each method before extractin