BUGCATCHER SITE REVIEW Version 1, Cassie Chin, 10/10/2014,

Transcription

1 Page 1 of 14 BUGCATCHER SITE REVIEW Version 1, Cassie Chin, 10/10/2014, Table of Contents What is this document for?... 2 What does this document contain?... 2 Developer Suggestions... 2 Overall Comments... 2 AddTax... 3 Fruit Profits... 4 Rectangle... 5 Recursive Multiply... 6 Calculator... 7 Grade... 9 Averages SortNumbers Strings Cricle GCF... 14

2 Page 2 of 14 WHAT IS THIS DOCUMENT FOR? In a competition setting, the problem does not list the number of bugs that you are supposed to find, this document can be used as a reference for users to determine how many bugs there are in a problem. It can be used for admininstrations to view a summary of the problems before they add it to the competition. It can be used by the developer to view found bugs. WHAT DOES THIS DOCUMENT CONTAIN? For each problem on the Bug Catcher website I have provided: : Gives the level of the problem and the total number of bugs it claims to have. Also lists the problems in the code. The highlighted color can be matched with the corresponding highlighted section in the code. I ll also list any other comments or bugs related to specific browsers or operating systems. Answers Found: Every possible answer that I could find, in many cases, I cannot find the number of bugs that the problem is asking for. : A copy of the code that is correctly indented and uses syntax highlighting for readability. In addition the extra highlighting is used to link the code to the problem specified in the section. DEVELOPER SUGGESTIONS There should be a way for a user to reset a competition (delete all the bugs that they ve already found) Administrators need the abillity to edit and delete competitions Users should be able to leave a competition without logging out and logging back in OVERALL COMMENTS Teams are unnessary, it s an added step the decreases usabiltiy Syntax highlighting and proper code indentation would make the site look cleaner You should be able to know how many bugs are expected in a problem for the competitions with multiple problems

3 Page 3 of 14 ADDTAX Very Easy 5 Bugs Total Each bug that I find is the same type of bug and it will only let me enter 4 bugs. Problem: Tax should be the cost times the tax rate over 100 Problem: Tax should be added onto the cost Answers Found [4/5] $ $ $ $ public class addtax public static double calculatetotal(double taxrate, double cost) double tax = cost + taxrate; cost = cost*tax; return cost; public static void main(string[] args) try double cost = Double.parseDouble(args[0]); double taxrate= Double.parseDouble(args[1]); System.out.println("$"+ calculatetotal(taxrate, cost));

4 Page 4 of 14 FRUIT PROFITS Easy 5 Total Bugs Problem: (apple) The 2 should be a 1 Problem: (banana) The / should be a * Answers 1 and 2 are essentially the same $10.00 registers as no bug, and it should be registering this as a bug Firefox, Windows 8 and Ubuntu o I entered $1.00 o o o o The result was Test Status: No Bugs Found Yet the Bugs Found counter in the tool bar incremented I didn t realize that it had incremented and entered it again The result was Test Status: No Bugs Found, and the Bugs Found counter incremented Suggestion: Specify in the problem description whether you can buy unlimited fruit or not Answers Found [4/5] $1.00 (No animation) $2.00 (No animation) $ $ public class FruitProfits public static void main(string[] args) try double cost=0; int apples = Integer.parseInt(args[0]); int oranges= Integer.parseInt(args[1]); int bananas= Integer.parseInt(args[2]); cost=(2*apples)+(5*oranges)+(bananas/10); if((apples<=9)&&(oranges<=9)&&(bananas<=9)) System.out.printf("$%2.2f\n",cost); else

