Fix: Enhance the warning of ntype (#7239)

* Fix: Enhance the warning of ntype

* Fix: update according to copilot

Author: sunliang98

source/source_io/module_parameter/read_input.cpp

@@ -9,6 +9,7 @@
 #include <array>
 #include <vector>
 #include <cassert>
+#include <cctype>
 #include <limits>
 #include "source_base/formatter.h"
 #include "source_base/global_file.h"
@@ -522,32 +523,32 @@ void ReadInput::check_ntype(const std::string& fn, int& param_ntype)
             {
                 break;
             }
-            else if (isalpha(temp[0]))
+            else if (!temp.empty() && std::isalpha(static_cast<unsigned char>(temp[0])))
             {
                 ntype_stru += 1;
             }
         }
     }
 
-    if (param_ntype == 0)
+    if (ntype_stru <= 0)
     {
-        param_ntype = ntype_stru;
-        GlobalV::ofs_running << " 'ntype' is no longer required in INPUT, and "
-                                "it will be ignored. "
-                             << std::endl;
+        ModuleBase::WARNING_QUIT("ReadInput::check_ntype",
+                                 "Failed to detect valid ntype from STRU: no valid ATOMIC_SPECIES entries were found.");
     }
-    else if (param_ntype != ntype_stru)
+
+    if (param_ntype < 0)
     {
-        ModuleBase::WARNING_QUIT("ReadInput",
-                                 "The ntype in INPUT is not equal to the ntype "
-                                 "counted in STRU, check it.");
+        ModuleBase::WARNING_QUIT("ReadInput::check_ntype", "The ntype in INPUT should not be less than 0.");
     }
-    if (param_ntype <= 0)
+    else if (param_ntype != 0 && param_ntype != ntype_stru)
     {
-        ModuleBase::WARNING_QUIT("ReadInput", "ntype should be greater than 0.");
+        ModuleBase::WARNING_QUIT("ReadInput::check_ntype",
+                                 "The ntype in INPUT is not equal to the ntype "
+                                 "counted in STRU, check it.");
     }
-    else
+    else if (param_ntype == 0)
     {
+        param_ntype = ntype_stru;
         GlobalV::ofs_running << " 'ntype' is automatically set to " << param_ntype << std::endl;
     }
 }