completed querystring.js

Author: Alex Jamshidi

Sprint-2/implement/querystring.js

@@ -3,14 +3,40 @@ function parseQueryString(queryString) {
   if (queryString.length === 0) {
     return queryParams;
   }
+
+  // Adds percentage-encoded characters
+  queryString = changePEC(queryString)
+
+  // Replaces + with space
+  queryString = queryString.replaceAll("+", " ")
+  
   const keyValuePairs = queryString.split("&");
 
   for (const pair of keyValuePairs) {
-    const [key, value] = pair.split("=");
+    if (pair !== "") {
+    let index = pair.indexOf("=")
+    if (!pair.includes("=")) {index = pair.length}
+    const [key, value] = [pair.slice(0, index), pair.slice(index + 1)]
     queryParams[key] = value;
+    }
+    else {};
   }
 
   return queryParams;
 }
 
+// this function below can have the object PEC expanded to include all percentage-encoded characters
+// currently without adding this database, an unknown PEC could cause the code to get stuck in an infinite loop
+
+function changePEC(string) {
+  const PEC = {"%24": "$", "%2F": "/"}
+  while (string.includes("%")) {
+    const index = string.indexOf("%");
+    const code = string.slice(index, index + 3);
+    if (PEC[code]) {string = string.replace(code, PEC[code]);}
+    else {break;}
+  }
+  return string
+}
+
 module.exports = parseQueryString;

Sprint-2/implement/querystring.test.js

@@ -40,9 +40,11 @@ test("should replace '+' by ' '", () => {
 // Stretch exercise: Handling query strings that contain identical keys
 
 // Delete this test if you are not working on this optional case
+/*
 test("should store values of a key in an array when the key has 2 or more values", () => {
   expect(parseQueryString("key=value1&key=value2&key=value3&foo=bar")).toEqual({
     key: ["value1", "value2", "value3"],
     foo: "bar",
   });
 });
+*/