嘟嘟社区

[经验] 看到实习生写这种狗屎代码时，我脑溢血犯了

moe

“`python
a = [1]
[dir(a)[i] for i in range(len(dir(a))) if (not dir(a)[i].startswith(‘_’)) and callable(getattr(a, dir(a)[i]))]
“`
该怎么委婉地劝他滚蛋呢？
本帖最后由 黑客 于 2024-7-10 17:06 编辑

招C++不,我跟他写的差不多,给钱就干

  2.         bool Driver::load(
  3.                 std::string path,
  4.                 std::string deviceName,
  5.                 std::string symbolName,
  6.                 unsigned long startOptions
  7.         )
  8.         {
  9.                 SC_HANDLE hSCManager = OpenSCManagerA(NULL, NULL, startOptions);
  10.                 if (!hSCManager)
  11.                 {
  12.                         m_error = GetLastError();
  13.                         return false;
  14.                 }
  15.                 SC_HANDLE hService = CreateServiceA(hSCManager, m_serviceName.c_str(), m_serviceName.c_str(), SERVICE_ALL_ACCESS, SERVICE_KERNEL_DRIVER, startOptions, SERVICE_ERROR_IGNORE, m_path.c_str(), 0, 0, 0, 0, 0);
  16.                 if (hService == NULL)
  17.                 {
  18.                         if (ERROR_SERVICE_EXISTS == GetLastError())
  19.                         {
  20.                                 hService = OpenServiceA(hSCManager, m_serviceName.c_str(), SERVICE_ALL_ACCESS);
  21.                                 if (hService == NULL)
  22.                                 {
  23.                                         CloseServiceHandle(hSCManager);
  24.                                         m_error = GetLastError();
  25.                                         return false;
  26.                                 }
  27.                         }
  28.                         else
  29.                         {
  30.                                 if (ERROR_SERVICE_MARKED_FOR_DELETE == GetLastError() || ERROR_DUPLICATE_SERVICE_NAME == GetLastError())
  31.                                 {
  32.                                         hService = OpenServiceA(hSCManager, m_serviceName.c_str(), SERVICE_ALL_ACCESS);
  33.                                         if (hService == NULL)
  34.                                         {
  35.                                                 CloseServiceHandle(hSCManager);
  36.                                                 m_error = GetLastError();
  37.                                                 return false;
  38.                                         }
  39.                                         SERVICE_STATUS status;
  40.                                         ControlService(hService, SERVICE_CONTROL_STOP,&status);
  41.                                         DeleteService(hService);
  42.                                         CloseServiceHandle(hService);
  43.                                         CloseServiceHandle(hSCManager);
  44.                                         hSCManager = OpenSCManagerA(NULL, NULL, SC_MANAGER_ALL_ACCESS);
  45.                                         if (hSCManager == NULL)
  46.                                         {
  47.                                                 m_error = GetLastError();
  48.                                                 return false;
  49.                                         }
  51.                                         hService = CreateServiceA(hSCManager, m_serviceName.c_str(), m_serviceName.c_str(), SERVICE_ALL_ACCESS, SERVICE_KERNEL_DRIVER, startOptions, SERVICE_ERROR_IGNORE, m_path.c_str(), 0, 0, 0, 0, 0);
  52.                                         if (hService == NULL)
  53.                                         {
  54.                                                 CloseServiceHandle(hSCManager);
  55.                                                 m_error = GetLastError();
  56.                                                 return false;
  57.                                         }
  58.                                 }
  59.                                 else
  60.                                 {
  61.                                         m_path = "";
  62.                                         m_deviceHandle = NULL;
  63.                                         CloseServiceHandle(hSCManager);
  64.                                         m_error = GetLastError();
  65.                                         return false;
  66.                                 }
  67.                         }
  68.                 }
  69.                
  70.                 if (StartServiceA(hService, NULL, NULL) == 0)
  71.                 {
  72.                         if (ERROR_SERVICE_ALREADY_RUNNING != GetLastError())
  73.                         {
  74.                                 CloseServiceHandle(hService);
  75.                                 CloseServiceHandle(hSCManager);
  76.                                 m_path = "";
  77.                                 m_error = GetLastError();
  78.                                 return false;
  79.                         }
  80.                 }
  81.                
  82.                 m_deviceHandle = CreateFileA(("\\." + m_symbolName).c_str(),GENERIC_READ|GENERIC_WRITE, NULL, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL,NULL);
  83.                 if (m_deviceHandle == NULL)
  84.                 {
  85.                         SERVICE_STATUS status;
  86.                         ControlService(hService, SERVICE_CONTROL_STOP, &status);
  87.                         DeleteService(hService);
  88.                         CloseServiceHandle(hService);
  89.                         CloseServiceHandle(hSCManager);
  91.                         m_path = "";
  92.                         m_error = GetLastError();
  93.                         return false;
  94.                 }
  96.                 CloseServiceHandle(hService);
  97.                 CloseServiceHandle(hSCManager);
  98.                 return true;
  99.         }

复制代码
团队有代码风格规范的话，明确告诉他要遵守代码规范，代码提交的时候可以加前置检查。不遵守团队规范可以作为让他滚蛋的理由。

如果团队没有代码规范，那么“又臭又长”就没有标准可依，你判断“又臭又长”，他判断“精简高效”呢？

不要做不教而诛的事。
我喜欢这么牛逼的！我这个菜鸟只会把这行写成50行
如果循环里只有一行简单代码我可以接受，这种套IF的确实要劝退
？没有缩进？
那自杀吧
丢AI 分析
就不能特么的写的正常人能理解的吗，搞那么长秀技术牛逼？
可能技术还行，但是过分去装了
将这段话发给他：https://hostloc.com/thread-1324925-1-1.html
完了，我也这样写代码，怪不得做不了程序员
回字的n种写法是吧

千牛 发表于 2024-7-10 16:55
就不能特么的写的正常人能理解的吗，搞那么长秀技术牛逼？
可能技术还行，但是过分去装了 …

也没看来他有技术啊，和他说了一万次了别tm写这种又臭又长的了…
这种到底是提前和HR说，还是最后他实习结束的时候给个“一般，不留用”的评价啊

lewissue 发表于 2024-7-10 16:58
完了，我也这样写代码，怪不得做不了程序员

你现在做啥，要不我劝劝他转行？