Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

同じ時刻がふたつ出力されることがある #4

Closed
pankona opened this issue Aug 14, 2015 · 6 comments
Closed

同じ時刻がふたつ出力されることがある #4

pankona opened this issue Aug 14, 2015 · 6 comments

Comments

@pankona
Copy link
Contributor

pankona commented Aug 14, 2015

2014.08.14 Fri 20:24 に実行したところ、

$ KeiseiBusTime
20:26
20:26
20:29

という出力結果を得ました。
おそらく期待結果ではないと思われますので、一応、ご報告致します。

@sassy
Copy link
Owner

sassy commented Aug 14, 2015

@pankona
これって実は時刻表では期待通りなんですよ。
上記は20:26分に2台バスが来ると言う意味で、時刻表にも26が二つ表記されています。
利用者側としては一つにまとめた方がいいですかね。。。

@pankona
Copy link
Contributor Author

pankona commented Aug 14, 2015

@sassy
その可能性は頭をよぎりましたが、、、まさか同着があるとは知らなんだです。
同着は〜、、、どうでしょうね。二つ一緒にくる、つまり後ろのに乗れば空いてる、とかそういう判断もありそうですので、まったく無駄な情報ではないと思います。。。

指摘しておいてあれですが、これは据え置きがいいかと思いますが、どうでしょうか?

@sassy
Copy link
Owner

sassy commented Aug 14, 2015

@pankona
例えば
20:26 (2)
みたいに2台と表示するとかですかねぇ。。。めんどくさいかな?

@pankona
Copy link
Contributor Author

pankona commented Aug 14, 2015

@sassy
それも良さそうですね!めんどくさくは、、、ない、、、と思います。

ただ3台以上はおそらくないと思われるので、
20:26* とか、20:26! みたいに、ちょっと記号をつけて「1台ではない」の意にするのもありかな、と。見た目スマートになる。

他のアイデアとしてはオプション利用で、

  • -u で同着を表示しない
  • 逆に、同着はデフォルトで表示するようにしていて、--verbose で同着も表示されるようになる

といったオプション利用も考えましたが、一生使わないオプションになる気がするので微妙かと思いましたです。

ちなみにオプション関連では、この件とあんまり関係ないですが、

  • -n [NUM] で NUMの数だけ表示する、とか便利かも。

@sassy
Copy link
Owner

sassy commented Aug 14, 2015

@pankona
わかりました。記号使うとかで対応します(優先度低で)
Pull Request大歓迎です。

-n オプションは考えてました。
よかったら別Issue立てていただけると。

pankona added a commit to pankona/KeiseiBusTime that referenced this issue Aug 15, 2015
pankona added a commit to pankona/KeiseiBusTime that referenced this issue Aug 16, 2015
pankona added a commit to pankona/KeiseiBusTime that referenced this issue Aug 16, 2015
sassy added a commit that referenced this issue Aug 16, 2015
[#4] mark "*" if there're two buses that arrives same time.
@sassy
Copy link
Owner

sassy commented Aug 16, 2015

resolve for #7
thx @pankona

@sassy sassy closed this as completed Aug 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants