Account.rb -- 53 warnings: Account has the variable name '@accountBalance' (UncommunicativeName) Account has the variable name '@accountName' (UncommunicativeName) Account has the variable name '@accountNumber' (UncommunicativeName) Account has the variable name '@subAccount' (UncommunicativeName) Account#addAccountEntry has 7 parameters (LongParameterList) Account#addAccountEntry has the name 'addAccountEntry' (UncommunicativeName) Account#addAccountEntry has the parameter name 'subAccNum' (UncommunicativeName) Account#addAccountEntry has the parameter name 'transNumber' (UncommunicativeName) Account#addAccountEntry has the variable name 'subAccNum' (UncommunicativeName) Account#addToSubAcc calls (@accountBalance + amount) twice (Duplication) Account#addToSubAcc has the name 'addToSubAcc' (UncommunicativeName) Account#addToSubAcc is controlled by argument number (ControlCouple) Account#createSubAcc has the name 'createSubAcc' (UncommunicativeName) Account#createSubAcc has the parameter name 'subAcc' (UncommunicativeName) Account#createSubAcc has the parameter name 'subAccNum' (UncommunicativeName) Account#createSubAcc has the parameter name 'subBal' (UncommunicativeName) Account#createSubAcc has the variable name 'subBal' (UncommunicativeName) Account#createSubAccInitializationOfAccount has the name 'createSubAccInitializationOfAccount' (UncommunicativeName) Account#createSubAccInitializationOfAccount has the parameter name 'subAcc' (UncommunicativeName) Account#createSubAccInitializationOfAccount has the parameter name 'subAccNum' (UncommunicativeName) Account#createSubAccInitializationOfAccount has the parameter name 'subBal' (UncommunicativeName) Account#examineAccount has the name 'examineAccount' (UncommunicativeName) Account#examineAccountOnDate calls ((@entries.size - 1) - i) twice (Duplication) Account#examineAccountOnDate calls (@entries.size - 1) twice (Duplication) Account#examineAccountOnDate calls @entries.size 3 times (Duplication) Account#examineAccountOnDate calls @entries[((@entries.size - 1) - i)] twice (Duplication) Account#examineAccountOnDate has the name 'examineAccountOnDate' (UncommunicativeName) Account#examineAccountOnDate has the variable name 'i' (UncommunicativeName) Account#examineAccountOnDate has the variable name 'returnString' (UncommunicativeName) Account#examineBalanceForPeriod calls entry.getDate twice (Duplication) Account#examineBalanceForPeriod has the name 'examineBalanceForPeriod' (UncommunicativeName) Account#examineBalanceForPeriod has the parameter name 'endDate' (UncommunicativeName) Account#examineBalanceForPeriod has the parameter name 'startDate' (UncommunicativeName) Account#examineBalanceForPeriod refers to entry more than self (LowCohesion) Account#examineForAnyPeriod calls entry.getDate 3 times (Duplication) Account#examineForAnyPeriod has the name 'examineForAnyPeriod' (UncommunicativeName) Account#examineForAnyPeriod has the parameter name 'endDate' (UncommunicativeName) Account#examineForAnyPeriod has the parameter name 'startDate' (UncommunicativeName) Account#examineForAnyPeriod refers to entry more than self (LowCohesion) Account#getAccountName has the name 'getAccountName' (UncommunicativeName) Account#getAccountNumber has the name 'getAccountNumber' (UncommunicativeName) Account#getBalance has the name 'getBalance' (UncommunicativeName) Account#getSubBal has the name 'getSubBal' (UncommunicativeName) Account#initialize has the parameter name 'subAccName' (UncommunicativeName) Account#initialize has the parameter name 'subAccNum' (UncommunicativeName) Account#initialize has the variable name 'd' (UncommunicativeName) Account#initialize has the variable name 'm' (UncommunicativeName) Account#initialize has the variable name 'subAccName' (UncommunicativeName) Account#initialize has the variable name 'subAccNum' (UncommunicativeName) Account#initialize has the variable name 'y' (UncommunicativeName) Account#initialize is controlled by argument subAccName (ControlCouple) Account#writeFile has the name 'writeFile' (UncommunicativeName) Account#writeFile has the variable name 'aFile' (UncommunicativeName) AccountEntry.rb -- 12 warnings: AccountEntry has the variable name '@subAccNum' (UncommunicativeName) AccountEntry has the variable name '@transNumber' (UncommunicativeName) AccountEntry#getBalance has the name 'getBalance' (UncommunicativeName) AccountEntry#getCredit has the name 'getCredit' (UncommunicativeName) AccountEntry#getDate has the name 'getDate' (UncommunicativeName) AccountEntry#getDebit has the name 'getDebit' (UncommunicativeName) AccountEntry#getDescription has the name 'getDescription' (UncommunicativeName) AccountEntry#getEntry has the name 'getEntry' (UncommunicativeName) AccountEntry#getTransNumber has the name 'getTransNumber' (UncommunicativeName) AccountEntry#initialize has 7 parameters (LongParameterList) AccountEntry#initialize has the parameter name 'subAcc' (UncommunicativeName) AccountEntry#initialize has the parameter name 'transNumber' (UncommunicativeName) AccountingSystem.rb -- 23 warnings: AccountingSystem has the variable name '@accountList' (UncommunicativeName) AccountingSystem#getAccounts has the name 'getAccounts' (UncommunicativeName) AccountingSystem#readFile contains iterators nested 2 deep (NestedIterators) AccountingSystem#readFile has approx 12 statements (LongMethod) AccountingSystem#readFile has the name 'readFile' (UncommunicativeName) AccountingSystem#readFile has the variable name 'aAccount' (UncommunicativeName) AccountingSystem#readFile has the variable name 'aFile' (UncommunicativeName) AccountingSystem#readFile has the variable name 'accountFile' (UncommunicativeName) AccountingSystem#readFile has the variable name 'd' (UncommunicativeName) AccountingSystem#readFile has the variable name 'entryDate' (UncommunicativeName) AccountingSystem#readFile has the variable name 'm' (UncommunicativeName) AccountingSystem#readFile has the variable name 'tokenDate' (UncommunicativeName) AccountingSystem#readFile has the variable name 'y' (UncommunicativeName) AccountingSystem#readFile refers to line more than self (LowCohesion) AccountingSystem#readQIF calls quick.account twice (Duplication) AccountingSystem#readQIF calls quick.transactions twice (Duplication) AccountingSystem#readQIF contains iterators nested 2 deep (NestedIterators) AccountingSystem#readQIF has approx 8 statements (LongMethod) AccountingSystem#readQIF has the name 'readQIF' (UncommunicativeName) AccountingSystem#readQIF has the variable name 'qifFile' (UncommunicativeName) AccountingSystem#readQIF refers to quick more than self (LowCohesion) AccountingSystem#saveAccounts calls account twice (Duplication) AccountingSystem#saveAccounts has the name 'saveAccounts' (UncommunicativeName) BalanceSheet.rb -- 7 warnings: BalanceSheet#calculate_total calls account.getBalance twice (Duplication) BalanceSheet#calculate_total has approx 8 statements (LongMethod) BalanceSheet#calculate_total has the parameter name 'accountArray' (UncommunicativeName) BalanceSheet#initialize has the parameter name 'creditAccountsArray' (UncommunicativeName) BalanceSheet#initialize has the parameter name 'debitAccountsArray' (UncommunicativeName) BalanceSheet#make_report has the variable name 'creditTotal' (UncommunicativeName) BalanceSheet#make_report has the variable name 'debitTotal' (UncommunicativeName) BookParser.rb -- 7 warnings: BookParser has no descriptive comment (IrresponsibleModule) BookParser has the variable name '@textFile' (UncommunicativeName) BookParser#createAccounts has the name 'createAccounts' (UncommunicativeName) BookParser#createAccounts has the parameter name 'accountArray' (UncommunicativeName) BookParser#createAccounts refers to accountArray more than self (LowCohesion) BookParser#parseFile has approx 7 statements (LongMethod) BookParser#parseFile has the name 'parseFile' (UncommunicativeName) CheckingJournal.rb -- 17 warnings: CheckingJournal#add_transaction calls (3 + @destinationAccounts.length) twice (Duplication) CheckingJournal#add_transaction calls @destinationAccounts.length twice (Duplication) CheckingJournal#add_transaction has approx 10 statements (LongMethod) CheckingJournal#add_transaction refers to trans more than self (LowCohesion) CheckingJournal#write_journal has approx 6 statements (LongMethod) CheckingJournal#write_journal has the parameter name 'fileName' (UncommunicativeName) CheckingJournal#write_journal has the variable name 'aFile' (UncommunicativeName) CheckingJournal#write_journal has the variable name 'e' (UncommunicativeName) CheckingJournal#write_journal has the variable name 'fileName' (UncommunicativeName) CheckingJournalEntry has no descriptive comment (IrresponsibleModule) CheckingJournalEntry#getFileOutString calls (text.size - 1) twice (Duplication) CheckingJournalEntry#getFileOutString calls self.other_values twice (Duplication) CheckingJournalEntry#getFileOutString calls text.size twice (Duplication) CheckingJournalEntry#getFileOutString calls text[0, (text.size - 1)] twice (Duplication) CheckingJournalEntry#getFileOutString has the name 'getFileOutString' (UncommunicativeName) CheckingJournalEntry#getFileOutString has the variable name 'i' (UncommunicativeName) CheckingJournalEntry#getFileOutString has the variable name 'p' (UncommunicativeName) GeneralJournal.rb -- 26 warnings: GeneralJournal#addTransaction has the name 'addTransaction' (UncommunicativeName) GeneralJournal#calculateTotal doesn't depend on instance state (LowCohesion) GeneralJournal#calculateTotal has the name 'calculateTotal' (UncommunicativeName) GeneralJournal#calculateTotal has the variable name 'i' (UncommunicativeName) GeneralJournal#calculateTotal refers to acc more than self (LowCohesion) GeneralJournal#close calls trans.get_date twice (Duplication) GeneralJournal#close has approx 10 statements (LongMethod) GeneralJournal#close has the variable name 'testString' (UncommunicativeName) GeneralJournal#close has the variable name 'transactionNumber' (UncommunicativeName) GeneralJournal#examineCreditDebit doesn't depend on instance state (LowCohesion) GeneralJournal#examineCreditDebit has the name 'examineCreditDebit' (UncommunicativeName) GeneralJournal#examineCreditDebit has the variable name 'i' (UncommunicativeName) GeneralJournal#examineCreditDebit refers to text more than self (LowCohesion) GeneralJournal#examineTransactions calls time.month 3 times (Duplication) GeneralJournal#examineTransactions calls time.year 3 times (Duplication) GeneralJournal#examineTransactions calls w.get_date 3 times (Duplication) GeneralJournal#examineTransactions has approx 11 statements (LongMethod) GeneralJournal#examineTransactions has the name 'examineTransactions' (UncommunicativeName) GeneralJournal#examineTransactions has the variable name 'w' (UncommunicativeName) GeneralJournal#examineTransactions refers to time more than self (LowCohesion) GeneralJournal#getTransactions has the name 'getTransactions' (UncommunicativeName) GeneralJournal#write_journal has approx 7 statements (LongMethod) GeneralJournal#write_journal has the parameter name 'fileString' (UncommunicativeName) GeneralJournal#write_journal has the variable name 'aFile' (UncommunicativeName) GeneralJournal#write_journal has the variable name 'fileString' (UncommunicativeName) GeneralJournal#write_journal has the variable name 't' (UncommunicativeName) Ledger.rb -- 28 warnings: Ledger#getAccount has the name 'getAccount' (UncommunicativeName) Ledger#getAccount_bynumber has the name 'getAccount_bynumber' (UncommunicativeName) Ledger#printFooter doesn't depend on instance state (LowCohesion) Ledger#printFooter has the name 'printFooter' (UncommunicativeName) Ledger#printFooter has the parameter name 'netIncome' (UncommunicativeName) Ledger#printFooter refers to netIncome more than self (LowCohesion) Ledger#printHeader has the name 'printHeader' (UncommunicativeName) Ledger#printIncomeStatement calls ((statement + account.getBalance.to_s) + "") twice (Duplication) Ledger#printIncomeStatement calls (statement + account.getBalance.to_s) twice (Duplication) Ledger#printIncomeStatement calls account.getBalance 4 times (Duplication) Ledger#printIncomeStatement calls account.getBalance.to_s twice (Duplication) Ledger#printIncomeStatement has approx 12 statements (LongMethod) Ledger#printIncomeStatement has the name 'printIncomeStatement' (UncommunicativeName) Ledger#printIncomeStatement has the variable name 'netIncome' (UncommunicativeName) Ledger#printIncomeStatementForAnyPeriod calls ((statement + account.getBalance.to_s) + "") twice (Duplication) Ledger#printIncomeStatementForAnyPeriod calls (statement + account.getBalance.to_s) twice (Duplication) Ledger#printIncomeStatementForAnyPeriod calls account.getBalance 4 times (Duplication) Ledger#printIncomeStatementForAnyPeriod calls account.getBalance.to_s twice (Duplication) Ledger#printIncomeStatementForAnyPeriod has approx 12 statements (LongMethod) Ledger#printIncomeStatementForAnyPeriod has the name 'printIncomeStatementForAnyPeriod' (UncommunicativeName) Ledger#printIncomeStatementForAnyPeriod has the variable name 'netIncome' (UncommunicativeName) Ledger#printLineExp doesn't depend on instance state (LowCohesion) Ledger#printLineExp has the name 'printLineExp' (UncommunicativeName) Ledger#printLineExp refers to acc more than self (LowCohesion) Ledger#printLineExp refers to statement more than self (LowCohesion) Ledger#printLineRev doesn't depend on instance state (LowCohesion) Ledger#printLineRev has the name 'printLineRev' (UncommunicativeName) Ledger#printLineRev refers to statement more than self (LowCohesion) SpecialtyJournal.rb -- 20 warnings: SpecialtyJournal has the variable name '@destinationAccounts' (UncommunicativeName) SpecialtyJournal#add_transaction calls (2 + @destinationAccounts.length) twice (Duplication) SpecialtyJournal#add_transaction calls @destinationAccounts.length twice (Duplication) SpecialtyJournal#add_transaction has approx 9 statements (LongMethod) SpecialtyJournal#add_transaction refers to trans more than self (LowCohesion) SpecialtyJournal#initialize has the parameter name 'destinationAccounts' (UncommunicativeName) SpecialtyJournal#initialize has the parameter name 'mainAccount' (UncommunicativeName) SpecialtyJournal#write_journal has approx 6 statements (LongMethod) SpecialtyJournal#write_journal has the parameter name 'fileName' (UncommunicativeName) SpecialtyJournal#write_journal has the variable name 'aFile' (UncommunicativeName) SpecialtyJournal#write_journal has the variable name 'e' (UncommunicativeName) SpecialtyJournal#write_journal has the variable name 'fileName' (UncommunicativeName) SpecialtyJournalEntry has no descriptive comment (IrresponsibleModule) SpecialtyJournalEntry#getFileOutString calls (text.size - 1) twice (Duplication) SpecialtyJournalEntry#getFileOutString calls self.other_values twice (Duplication) SpecialtyJournalEntry#getFileOutString calls text.size twice (Duplication) SpecialtyJournalEntry#getFileOutString calls text[0, (text.size - 1)] twice (Duplication) SpecialtyJournalEntry#getFileOutString has the name 'getFileOutString' (UncommunicativeName) SpecialtyJournalEntry#getFileOutString has the variable name 'i' (UncommunicativeName) SpecialtyJournalEntry#getFileOutString has the variable name 'p' (UncommunicativeName) Transaction.rb -- 15 warnings: Transaction#examineCreditDebit doesn't depend on instance state (LowCohesion) Transaction#examineCreditDebit has the name 'examineCreditDebit' (UncommunicativeName) Transaction#examineCreditDebit has the variable name 'i' (UncommunicativeName) Transaction#examineCreditDebit refers to string more than self (LowCohesion) Transaction#getComment has the name 'getComment' (UncommunicativeName) Transaction#getFileOutString has approx 8 statements (LongMethod) Transaction#getFileOutString has the name 'getFileOutString' (UncommunicativeName) Transaction#postTo calls Time.now 6 times (Duplication) Transaction#postTo calls Time.now.day twice (Duplication) Transaction#postTo calls Time.now.month twice (Duplication) Transaction#postTo calls Time.now.year twice (Duplication) Transaction#postTo calls getComment twice (Duplication) Transaction#postTo has approx 6 statements (LongMethod) Transaction#postTo has the name 'postTo' (UncommunicativeName) Transaction#postTo has the parameter name 'transactionNumber' (UncommunicativeName)