rusty-donkey/oxidice
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

review: change roll result type to i32

Browse Source
This commit is contained in:
Kate Korsaro
2025-11-16 09:52:44 +01:00
parent e12e954d19
commit 4d4810b52b
6 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
01.workspace/oxidice_lib/src/imp/die_roll.rs
View File

@@ -1,8 +1,8 @@
use crate::*;
impl Die {
pub fn roll(&self) -> u16 {
pub fn roll(&self) -> i32 {
let mut rng = rand::rng();
rng.random_range(1..=self.sides) as u16
rng.random_range(1..=self.sides) as i32
}
}

2
01.workspace/oxidice_lib/src/imp/handful_roll.rs
View File

@@ -2,7 +2,7 @@ use crate::*;
impl Handful {
pub fn roll(self) -> Outcome {
let mut die_rolls = Vec::<u16>::new();
let mut die_rolls = Vec::<i32>::new();
for die in self.dice {
die_rolls.push(die.roll());
}

4
01.workspace/oxidice_lib/src/imp/outcome_max.rs
View File

@@ -1,10 +1,10 @@
use crate::*;
fn max_of(values: Vec<u16>) -> Result<Outcome, FailedTo> {
fn max_of(values: Vec<i32>) -> Result<Outcome, FailedTo> {
if values.is_empty() {
return Err(FailedTo::ProcessInput);
}
let ret = Ok(Outcome::Scalar(u16::MIN));
let ret = Ok(Outcome::Scalar(i32::MIN));
ret.and_then(|_| {
values
.iter()

4
01.workspace/oxidice_lib/src/imp/outcome_min.rs
View File

@@ -1,10 +1,10 @@
use crate::*;
fn min_of(values: Vec<u16>) -> Result<Outcome, FailedTo> {
fn min_of(values: Vec<i32>) -> Result<Outcome, FailedTo> {
if values.is_empty() {
return Err(FailedTo::ProcessInput);
}
Ok(Outcome::Scalar(u16::MIN)).and_then(|_| {
Ok(Outcome::Scalar(i32::MIN)).and_then(|_| {
values
.iter()
.min()

4
01.workspace/oxidice_lib/src/imp/outcome_sum.rs
View File

@@ -1,6 +1,6 @@
use crate::*;
fn sum_of(values: Vec<u16>) -> Result<Outcome, FailedTo> {
fn sum_of(values: Vec<i32>) -> Result<Outcome, FailedTo> {
if values.is_empty() {
return Ok(Outcome::Scalar(0));
}
@@ -10,7 +10,7 @@ fn sum_of(values: Vec<u16>) -> Result<Outcome, FailedTo> {
Outcome::List(_) => Err(FailedTo::ProcessInput),
Outcome::Scalar(acc) => acc
.checked_add(value)
.map(|new_acc: u16| Outcome::Scalar(new_acc))
.map(|new_acc: i32| Outcome::Scalar(new_acc))
.ok_or(FailedTo::SumValues),
})
}