1

I am writing a program that does probably a little bit too much with string processing. I moved most of the literal messages to constants; I'm not sure if that is the proper way in Rust, but I'm used to writing that in C.

I figured out that I cannot easily use my static &str inside a match expression. I can use the text itself, but cannot figure out how to do that properly.

I understand it's a compiler issue, but don't know how to write that construction properly in Rust style. Should I use enums instead of C-like static variables?

static SECTION_TEST: &str = "test result:";
static STATUS_TEST_OK: &str = "PASSED";

fn match_out(out: &String) -> bool {
    let s = &out[out.find(SECTION_TEST).unwrap() + SECTION_TEST.len()..];

    match s {
        STATUS_TEST_OK => {
            println!("Yes");
            true
        }
        _ => {
            println!("No");
            false
        }
    }
}
error[E0530]: match bindings cannot shadow statics
 --> src/lib.rs:8:9
  |
2 | static STATUS_TEST_OK: &str = "PASSED";
  | --------------------------------------- the static `STATUS_TEST_OK` is defined here
...
8 |         STATUS_TEST_OK => {
  |         ^^^^^^^^^^^^^^ cannot be named the same as a static
Shepmaster
  • 326,504
  • 69
  • 892
  • 1,159
Mazeryt
  • 765
  • 1
  • 13
  • 36
  • [Why is it discouraged to accept a reference to a String (&String), Vec (&Vec), or Box (&Box) as a function argument?](https://stackoverflow.com/q/40006219/155423) – Shepmaster Apr 14 '19 at 20:38
  • Hmm, yes in my code I used: &out[out.find(SECTION_TEST).unwrap()+SECTION_TEST.len()..] Just posted simplified version, which as you mentioned wont compile. Is that construction ok? I am just reading the link that you have sent – Mazeryt Apr 14 '19 at 20:43

1 Answers1

4

Use a constant instead of a static:

const STATUS_TEST_OK: &str = "PASSED";

fn match_out(s: &str) -> bool {
    match s {
        STATUS_TEST_OK => {
            println!("Yes");
            true
        }
        _ => {
            println!("No");
            false
        }
    }
}

See also:

Shepmaster
  • 326,504
  • 69
  • 892
  • 1,159