[WebServer] Mark functions as const + reduce copy of strings (#10339)

TD-er · pre-commit-ci-lite[bot] · web-flow · commit 9e60bbe4bc05 · 2024-09-13T11:39:54.000+03:00
* [WebServer] Mark functions as const + reduce copy of strings

* ci(pre-commit): Apply automatic fixes

---------

Co-authored-by: pre-commit-ci-lite[bot] &lt;117423508+pre-commit-ci-lite[bot]@users.noreply.github.com&gt;
diff --git a/libraries/WebServer/src/Parsing.cpp b/libraries/WebServer/src/Parsing.cpp
@@ -280,7 +280,7 @@ bool WebServer::_collectHeader(const char *headerName, const char *headerValue)
   return false;
 }
 
-void WebServer::_parseArguments(String data) {
+void WebServer::_parseArguments(const String &data) {
   log_v("args: %s", data.c_str());
   if (_currentArgs) {
     delete[] _currentArgs;
@@ -387,7 +387,7 @@ int WebServer::_uploadReadByte(NetworkClient &client) {
   return res;
 }
 
-bool WebServer::_parseForm(NetworkClient &client, String boundary, uint32_t len) {
+bool WebServer::_parseForm(NetworkClient &client, const String &boundary, uint32_t len) {
   (void)len;
   log_v("Parse Form: Boundary: %s Length: %d", boundary.c_str(), len);
   String line;
diff --git a/libraries/WebServer/src/WebServer.cpp b/libraries/WebServer/src/WebServer.cpp
@@ -674,14 +674,14 @@ void WebServer::_streamFileCore(const size_t fileSize, const String &fileName, c
   send(code, contentType, "");
 }
 
-String WebServer::pathArg(unsigned int i) {
+String WebServer::pathArg(unsigned int i) const {
   if (_currentHandler != nullptr) {
     return _currentHandler->pathArg(i);
   }
   return "";
 }
 
-String WebServer::arg(String name) {
+String WebServer::arg(const String &name) const {
   for (int j = 0; j < _postArgsLen; ++j) {
     if (_postArgs[j].key == name) {
       return _postArgs[j].value;
@@ -695,25 +695,25 @@ String WebServer::arg(String name) {
   return "";
 }
 
-String WebServer::arg(int i) {
+String WebServer::arg(int i) const {
   if (i < _currentArgCount) {
     return _currentArgs[i].value;
   }
   return "";
 }
 
-String WebServer::argName(int i) {
+String WebServer::argName(int i) const {
   if (i < _currentArgCount) {
     return _currentArgs[i].key;
   }
   return "";
 }
 
-int WebServer::args() {
+int WebServer::args() const {
   return _currentArgCount;
 }
 
-bool WebServer::hasArg(String name) {
+bool WebServer::hasArg(const String &name) const {
   for (int j = 0; j < _postArgsLen; ++j) {
     if (_postArgs[j].key == name) {
       return true;
@@ -727,7 +727,7 @@ bool WebServer::hasArg(String name) {
   return false;
 }
 
-String WebServer::header(String name) {
+String WebServer::header(const String &name) const {
   for (int i = 0; i < _headerKeysCount; ++i) {
     if (_currentHeaders[i].key.equalsIgnoreCase(name)) {
       return _currentHeaders[i].value;
@@ -749,25 +749,25 @@ void WebServer::collectHeaders(const char *headerKeys[], const size_t headerKeys
   }
 }
 
-String WebServer::header(int i) {
+String WebServer::header(int i) const {
   if (i < _headerKeysCount) {
     return _currentHeaders[i].value;
   }
   return "";
 }
 
-String WebServer::headerName(int i) {
+String WebServer::headerName(int i) const {
   if (i < _headerKeysCount) {
     return _currentHeaders[i].key;
   }
   return "";
 }
 
-int WebServer::headers() {
+int WebServer::headers() const {
   return _headerKeysCount;
 }
 
-bool WebServer::hasHeader(String name) {
+bool WebServer::hasHeader(const String &name) const {
   for (int i = 0; i < _headerKeysCount; ++i) {
     if ((_currentHeaders[i].key.equalsIgnoreCase(name)) && (_currentHeaders[i].value.length() > 0)) {
       return true;
@@ -776,7 +776,7 @@ bool WebServer::hasHeader(String name) {
   return false;
 }
 
-String WebServer::hostHeader() {
+String WebServer::hostHeader() const {
   return _hostHeader;
 }
 
diff --git a/libraries/WebServer/src/WebServer.h b/libraries/WebServer/src/WebServer.h
@@ -158,10 +158,10 @@ class WebServer {
   void onNotFound(THandlerFunction fn);     //called when handler is not assigned
   void onFileUpload(THandlerFunction ufn);  //handle file uploads
 
-  String uri() {
+  String uri() const {
     return _currentUri;
   }
-  HTTPMethod method() {
+  HTTPMethod method() const {
     return _currentMethod;
   }
   virtual NetworkClient &client() {
@@ -174,24 +174,24 @@ class WebServer {
     return *_currentRaw;
   }
 
-  String pathArg(unsigned int i);                                               // get request path argument by number
-  String arg(String name);                                                      // get request argument value by name
-  String arg(int i);                                                            // get request argument value by number
-  String argName(int i);                                                        // get request argument name by number
-  int args();                                                                   // get arguments count
-  bool hasArg(String name);                                                     // check if argument exists
+  String pathArg(unsigned int i) const;                                         // get request path argument by number
+  String arg(const String &name) const;                                         // get request argument value by name
+  String arg(int i) const;                                                      // get request argument value by number
+  String argName(int i) const;                                                  // get request argument name by number
+  int args() const;                                                             // get arguments count
+  bool hasArg(const String &name) const;                                        // check if argument exists
   void collectHeaders(const char *headerKeys[], const size_t headerKeysCount);  // set the request headers to collect
-  String header(String name);                                                   // get request header value by name
-  String header(int i);                                                         // get request header value by number
-  String headerName(int i);                                                     // get request header name by number
-  int headers();                                                                // get header count
-  bool hasHeader(String name);                                                  // check if header exists
+  String header(const String &name) const;                                      // get request header value by name
+  String header(int i) const;                                                   // get request header value by number
+  String headerName(int i) const;                                               // get request header name by number
+  int headers() const;                                                          // get header count
+  bool hasHeader(const String &name) const;                                     // check if header exists
 
-  int clientContentLength() {
+  int clientContentLength() const {
     return _clientContentLength;
   }  // return "content-length" of incoming HTTP header from "_currentClient"
 
-  String hostHeader();  // get request host header if available or empty String if not
+  String hostHeader() const;  // get request host header if available or empty String if not
 
   // send response to the client
   // code - HTTP response code, can be 200 or 404
@@ -240,9 +240,9 @@ class WebServer {
   void _handleRequest();
   void _finalizeResponse();
   bool _parseRequest(NetworkClient &client);
-  void _parseArguments(String data);
+  void _parseArguments(const String &data);
   static String _responseCodeToString(int code);
-  bool _parseForm(NetworkClient &client, String boundary, uint32_t len);
+  bool _parseForm(NetworkClient &client, const String &boundary, uint32_t len);
   bool _parseFormUploadAborted();
   void _uploadWriteByte(uint8_t b);
   int _uploadReadByte(NetworkClient &client);

Original file line number	Diff line number	Diff line change
`@@ -674,14 +674,14 @@ void WebServer::_streamFileCore(const size_t fileSize, const String &fileName, c`
`674`	`674`	`send(code, contentType, "");`
`675`	`675`	`}`
`676`	`676`
`677`		`-String WebServer::pathArg(unsigned int i) {`
	`677`	`+String WebServer::pathArg(unsigned int i) const {`
`678`	`678`	`if (_currentHandler != nullptr) {`
`679`	`679`	`return _currentHandler->pathArg(i);`
`680`	`680`	`}`
`681`	`681`	`return "";`
`682`	`682`	`}`
`683`	`683`
`684`		`-String WebServer::arg(String name) {`
	`684`	`+String WebServer::arg(const String &name) const {`
`685`	`685`	`for (int j = 0; j < _postArgsLen; ++j) {`
`686`	`686`	`if (_postArgs[j].key == name) {`
`687`	`687`	`return _postArgs[j].value;`
`@@ -695,25 +695,25 @@ String WebServer::arg(String name) {`
`695`	`695`	`return "";`
`696`	`696`	`}`
`697`	`697`
`698`		`-String WebServer::arg(int i) {`
	`698`	`+String WebServer::arg(int i) const {`
`699`	`699`	`if (i < _currentArgCount) {`
`700`	`700`	`return _currentArgs[i].value;`
`701`	`701`	`}`
`702`	`702`	`return "";`
`703`	`703`	`}`
`704`	`704`
`705`		`-String WebServer::argName(int i) {`
	`705`	`+String WebServer::argName(int i) const {`
`706`	`706`	`if (i < _currentArgCount) {`
`707`	`707`	`return _currentArgs[i].key;`
`708`	`708`	`}`
`709`	`709`	`return "";`
`710`	`710`	`}`
`711`	`711`
`712`		`-int WebServer::args() {`
	`712`	`+int WebServer::args() const {`
`713`	`713`	`return _currentArgCount;`
`714`	`714`	`}`
`715`	`715`
`716`		`-bool WebServer::hasArg(String name) {`
	`716`	`+bool WebServer::hasArg(const String &name) const {`
`717`	`717`	`for (int j = 0; j < _postArgsLen; ++j) {`
`718`	`718`	`if (_postArgs[j].key == name) {`
`719`	`719`	`return true;`
`@@ -727,7 +727,7 @@ bool WebServer::hasArg(String name) {`
`727`	`727`	`return false;`
`728`	`728`	`}`
`729`	`729`
`730`		`-String WebServer::header(String name) {`
	`730`	`+String WebServer::header(const String &name) const {`
`731`	`731`	`for (int i = 0; i < _headerKeysCount; ++i) {`
`732`	`732`	`if (_currentHeaders[i].key.equalsIgnoreCase(name)) {`
`733`	`733`	`return _currentHeaders[i].value;`
`@@ -749,25 +749,25 @@ void WebServer::collectHeaders(const char *headerKeys[], const size_t headerKeys`
`749`	`749`	`}`
`750`	`750`	`}`
`751`	`751`
`752`		`-String WebServer::header(int i) {`
	`752`	`+String WebServer::header(int i) const {`
`753`	`753`	`if (i < _headerKeysCount) {`
`754`	`754`	`return _currentHeaders[i].value;`
`755`	`755`	`}`
`756`	`756`	`return "";`
`757`	`757`	`}`
`758`	`758`
`759`		`-String WebServer::headerName(int i) {`
	`759`	`+String WebServer::headerName(int i) const {`
`760`	`760`	`if (i < _headerKeysCount) {`
`761`	`761`	`return _currentHeaders[i].key;`
`762`	`762`	`}`
`763`	`763`	`return "";`
`764`	`764`	`}`
`765`	`765`
`766`		`-int WebServer::headers() {`
	`766`	`+int WebServer::headers() const {`
`767`	`767`	`return _headerKeysCount;`
`768`	`768`	`}`
`769`	`769`
`770`		`-bool WebServer::hasHeader(String name) {`
	`770`	`+bool WebServer::hasHeader(const String &name) const {`
`771`	`771`	`for (int i = 0; i < _headerKeysCount; ++i) {`
`772`	`772`	`if ((_currentHeaders[i].key.equalsIgnoreCase(name)) && (_currentHeaders[i].value.length() > 0)) {`
`773`	`773`	`return true;`
`@@ -776,7 +776,7 @@ bool WebServer::hasHeader(String name) {`
`776`	`776`	`return false;`
`777`	`777`	`}`
`778`	`778`
`779`		`-String WebServer::hostHeader() {`
	`779`	`+String WebServer::hostHeader() const {`
`780`	`780`	`return _hostHeader;`
`781`	`781`	`}`
`782`	`782`