<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div><blockquote type="cite" class=""><div class="">On Aug 2, 2017, at 10:49 AM, Xiaodi Wu via swift-evolution <<a href="mailto:swift-evolution@swift.org" class="">swift-evolution@swift.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><div class="gmail_quote"><div dir="auto" class="">Brent had a draft proposal to revise the names of collection methods to improve the situation here. There is room for improvement.</div></div></div></div></blockquote><br class=""></div><div>It didn't change `remove(at:)`, though. (Well, it might have affected its return value or something, I don't remember.) It mostly addressed the `dropLast()`, `prefix(_:)` etc. calls.</div><div><br class=""></div><div>To respond to the original post:</div><div><br class=""></div><div>Some of the APIs you cite are not very well named, but I think your diagnosis is incorrect and so your prescription isn't going to help.</div><div><br class=""></div><div><div class="" style="font-family: HelveticaNeue;"></div><blockquote type="cite" class=""><div class="" style="font-family: HelveticaNeue;">The reason for this is, whenever a preposition is used in English, it almost always takes a dyadic form, relating a subject to the preposition's object. The two most common dyadic formats are:</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;"><b class=""><subject> [<preposition> <object of preposition>]</b></div><div class="" style="font-family: HelveticaNeue;"><The boy> [<with> <the dog>] crossed the street. </div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;"><b class="">[<preposition> <</b><b class=""> object of preposition</b><b class="">>] <subject></b></div><div class="" style="font-family: HelveticaNeue;">[<In> <space>], <no one> can hear you scream.</div><div class="" style="font-family: HelveticaNeue;">[<On> <the Moon>] are <many craters>.</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">Now, in Objective C through Swift 1 and 2, prepositions' dyadic nature were generally respected in method signatures. However, Swift 3's migration of the preposition inside the parentheses also seems to have been accompanied by the stripping away of either the subject, the prepositional object, or both—according to no discernible pattern. For example: </div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">(1) CloudKit:</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><blockquote class="" style="font-family: HelveticaNeue; margin: 0px 0px 0px 40px; border: none; padding: 0px;"><div class="">old: myCKDatabase.fetchRecordWithID(recordID)</div><div class="">new: myCKDatabase.fetch(withRecordID: recordID)</div><div class=""><b class="">(subject "Record" got removed)</b></div></blockquote><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">(2) String:</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><blockquote class="" style="font-family: HelveticaNeue; margin: 0px 0px 0px 40px; border: none; padding: 0px;"><div class="">old: myString.capitalizedStringWithLocale(_: myLocale)</div><div class="">new: myString.capitalized(with: myLocale)</div><div class=""><b class="">(subject "String" and prep. object "Locale" both got removed)</b></div></blockquote><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">(3) Dictionary:</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><blockquote class="" style="font-family: HelveticaNeue; margin: 0px 0px 0px 40px; border: none; padding: 0px;"><div class="">old: myDict.removeAtIndex(myIndex)</div><div class="">new: myDict.remove(at: myIndex)</div><div class=""><b class="">(subject "element" already missing from both; prep. object "Index" got removed)</b></div></blockquote><div class="" style="font-family: HelveticaNeue;"></div></blockquote><div><br class=""></div><div>The thing is, the subjects and prepositional objects *are* present in all of these—they are the parameters and targets of the calls. </div><div><br class=""></div><div>In your own example, you say "In space, no one can hear you scream", not "In location space, group-of-people no one can hear you scream". So why is it a problem that we say "myString, capitalized with myLocale" instead of "myString, string capitalized with locale myLocale"? These are redundancies that we would never tolerate in natural language; I don't see why you think code should be different.</div><br class=""><blockquote type="cite" class=""><div class="" style="font-family: HelveticaNeue;">(4) Date:</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><blockquote class="" style="font-family: HelveticaNeue; margin: 0px 0px 0px 40px; border: none; padding: 0px;"><div class="">old: myDate.timeIntervalSinceDate(myDate)</div><div class="">new: myDate.timeIntervalSince(date: myDate)</div><div class=""><b class="">(subject "timeInterval" and object "Date" both still present; but oddly, preposition "since" got left outside of the parentheses)</b></div></blockquote></blockquote><div><br class=""></div><div>This is actually inaccurate—the parameter to `timeIntervalSince(_:)` is unlabeled, so it's:</div><div><br class=""></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>new: myDate.timeIntervalSince(myDate)</div><br class=""><blockquote type="cite" class=""><font face="HelveticaNeue" class="">(5) Array:</font><div class=""><font face="HelveticaNeue" class=""><br class=""></font></div><div class=""><font face="HelveticaNeue" class=""><span class="Apple-tab-span" style="white-space: pre;"> </span> old: myArray.forEach({ thing in code})</font></div><blockquote class="" style="margin: 0px 0px 0px 40px; border: none; padding: 0px;"><div class=""><font face="HelveticaNeue" class="">new: myArray.forEach() { thing in //code }</font></div></blockquote><div class=""><font face="HelveticaNeue" class=""> <b class="">(preposition “for” is outside of the parentheses)</b><br class=""></font></div></blockquote><div class=""></div><div class=""><font face="HelveticaNeue" class=""><b class=""><br class=""></b></font></div></div><div>Yes, because the preposition does not apply to the parameter—it applies to the operation as a whole. I'll have more to say on this in a moment.</div><div><br class=""></div><div><blockquote type="cite" class=""><span style="font-family: HelveticaNeue;" class="">The inconsistency between the examples is shown in the bold text of each example, but lets go over why this matters. It matters because any language is easier to learn the more consistently it sticks to its own rules.</span></blockquote></div><div><br class=""></div><div>This is true, but you aren't just proposing sticking more closely to our existing standards—you're proposing *changing* our standards. And I don't think the changes you propose are an improvement. In fact, I'd say each of these examples is worse:</div><div><br class=""></div><div><div class="" style="font-family: HelveticaNeue;"></div><blockquote type="cite" class=""><div class="" style="font-family: HelveticaNeue;">(1) myCKDatabase.fetchRecord(withRecordID:)</div></blockquote><div><br class=""></div><div>"Fetch record with record ID"? I mean, you could at least remove the `Record` before `ID`. What other ID do you suppose it would be?</div><div><br class=""></div><div>I *can* see the case for going back to `fetchRecord` instead of just `fetch`, though. On the other hand, I do think that, if you know it's a database, the most obvious thing for `fetch` to be fetching is a record from that database. It's not a dog—it won't be fetching a stick.</div><br class=""><blockquote type="cite" class=""><div class="" style="font-family: HelveticaNeue;">(2) myString.stringCapitalized(withLocale:)</div></blockquote><div><br class=""></div><div>Let's translate this to an actual use site, which is what we care about.</div><div><br class=""></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>func tableView(_: UITableView, titleForHeaderInSection section: Int) -> String? {</div><div><span class="Apple-tab-span" style="white-space:pre"> </span>return sections[section].title.stringCapitalized(withLocale: .current)</div><div><span class="Apple-tab-span" style="white-space:pre"> </span>}</div><div><br class=""></div><div>What is `string` contributing here? We already know it's a "title", which sounds a lot like a string. If you asked for a "capitalized" string, what else would you get back if not another string?</div><div><br class=""></div><div>The locale parameter is a little more tricky. You're right that `(with: .current)` is vague, but I can imagine plenty of call sites where `with` wouldn't be:</div><div><br class=""></div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>title.capitalized(with: german)</div><div><span class="Apple-tab-span" style="white-space:pre"> </span>title.capitalized(with: docLocale)</div><div><span class="Apple-tab-span" style="white-space:pre"> </span>title.capitalized(with: otherUser.locale)</div><div><br class=""></div><div>Something at the call site needs to imply this is a locale, and there's nothing in `(with: .current)` that does so. This is arguably just a style issue, though: even though the language allows you to say `(with: .current)`, you really ought to say `(with: Locale.current)` to be clearer. Or perhaps the problem is with the name `current`—it ought to be `currentLocale` (even though that's redundant when you write it out as `Locale.currentLocale`), or it should use some location-ish terminology like `home` or `native`.</div><div><br class=""></div><div>(Actually, I think there might be a new guideline there: Variable and property names should at least hint at the type of the value they contain. Names like `current` or `shared` or `default` are too vague, and should usually be paired with a word that implies the type.)</div><div><br class=""></div><div>It might also help to change the `with` preposition to `in`, which would at least imply that the parameter is related to some kind of location.</div><div><br class=""></div><div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>title.capitalized(in: german)</div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>title.capitalized(in: docLocale)</div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>title.capitalized(in: otherUser.locale)</div><div class=""><div><span class="Apple-tab-span" style="white-space: pre;"> </span>title.capitalized(in: .current)<span class="Apple-tab-span" style="white-space:pre"> </span>// Still not great, but better</div></div><div><br class=""></div></div><blockquote type="cite" class=""><div class="" style="font-family: HelveticaNeue;">(3) myDictionary.elementRemoved(atIndex:)</div></blockquote><div><br class=""></div><div>This naming is exactly backwards, and is a perfect example of why we *don't* want rigid consistency:</div><div><br class=""></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>1. It emphasizes the element being returned, while making the "Removed" operation an afterthought, even though the removal is the main thing you want to happen and the element is returned as an afterthought.</div><div><br class=""></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>2. It mentions the "Index", even though there is no other plausible thing that could be labeled "at". (The only other plausible parameters to a `remove` method are an element, an array of elements, a predicate, or a range of indices. Of those four, only the range of indices could possibly make sense with "at", but that ambiguity is a harmless overloading.)</div><div><br class=""></div><div>Again, think about a use site:</div><div><br class=""></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>func tableView(_: UITableView, commit edit: UITableViewCellEditingStyle, forRowAt indexPath: IndexPath) {</div><div><span class="Apple-tab-span" style="white-space:pre"> </span>assert(edit == .delete)</div><div><span class="Apple-tab-span" style="white-space:pre"> </span></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>sections[indexPath.section].rows.<span style="font-family: HelveticaNeue;" class="">elementRemoved</span>(atIndex: indexPath.row)</div><div><span class="Apple-tab-span" style="white-space:pre"> </span>// vs.</div><div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>sections[indexPath.section].rows.<span style="font-family: HelveticaNeue;" class="">remove</span>(at: indexPath.row)</div></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>}</div><div><br class=""></div><div>In context, `elementRemoved` obscures the purpose of the line, and there is no ambiguity about what `at` means. The current name is *far* better.</div><br class=""><blockquote type="cite" class=""><div class="" style="font-family: HelveticaNeue;">(4) myDate.timeInterval(sinceDate:)</div></blockquote><div><br class=""></div><div>I have a hard time thinking of a call site where `sinceDate` would be an improvement over `since`; the preposition already *strongly* implies a temporal aspect to the parameter.</div><div><br class=""></div><div>If we remove `Date`, then in isolation I kind of like this change. The problem arrives when you step out of isolation and think about the other methods in its family. It would be strange to have `myDate.timeInterval(since: otherDate)`, but `myDate.timeIntervalSinceNow()`.</div><div><br class=""></div><div>One solution would be to add `now`, `referenceDate`, and `unixEpoch` (or `utc1970`) as static properties on `Date`. Then you could have just the one `timeInterval(since:)` method:</div><div><br class=""></div><div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>myDate.timeInterval(since: otherDate)</div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>myDate.timeInterval(since: .now)</div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>myDate.timeInterval(since: .referenceDate)</div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>myDate.timeInterval(since: .unixEpoch)</div><div class=""><br class=""></div><div class="">Another solution would be to add a `-` operator that takes two `Date`s and returns a `TimeInterval`, sidestepping the wording issue entirely.</div></div><div><br class=""></div><blockquote type="cite" class=""><div class="" style="font-family: HelveticaNeue;">(5) myArray.each(inClosure: ) </div></blockquote><div class=""><br class=""></div><div class="">I don't get this name at all. This operation is completely imperative and side-effectful, but there's no verb? How is it "in" the closure? Why "closure" when you can pass a function, and in fact you probably will *only* see this label if you're passing a function?</div><div class=""><br class=""></div></div><div>I do think there's a problem with `forEach(_:)`—it ought to be `forEach(do:)`. This is much like `DispatchQueue.async(execute:)` or the `withoutActuallyEscaping(_:do:)` function in the standard library. When you pass a function parameter, and the call's primary purpose is to run that parameter, it's often best to attach the verb to that parameter instead of putting it in the base name. Of course, the complication there is that the verb doesn't actually get written if you use trailing closure syntax, but the curlies sort of fill that role. Kind of.</div><div><br class=""></div><div><div class="" style="font-family: HelveticaNeue;"><blockquote type="cite" class="">Although I do understand removing "string" from the latter was to reduce redundancy in function/method declarations, we only make one declaration, yet we make many calls. So increasing ambiguity in calls does not seem like a good trade-off for decreased boilerplate in declarations. More often than not it's calls that we're reading, not the declarations—unless of course the call was ambiguous and we had to read the declaration to make sense out of it. So perhaps we might question if increased ambiguity is an overall good thing. </blockquote></div><div class=""><br class=""></div><div class="">I think you misunderstand the current Guidelines' goals. The Guidelines are not trying to reduce redundancy at declaration sites—they're trying to reduce redundancy at *call sites*. The idea is that, if the variable names for the method's target and parameters imply something about the types they contain, those names along with the prepositions will imply the purpose of each parameter, and therefore the call. The types are just a more formal version of that check.</div><div class=""><br class=""></div><div class="">That's why the very first paragraph of the API Design Guidelines <<a href="https://swift.org/documentation/api-design-guidelines/" class="">https://swift.org/documentation/api-design-guidelines/</a>> says:</div><div class=""><br class=""></div><div class=""><div class=""><blockquote type="cite" class=""><blockquote type="cite" class=""><blockquote type="cite" class="">Clarity at the point of use is your most important goal. Entities such as methods and properties are declared only once but used repeatedly. Design APIs to make those uses clear and concise. When evaluating a design, reading a declaration is seldom sufficient; always examine a use case to make sure it looks clear in context.<br class=""></blockquote></blockquote></blockquote><br class=""></div></div><div class="">So the tradeoff is not between *declaration redundancy* and call site clarity—it is between *call site redundancy* and call site ambiguity. Because their parameters are unlabeled, most languages have severe call site ambiguity problems. Objective-C has a pretty serious call site redundancy problem. Swift's design is trying to hit the baby bear "just right" point.</div><div class=""><br class=""></div><div class="">It is quite possible that, in some areas, we have swung too far back towards ambiguity. But I don't think `elementRemoved(atIndex:)` is going to fix that.</div><div class=""><br class=""></div><div class=""><div class="" style="font-family: HelveticaNeue;"></div></div><blockquote type="cite" class=""><div class=""><div class="" style="font-family: HelveticaNeue;">However this removal of explicit contextual cues from the method signature harms readability, since now, the compiler will let people write code like:</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">{ return $0.fetch(withRecordID:$1) }</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">Clearly, the onus is now on the developer not to use cryptic, short variable names or NO variable names. However, spend much time on GitHub or in CocoaPods and you will see an increasing number of codebases where that's exactly what they do, especially in closures.</div><div class="" style="font-family: HelveticaNeue;"></div></div></blockquote><div><br class=""></div><div>What I think you're missing with this example—and in fact with all of your closure-based examples—is that closures don't exist in isolation; they're in some larger context. (Otherwise, they won't compile.) For instance, the above closure might be in a line like:</div><div><br class=""></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>return zip(databases, recordIDs)</div><div><span class="Apple-tab-span" style="white-space:pre"> </span>.map <span style="font-family: HelveticaNeue;" class="">{ return $0.fetch(withRecordID:$1) }</span></div><div><br class=""></div><div>Read in the context of its line, the meanings of $0 and $1 are fairly clear.</div><br class=""><blockquote type="cite" class=""><div class=""><div class="" style="font-family: HelveticaNeue;">Another problem is that the compiler doesn't care if you write:</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">{ ambiguousName in</div><div class="" style="font-family: HelveticaNeue;">let myRecordID = ambiguousName.fetch(withRecordID:myID) </div><div class="" style="font-family: HelveticaNeue;">return myRecordID }</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">This is highly problematic because someone reading this code will have no reason to expect the type of "myRecordID" not to be CKRecordID. (In fact, it's CKRecord.)</div></div></blockquote><div class=""><br class=""></div><div class="">Again, in the larger context, this line will end up generating a `[CKRecord]` array instead of a `[CKRecordID]` array, which is probably going to cause a type mismatch once you try to actually use the alleged record IDs. (But as I said earlier, I can see the case for using `fetchRecord(withID:)` or `fetchRecord(with:)` instead of `fetch(withRecordID:)`.)</div><div class=""><br class=""></div><div class="">Ambiguous names can hide bugs in their ambiguity, but verbose names can also hide bugs in the sheer mass of code they generate. The difference is, developers can manage the ambiguity in their code by naming variables well, but they can't manage verbosity if verbose names are imposed on them.</div></div><div class=""><br class=""></div><div class=""><div class="" style="font-family: HelveticaNeue;"></div></div><blockquote type="cite" class=""><div class=""><div class="" style="font-family: HelveticaNeue;">We also have examples like: </div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">{ return $0.draw(with:$1) }</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">What is $0? What is $1? This is a real Apple API, BTW.</div></div></blockquote><div class=""><br class=""></div><div class=""></div><div class="">Again, the context of the closure would tell you, but part of the problem here is that they held onto an Objective-C preposition which was poorly chosen. If the line were `<span style="font-family: HelveticaNeue;" class="">$0.draw(in:$1)`, you would know that `$1` specified an area of the screen and `$0` was something that could be drawn, which frankly is all you really *need* to know to understand what this line does.</span></div><div class=""><span style="font-family: HelveticaNeue;" class=""><br class=""></span></div><blockquote type="cite" class=""><div class=""><div class="" style="font-family: HelveticaNeue;">{array, key in </div><div class="" style="font-family: HelveticaNeue;">let number = array.remove(at:key)</div><div class="" style="font-family: HelveticaNeue;">return number }</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">This will compile and run even though number will be a tuple key-value pair, array will be a dict, and key will be an index type! This may seem like a ridiculous example, but I have literally seen things like this.</div></div></blockquote><div class=""></div><div class=""><br class=""></div><div class="">Where have you seen something like this? `array` would have to be passed `inout` for this to work at all.</div><div class=""><br class=""></div><div class="">Nevertheless, how would more verbose names help with this problem? This is every bit as incorrect, and the compiler will still accept it:</div><div class=""><br class=""></div><div class=""><div class="" style="font-family: HelveticaNeue;"><span class="Apple-tab-span" style="white-space:pre"> </span>{array, key in </div><div class="" style="font-family: HelveticaNeue;"><span class="Apple-tab-span" style="white-space:pre"> </span>let number = array.elementRemoved(atIndex:key)</div><div class="" style="font-family: HelveticaNeue;"><span class="Apple-tab-span" style="white-space:pre"> </span>return number }</div></div><div class=""><div class="" style="font-family: HelveticaNeue;"><br class=""></div></div><div class="" style="font-family: HelveticaNeue;">Are you thinking that they'll notice that `atIndex` is not `atKey`? There is already a much stronger safeguard against that: `Dictionary.Index` and `Dictionary.Key` are different, totally incompatible types. Every mechanism I can think of to get a `Dictionary.Index` has "index" or "indices" in its name, so you could only make this mistake if you confused dictionary indices with dictionary keys, in which case `atIndex:` would not stand out to you either.</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;">Ultimately, unless the compiler actually understands identifiers, it's just not going to be able to catch mistakes like calling a dictionary an "array", or many of the other problems you describe here. But the type system can and often does flag these kinds of problems pretty close to the source.</div><div class="" style="font-family: HelveticaNeue;"><br class=""></div><div class="" style="font-family: HelveticaNeue;"><blockquote type="cite" class="">Orphaning method signatures by stripping useful return type and argument type information wouldn't be so bad if variables were all named descriptively, but that is a strangely optimistic hope for a language that's as paranoid about safety that it was specifically designed to prevent many categories of common mistakes.</blockquote></div><div class=""><br class=""></div><div class="">Personally, I think of Swift's approach to safety as similar to Reagan's "trust, but verify". Our naming conventions trust the programmer to write code with clear names; our picky type system verifies that the code is plausible. We don't force the programmer to explain herself to us until we notice that something doesn't seem right.</div><div class=""><br class=""></div><div class="">The bottom line is, a language can't force you to write clear and correct code, but it has many tools it can use to encourage it. Swift chooses not to use the "heavily redundant names" tool because its cost to good code is too high. We instead rely more on other tools, like strong typing, value types, and definite initialization, to encourage high-quality code at a lower cost.</div><div class=""><br class=""></div><div class="">
<span class="Apple-style-span" style="border-collapse: separate; font-variant-ligatures: normal; font-variant-east-asian: normal; font-variant-position: normal; line-height: normal; border-spacing: 0px;"><div class=""><div style="font-size: 12px; " class="">-- </div><div style="font-size: 12px; " class="">Brent Royal-Gordon</div><div style="font-size: 12px; " class="">Architechies</div></div></span>
</div>
<br class=""></body></html>