Skip to content

fix: 修复应用发布后无法使用最新的应用设置 #1554

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 6, 2024
Merged

fix: 修复应用发布后无法使用最新的应用设置 #1554

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion apps/common/cache/file_cache.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ def clear_by_application_id(self, application_id):
value.application.id) == application_id):
delete_keys.append(key)
for key in delete_keys:
self.delete(key)
self.cache.delete(key)

def clear_timeout_data(self):
for key in self.cache.iterkeys():
Copy link

@fit2cloudrd fit2cloudrd Nov 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

该Python代码存在一些语法或函数调用方式上的错误。

首先：
使用 key 对象来直接访问value.application.id可能会导致问题。由于对象引用和值之间的关系不是线性的，key 的方法不能获取到其内部存储的字典键对应的值（因为对象没有一个内置的方法可以用于获取内部属性）。正确处理应该使用dict()方法或者.get() 方法将数据格式化为列表的方式，比如：for item in value['application']['id']；

其次，在删除操作中，delete_keys = append_key_to_list(keys, 'delete') 错误地尝试将另一个列表append_key_to_list()传入给变量delete_keys，在循环内对同一个列表进行重复操作可能导致内存冲突甚至运行时异常。

最后，如果想要清除缓存中已经存在的数据，应当考虑更新整个self.cache的数据结构而不是逐一删除每个条目的方式。

以上都是一些比较基本的设计规范方面的问题，可能需要根据上下文背景去理解作者意图及编写风格等细节。

修正完后更合适的Python代码如下：

def clear_by_application_id(self, application_id):
    keys_in_cache = list(self.cache.keys())
    
    delete_keys = []  # 假设这是一个初始化空list的地方
    
    for key in keys_in_cache:
        if (getattr(key, '__iter__', None) is not False and hasattr(
                key.__class__, "keys") and getattr(key, "__getitem__", -1)() == str(application_id)):
            # 注意这里要用str转换一下，确保能匹配上key里面保存的是什么类型的内容            
             delete_keys.append(key)

    for key in delete_keys:
        del self.cache[key]

# 更改以下部分：
for key in self.cache.iterkeys():

请注意，上述修改中的“此处”指代的具体内容是依据你的实际需求而定，需要根据具体环境或文档调整。

另外，如果你正在构建一种数据库应用，则有可能会遇到大量复杂的业务逻辑与安全限制条件，这时候就需要引入事务管理、权限控制等额外的考量和设计了。不过基于上面提到的一些最基础的概念，你可以自行思考如何改善当前代码片段，并结合实际情况作出改进。

Expand Down
Loading