Menu
Search

Forums

Forum Index

Thread overview
A better way than foreach with this?
Aug 24, 2015
Joel
Aug 24, 2015
Jacob Carlborg
Aug 24, 2015
Joel
Aug 24, 2015
ted
Aug 24, 2015
Joel
August 24, 2015
Gravatar of JoelPosted by JoelReply
Joel
Gravatar of Joel


Reply
	auto names =
			"Alef Bet Gimel Dalet He Vav Zayen Het Tet Yod Final_Kaf "
			"Kaf Lamed Final_Mem Mem Final_Nun Nun Samekh Ayin Final_Pe "
			"Pe Final_Tsadi Tsadi Qof Resh Shin Tav".split;

	foreach (ref name; names)
		name = replace(name, "_", " ");
August 24, 2015
Gravatar of Jacob CarlborgPosted by Jacob Carlborg
in reply to Joel		Reply
Jacob Carlborg
Gravatar of Jacob Carlborg
Posted in reply to Joel


Reply
On 2015-08-24 07:58, Joel wrote:
>      auto names =
>              "Alef Bet Gimel Dalet He Vav Zayen Het Tet Yod Final_Kaf "
>              "Kaf Lamed Final_Mem Mem Final_Nun Nun Samekh Ayin Final_Pe "
>              "Pe Final_Tsadi Tsadi Qof Resh Shin Tav".split;
>
>      foreach (ref name; names)
>          name = replace(name, "_", " ");
>

Why not call the "replace" function directly on "names"?

-- 
/Jacob Carlborg
August 24, 2015
Gravatar of tedPosted by ted
in reply to Joel		Reply
ted
Gravatar of ted
Posted in reply to Joel


Reply
try:
  auto names1 = names.map!( a => replace(a, "_"," "));

...not sure how to do it in-place though.

Joel wrote:

> auto names =
> "Alef Bet Gimel Dalet He Vav Zayen Het Tet Yod Final_Kaf "
> "Kaf Lamed Final_Mem Mem Final_Nun Nun Samekh Ayin Final_Pe "
> "Pe Final_Tsadi Tsadi Qof Resh Shin Tav".split;
> 
> foreach (ref name; names)
> name = replace(name, "_", " ");
>
August 24, 2015
Gravatar of JoelPosted by Joel
in reply to Jacob Carlborg		Reply
Joel
Gravatar of Joel
Posted in reply to Jacob Carlborg


Reply
On Monday, 24 August 2015 at 06:13:50 UTC, Jacob Carlborg wrote:
> On 2015-08-24 07:58, Joel wrote:
>>      auto names =
>>              "Alef Bet Gimel Dalet He Vav Zayen Het Tet Yod Final_Kaf "
>>              "Kaf Lamed Final_Mem Mem Final_Nun Nun Samekh Ayin Final_Pe "
>>              "Pe Final_Tsadi Tsadi Qof Resh Shin Tav".split;
>>
>>      foreach (ref name; names)
>>          name = replace(name, "_", " ");
>>
>
> Why not call the "replace" function directly on "names"?

They would split wrong.
August 24, 2015
Gravatar of JoelPosted by Joel
in reply to ted		Reply
Joel
Gravatar of Joel
Posted in reply to ted


Reply
On Monday, 24 August 2015 at 06:17:02 UTC, ted wrote:
> try:
>   auto names1 = names.map!( a => replace(a, "_"," "));
>
> ...not sure how to do it in-place though.
>
> Joel wrote:
>
>> auto names =
>> "Alef Bet Gimel Dalet He Vav Zayen Het Tet Yod Final_Kaf "
>> "Kaf Lamed Final_Mem Mem Final_Nun Nun Samekh Ayin Final_Pe "
>> "Pe Final_Tsadi Tsadi Qof Resh Shin Tav".split;
>> 
>> foreach (ref name; names)
>> name = replace(name, "_", " ");

Thanks, worked it out, (though has 'array'):
	names = names.map!( a => replace(a, "_"," ")).array;