Browse Source

create a maintainable way to add HTTP Status codez

Again, nil is default for attrs not present in a table, so if you're not nil, you should just return what you know is defined.  Otherwise, return the "Unknown HTTP Status" default string.
TJ Borromeo 9 years ago
parent
commit
472e397429
1 changed files with 3 additions and 5 deletions
  1. 3 5
      httpserver-header.lua

+ 3 - 5
httpserver-header.lua

@@ -5,11 +5,9 @@
 return function (connection, code, extension)
 
    local function getHTTPStatusString(code)
-      if code == 200 then return "OK" end
-      if code == 404 then return "Not Found" end
-      if code == 400 then return "Bad Request" end
-      if code == 501 then return "Not Implemented" end
-      return "Unknown HTTP status"
+      local codez = {200="OK", 400="Bad Request", 404="Not Found", 501="Not Implemented"}
+      local myResult = codez[code]
+      if myResult then return myResult else return "Unknown HTTP Status" end
    end
 
    local function getMimeType(ext)