Там 2/3 сначала можно не читать.
Суть в том, что автор редактировал код крейта, чтобы добавить туда Send

Добрый день,

Подскажите пожалуйста
Нужно сформировать новую строку исходя из наличия 'http' в &String
Делаю через clone но не уверен что это правильно хотя и работает.

host.domain является &String


let mut t = host.domain.clone();
if !host.domain.contains("http") {
   t =  format!("{}http://{}", url, t);
}

то есть надо получить в результате url c "http"

Вообще выглядит так, будто я могу это закостылять через typetag, но там толи ron не умеет толи чего. В итоге:

[
  {"A": (1, 2, 3)},
  {"B": ("abc")},
]

Вместо

[
  A(1, 2, 3),
  B("abc"),
]

Впрочем и так сойдёт

contains слишком расточительно, лучше использовать .starts_with().
И "http" - не достаточно конкретно (вдруг у меня хост называется http_host). Надо проверять хотя бы на "http:/", если не предполагается наличие https

@cykooz спасибо, то есть clone() в данном случае это ок ?

let t = if !host.domain.starts_with("http:/") {
    format!("{}http://{}", url, host.domain)
} 
else {
    host.domain.clone()
};

Только я не понял что такое url и почему его не должно быть в else ветке.

format!() сам создаёт новый String, не обязательно ему передавать клон исходной строки

@cykooz без clone проблема была такой

Через match будет на пару строк меньше:

let t = match host.domain.starts_with("http:/") {
    true => host.domain.clone(),
    false => format!("{}http://{}", url, host.domain),
};

Вы случайно не из Python в Rust пришли? 😊
Это обычно там такое практикуют - сначала присвоить переменной дефолтное значение, а потом одним if без else-ветки поменять его на другое значение.

да, через match тоже делал, я хотел избавиться от clone(). :).  Точно!  из python

Ну вот в Rust - это преждевременная операция, а в случае с clone() ещё и вероятно операция с системным вызовом и аллокацией памяти. Лучше сначала определиться что нужно, а потом делать это.

я перепробовал уже несколько конструкций пока оставил привычную со временем адаптируюсь

я это понимаю 🙂 спасибо

Ну вот я вам привёл вариант с match - используйте его, и не будет у вас лишнего clone() если он не нужен.

пасиб тока сейчас я получаю
  true => &host.domain,
  | |                 ------------ this is found to be of type &std::string::String
95 | |         false => format!("{}http://{}", url, host.domain),
  | |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected &std::string::String, found struct std::string::String

Да, потому что у вас типы данных в ветках матча разные. Поэтому в моём примере используется clone()

оки привел  to_string() и прошел дальше спасибо

Если хотите уменьшить число клонов и использовать ссылки, то можно вот так:

  let tmp: String;
  let host = "domain".to_string();
  let t = match host.starts_with("http:/") {
     true => &host,
     false => {
       tmp = format!("http://{}", host);
       &tmp
     }
  };