Details
services build and all tests pass (integration, performance).
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
services/commtest/tests/backup_integration_test.rs | ||
---|---|---|
112–130 | Feel like this could just be a simple loop with a zip. No need to construct two hashmaps with the same structure over the data you're iterating. |
services/commtest/tests/backup_integration_test.rs | ||
---|---|---|
112–130 | This doesn't work, because zip doesn't overcome the order problem that I pointed out in the description of D5072 (which is referred to in the description of this very diff):
This is why I used maps in the first place. To demonstrate what's the issue with your code, I created a minimal code so you can see: use std::collections::HashMap; struct S { id: i32, value: i32, } fn main() { let v1 = vec![ S { id: 0, value: 0 }, S { id: 1, value: 1 }, S { id: 2, value: 2 }, S { id: 3, value: 3 }, ]; let v2 = vec![ S { id: 3, value: 3 }, S { id: 0, value: 0 }, S { id: 2, value: 2 }, S { id: 1, value: 1 }, ]; // this works { let mut m1 = HashMap::new(); let mut m2 = HashMap::new(); assert_eq!(v1.len(), v2.len()); for i in 0..v1.len() { m1.insert(v1[i].id, v1[i].value); m2.insert(v2[i].id, v2[i].value); } for (id1, val1) in m1 { let val2 = m2.get(&id1).unwrap().clone(); assert_eq!(val1, val2); } } // this doesn't work { for (item1, item2) in v1.iter().zip(v2) { assert_eq!(item1.value, item2.value); } } } |
services/commtest/tests/backup_integration_test.rs | ||
---|---|---|
112–130 | Does Rust have a group by? It could be the simplest approach. |
services/commtest/tests/backup_integration_test.rs | ||
---|---|---|
112–130 | Just a reminder about this question |
services/commtest/tests/backup_integration_test.rs | ||
---|---|---|
112–130 | There's something, but I don't really think this is a top priority. For me, the code is fairly clean right now, it doesn't have to be the most efficient on the planet, since there are just tests. The goal is to ship in this stack ASAP, this code can always be improved. If you feel like this is critical nevertheless, feel free to create a task for this. |