<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="">Hi,&nbsp;<div class=""><br class=""></div><div class="">As I understand it, the argument for remove forEach is that it’s easy to make confusing code that doesn’t do what it looks like it should do.</div><div class=""><br class=""></div><div class="">In this particular example:</div><div class=""><br class=""></div><div class=""><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp;&nbsp;<span class="" style="color: rgb(4, 51, 255);">func</span>&nbsp;indexOf_foreach(element:&nbsp;<span class="" style="color: rgb(52, 149, 175);">Element</span>) -&gt;&nbsp;<span class="" style="color: rgb(52, 149, 175);">Int</span>? {</div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; &nbsp; &nbsp;&nbsp;<span class="" style="color: rgb(4, 51, 255);">self</span>.<span class="" style="color: rgb(52, 149, 175);">indices</span>.<span class="" style="color: rgb(52, 149, 175);">filter</span>&nbsp;{ idx&nbsp;<span class="" style="color: rgb(4, 51, 255);">in</span>&nbsp;<span class="" style="color: rgb(4, 51, 255);">self</span>[idx]&nbsp;<span class="" style="color: rgb(52, 149, 175);">==</span>&nbsp;element }.<span class="" style="color: rgb(52, 149, 175);">forEach</span>&nbsp;{ idx&nbsp;<span class="" style="color: rgb(4, 51, 255);">in</span></div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;&nbsp;<span class="" style="color: rgb(4, 51, 255);">return</span>&nbsp;idx</div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; &nbsp; &nbsp; }</div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; &nbsp; &nbsp;&nbsp;<span class="" style="color: rgb(4, 51, 255);">return</span>&nbsp;<span class="" style="color: rgb(4, 51, 255);">nil</span></div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; }</div></div><div class=""><br class=""></div><div class="">I think the compiler should give us two warnings:</div><div class="">- closure returns type Int when it should return type ()</div><div class="">- function always return nil (which is a unit type just like () ) when it expects Option&lt;Int&gt;</div><div class=""><br class=""></div><div class="">In my opinion those warnings should be enough to make the programmer aware that this code doesn’t do what it looks like it does. Typically, returning a type when () is expected shouldn’t go silently.</div><div class=""><br class=""></div><div class="">As such I think keeping forEach is important as it allows for every elegant expressions and convey already the already established convention when you use forEach that “I”m only interested in side-effects with elements in this collection”</div><div class=""><br class=""></div><div class="">My proposition is then that instead of removing forEach, we should have better type check and warn when the cardinality of the expected type and the returned type don’t match.</div><div class=""><br class=""></div><div class="">-André</div><div class=""><br class=""></div><div><blockquote type="cite" class=""><div class="">On 08 Dec 2015, at 20:07, Chris Eidhof via swift-evolution &lt;<a href="mailto:swift-evolution@swift.org" class="">swift-evolution@swift.org</a>&gt; wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html charset=utf-8" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi all,<div class=""><br class=""></div><div class="">As long as the `for` loop is in the language, I don’t really see the use of `forEach`. I understand that it can read much nicer to write something like `a.map { something }.filter { somethingElse }.forEach {` rather than a `for` loop. However, I think the costs don’t outweigh the benefits. It might look like a for loop can just be replaced by a `forEach`, however, this is not true in the general case. For example, consider the following example:</div><div class=""><br class=""></div><div class=""><div class=""><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp;&nbsp;<span class="" style="color: rgb(4, 51, 255);">func</span>&nbsp;indexOf_foreach(element:&nbsp;<span class="" style="color: rgb(52, 149, 175);">Element</span>) -&gt;&nbsp;<span class="" style="color: rgb(52, 149, 175);">Int</span>? {</div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; &nbsp; &nbsp;&nbsp;<span class="" style="color: rgb(4, 51, 255);">self</span>.<span class="" style="color: rgb(52, 149, 175);">indices</span>.<span class="" style="color: rgb(52, 149, 175);">filter</span>&nbsp;{ idx&nbsp;<span class="" style="color: rgb(4, 51, 255);">in</span>&nbsp;<span class="" style="color: rgb(4, 51, 255);">self</span>[idx]&nbsp;<span class="" style="color: rgb(52, 149, 175);">==</span>&nbsp;element }.<span class="" style="color: rgb(52, 149, 175);">forEach</span>&nbsp;{ idx&nbsp;<span class="" style="color: rgb(4, 51, 255);">in</span></div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;&nbsp;<span class="" style="color: rgb(4, 51, 255);">return</span>&nbsp;idx</div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; &nbsp; &nbsp; }</div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; &nbsp; &nbsp;&nbsp;<span class="" style="color: rgb(4, 51, 255);">return</span>&nbsp;<span class="" style="color: rgb(4, 51, 255);">nil</span></div><div class="" style="margin: 0px; font-size: 13px; line-height: normal; font-family: 'Akkurat TT';">&nbsp; &nbsp; }</div></div></div><div class=""><br class=""></div><div class="">The code above (I realise it’s quite inefficient) might look like it’s returning the first index for which the filter’s condition returned true. However, the first occurrence of return is actually returning inside the closure, not the outer function. So the result of this function is always nil.</div><div class=""><br class=""></div><div class="">Another solution would be to give a good warning/error message here (you’re trying to return an Optional value where we expect a ()). However, this is also problematic when dealing with side-effects. For example:</div><div class=""><br class=""></div><div class="">[1,2,3,4,5].forEach { num in</div><div class="">&nbsp; &nbsp;print(num)</div><div class="">&nbsp; &nbsp;if num &gt; 3 { return }</div><div class="">}</div><div class=""><br class=""></div><div class="">I think it’s quite easy to get things wrong with forEach, so I’d propose removing it and rather having a regular for loop. (Erica-style).</div><div class=""><br class=""></div><div class="">Chris</div>
<img src="https://u2002410.ct.sendgrid.net/wf/open?upn=41N46VivWCZSszJfIBo5Gq4IkbqtRm7tCLAjdnNzXvxi8ViafKS4bfI3ujyuSG3GJ3ldq2EtbzEDjbnI5euvHOHaBWE8kdbzXSNr1rapHVxx9VoRKNZ5mrK9HomYJF60CifKeEN6X7COKdy3jwIYSdcxnZomrV1NNDp-2Bj8O3cn-2BfjhTnb9PlqLhSyuwKNKeCfHvIedQ4K5XhXkRgcU2Hqw-3D-3D" alt="" width="1" height="1" border="0" style="height:1px !important;width:1px !important;border-width:0 !important;margin-top:0 !important;margin-bottom:0 !important;margin-right:0 !important;margin-left:0 !important;padding-top:0 !important;padding-bottom:0 !important;padding-right:0 !important;padding-left:0 !important;" class="">
</div>
_______________________________________________<br class="">swift-evolution mailing list<br class=""><a href="mailto:swift-evolution@swift.org" class="">swift-evolution@swift.org</a><br class="">https://lists.swift.org/mailman/listinfo/swift-evolution<br class=""></div></blockquote></div><br class=""></body></html>