fixed codes

Author: itsanshulverma

ques_2.py

@@ -1,6 +1,6 @@
 
-def calc(sales):
-    total_sales = 4*sales
+def calc(sale1, sale2, sale3, sale4):
+    total_sales = sale1+sale2+sale3+sale4
     com_amt = 0 #Commission
     if total_sales >= 50000:
         com_amt = total_sales*0.05
@@ -17,9 +17,12 @@ def calc(sales):
     return total_sales, com_amt, remarks
 
 if __name__ == "__main__":
-    sales = int(input("Enter the sales per week: "))
-    t_sales, comm, remarks = calc(sales)
+    sale1 = int(input("Enter the sales for week 1: "))
+    sale2 = int(input("Enter the sales for week 2: "))
+    sale3 = int(input("Enter the sales for week 3: "))
+    sale4 = int(input("Enter the sales for week 4: "))
+    t_sales, comm, remarks = calc(sale1, sale2, sale3, sale4)
 
     print("Total Sales: Rs.{:.2f}".format(t_sales))
     print("Commission: Rs.{:.2f}".format(comm))
-    print("Remarks: ", remarks)
+    print("Remarks:", remarks)

ques_3.py

@@ -6,7 +6,9 @@ def factorial(n):
 
 # Fibonacci function to find nth term of series
 def fib(n): #Recursive
-    if n <= 1:
+    if n <= 0:
+        return 0
+    if n == 1:
         return 1
     return fib(n-1) + fib(n-2)
 

ques_5.py

@@ -5,11 +5,8 @@
 def sum_series(x, n):
     sum = 0
     for i in range(1, n+1, 1):
-        term = x**(2*i-2)/factorial(2*i-2) #Calculating nth term
-        if i % 2 == 0:
-            sum -= term
-        else:
-            sum += term
+        term = ((-1)**(i+1))*(x**(2*i-2)/factorial(2*i-2)) #Calculating nth term
+        sum += term
     return sum
 
 if __name__ == "__main__":

ques_7.py

@@ -7,9 +7,9 @@ def maxof_three():
     str2= input("Enter string 2: ")
     str3 = input("Enter string 3: ")
     maxstr = ""
-    if len(str1) > len(str2) and len(str1) > len(str3):
+    if str1 > str2 and str1 > str3:
         maxstr = str1
-    elif len(str2) > len(str1) and len(str2) > len(str3):
+    elif str2 > str1 and str2 > str3:
         maxstr = str2
     else:
         maxstr = str3
@@ -45,31 +45,33 @@ def palindrome():
     else:
         print(f"{str} is not a palindrome")
 
+def main():
+    print("\nMenu")
+    print("-"*20)
+    print("1. Length of string")
+    print("2. Maximum of three strings")
+    print("3. Replace vowels with '#'")
+    print("4. No. of words")
+    print("5. Check Palindrome")
+    print("6. Exit")
+    print("-"*20)
+    option = input("Your choice: ")
+
+    switcher = {
+        '1' : len_str,
+        '2' : maxof_three,
+        '3' : replace_vowels,
+        '4' : numofwords,
+        '5' : palindrome,
+        '6' : quit
+    }
+    func = switcher.get(option, lambda: print("Invalid Choice!"))
+    func()
+
 if __name__ == "__main__":
 
     ch = 'y'
     while ch.lower() == 'y':
-        print("\nMenu")
-        print("-"*20)
-        print("1. Length of string")
-        print("2. Maximum of three strings")
-        print("3. Replace vowels with '#'")
-        print("4. No. of words")
-        print("5. Check Palindrome")
-        print("6. Exit")
-        print("-"*20)
-        option = input("Your choice: ")
-
-        switcher = {
-            '1' : len_str,
-            '2' : maxof_three,
-            '3' : replace_vowels,
-            '4' : numofwords,
-            '5' : palindrome,
-            '6' : quit
-        }
-        func = switcher.get(option, lambda: print("Invalid Choice!"))
-        func()
-
+        main()
         ch = input("\nWant to continue? [y/n]: ")
 

ques_8.py

@@ -78,25 +78,18 @@ def common(l1, l2):
 		print("All elements are numbers")
 	else:
 		print("All elements are not numbers")
-	
 	count_odd(l)
-	
 	largest_str(l)
-	
 	print("Reversed list:", end=" ")
 	display_reverse(l)
-	
 	item = input("\nEnter an element: ")
 	find_item(item,l)
-	
 	remove_item(item, l)
-	
 	print("Sorted in Descending Order: ")
 	print(sort_desc(l))
 	l2 = []
 	n = int(input("Enter the size of new list: "))
 	for i in range(0, n, 1):
 		l2.append(input("Enter element: "))
-
 	common(l, l2)