Refactor routes and utils/KV modules

routes/index.tsx

@@ -21,15 +21,15 @@ export const handler: Handlers<Props> = {
   async GET(_req, ctx) {
     const courses = await getCourses(cache);
     const session = getCookies(_req.headers)["sessionId"];
+    const total = getNumberOfCourses(courses.courses);
     if (!session) {
       return ctx.render({
         completed: [],
-        total: getNumberOfCourses(courses.courses),
+        total: total,
         courses: courses.courses,
       });
     }
     const completed = (await getStudent(session)).completedCourses;
-    const total = getNumberOfCourses(courses.courses);
     return ctx.render({
       completed,
       total,

utils/KV.ts

@@ -10,23 +10,23 @@ export const createStudent = async (): Promise<student> => {
     if (res.ok) {
         return student
     } else {
-        throw new Error("Failed to create student")
+        return { sessionId: "", completedCourses: [] } as student
     }
 }
 export const getStudent = async (sessionId: string): Promise<student> => {
     const student = await kv.get(["student", sessionId])
     if (student) {
         return student.value as student
     } else {
-        throw new Error("Failed to get student")
+        return { sessionId: "", completedCourses: [] } as student
     }
 }
 export const updateStudent = async (sessionId: string, student: student): Promise<Result> => {
     const res = await kv.set(["student", sessionId], student)
     if (res.ok) {
         return res
     } else {
-        throw new Error("Failed to update student")
+        return {ok: false} as Result
     }
 }
 export const addCompletedCourse = async (sessionId: string, course: string) : Promise<Result> => {
@@ -41,6 +41,6 @@ export const addCompletedCourse = async (sessionId: string, course: string) : Pr
         const res = await updateStudent(sessionId, student)
         return res
     } catch {
-        return { ok: false }
+        return { ok: false } as Result
     }
 }