cloudsim-plus-testbeds/src/main/java/org/cloudsimplus/performance/PerformanceTestbed1.java
Private field 'broker0' could be made final; it is only initialized in the declaration or constructor.
private DatacenterBroker broker0;
cloudsim-plus-testbeds/src/main/java/org/cloudsimplus/testbeds/dynamiccloudlets/DynamicCloudletsArrivalRunner.java
Document empty method body
protected void setup() {}
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/cloudlets/CloudletAbstract.java
Avoid excessively long variable names like lastExecutedDatacenterIdx
private int lastExecutedDatacenterIdx;
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/cloudlets/CloudletDatacenterExecution.java
Use explicit scoping instead of the default package private level
long getFinishedSoFar() {
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/datacenters/network/NetworkDatacenter.java
Avoid using Literals in Conditional Statements
if (estimatedFinishTime > 0.0) { // if this cloudlet is in the exec
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/hosts/Host.java
This class has too many methods, consider refactoring it.
public interface Host extends Machine, Comparable<Host> {
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/hosts/HostSimple.java
This class has a bunch of public methods and attributes
package org.cloudbus.cloudsim.hosts;
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/hosts/power/PowerHostSimple.java
Local variable 'fromPower' could be declared final
double fromPower = getPower(fromUtilization);
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/schedulers/cloudlet/CloudletSchedulerAbstract.java
Local variable 'finishCloudletMigration' could be declared final
Function<CloudletExecution, Cloudlet> finishCloudletMigration = rcl -> {
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/util/DataCloudTags.java
Avoid excessively long variable names like FILE_ADD_ERROR_STORAGE_FULL
public static final int FILE_ADD_ERROR_STORAGE_FULL = RM_BASE + 21;
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/util/WorkloadFileReader.java
Avoid using redundant field initializer for 'jobNum'
private int jobNum = 0;
cloudsim-plus/src/main/java/org/cloudbus/cloudsim/utilizationmodels/UtilizationModelDynamic.java
Avoid excessively long variable names like initialUtilizationPercent
public UtilizationModelDynamic(final double initialUtilizationPercent) {
cloudsim-plus/src/test/java/org/cloudbus/cloudsim/resources/FileTest.java
JUnit tests should not contain more than 1 assert(s).
public void testMakeMasterCopy() {
JUnit tests should not contain more than 1 assert(s).
public void testSetTransactionTime() {
cloudsim-plus/src/test/java/org/cloudbus/cloudsim/resources/HarddriveStorageTest.java
Method names should not contain underscores
public void testNewHarddriveStorage_onlyWhiteSpacesName() {
JUnit tests should include assert() or fail()
public void testSetAvgSeekTime_double() {
JUnit tests should not contain more than 1 assert(s).
public void testDeleteFile_String() {
cloudsim-plus/src/test/java/org/cloudbus/cloudsim/resources/ProcessorTest.java
Avoid unused private methods such as 'createCloudletExecList(int)'.
private List<CloudletExecution> createCloudletExecList(int numberOfCloudlets) {
cloudsim-plus/src/test/java/org/cloudbus/cloudsim/schedulers/cloudlet/CloudletSchedulerSpaceSharedTest.java
Method names should not contain underscores
public void testGetCloudletToMigrate_WhenTwoCloudletsAreRunning_AfterMigrationUsedPesReturnToOne() {
docs/conf.py
Redefining built-in 'copyright'
copyright = u'2017, Manoel C. Silva Filho, Raysa L. Oliveira, Claudio C. Monteiro, Pedro R. M. Inácio, Mário M. Freire'
More
Previous Next