(05) ticket_v2 - (07) unwrap has nothing (necessarily) to do with unwrapping

[shblack]

This one seems completely disconnected from the parallel text. Unwrapping is not necessarily involved, at all, in solving the problem. It's not even particularly suggested by the way the problem is presented.

// TODO: `easy_ticket` should panic when the title is invalid.
//   When the description is invalid, instead, it should use a default description:
//   "Description not provided".
fn easy_ticket(title: String, description: String, status: Status) -> Ticket {
    // Validate title and panic if invalid
    if title.is_empty() {
        panic!("Title cannot be empty");
    }
    if title.len() > 50 {
        panic!("Title cannot be longer than 50 bytes");
    }

    // Use default description if description is invalid
    let final_description = if description.is_empty() || description.len() > 500 {
        "Description not provided".to_string()
    } else {
        description
    };

    Ticket {
        title,
        description: final_description,
        status,
    }
}

[shblack]

The problem could be improved to demonstrate the intended material:

// TODO: `easy_ticket` should panic when the title is invalid.
//   When the description is invalid, instead, it should use a default description:
//   "Description not provided".
fn easy_ticket(title: String, description: String, status: Status) -> Ticket {
    todo!()
}

// Helper function that validates titles and returns a Result
// You should call this function and use unwrap() on its result
fn validate_title(title: &str) -> Result<String, String> {
    if title.is_empty() {
        return Err("Title cannot be empty".to_string());
    }
    if title.len() > 50 {
        return Err("Title cannot be longer than 50 bytes".to_string());
    }
    Ok(title.to_string())
}

// Helper function that validates descriptions and returns a Result
// You should call this function and use match on its result
fn validate_description(description: &str) -> Result<String, String> {
    if description.is_empty() {
        return Err("Description cannot be empty".to_string());
    }
    if description.len() > 500 {
        return Err("Description cannot be longer than 500 bytes".to_string());
    }
    Ok(description.to_string())

Solution:

fn easy_ticket(title: String, description: String, status: Status) -> Ticket {
    let title = validate_title(&title).unwrap();
    match validate_description(&description) {
        Ok(description) => Ticket {
            title,
            description,
            status,
        },
        Err(_) => Ticket {
            title,
            description: "Description not provided".to_string(),
            status,
        },
    }
}