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

^^^

```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 分析

千牛

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

lotnpp

感觉这段代码写的还行，大多数情况下，列表推导式效率更高且更易懂，缺少注释勉强称为缺点，毕竟写代码的人需要把看代码的当作傻瓜

我是大白菜

^^^ 发表于 2024-7-10 17:32
是啊，也不需要搞列表索引那些乱七八糟的，不知道他怎么想的。

其实我完全不懂代码，一丁点都不懂，这是我让AI简化的。。。

^^^

黑客 发表于 2024-7-10 23:43
社招待遇咋样啊大佬 有意向

私你了。应该是有HC的。

^^^

Eric.c 发表于 2024-7-10 22:49
我喜欢这个，说明对技术有追求

不是，这怎么就有技术了？我没看出来。
编程技术，至少是算法落地的层次吧。

BigBug

除了可读性差 但是简洁高效

^^^

icon 发表于 2024-7-10 22:09
让gpt优化了一下

看到这种我辞职，谁愿意谁伺候！

^^^

yanzhiling2002 发表于 2024-7-10 21:50
def is_public(name):
    return not name.startswith('_')

发给他了。让他好好看看。
但是，我不懂，列表推导式是能让人高潮吗为什么一定要用。

黑客

^^^ 发表于 2024-7-10 17:27
之前和他一起招了一个C++的实习生，不过我不清楚情况怎么样。他们这一批实习结束还有两个月。 ...

社招待遇咋样啊大佬 有意向

黑客

weihu120 发表于 2024-7-10 17:59
膜拜驱动大佬

我是萌新..驱动大佬哪有事件逛loc