Решение на FMI Buzz от Симеон Николов
Резултати
- 15 точки от тестове
- 0 бонус точки
- 15 точки общо
- 10 успешни тест(а)
- 0 неуспешни тест(а)
Код
Лог от изпълнението
Compiling solution v0.1.0 (/tmp/d20201028-2816268-qzh8sf/solution)
Finished test [unoptimized + debuginfo] target(s) in 2.07s
Running target/debug/deps/solution-ebb42508826ef2b4
running 0 tests
test result: ok. 0 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
История (2 версии и 1 коментар)
Симеон качи решение на 26.10.2020 08:38 (преди почти 5 години)
pub fn fizzbuzz(n: usize) -> Vec<String> {
- let mut res = Vec::<String>::new();
- let fizz = String::from("Fizz");
- let buzz = String::from("Buzz");
- let fizzbuzz1 = String::from("Fizzbuzz");
- for i in 1..n + 1{
- if i % 3 == 0 && i% 5 != 0{
- res.push(fizz.clone());
- }
- else if i %3 !=0 && i % 5 == 0{
- res.push(buzz.clone());
- }
- else if i % 3 == 0 && i % 5 == 0{
- res.push(fizzbuzz1.clone());
- }
- else{
- res.push(i.to_string());
- }
+let mut res = Vec::<String>::new();
+let fizz = String::from("Fizz");
+let buzz = String::from("Buzz");
+let fizzbuzz1 = String::from("Fizzbuzz");
Не би трябвало да е проблем да използваш fizzbuzz за име на променлива -- Функциите и променливите са в различни namespaces. Може би е добра идея да са различни, за да се избегне объркване и да се комуникира ясно разликата, но тогава бих избрал някакво по-комуникативно име :). Примерно, fizz_label, buzz_label, fizzbuzz_label или нещо в тоя дух.
+
+for i in 1..n + 1{
+ if i % 3 == 0 && i% 5 != 0{
+ res.push(fizz.clone());
}
- res
+ else if i %3 !=0 && i % 5 == 0{
+ res.push(buzz.clone());
+ }
+ else if i % 3 == 0 && i % 5 == 0{
+ res.push(fizzbuzz1.clone());
+ }
+ else{
+ res.push(i.to_string());
+ }
+}
+res
}
pub fn custom_buzz(n: usize, k1: u8, k2: u8) -> Vec<String> {
- let mut res = Vec::<String>::new();
- if k1 == 0 || k1 == 1 || k2 == 0 || k2 == 1{
- panic!("Division by 0 is a philosophical problem,not maths one and there is no point in division by 1 :) ");
- }
+let mut res = Vec::<String>::new();
+let fizz = String::from("Fizz");
+let buzz = String::from("Buzz");
+let fizzbuzz1 = String::from("Fizzbuzz");
- let fizz = String::from("Fizz");
- let buzz = String::from("Buzz");
- let fizzbuzz1 = String::from("Fizzbuzz");
- for i in 1..n + 1{
- if i as u8 % k1 == 0 && i as u8 % k2 != 0{
- res.push(fizz.clone());
- }
- else if i as u8 % k1!=0 && i as u8 %k2 == 0{
- res.push(buzz.clone());
- }
- else if i as u8 % k1 == 0 && i as u8 %k2 == 0{
- res.push(fizzbuzz1.clone());
- }
- else{
- res.push(i.to_string());
- }
+
+if k1 == 0 || k1 == 1 || k2 == 0 || k2 == 1{
+ panic!("Division by 0 is a philosophical problem,not maths one and there is no point in division by 1 :) ");
+}
+
+
+for i in 1..n + 1{
+ if i as u8 % k1 == 0 && i as u8 % k2 != 0{
+ res.push(fizz.clone());
}
- res
+ else if i as u8 % k1!=0 && i as u8 %k2 == 0{
+ res.push(buzz.clone());
+ }
+ else if i as u8 % k1 == 0 && i as u8 %k2 == 0{
+ res.push(fizzbuzz1.clone());
+ }
+ else{
+ res.push(i.to_string());
+ }
}
+res
+}
pub struct FizzBuzzer {
- pub k1: u8,
- pub k2: u8,
- pub labels: [String; 3],
+pub k1: u8,
+pub k2: u8,
+pub labels: [String; 3],
}
impl FizzBuzzer {
-
- pub fn take(&self, n: usize) -> Vec<String> {
- let mut res = Vec::<String>::new();
+
+pub fn take(&self, n: usize) -> Vec<String> {
+ let mut res = Vec::<String>::new();
if self.k1 == 0 || self.k1 == 1 || self.k2 == 0 || self.k2 == 1{
panic!("Division by 0 is a philosophical problem,not maths one and there is no point in division by 1 :) ");
}
for i in 1..n + 1{
if i as u8 % self.k1 == 0 && i as u8 % self.k2 != 0{
- res.push(self.labels[0].to_string());
+ res.push(self.labels[0].clone());
}
else if i as u8 % self.k1!=0 && i as u8 % self.k2 == 0{
- res.push(self.labels[1].to_string());
+ res.push(self.labels[1].clone());
}
else if i as u8 % self.k1 == 0 && i as u8 % self.k2 == 0{
- res.push(self.labels[2].to_string());
+ res.push(self.labels[2].clone());
}
else{
- res.push(i.to_string());
+ res.push(i.to_string());
}
}
- res
- }
+res
+}
- pub fn change_label(&mut self, index: usize, value: &String) {
- if index > 2{
- panic!("Too big of an index.Try with one less or equal to 2");
- }
- let str_tmp = value;
- self.labels[index] = str_tmp.to_string();
-
+pub fn change_label(&mut self, index: usize, value: &String) {
+ if index > 2{
+ panic!("Too big of an index.Try with a number from [0,1,2]");
}
+ self.labels[index] = value.clone();
+
+ }
}
Не би трябвало да е проблем да използваш
fizzbuzzза име на променлива -- Функциите и променливите са в различни namespaces. Може би е добра идея да са различни, за да се избегне объркване и да се комуникира ясно разликата, но тогава бих избрал някакво по-комуникативно име :). Примерно,fizz_label,buzz_label,fizzbuzz_labelили нещо в тоя дух.