Go Code It

4 responses to “Detailed Testing Feedback”

1. 

   Pete Silk

   September 16, 2024

   Apparent bug on CGi Unit 4, “Teachers Test Score Calculator”:
   Code written:
   print(“How many scores do you want to enter?”)
   numScores = int(input(“”))
   totalScore = 0
   i = 0
   testScore = 0

   isEnd = False
   while not isEnd:
   print(“Please enter a score”)
   testScore = int(input())
   if testScore > 100 or testScore < 0:
   print("Please enter a score")
   testScore = int(input())
   else:
   totalScore += testScore
   i += 1
   if i == numScores:
   isEnd = True

   averageScore = totalScore / numScores

   print(f"Number of tests was {numScores}")
   print(f"Total score was {totalScore}")
   print(f"Average score was {int(averageScore)}")

   The result when clicking "Test":

   ValueError: invalid literal for int() with base 10: '' on line 10

   It runs just fine when pressing "Run".

   Reply
   1. 

      Pete (GoCodeIt)

      September 16, 2024

      Hi,

      Thanks for the testing feedback. There is an issue with your code that means it does not work in all cases. The tests are designed to ensure that all cases need to be passed.

      In your code, within the while loop, you are allowing the value to be entered twice, once initially, which is okay if the code passes, but then again if the code does not pass. However, because the second number entry is done inside the first part of the if, if that second number IS valid then it is not added to the total or counted as an entry (so i is also not incremented).

      You can test this with the following values:
      3, 101,1,-2,80,99
      This is 3 numbers to enter followed by 101 etc.

      You will note that your code ask for another number to enter even though 3 valid numbers have already been entered (1,80 and 99)
      The resolution to this and the way I teach validation is to initialise the value to be validated first so that it is not valid and do this outside the loop – e.g

      number = -1
      while number 100:
      number = int(input(“Enter number”)

      This forces the while to execute at least once.

      Reply
2. 

   Pete Silk

   September 16, 2024

   Apparent bug found on Circle Properties question on CGi Time2Code questions:
   When testing, two of the tests fail as the C&D solution is successfully run in IDLE but not in CGI. CGI expects two tests to be 3 decimal places and two to be 10dp

   # Circle properties program

   # ————————-
   # Subprograms
   # ————————-
   # Calculate the area of a circle.
   def circle_area(diameter):
   radius = diameter / 2
   area = 3.14 * (radius * radius)
   return area

   # Calculate the circumference of a circle.
   def circle_circumference(diameter):
   circumference = 3.14 * diameter
   return circumference

   # ————————-
   # Main program
   # ————————-
   diameter = float(input(“Enter the diameter of the circle: “))
   area = circle_area(diameter)
   circumference = circle_circumference(diameter)
   print(“Area:”, area)
   print(“Circumference:”, circumference)

   Reply
   1. 

      Pete (GoCodeIt)

      September 16, 2024

      Hi again,

      Good spot. This is one of those edge cases for the platform.

      In most Python interpreters, floating-point arithmetic can display numbers with long decimal places, like 31.400000000000002. However, Skulpt and JavaScript avoid showing unnecessarily long floating-point precision by default for readability, typically rounding to a reasonable number of decimal places (2 in this case).

      Trinket also uses the same interpreter and rounds numbers as do many online python interpreters that use Javascript as their base as opposed to a virtual machine running python directly.

      You can force it to display the necessary decimal points using formatting functions, but this will probably be outside of the knowledge of most students using GoCodeIT.

      I have adjusted the test to account for the rounding.

      Reply