Ревью 🔍

Несколько ключевых моментов, которые помогут в процессе проверки:

⏱️ Уровень сложности ревью: 5 🔵🔵🔵🔵🔵

🧪 Нет релевантных тестов

🔒 Проблем с безопасностью не выявлено

⚡ Рекомендовано к рассмотрению

Возможная проблема

Класс Task содержит синтаксически неверный код, что делает невозможным создание экземпляров задач и приведёт к падению приложения.

class Task:
vfmv;fdvmfvlfmv;cv c,.vc

Возможная проблема

Метод Task.to_dict() отсутствует, но вызывается в save_tasks(), что вызовет AttributeError при сохранении задач.

json.dump([task.to_dict() for task in self.tasks], f,

Critical Issue

Весь функционал управления задачами сломан из-за отсутствия корректного определения класса Task, включая загрузку, сохранение и экспорт данных.

class TaskManager:
    def __init__(self, storage_file: str = "tasks.json"):
        self.storage_file = storage_file
        self.tasks: List[Task] = []
        self.lock_file = f"{storage_file}.lock"
        self.load_tasks()

    def _acquire_lock(self, timeout: int = 5) -> bool:
        start = time.time()
        while os.path.exists(self.lock_file) and time.time() - start < timeout:
            time.sleep(0.1)
        if os.path.exists(self.lock_file):
            return False
        with open(self.lock_file, "w") as f:
            f.write(str(os.getpid()))
        return True

    def _release_lock(self) -> None:
        if os.path.exists(self.lock_file):
            os.remove(self.lock_file)

    def load_tasks(self) -> None:
        if not self._acquire_lock():
            raise RuntimeError("Не удалось получить доступ к хранилищу задач")
        try:
            if os.path.exists(self.storage_file):
                with open(self.storage_file, "r", encoding="utf-8") as f:
                    data = json.load(f)
                    self.tasks = [Task.from_dict(item) for item in data]
            else:
                self.tasks = []
        finally:
            self._release_lock()

    def save_tasks(self) -> None:
        if not self._acquire_lock():
            raise RuntimeError("Не удалось сохранить задачи: файл заблокирован")
        try:
            with open(self.storage_file, "w", encoding="utf-8") as f:
                json.dump([task.to_dict() for task in self.tasks], f, 
                         ensure_ascii=False, indent=2)
        finally:
            self._release_lock()

    def add_task(self, title: str, description: str, category: str, 
                 priority: Priority, deadline: Optional[str]) -> Task:
        title = Validator.validate_title(title)
        category = Validator.validate_category(category)
        deadline = Validator.validate_deadline(deadline) if deadline else None

        task_id = max((t.task_id for t in self.tasks), default=0) + 1
        task = Task(
            task_id=task_id,
            title=title,
            description=description[:Validator.MAX_DESC_LENGTH],
            category=category,
            priority=priority,
            deadline=deadline,
            created_at=datetime.now().isoformat(),
            completed=False
        )
        self.tasks.append(task)
        self.save_tasks()
        return task

    def complete_task(self, task_id: int) -> bool:
        task = self.get_task(task_id)
        if task and not task.completed:
            task.completed = True
            self.save_tasks()
            return True
        return False

    def remove_task(self, task_id: int) -> bool:
        for i, task in enumerate(self.tasks):
            if task.task_id == task_id:
                del self.tasks[i]
                self.save_tasks()
                return True
        return False

    def get_task(self, task_id: int) -> Optional[Task]:
        return next((t for t in self.tasks if t.task_id == task_id), None)

    def search_tasks(self, query: str) -> List[Task]:
        query = query.lower()
        return [t for t in self.tasks 
                if query in t.title.lower() or query in t.description.lower()]

    def filter_tasks(self, status: Optional[Status] = None, 
                    category: Optional[str] = None, 
                    priority: Optional[Priority] = None,
                    hide_completed: bool = True) -> List[Task]:
        result = self.tasks[:]
        if hide_completed:
            result = [t for t in result if not t.completed]
        if status:
            result = [t for t in result if t.get_status() == status]
        if category:
            result = [t for t in result if t.category == category.lower()]
        if priority:
            result = [t for t in result if t.priority == priority]
        return result

    def get_statistics(self) -> Dict:
        total = len(self.tasks)
        completed = sum(1 for t in self.tasks if t.completed)
        overdue = sum(1 for t in self.tasks 
                     if not t.completed and t.get_status() == Status.OVERDUE)
        by_category = {}
        for task in self.tasks:
            by_category[task.category] = by_category.get(task.category, 0) + 1
        return {
            "total": total,
            "completed": completed,
            "active": total - completed,
            "overdue": overdue,
            "completion_rate": round(completed / total * 100, 1) if total > 0 else 0.0,
            "by_category": by_category
        }

    def export_report(self, filename: str = "report.txt") -> bool:
        try:
            stats = self.get_statistics()
            with open(filename, "w", encoding="utf-8") as f:
                f.write(f"Отчёт по задачам ({datetime.now().strftime('%Y-%m-%d %H:%M')})\n")
                f.write("=" * 60 + "\n\n")
                f.write(f"Всего задач: {stats['total']}\n")
                f.write(f"Выполнено: {stats['completed']} ({stats['completion_rate']}%)\n")
                f.write(f"Просрочено: {stats['overdue']}\n\n")
                f.write("Задачи по категориям:\n")
                for cat, count in stats["by_category"].items():
                    f.write(f"  • {cat}: {count}\n")
                f.write("\nДетали задач:\n")
                for task in sorted(self.tasks, key=lambda t: t.task_id):
                    status = "✓" if task.completed else ("!" if task.get_status() == Status.OVERDUE else " ")
                    f.write(f"[{status}] #{task.task_id} [{task.priority.value}] {task.title}\n")
                    if task.deadline:
                        f.write(f"    Дедлайн: {task.deadline}\n")
            return True
        except Exception:
            return False

BranchBuddies

Популярные репозитории