Tintenfisch3 , 2 hours ago

@became_fish
This is what let else was made for:

let Some(val) = some_function() else {  
 return Ok(...);  
};  

Muddobbers , 5 hours ago

@became_fish

if I'm understanding it correctly..

some_function() is returning an Option, if it's None then you return out from where you are, otherwise you continue and do your thing with the contents of the value 'val'?

Maybe:

if let Some(val) = some_function() {<br></br> println!{"{}", val)<br></br>} else {<br></br> return Ok(...)<br></br>

became_fish OP , 5 hours ago

@Muddobbers this works, but i'm looking for something that won't increase the indent level (if i do this five times in a function, i don't want my code to look like a pyramid). i think let... else that other replies have brought up is what i'm looking for here

mendelsshop , 6 hours ago

@became_fish

You can use a let else.

let val = some_function();  
let Some(val) = val else {  
 return Err(...);  
}  

bryan , 6 hours ago

@became_fish

Not much better, but we use

let Some(val) = some_function() else {  
 return OK(…);  
};  

pretty regularly.

If you’re returning an Err instead of an Ok then

let val = some_function().ok_or(Error::…)?;  

works nicely.

livingcoder , 6 hours ago

@became_fish I'm generally okay with this pattern, but I change it up slightly to assignments from match or blocks:

From match:

let value = match some_function() {  
 Some(value) => value,  
 None => return Ok(...),  
};  

From block:

let value = {  
 let value = some_function();  
 if value.is_none() {  
 return Ok(...);  
 }  
 value.unwrap()  
};  

became_fish OP , 6 hours ago

@livingcoder wait, that second one - you can use a block as a value?

livingcoder , 6 hours ago

@became_fish Yeah, I find it quite helpful to separate parsing logic out from the root of the function to keep the variable names clean in the root. I end up creating steps as blocks.