Програмиране с Rust

Курс във Факултета по Математика и Информатика към СУ

Решение на FMI Buzz от Костадин Пеков

Обратно към всички решения

Към профила на Костадин Пеков

Резултати

  • 15 точки от тестове
  • 0 бонус точки
  • 15 точки общо
  • 10 успешни тест(а)
  • 0 неуспешни тест(а)

Код

/// Вход: променлива `n`, която описва броя елементи, които ще генерираме в резултата.
///
/// За всяко число от 1 до `n` включително, искаме съответстващия елемент в резултата да е:
///
/// - String със съдържание "Fizz" ако числото се дели на 3, но не на 5
/// - String със съдържание "Buzz" ако числото се дели на 5, но не на 3
/// - String със съдържание "Fizzbuzz" ако числото се дели и на 3, и на 5
/// - Числото конвертирано до низ, във всички други случаи
///
/// Тоест, във `fizzbuzz(3)`, първия елемент ще бъде `String::from("1")`, втория
/// `String::from("2")`, последния `String::from("Fizz")`.
///
/// Ако `n` е 0, очакваме празен вектор за резултат.
///
pub fn fizzbuzz(n: usize) -> Vec<String> {
let mut result: Vec<String> = Vec::new();
if n == 0
{
return result;
}
println!("here is n {}", n);
for i in 1..n+1
{
println!("here is i {}", i);
if i%3 == 0 && i%5 == 0
{
result.push(String::from("Fizzbuzz"));
}
else if i%3 == 0
{
result.push(String::from("Fizz"));
}
else if i%5 == 0
{
result.push(String::from("Buzz"));
}
else
{
result.push(i.to_string());
}
}
return result
}
/// Вход:
/// - променлива `n`, която описва броя елементи, които ще генерираме в резултата.
/// - променливи `k1` и `k2`, които са двата делителя, които ще използваме за заместване.
///
/// За всяко число от 1 до `n` включително, искаме съответстващия елемент в резултата да е:
///
/// - String със съдържание "Fizz" ако числото се дели на k1, но не на k2
/// - String със съдържание "Buzz" ако числото се дели на k2, но не на k1
/// - String със съдържание "Fizzbuzz" ако числото се дели и на k1, и на k2
/// - Числото конвертирано до низ, във всички други случаи
///
/// Ако `n` е 0, очакваме празен вектор за резултат.
/// Ако `k1` или `k2` са 0 или 1, очакваме функцията да panic-не с каквото съобщение изберете.
///
pub fn custom_buzz(n: usize, k1: u8, k2: u8) -> Vec<String> {
let mut result: Vec<String> = Vec::new();
if k1 == 0 || k1 == 1 || k2 == 0 || k2 == 1
{
panic!("k1 = {} and k2 = {} should not be 1 or 0", k1, k2);
}
if n == 0
{
return result;
}
println!("here is n {}", n);
for i in 1..n+1
{
println!("here is i {}", i);
if i % k1 as usize == 0 && i % k2 as usize == 0
{
result.push(String::from("Fizzbuzz"));
}
else if i % k1 as usize == 0
{
result.push(String::from("Fizz"));
}
else if i % k2 as usize == 0
{
result.push(String::from("Buzz"));
}
else
{
result.push(i.to_string());
}
}
return result
}
/// Параметри:
/// - полета `k1` и `k2`, които са двата делителя, които ще използваме за заместване.
/// - поле `labels`, които са трите етикета, които съответстват на коефициентите
///
pub struct FizzBuzzer {
pub k1: u8,
pub k2: u8,
pub labels: [String; 3],
}
impl FizzBuzzer {
/// За всяко число от 1 до `n` включително, искаме съответстващия елемент в резултата да е:
///
/// - Първия String от полето `labels` ако числото се дели на k1, но не на k2
/// - Втория String от полето `labels` ако числото се дели на k2, но не на k1
/// - Третия String от полето `labels` ако числото се дели и на k1, и на k2
/// - Числото конвертирано до низ, във всички други случаи
///
/// Ако `n` е 0, очакваме празен вектор за резултат.
/// Ако `k1` или `k2` са 0 или 1, очакваме функцията да panic-не с каквото съобщение изберете.
///
pub fn take(&self, n: usize) -> Vec<String> {
let mut result: Vec<String> = Vec::new();
if self.k1 == 0 || self.k1 == 1 || self.k2 == 0 || self.k2 == 1
{
panic!("k1 = {} and k2 = {} should not be 1 or 0", self.k1, self.k2);
}
if n == 0
{
return result;
}
for i in 1..n+1
{
if i % self.k1 as usize == 0 && i % self.k2 as usize == 0
{
result.push(String::from(&self.labels[2]));
}
else if i % self.k1 as usize == 0
{
result.push(String::from(&self.labels[0]));
}
else if i % self.k2 as usize == 0
{
result.push(String::from(&self.labels[1]));
}
else
{
result.push(i.to_string());
}
}
return result
}
/// Параметъра `index` указва кой етикет от полето `labels` променяме, от 0 до 2. Ако подадения
/// `index` е извън тези рамки, очакваме функцията да panic-не.
///
/// Стойността `value` е низа, който ще сложим на този индекс в полето `labels`.
///
pub fn change_label(&mut self, index: usize, value: &String) {
if index > 2
{
panic!("index is {} - out of range", index);
}
self.labels[index] = String::from(value);
}
}
mod tests {
#[test]
fn test_fizzbuzz() {
use crate::fizzbuzz;
let expected = vec![1.to_string(), 2.to_string(), String::from("Fizz")];
let expected2 = vec![1.to_string(), 2.to_string(), String::from("Fizz"),4.to_string(), String::from("Buzz"),String::from("Fizz"), 7.to_string(),8.to_string(),String::from("Fizz"), String::from("Buzz"),11.to_string(),String::from("Fizz"),13.to_string(),14.to_string(),String::from("Fizzbuzz")];
assert_eq!(fizzbuzz(3), expected);
assert_eq!(Vec::<String>::new(),fizzbuzz(0));
assert_eq!(expected2,fizzbuzz(15));
}
#[test]
fn test_custom_buzz_gw() {
use crate::custom_buzz;
let expected = vec![1.to_string(), 2.to_string(), String::from("Fizz")];
let expected2 = vec![1.to_string(), 2.to_string(), String::from("Fizz"),4.to_string(), String::from("Buzz"),String::from("Fizz"), 7.to_string(),8.to_string(),String::from("Fizz"), String::from("Buzz"),11.to_string(),String::from("Fizz"),13.to_string(),14.to_string(),String::from("Fizzbuzz")];
let expected_custom = vec![1.to_string(), 2.to_string(), String::from("Fizzbuzz"),4.to_string(),5.to_string(),String::from("Fizzbuzz")];
assert_eq!(custom_buzz(3, 3, 5), expected);
assert_eq!(custom_buzz(15, 3, 5), expected2);
assert_eq!(Vec::<String>::new(),custom_buzz(0,3,5));
assert_eq!(custom_buzz(6, 3, 3), expected_custom);
}
#[test]
#[should_panic]
fn test_custom_buzz_bw() {
use crate::custom_buzz;
custom_buzz(5,1,0);
}
#[test]
fn test_fizzbuzzer_gw() {
use crate::FizzBuzzer;
let expected = vec![1.to_string(), 2.to_string(), String::from("Fizz")];
let expected2 = vec![1.to_string(), 2.to_string(), String::from("Fizz"),4.to_string(), String::from("Buzz"),String::from("Fizz"), 7.to_string(),8.to_string(),String::from("Fizz"), String::from("Buzz"),11.to_string(),String::from("Fizz"),13.to_string(),14.to_string(),String::from("Fizzbuzz")];
let expected_custom = vec![1.to_string(), 2.to_string(), String::from("Fizzbuzz"),4.to_string(),5.to_string(),String::from("Fizzbuzz")];
let mut fizzbuzzer1 = FizzBuzzer {
k1: 3,
k2: 5,
labels: [
String::from("Fizz"),
String::from("Buzz"),
String::from("Fizzbuzz")
],
};
let mut fizzbuzzer2 = FizzBuzzer {
k1: 3,
k2: 3,
labels: [
String::from("Fizz"),
String::from("Buzz"),
String::from("Fizzbuzz")
],
};
assert_eq!(fizzbuzzer1.take(3), expected);
assert_eq!(fizzbuzzer1.take(15), expected2);
assert_eq!(Vec::<String>::new(),fizzbuzzer1.take(0));
assert_eq!(fizzbuzzer2.take(6), expected_custom);
let expected_after_change = vec![1.to_string(), 2.to_string(), String::from("Fiz")];
fizzbuzzer1.change_label(0, &String::from("Fiz"));
assert_eq!(fizzbuzzer1.take(3), expected_after_change);
let expected_after_change2 = vec![1.to_string(), 2.to_string(), String::from("Fiz"),4.to_string(),String::from("Fiz")];
fizzbuzzer1.change_label(1, &String::from("Fiz"));
assert_eq!(fizzbuzzer1.take(5), expected_after_change2);
let expected_after_change3 = vec![1.to_string(), 2.to_string(), String::from("")];
fizzbuzzer1.change_label(0, &String::new());
assert_eq!(fizzbuzzer1.take(3), expected_after_change3);
fizzbuzzer2.change_label(0, &String::new());
assert_eq!(fizzbuzzer2.take(6), expected_custom);
}
#[test]
#[should_panic]
fn test_fizzbuzzer_bw() {
use crate::FizzBuzzer;
let fizzbuzzer = FizzBuzzer {
k1: 1,
k2: 1,
labels: [
String::from("Fizz"),
String::from("Buzz"),
String::from("Fizzbuzz")
],
};
fizzbuzzer.take(3);
}
}

