Running example on Rust Playground.
Note: the example above uses two external libraries. This adds some complexity in the event that you want to avoid cargo for some reason.
Summary of changes
Use byte literals instead of casting byte to char.
- match pattern[p] as char {
- '*' => {
+ match pattern[p] {
+ b'*' => {Use an existing path walker instead of rolling our own. Yours works fine, but using an existing function is more in line with i.e. the go implementation.
- for entry in fs::read_dir(dir)? {
- let path = entry?.path();
- if path.is_dir() {
- walk(&path, &pattern)?;
- continue;
- }
+ for entry in WalkDir::new(".") {
+ let entry = entry?;
+ if entry.file_type().is_dir() {
+ continue;
+ }Use standard convenience combinators:
- let line = match line {
- Ok(s) => s,
- Err(_) => "".to_string(),
- };
+ let line = line.unwrap_or_default();Show an approximation of the path instead of an empty string if the path contains non-unicode characters:
- println!("{}:{}\t{}", path.to_str().unwrap_or(""), lineno, line);
+ println!("{}:{}\t{}", path.display(), lineno, line);Print any error message to stderr and set an appropriate error code instead of panicing, printing to stdout, etc.
- match walk(Path::new("."), &argv[1]) {
- Ok(()) => (),
- Err(error) => panic!("Error: {:?}", error),
- };
+ let exit_code = match main_inner() {
+ Ok(_) => 0,
+ Err(err) => {
+ eprintln!("{}", err);
+ 1
+ }
+ };
+ std::process::exit(exit_code);Run each unit test case independently.
- fn test_glob() {
- assert!(glob(b"", b""));
- assert!(glob(b"hello", b"hello"));
+ tests! {
+ empty(b"", b"");
+ literal(b"hello", b"hello");
React
Typescript
Django
Laravel
Facebook
Microsoft
Google
Alibaba
D3
Tencent