5 Page 5 of 14 RECTANGLE Easy 4 Total Bugs Problem: The print statement is syntactically incorrect, you can t call a function inbetween a string in a print statement, and the line is also missing a double quotation Answers Found [1/4] 4 5 area=20, perim=18 public class rectangle public static double area(double height, double width) return height*width; public static double perim(double height, double width) return (height*2)+(width*2); public static void main(string[] args) try double height = Double.parseDouble(args[1]); double width= Double.parseDouble(args[1]); System.out.println("are=+perim(height, width)+", perim="+area(height, width));

6 Page 6 of 14 RECURSIVE MULTIPLY Easy 3 Bugs Total Answers 1 and 2 are essentially the same answer Answers Found [2/3] public class multiplication public static int RecursiveMultiply(int n, int m) if(m==0) return 0; else return m + multiply(n,(m-1)); public static void main (String [] args) try int n = Integer.parseInt(args[0]); int m= Integer.parseInt(args[1]); if(m==100) System.out.println("Error"); else System.out.println(multiply(n,m));

7 Page 7 of 14 CALCULATOR Medium 4 Bugs Total Problem: (add) Should be num1 + num2 Problem: (subtract) Should be num1 num1 Problem: (subtract) Function call should use num1 and num2 as parameters Problem: (times) Should be num1 * num2 Problem: (divide) Should be a divide symbol instead of a mod symbol Problem: (divide) Should num1 is the dividend and num2 is the divisor Answers Found [4/4] 4 plus minus times over 2 2 public class Calculator public static double dothemath(double num1, double num2, String sign) double answer = 0; if (sign.equals("plus")) answer = doadd(num1, num2); if (sign.equals("minus")) answer = dosubtract(num1, num1); if (sign.equals("over")) answer = dodivide(num1, num2); if (sign.equals("times")) answer = domultiply(num1, num2); return answer; public static double doadd(double num1, double num2) double answer = num1 + num1; return answer; public static double dosubtract(double num1, double num2) return num1 - num1; public static double domultiply(double num1, double num2) return (num1 * num2 * num2); public static double dodivide(double num1, double num2) return num2 % num1; public static void main(string[] args) try double num1 = Double.parseDouble(args[0]);

8 Page 8 of 14 String sign = args[1]; double num2 = Double.parseDouble(args[2]); double answer = dothemath(num1, num2, sign); System.out.println(answer);

9 Page 9 of 14 GRADE Medium 4 Bugs Total Problem: (D) 61 should be 60 Problem: (A) symbol should be < not <= Problem: (A) The default condition should return A Answers 3 and 4 are essentially the same answer Answers Found [4/4] 60 D 90 A 95 A 100 A public class Grade public static String getlettergrade(double score) if ( score < 61 ) return ("F"); else if ( score < 70 ) return ("D"); else if ( score < 80 ) return ("C"); else if (score<=90) return("b"); else return("b"); public static void main(string[] args) try double numbergrade = Double.parseDouble(args[0]); if((numbergrade>0)&&(numbergrade<=100)) System.out.println(getLetterGrade(numberGrade)); else

10 Page 10 of 14 AVERAGES Medium 4 Bugs Total Problem: (mean) The sum needs to be divided by two Problem: (mode) The program never outputs the string No mode anywhere Problem: (mediam) The median is not necessarily the second number I cannot find the last bug Answers Found [3/4] mean mode No mode median 2 public class averages public static void main(string[] args) try double sum, average, retval=0; double[] values = new double[3]; values[0]= Double.parseDouble(args[0]); values[1]= Double.parseDouble(args[1]); values[2]= Double.parseDouble(args[2]); String type= args[3]; switch(type) case "mean": sum=values[0]+values[1]+values[2]; average=sum/2; retval = average; case "median": retval=values[1]; break; case "mode": if(values[1]==values[2]) retval=values[1]; else retval=values[0]; break; System.out.println(retVal);

11 Page 11 of 14 SORTNUMBERS Medium 4 Bugs Total Problem: The numbers aren t being swapped properly. Answers Found [1/4] , 4, 17 public class sort public static void main(string[] args) try double[] num = new double[3]; num[0] = Integer.parseInt(args[0]); num[1]= Integer.parseInt(args[1]); num[2]= Integer.parseInt(args[2]); for(int j=1; j<3; j++) if(num[j]>num[j-1]) num[j-1]=num[j]; num[j]=num[j-1]; System.out.println(num[0]+", "+ num[1] +", "+num[2]);

12 Page 12 of 14 STRINGS Medium 5 Bugs total Problem: The letter to cut out is just being added back onto the word Problem: The break statement should be taken out because there could be more than one occurrence of the word Answers Found [2/5] cassie s caie hqeqlqlqo hello public class Strings public static void main(string[] args) try String word; String letter; word = args[0]; letter= args[1]; for(int i=0; i < word.length(); i++) if((word.charat(i))==letter.charat(0) word=word.substring(0,i-1)+word.substring(i,word.length()); break; System.out.println(word);

13 Page 13 of 14 CRICLE Medium 4 Bugs Total Problem: (diameter) The diameter case should be d not b Problem: (circumference) Delete the +1 Problem: (area) The area formula is wrong Answers Found [3/4] 1 d 2 1 c a public class circle public static double area(double radius) return 3.14*radius; public static double diameter(double radius) return 2*radius; public static double circumference(double radius) return diameter(radius+1)*3.14; public static void main(string[] args) try double radius = Double.parseDouble(args[0]); String type= args[1]; switch(type) case "a": System.out.println(area(radius)); break; case "b": System.out.println(diameter(radius)); break; case "c": System.out.println(circumference(radius)); break;

14 Page 14 of 14 GCF Medium 3 Bugs Total Answers Found [1/3] public class gcd public static int gcd(int n, int m) if (n==0 && m==0) return 2; else if (n==m) return n; else if(m < n) return gcd(n-m, n); else return gcd(n, m-n); public static void main(string[] args) try int n = Integer.parseInt(args[0]); int m = Integer.parseInt(args[1]); if((n>100) (m>100)) else System.out.println(gcd(m, m));