Лог от изпълнението

Compiling solution v0.1.0 (/tmp/d20201028-2816268-14vfku7/solution)
    Finished test [unoptimized + debuginfo] target(s) in 2.76s
     Running target/debug/deps/solution-ebb42508826ef2b4

running 5 tests
test tests::test_custom_buzz_bw ... ok
test tests::test_custom_buzz_gw ... ok
test tests::test_fizzbuzz ... ok
test tests::test_fizzbuzzer_bw ... ok
test tests::test_fizzbuzzer_gw ... ok

test result: ok. 5 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running target/debug/deps/solution_test-9e954a53ed808c89

running 10 tests
test solution_test::test_change_label_basic ... ok
test solution_test::test_change_label_invalid ... ok
test solution_test::test_classic1 ... ok
test solution_test::test_classic2 ... ok
test solution_test::test_coefficients1 ... ok
test solution_test::test_coefficients2 ... ok
test solution_test::test_coefficients_invalid ... ok
test solution_test::test_struct_basic ... ok
test solution_test::test_struct_invalid ... ok
test solution_test::test_zeroes ... ok

test result: ok. 10 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

   Doc-tests solution

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

История (1 версия и 1 коментар)

Костадин качи първо решение на 27.10.2020 22:50 (преди почти 5 години)

Оценявам опита за тестове, но не си пробвал нито веднъж да генерираш успешна поредица с етикети, различни от Fizz/Buzz/Fizzbuzz. Веднъж сменяш етикет, но само първия. Не бих казал, че е нужно да правиш пълна проверка на всички стойности, примерно валиден тест би било да вземеш елемент с .get( и да провериш, че наистина е елемента от етикета, а не елемент от default-ната поредица Fizz/Buzz/Fizzbuzz.

Допълнително, добра идея е да си провериш настройките на текстовия редактор, понеже имаш доста неконсистентна индентация. Не правим проблеми за такива неща, но определено затруднява четенето :). Може да видиш как изглежда текста тук например.