文字列の奇数番目の文字を出力

Question

###与えられた文字列の、奇数番目の文字だけを出力

某プログラミング学習サイトの問題を解いていて、不正解の原因がわからないのでこちらで質問させていただきます。（公開されている解答はpythonのみ）

問題：入力される文字列の奇数番目の文字だけを出力する
例：入力 → aXaXaXaXaX　　　出力 → aaaaa
※最後に改行の出力指定は無し

発生している問題・エラーメッセージ

// エディタでテストしてもエラーにならず、正しく出力されているように見えます。

該当のソースコード

process.stdin.resume();
process.stdin.setEncoding('utf8');

const reader = require('readline').createInterface( {
  input: process.stdin,
  output: process.stdout
});

var lines;
var result;

reader.on('line', (line) => {
  lines = line.split('');
});
reader.on('close', () => {
  result = lines.filter(item => (lines.indexOf(item) + 1) % 2 === 1);
　console.log(result.join(''));
});

試したこと

closeイベントの中を以下のようにfor ifで書いたら正解と判定されました

reader.on('close', () => {
  for (var i = 0; i < lines.length; i++) {
    if ((i + 1) % 2 === 1) {
      process.stdout.write(lines[i]);
    }
  }
});

補足

初めて質問させて頂きます。
filterとforで出力される内容は同じように見えますが、違いがあれば教えて頂きたいです(改行有無以外で)｡
質問の仕方についても不備があればご指摘ください。
よろしくお願い致します。

Comments

[y_waiwai]

その提示のコードではどういう動作になるんでしょうか

Answer 1

テストケース次第ですが、例えばテストケースが"abbcd"のとき、正解は"abd"のはずですがindexOfで位置を求めているために"ad"と出力されるのではないでしょうか。また"aaa"だと"aa"が正解のところおそらく"aaa"となると思います。

filter((_,index) => (index+1) % 2 === 1 )のように、配列のインデクスで確認したほうがよいように思います。

Comments

[Dokumoot]

大変わかりやすい回答をありがとうございます。仰る通りでした。
indexOfの仕様をしっかり理解していませんでした。勉強になりました。

Answer 2

filterとforで出力される内容は同じように見えますが、違いがあれば教えて頂きたいです

ざっくりでよければ。
昔の javascript には Array の各要素を走査する関数が無かったんですよ。
それまでは for 文で様々な走査用のコードが書かれていました。

現在の forEache(), map(), filter(), ... etc は
「こういうのあると、可読性の高いコードで走査できるよね」ってことで、追加されました。

同じように見えますが走査の目的が違うため、これらのメソッド（関数）を使うと
どのような結果になるのか？仕様を理解して使う必要があります。

---

因みに、papinian us さんの発展：
filter((_,index)=> ~index&1) のような（ビット演算）も使えます。

正解と判定された理由は stdout への出力というumejiさんの採点基準の考察も十分に頷けます。
ですので、次のように書いても正解だったかもしれませんね。

// 1文字ずつ出力ストリーム(stdout)に書き出す。
reader.on('close', () => {
  lines.filter((_,index)=> ~index&1)
    .forEach( char => process.stdout.write(char) );
});

Comments

[Dokumoot]

詳しくご説明頂きありがとうございます。
ビット演算についても勉強したいと思います。

Answer 3

lines.filter(item => (lines.indexOf(item) + 1) % 2 === 1)

だと、abcabcがうまくいかなくない？4つ目のaも検知してしまうやん？

lines.filter((_, i) => i % 2 === 0)

だな！答え言ってしまうと

Comments

[退会済みユーザー]

すんません、オレンジの人と回答どっかぶりでした。。。

[Dokumoot]

回答頂きありがとうございます。
同じ内容ではありますが、｢4つ目のaも検知してまう｣とのご指摘を頂けたおかげで理解が早まりました。

Answer 4

不正解となったものはconsole.log()で、正解となったものはprocess.stdout.write()で出力されてますね。そのプログラミング学習サイトではprocess.stdout.write()で出力したものしか採点しない仕様になっているのではないでしょうか。

Comments

[Dokumoot]

回答ありがとうございます。
今回は出力方法による採点判断はありませんでした。