[bug] Уничтоженная ТЗМ навсегда резервирует заявку на подвоз боеприпасов #2

New Issue

2026-10-29T11:43:25Z

chexit commented

2026-10-29 11:43:25 +00:00

Проверено на 90379f6.

Список taken собирает job всех ТЗМ, включая уничтоженные. Обработка уничтоженных грузовиков пропускается, а их job при уничтожении не снимается. Другие исправные машины исключают эту цель из выбора.

Воспроизведение: есть склад далеко от установки; установка ждёт БК; уничтоженная ТЗМ имеет job на эту установку; рядом есть живая ТЗМ с грузом. Живая машина не принимает заявку.

Предложение: освобождать заявку при уничтожении/удалении машины и при построении taken учитывать только живые машины с действительными заданиями.

Код:

		airdef/airdef-mod/src/sim/logistics.rs
		Line 75 in 90379f6
	
				            let taken: Vec<u32> = self.sams.iter().filter(|s| s.kind == SamKind::Tzm).filter_map(|s| s.job).collect();

Воспроизведено на настоящем коде sim и wave в отдельном native Rust crate (edition=2024, glam=0.30.10, cargo test --release). Движок и GPU не участвовали. Тест ниже проверяет ожидаемое исправленное поведение и сейчас падает; поместить в дочерний #[cfg(test)]-модуль соответствующего файла с use super::*;.

#[test]
fn dead_truck_must_not_reserve_a_waiting_system() {
        let mut sim=Sim::new(1);
        let depot=sim.id(); sim.sams.push(Sam::new(depot,SamKind::Depot,DVec3::new(500.,70.,0.),0.));
        let target=sim.id(); let mut s=Sam::new(target,SamKind::Tor,DVec3::new(0.,70.,0.),0.);s.empty=true;s.missiles=0;sim.sams.push(s);
        let dead=sim.id();let mut s=Sam::new(dead,SamKind::Tzm,DVec3::new(100.,70.,0.),0.);s.job=Some(target);s.health=0.;sim.sams.push(s);
        let live=sim.id();sim.sams.push(Sam::new(live,SamKind::Tzm,DVec3::new(30.,70.,0.),0.));
        sim.tick=3;sim.step_supply(&mut Out::default());
        assert_eq!(sim.sams[3].job,Some(target),"a live loaded truck must take the waiting system");
    }

Проверено на `90379f6`. Список `taken` собирает `job` всех ТЗМ, включая уничтоженные. Обработка уничтоженных грузовиков пропускается, а их `job` при уничтожении не снимается. Другие исправные машины исключают эту цель из выбора. Воспроизведение: есть склад далеко от установки; установка ждёт БК; уничтоженная ТЗМ имеет `job` на эту установку; рядом есть живая ТЗМ с грузом. Живая машина не принимает заявку. Предложение: освобождать заявку при уничтожении/удалении машины и при построении `taken` учитывать только живые машины с действительными заданиями. Код: https://gitea.maoniao.team/chexit/airdef/src/commit/90379f6/airdef-mod/src/sim/logistics.rs#L75 Воспроизведено на настоящем коде `sim` и `wave` в отдельном native Rust crate (`edition=2024`, `glam=0.30.10`, `cargo test --release`). Движок и GPU не участвовали. Тест ниже проверяет ожидаемое исправленное поведение и сейчас падает; поместить в дочерний `#[cfg(test)]`-модуль соответствующего файла с `use super::*;`. ```rust #[test] fn dead_truck_must_not_reserve_a_waiting_system() { let mut sim=Sim::new(1); let depot=sim.id(); sim.sams.push(Sam::new(depot,SamKind::Depot,DVec3::new(500.,70.,0.),0.)); let target=sim.id(); let mut s=Sam::new(target,SamKind::Tor,DVec3::new(0.,70.,0.),0.);s.empty=true;s.missiles=0;sim.sams.push(s); let dead=sim.id();let mut s=Sam::new(dead,SamKind::Tzm,DVec3::new(100.,70.,0.),0.);s.job=Some(target);s.health=0.;sim.sams.push(s); let live=sim.id();sim.sams.push(Sam::new(live,SamKind::Tzm,DVec3::new(30.,70.,0.),0.)); sim.tick=3;sim.step_supply(&mut Out::default()); assert_eq!(sim.sams[3].job,Some(target),"a live loaded truck must take the waiting system"); } ```

chexit added the area:logistics priority:P1 type:bug labels 2026-10-29 12:56:02 +00:00

chexit referenced this issue from a commit

2026-10-29 13:28:10 +00:00

P1 issues: ground queries keep room for unknown land (#1), a destroyed truck frees its job (#2), trucks follow a system that moved (#3), repair cancels cook-offs (#5), a failed save is retried and autosaves are paced (#6, #7), a combined system reloads when either weapon is spent (#20), munitions go down to the land left under a destroyed target (#34); regression tests from the issues

chexit commented

2026-10-29 13:29:20 +00:00