Да ладно, хрен с этим, ибо лайфтаймы там таки нужны

@AtomicInteger давай ошибку-то уже =)

@AtomicInteger давай ошибку-то уже =)

Держи:D error[E0597]: `all_nodes` does not live long enough --> src\tree_node.rs:42:21 | 42 | for node in all_nodes[0].get_mut_children().iter_mut() { | ^^^^^^^^^ borrowed value does not live long enough ... 51 | } | - borrowed value only lives until here | note: borrowed value must be valid for the anonymous lifetime #1 defined on the method body at 40:5... --> src\tree_node.rs:40:5 | 40 | / pub fn nodes_mut(&mut self) -> Vec<&mut TreeNode<T>> { 41 | | let mut all_nodes = vec![self]; 42 | | for node in all_nodes[0].get_mut_children().iter_mut() { 43 | | let is_empty = node.get_children().is_empty(); ... | 50 | | all_nodes 51 | | } | |_____^

вот просто пальцем в небо, сделай for &node in all_nodes[...

может у тебя код целиком где-то выложен? у меня пока мозг не научился компилять раст :(

вот просто пальцем в небо, сделай for &node in all_nodes[...

ээ, чёт неа)

может у тебя код целиком где-то выложен? у меня пока мозг не научился компилять раст :(

Конкретно этот - нет

Я его ещё не запушил банально, ибо не работает ничего))

пуш как есть)

Ладно, сейчас. Кстати, с pop работает. есть вариант заюзать вместо вектора мой лист, там есть peek)

пуш как есть)

https://github.com/AtomicInteger/one_more_dstruct/tree/trying

а разве все равно не будет проблемы с лайфтаймами?

а разве все равно не будет проблемы с лайфтаймами?

Да, будут

то есть ты делаешь peek, ну или last_mut то референс будет валиден только в этом скоупе, следовательно тебе не дадут вернуть твой вектор

Ну да, да

изи вариант это клонить сразу)

но как то неоч оно, мб и можно как то это собрать

Клонить очень не хочу, ибо мне потом нужно менять эти данные, вот и получится, что я клон меняю

Там, если в коммитах глянуть, так и было раньше:)

https://github.com/AtomicInteger/one_more_dstruct/tree/trying

ты походу без свежих изменений запушил

у меня ни get_mut_children ни nodes_mut нет

Ну ты ветки переключи

https://github.com/AtomicInteger/one_more_dstruct/commit/85fe2354a86f2bc7868af5e96121b9f40b4acd7d

Вот он, коммит

А в мастер я завальный код не пушил:)

cargo check Compiling one_more_dstruct v0.0.1 (file:///home/mexus/test/rust/one_more_dstruct) Finished dev [unoptimized + debuginfo] target(s) in 0.23 secs

Ты ветку точно переключил?

git log -1 commit 85fe2354a86f2bc7868af5e96121b9f40b4acd7d Author: AtomicInteger <rostyk1@hotmail.com> Date: Thu Apr 12 23:30:01 2018 +0300 Some fail code

Ну да, верно

Чёт я может не выспался, но у меня падает

какая версия?

какая версия?

cargo? cargo 1.26.0-nightly (d63299b6e 2018-03-28)

то же самое было

ща обновлю найтли на всякий :)

всё билдится, тесты проходят

Но ты методы то видишь?Видишь же, что они хреновые?

nodes_mut есть, если добавляют туда АХАХАХА, билд фейлится)

pub fn nodes_mut(&mut self) -> Vec<&mut TreeNode<T>> { let mut all_nodes = vec![self]; for node in all_nodes.pop().unwrap().get_mut_children().iter_mut() { let is_empty = node.get_children().is_empty(); all_nodes.push(node); if !is_empty { let mut inner_nodes = all_nodes.pop().unwrap().nodes_mut(); all_nodes.append(&mut inner_nodes); } } all_nodes }

Так же, ты же знаешь что поп в формате цикле и внутри ифа по сути убрал оттуда Селф и ноду которую ты только запушил? Так fyi, мб так и нужно

nodes_mut есть, если добавляют туда АХАХАХА, билд фейлится)

чудеса)

@lundibundi у тебя тоже билд фейлится?

Так же, ты же знаешь что поп в формате цикле и внутри ифа по сути убрал оттуда Селф и ноду которую ты только запушил? Так fyi, мб так и нужно

Что-то несколько раз прочитал и ничего не понял

Я кушать делаю) я не клонил я просто локально написал стабы с нужными фями которые unimplemented!

Мб ппже гляну

ща на другой машине попробую

ща на другой машине попробую

Есть у карго клин какой то?

Походу он не думает что файл не менялся

чот сомневаюсь)

Надо почистить кеш Билда я думаю

@AtomicInteger git clone https://github.com/AtomicInteger/one_more_dstruct.git git checkout trying cargo check cargo test

всё отработало окей

Гугл говорит что есть cargo clean

На крайняк попробуй удалить таргет если она есть

cargo clean по-моему именно это и делает)

strace подтверждает :)

)

Во магия

И у меня не падает

Как так то:D

Может это и валидный код

собирается — значит валидный =)

Погоди ка, там же в коммите с поп код

Так он вроде ок)

Ну да, я же с поп запушил, но иде подсветила красным, видмо не успело обновиться

Что-то несколько раз прочитал и ничего не понял

Короче, в цикле, ты делаешь пуш в вектор, а потом если есть дети то ты делаешь поп

Ты убираешь этот элемент который запушил из вектора

В смысле, это неправильно?

pop() удаляет последний элемент из вектора — тебе так и нужно или нет? :)

Screenshot (Apr 13, 2018 00:01:39)

На самом деле нет, иначе зачем я его пушу тогда. Если вы хотит сказать, что это ошибка, то да, так оно и есть. Это я уже просто как заглушку поставил.

pop() удаляет последний элемент из вектора — тебе так и нужно или нет? :)

Вот это я пытался написать, ток что то переборщил с деталями)

Просто со стороны раста то все ок, а логически это ошибка вроде как, я потому и отписал)

А, ну ок, я понял. Ну да, ошибка, но это я поправлю. Просто так оно хотя бы билдится. Ладно, ребят, спасибо за помощь, дальше я уже сам буду копать

Что-то слишком много времени отняла такая глупость:)

Что-то слишком много времени отняла такая глупость:)

Это нормально

в общем

ничего не получится :)

нужно Rc использовать

нужно Rc использовать

Вот, кстати, насчет Rc, Arc и Box я уже не раз думал, но так почему-то и не разобрался