src/librustc_typeck/check/method/suggest.rs

   1 // Copyright 2014 The Rust Project Developers. See the COPYRIGHT
   2 // file at the top-level directory of this distribution and at
   3 // http://rust-lang.org/COPYRIGHT.
   4 //
   5 // Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
   6 // http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
   7 // <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
   8 // option. This file may not be copied, modified, or distributed
   9 // except according to those terms.
  10
  11 //! Give useful errors and suggestions to users when an item can't be
  12 //! found or is otherwise invalid.
  13
  14 use check::FnCtxt;
  15 use rustc::hir::map as hir_map;
  16 use rustc::ty::{self, Ty, TyCtxt, ToPolyTraitRef, ToPredicate, TypeFoldable};
  17 use hir::def::Def;
  18 use hir::def_id::{CRATE_DEF_INDEX, DefId};
  19 use middle::lang_items::FnOnceTraitLangItem;
  20 use rustc::traits::{Obligation, SelectionContext};
  21 use util::nodemap::FxHashSet;
  22
  23 use syntax::ast;
  24 use errors::DiagnosticBuilder;
  25 use syntax_pos::Span;
  26
  27 use rustc::hir;
  28 use rustc::hir::print;
  29 use rustc::infer::type_variable::TypeVariableOrigin;
  30
  31 use std::cell;
  32 use std::cmp::Ordering;
  33
  34 use super::{MethodError, NoMatchData, CandidateSource};
  35 use super::probe::Mode;
  36
  37 impl<'a, 'gcx, 'tcx> FnCtxt<'a, 'gcx, 'tcx> {
  38     fn is_fn_ty(&self, ty: &Ty<'tcx>, span: Span) -> bool {
  39         let tcx = self.tcx;
  40         match ty.sty {
  41             // Not all of these (e.g. unsafe fns) implement FnOnce
  42             // so we look for these beforehand
  43             ty::TyClosure(..) |
  44             ty::TyFnDef(..) |
  45             ty::TyFnPtr(_) => true,
  46             // If it's not a simple function, look for things which implement FnOnce
  47             _ => {
  48                 let fn_once = match tcx.lang_items.require(FnOnceTraitLangItem) {
  49                     Ok(fn_once) => fn_once,
  50                     Err(..) => return false,
  51                 };
  52
  53                 self.autoderef(span, ty).any(|(ty, _)| {
  54                     self.probe(|_| {
  55                         let fn_once_substs = tcx.mk_substs_trait(ty,
  56                             &[self.next_ty_var(TypeVariableOrigin::MiscVariable(span))]);
  57                         let trait_ref = ty::TraitRef::new(fn_once, fn_once_substs);
  58                         let poly_trait_ref = trait_ref.to_poly_trait_ref();
  59                         let obligation =
  60                             Obligation::misc(span,
  61                                              self.body_id,
  62                                              self.param_env,
  63                                              poly_trait_ref.to_predicate());
  64                         SelectionContext::new(self).evaluate_obligation(&obligation)
  65                     })
  66                 })
  67             }
  68         }
  69     }
  70
  71     pub fn report_method_error(&self,
  72                                span: Span,
  73                                rcvr_ty: Ty<'tcx>,
  74                                item_name: ast::Name,
  75                                rcvr_expr: Option<&hir::Expr>,
  76                                error: MethodError<'tcx>,
  77                                args: Option<&'gcx [hir::Expr]>) {
  78         // avoid suggestions when we don't know what's going on.
  79         if rcvr_ty.references_error() {
  80             return;
  81         }
  82
  83         let report_candidates = |err: &mut DiagnosticBuilder, mut sources: Vec<CandidateSource>| {
  84
  85             sources.sort();
  86             sources.dedup();
  87             // Dynamic limit to avoid hiding just one candidate, which is silly.
  88             let limit = if sources.len() == 5 { 5 } else { 4 };
  89
  90             for (idx, source) in sources.iter().take(limit).enumerate() {
  91                 match *source {
  92                     CandidateSource::ImplSource(impl_did) => {
  93                         // Provide the best span we can. Use the item, if local to crate, else
  94                         // the impl, if local to crate (item may be defaulted), else nothing.
  95                         let item = self.associated_item(impl_did, item_name)
  96                             .or_else(|| {
  97                                 self.associated_item(
  98                                     self.tcx.impl_trait_ref(impl_did).unwrap().def_id,
  99
 100                                     item_name
 101                                 )
 102                             }).unwrap();
 103                         let note_span = self.tcx.hir.span_if_local(item.def_id).or_else(|| {
 104                             self.tcx.hir.span_if_local(impl_did)
 105                         });
 106
 107                         let impl_ty = self.impl_self_ty(span, impl_did).ty;
 108
 109                         let insertion = match self.tcx.impl_trait_ref(impl_did) {
 110                             None => format!(""),
 111                             Some(trait_ref) => {
 112                                 format!(" of the trait `{}`",
 113                                         self.tcx.item_path_str(trait_ref.def_id))
 114                             }
 115                         };
 116
 117                         let note_str = format!("candidate #{} is defined in an impl{} \
 118                                                 for the type `{}`",
 119                                                idx + 1,
 120                                                insertion,
 121                                                impl_ty);
 122                         if let Some(note_span) = note_span {
 123                             // We have a span pointing to the method. Show note with snippet.
 124                             err.span_note(note_span, &note_str);
 125                         } else {
 126                             err.note(&note_str);
 127                         }
 128                     }
 129                     CandidateSource::TraitSource(trait_did) => {
 130                         let item = self.associated_item(trait_did, item_name).unwrap();
 131                         let item_span = self.tcx.def_span(item.def_id);
 132                         span_note!(err,
 133                                    item_span,
 134                                    "candidate #{} is defined in the trait `{}`",
 135                                    idx + 1,
 136                                    self.tcx.item_path_str(trait_did));
 137                         err.help(&format!("to disambiguate the method call, write `{}::{}({}{})` \
 138                                           instead",
 139                                           self.tcx.item_path_str(trait_did),
 140                                           item_name,
 141                                           if rcvr_ty.is_region_ptr() && args.is_some() {
 142                                               if rcvr_ty.is_mutable_pointer() {
 143                                                   "&mut "
 144                                               } else {
 145                                                   "&"
 146                                               }
 147                                           } else {
 148                                               ""
 149                                           },
 150                                           args.map(|arg| arg.iter()
 151                                               .map(|arg| print::to_string(print::NO_ANN,
 152                                                                           |s| s.print_expr(arg)))
 153                                               .collect::<Vec<_>>()
 154                                               .join(", ")).unwrap_or("...".to_owned())));
 155                     }
 156                 }
 157             }
 158             if sources.len() > limit {
 159                 err.note(&format!("and {} others", sources.len() - limit));
 160             }
 161         };
 162
 163         match error {
 164             MethodError::NoMatch(NoMatchData { static_candidates: static_sources,
 165                                                unsatisfied_predicates,
 166                                                out_of_scope_traits,
 167                                                mode,
 168                                                .. }) => {
 169                 let tcx = self.tcx;
 170
 171                 let actual = self.resolve_type_vars_if_possible(&rcvr_ty);
 172                 let mut err = if !actual.references_error() {
 173                     struct_span_err!(tcx.sess, span, E0599,
 174                                      "no {} named `{}` found for type `{}` in the \
 175                                       current scope",
 176                                      if mode == Mode::MethodCall {
 177                                          "method"
 178                                      } else {
 179                                          match item_name.as_str().chars().next() {
 180                                              Some(name) => {
 181                                                  if name.is_lowercase() {
 182                                                      "function or associated item"
 183                                                  } else {
 184                                                      "associated item"
 185                                                  }
 186                                              },
 187                                              None => {
 188                                                  ""
 189                                              },
 190                                          }
 191                                      },
 192                                      item_name,
 193                                      self.ty_to_string(actual))
 194                 } else {
 195                     self.tcx.sess.diagnostic().struct_dummy()
 196                 };
 197
 198                 // If the method name is the name of a field with a function or closure type,
 199                 // give a helping note that it has to be called as (x.f)(...).
 200                 if let Some(expr) = rcvr_expr {
 201                     for (ty, _) in self.autoderef(span, rcvr_ty) {
 202                         match ty.sty {
 203                             ty::TyAdt(def, substs) if !def.is_enum() => {
 204                                 if let Some(field) = def.struct_variant()
 205                                     .find_field_named(item_name) {
 206                                     let snippet = tcx.sess.codemap().span_to_snippet(expr.span);
 207                                     let expr_string = match snippet {
 208                                         Ok(expr_string) => expr_string,
 209                                         _ => "s".into(), // Default to a generic placeholder for the
 210                                         // expression when we can't generate a
 211                                         // string snippet
 212                                     };
 213
 214                                     let field_ty = field.ty(tcx, substs);
 215                                     let scope = self.tcx.hir.get_module_parent(self.body_id);
 216                                     if field.vis.is_accessible_from(scope, self.tcx) {
 217                                         if self.is_fn_ty(&field_ty, span) {
 218                                             err.help(&format!("use `({0}.{1})(...)` if you \
 219                                                                meant to call the function \
 220                                                                stored in the `{1}` field",
 221                                                               expr_string,
 222                                                               item_name));
 223                                         } else {
 224                                             err.help(&format!("did you mean to write `{0}.{1}` \
 225                                                                instead of `{0}.{1}(...)`?",
 226                                                               expr_string,
 227                                                               item_name));
 228                                         }
 229                                         err.span_label(span, "field, not a method");
 230                                     } else {
 231                                         err.span_label(span, "private field, not a method");
 232                                     }
 233                                     break;
 234                                 }
 235                             }
 236                             _ => {}
 237                         }
 238                     }
 239                 }
 240
 241                 if self.is_fn_ty(&rcvr_ty, span) {
 242                     macro_rules! report_function {
 243                         ($span:expr, $name:expr) => {
 244                             err.note(&format!("{} is a function, perhaps you wish to call it",
 245                                               $name));
 246                         }
 247                     }
 248
 249                     if let Some(expr) = rcvr_expr {
 250                         if let Ok(expr_string) = tcx.sess.codemap().span_to_snippet(expr.span) {
 251                             report_function!(expr.span, expr_string);
 252                         } else if let hir::ExprPath(hir::QPath::Resolved(_, ref path)) = expr.node {
 253                             if let Some(segment) = path.segments.last() {
 254                                 report_function!(expr.span, segment.name);
 255                             }
 256                         }
 257                     }
 258                 }
 259
 260                 if !static_sources.is_empty() {
 261                     err.note("found the following associated functions; to be used as methods, \
 262                               functions must have a `self` parameter");
 263
 264                     report_candidates(&mut err, static_sources);
 265                 }
 266
 267                 if !unsatisfied_predicates.is_empty() {
 268                     let bound_list = unsatisfied_predicates.iter()
 269                         .map(|p| format!("`{} : {}`", p.self_ty(), p))
 270                         .collect::<Vec<_>>()
 271                         .join("\n");
 272                     err.note(&format!("the method `{}` exists but the following trait bounds \
 273                                        were not satisfied:\n{}",
 274                                       item_name,
 275                                       bound_list));
 276                 }
 277
 278                 self.suggest_traits_to_import(&mut err,
 279                                               span,
 280                                               rcvr_ty,
 281                                               item_name,
 282                                               rcvr_expr,
 283                                               out_of_scope_traits);
 284                 err.emit();
 285             }
 286
 287             MethodError::Ambiguity(sources) => {
 288                 let mut err = struct_span_err!(self.sess(),
 289                                                span,
 290                                                E0034,
 291                                                "multiple applicable items in scope");
 292                 err.span_label(span, format!("multiple `{}` found", item_name));
 293
 294                 report_candidates(&mut err, sources);
 295                 err.emit();
 296             }
 297
 298             MethodError::ClosureAmbiguity(trait_def_id) => {
 299                 let msg = format!("the `{}` method from the `{}` trait cannot be explicitly \
 300                                    invoked on this closure as we have not yet inferred what \
 301                                    kind of closure it is",
 302                                   item_name,
 303                                   self.tcx.item_path_str(trait_def_id));
 304                 let msg = if let Some(callee) = rcvr_expr {
 305                     format!("{}; use overloaded call notation instead (e.g., `{}()`)",
 306                             msg,
 307                             self.tcx.hir.node_to_pretty_string(callee.id))
 308                 } else {
 309                     msg
 310                 };
 311                 self.sess().span_err(span, &msg);
 312             }
 313
 314             MethodError::PrivateMatch(def) => {
 315                 struct_span_err!(self.tcx.sess, span, E0624,
 316                                  "{} `{}` is private", def.kind_name(), item_name).emit();
 317             }
 318
 319             MethodError::IllegalSizedBound(candidates) => {
 320                 let msg = format!("the `{}` method cannot be invoked on a trait object", item_name);
 321                 let mut err = self.sess().struct_span_err(span, &msg);
 322                 if !candidates.is_empty() {
 323                     let help = format!("{an}other candidate{s} {were} found in the following \
 324                                         trait{s}, perhaps add a `use` for {one_of_them}:",
 325                                     an = if candidates.len() == 1 {"an" } else { "" },
 326                                     s = if candidates.len() == 1 { "" } else { "s" },
 327                                     were = if candidates.len() == 1 { "was" } else { "were" },
 328                                     one_of_them = if candidates.len() == 1 {
 329                                         "it"
 330                                     } else {
 331                                         "one_of_them"
 332                                     });
 333                     self.suggest_use_candidates(&mut err, help, candidates);
 334                 }
 335                 err.emit();
 336             }
 337         }
 338     }
 339
 340     fn suggest_use_candidates(&self,
 341                               err: &mut DiagnosticBuilder,
 342                               mut msg: String,
 343                               candidates: Vec<DefId>) {
 344         let limit = if candidates.len() == 5 { 5 } else { 4 };
 345         for (i, trait_did) in candidates.iter().take(limit).enumerate() {
 346             msg.push_str(&format!("\ncandidate #{}: `use {};`",
 347                                     i + 1,
 348                                     self.tcx.item_path_str(*trait_did)));
 349         }
 350         if candidates.len() > limit {
 351             msg.push_str(&format!("\nand {} others", candidates.len() - limit));
 352         }
 353         err.note(&msg[..]);
 354     }
 355
 356     fn suggest_traits_to_import(&self,
 357                                 err: &mut DiagnosticBuilder,
 358                                 span: Span,
 359                                 rcvr_ty: Ty<'tcx>,
 360                                 item_name: ast::Name,
 361                                 rcvr_expr: Option<&hir::Expr>,
 362                                 valid_out_of_scope_traits: Vec<DefId>) {
 363         if !valid_out_of_scope_traits.is_empty() {
 364             let mut candidates = valid_out_of_scope_traits;
 365             candidates.sort();
 366             candidates.dedup();
 367             err.help("items from traits can only be used if the trait is in scope");
 368             let msg = format!("the following {traits_are} implemented but not in scope, \
 369                                perhaps add a `use` for {one_of_them}:",
 370                             traits_are = if candidates.len() == 1 {
 371                                 "trait is"
 372                             } else {
 373                                 "traits are"
 374                             },
 375                             one_of_them = if candidates.len() == 1 {
 376                                 "it"
 377                             } else {
 378                                 "one of them"
 379                             });
 380
 381             self.suggest_use_candidates(err, msg, candidates);
 382             return;
 383         }
 384
 385         let type_is_local = self.type_derefs_to_local(span, rcvr_ty, rcvr_expr);
 386
 387         // there's no implemented traits, so lets suggest some traits to
 388         // implement, by finding ones that have the item name, and are
 389         // legal to implement.
 390         let mut candidates = all_traits(self.tcx)
 391             .filter(|info| {
 392                 // we approximate the coherence rules to only suggest
 393                 // traits that are legal to implement by requiring that
 394                 // either the type or trait is local. Multidispatch means
 395                 // this isn't perfect (that is, there are cases when
 396                 // implementing a trait would be legal but is rejected
 397                 // here).
 398                 (type_is_local || info.def_id.is_local())
 399                     && self.associated_item(info.def_id, item_name).is_some()
 400             })
 401             .collect::<Vec<_>>();
 402
 403         if !candidates.is_empty() {
 404             // sort from most relevant to least relevant
 405             candidates.sort_by(|a, b| a.cmp(b).reverse());
 406             candidates.dedup();
 407
 408             // FIXME #21673 this help message could be tuned to the case
 409             // of a type parameter: suggest adding a trait bound rather
 410             // than implementing.
 411             err.help("items from traits can only be used if the trait is implemented and in scope");
 412             let mut msg = format!("the following {traits_define} an item `{name}`, \
 413                                    perhaps you need to implement {one_of_them}:",
 414                                   traits_define = if candidates.len() == 1 {
 415                                       "trait defines"
 416                                   } else {
 417                                       "traits define"
 418                                   },
 419                                   one_of_them = if candidates.len() == 1 {
 420                                       "it"
 421                                   } else {
 422                                       "one of them"
 423                                   },
 424                                   name = item_name);
 425
 426             for (i, trait_info) in candidates.iter().enumerate() {
 427                 msg.push_str(&format!("\ncandidate #{}: `{}`",
 428                                       i + 1,
 429                                       self.tcx.item_path_str(trait_info.def_id)));
 430             }
 431             err.note(&msg[..]);
 432         }
 433     }
 434
 435     /// Checks whether there is a local type somewhere in the chain of
 436     /// autoderefs of `rcvr_ty`.
 437     fn type_derefs_to_local(&self,
 438                             span: Span,
 439                             rcvr_ty: Ty<'tcx>,
 440                             rcvr_expr: Option<&hir::Expr>)
 441                             -> bool {
 442         fn is_local(ty: Ty) -> bool {
 443             match ty.sty {
 444                 ty::TyAdt(def, _) => def.did.is_local(),
 445
 446                 ty::TyDynamic(ref tr, ..) => tr.principal()
 447                     .map_or(false, |p| p.def_id().is_local()),
 448
 449                 ty::TyParam(_) => true,
 450
 451                 // everything else (primitive types etc.) is effectively
 452                 // non-local (there are "edge" cases, e.g. (LocalType,), but
 453                 // the noise from these sort of types is usually just really
 454                 // annoying, rather than any sort of help).
 455                 _ => false,
 456             }
 457         }
 458
 459         // This occurs for UFCS desugaring of `T::method`, where there is no
 460         // receiver expression for the method call, and thus no autoderef.
 461         if rcvr_expr.is_none() {
 462             return is_local(self.resolve_type_vars_with_obligations(rcvr_ty));
 463         }
 464
 465         self.autoderef(span, rcvr_ty).any(|(ty, _)| is_local(ty))
 466     }
 467 }
 468
 469 pub type AllTraitsVec = Vec<DefId>;
 470
 471 #[derive(Copy, Clone)]
 472 pub struct TraitInfo {
 473     pub def_id: DefId,
 474 }
 475
 476 impl TraitInfo {
 477     fn new(def_id: DefId) -> TraitInfo {
 478         TraitInfo { def_id: def_id }
 479     }
 480 }
 481 impl PartialEq for TraitInfo {
 482     fn eq(&self, other: &TraitInfo) -> bool {
 483         self.cmp(other) == Ordering::Equal
 484     }
 485 }
 486 impl Eq for TraitInfo {}
 487 impl PartialOrd for TraitInfo {
 488     fn partial_cmp(&self, other: &TraitInfo) -> Option<Ordering> {
 489         Some(self.cmp(other))
 490     }
 491 }
 492 impl Ord for TraitInfo {
 493     fn cmp(&self, other: &TraitInfo) -> Ordering {
 494         // local crates are more important than remote ones (local:
 495         // cnum == 0), and otherwise we throw in the defid for totality
 496
 497         let lhs = (other.def_id.krate, other.def_id);
 498         let rhs = (self.def_id.krate, self.def_id);
 499         lhs.cmp(&rhs)
 500     }
 501 }
 502
 503 /// Retrieve all traits in this crate and any dependent crates.
 504 pub fn all_traits<'a, 'gcx, 'tcx>(tcx: TyCtxt<'a, 'gcx, 'tcx>) -> AllTraits<'a> {
 505     if tcx.all_traits.borrow().is_none() {
 506         use rustc::hir::itemlikevisit;
 507
 508         let mut traits = vec![];
 509
 510         // Crate-local:
 511         //
 512         // meh.
 513         struct Visitor<'a, 'tcx: 'a> {
 514             map: &'a hir_map::Map<'tcx>,
 515             traits: &'a mut AllTraitsVec,
 516         }
 517         impl<'v, 'a, 'tcx> itemlikevisit::ItemLikeVisitor<'v> for Visitor<'a, 'tcx> {
 518             fn visit_item(&mut self, i: &'v hir::Item) {
 519                 match i.node {
 520                     hir::ItemTrait(..) => {
 521                         let def_id = self.map.local_def_id(i.id);
 522                         self.traits.push(def_id);
 523                     }
 524                     _ => {}
 525                 }
 526             }
 527
 528             fn visit_trait_item(&mut self, _trait_item: &hir::TraitItem) {
 529             }
 530
 531             fn visit_impl_item(&mut self, _impl_item: &hir::ImplItem) {
 532             }
 533         }
 534         tcx.hir.krate().visit_all_item_likes(&mut Visitor {
 535             map: &tcx.hir,
 536             traits: &mut traits,
 537         });
 538
 539         // Cross-crate:
 540         let mut external_mods = FxHashSet();
 541         fn handle_external_def(tcx: TyCtxt,
 542                                traits: &mut AllTraitsVec,
 543                                external_mods: &mut FxHashSet<DefId>,
 544                                def: Def) {
 545             let def_id = def.def_id();
 546             match def {
 547                 Def::Trait(..) => {
 548                     traits.push(def_id);
 549                 }
 550                 Def::Mod(..) => {
 551                     if !external_mods.insert(def_id) {
 552                         return;
 553                     }
 554                     for child in tcx.sess.cstore.item_children(def_id, tcx.sess) {
 555                         handle_external_def(tcx, traits, external_mods, child.def)
 556                     }
 557                 }
 558                 _ => {}
 559             }
 560         }
 561         for cnum in tcx.sess.cstore.crates() {
 562             let def_id = DefId {
 563                 krate: cnum,
 564                 index: CRATE_DEF_INDEX,
 565             };
 566             handle_external_def(tcx, &mut traits, &mut external_mods, Def::Mod(def_id));
 567         }
 568
 569         *tcx.all_traits.borrow_mut() = Some(traits);
 570     }
 571
 572     let borrow = tcx.all_traits.borrow();
 573     assert!(borrow.is_some());
 574     AllTraits {
 575         borrow: borrow,
 576         idx: 0,
 577     }
 578 }
 579
 580 pub struct AllTraits<'a> {
 581     borrow: cell::Ref<'a, Option<AllTraitsVec>>,
 582     idx: usize,
 583 }
 584
 585 impl<'a> Iterator for AllTraits<'a> {
 586     type Item = TraitInfo;
 587
 588     fn next(&mut self) -> Option<TraitInfo> {
 589         let AllTraits { ref borrow, ref mut idx } = *self;
 590         // ugh.
 591         borrow.as_ref().unwrap().get(*idx).map(|info| {
 592             *idx += 1;
 593             TraitInfo::new(*info)
 594         })
 595     }
 596 }