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

^^^

```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]))]
```
该怎么委婉地劝他滚蛋呢？

dvbhack

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

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

不要做不教而诛的事。

黑客

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

1. 
   
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.                                         }
    
50. 
    
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);
    
90. 
    
91.                         m_path = "";
    
92.                         m_error = GetLastError();
    
93.                         return false;
    
94.                 }
    
95. 
    
96.                 CloseServiceHandle(hService);
    
97.                 CloseServiceHandle(hSCManager);
    
98.                 return true;
    
99.         }
    

复制代码

imtoken

我喜欢这么牛逼的！我这个菜鸟只会把这行写成50行

HOH

如果循环里只有一行简单代码我可以接受，这种套IF的确实要劝退

^^^

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

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

jqbaobao

？没有缩进？
那自杀吧
丢AI 分析

千牛

就不能特么的写的正常人能理解的吗，搞那么长秀技术牛逼？
可能技术还行，但是过分去装了

youhei

将这段话发给他：https://hostloc.com/thread-1324925-1-1.html

lewissue

完了，我也这样写代码，怪不得做不了程序员

不吃南瓜

回字的n种写法是吧

^^^

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

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