Dynamic / resizable array type, and a crash problem

[anonymous via Digitalmars-d-learn]

On Thursday, 14 May 2015 at 20:50:05 UTC, ivoras wrote:
> I'm experimenting to get a feel for the language. Do you have a 
> suggestion about this example code: https://goo.gl/F7LCAg to 
> make it more "D-like", idiomatic?

Quoting from the code:

>  for (int i = 0; i < count; i++) {

foreach(i; 0 .. count)

>    try {
>      auto choices = markov[current];
[...]
>    } catch (RangeError e) {
>      break;
>    }

Don't catch Errors. Use the `in` operator to check if `current` 
is in `markov`:

if(current !in markov) break;

Or avoiding double lookup:

string[]* choicesp = current in markov;
if(choicesp is null) break;
auto choices = *choicesp;

> int main(string[] args)

You cam omit the return type and `args` if you're not going to 
use them:

void main()

>  foreach (c_line; stdin.byLine()) {
>    auto line = to!string(c_line);

Could use byLineCopy:

foreach(line; stdin.byLineCopy)

>      if (prev_token in markov) {
>        markov[prev_token] ~= token;
>      } else {
>        markov[prev_token] = [token];
>      }

I think you can just go with appending here:

markov[prev_token] ~= token;

But I'm not 100% sure about the implications.