tDiary

どうもみずほです。

At Sat, 24 Aug 2002 01:06:03 +0900,
NISHIMURA Takashi wrote:
> 
> NT です。
> 
> 自分の日記にも書いた、非表示のツッコミがあると変換がおかしい点ですが、
> 状況を変えながら試してみたところ、下のようになりました。
> 
> 1. 非表示のツッコミなし
> 	問題ありません。
> 
> 2. その日の一件目のツッコミを非表示
> 	その日の最後のツッコミが tdc ファイルにない。
> 	変換できたツッコミの表示設定は正しい。
> 
> 3. その日の一件目、二件目のツッコミを非表示
> 	一件目のツッコミしか tdc ファイルにない。
> 	変換できたツッコミの表示設定は正しい。
> 
> ということで、何か問題があることだけは確かなようです。
自分で試していませんが、

tdiary/defaultio.rbの CommentIO#store_comment

  diaries.each do |date,diary|
    diary.each_comment( diary.count_comments(true) ) do |com|  # << この行のtrueの部分を追加
      fhc.puts( "Date: #{date}" )
      fhc.puts( "Name: #{com.name}" )
      fhc.puts( "Mail: #{com.mail}" )

で直るかもしれません。


CommentManager#count_comments は引数なしで実行すると、見えないコメントが
カウントされないので、実際に存在するコメントより少ない値が返ります。
そのため、コメント書き込み時に存在するコメントより少ない数のコメントしか
書き込まれません。これが原因でご指摘の 2 のようなことが起きます。

次に、3 ですが、これは私には分かりません。
修正前に3の条件でconvert2.rbを使えば、最後と最後から二つ目の
コメントがtdcファイルに存在しないようになると思うのですが…。

多少怪しいですが、もし良かったらバックアップを取った上で試してみてください。




ところで、ただただしさん、CommentManager#each_visible_comment は作らないのですか?

***********************
NISHIO Mizuho
e-mail : gha@...

NT です。

On Sat, 24 Aug 2002 15:25:51 +0900
NISHIO Mizuho <gha@...> wrote:

> > 自分の日記にも書いた、非表示のツッコミがあると変換がおかしい点ですが、
> > 状況を変えながら試してみたところ、下のようになりました。
> > 
> > 1. 非表示のツッコミなし
> > 	問題ありません。
> > 
> > 2. その日の一件目のツッコミを非表示
> > 	その日の最後のツッコミが tdc ファイルにない。
> > 	変換できたツッコミの表示設定は正しい。
> > 
> > 3. その日の一件目、二件目のツッコミを非表示
> > 	一件目のツッコミしか tdc ファイルにない。
> > 	変換できたツッコミの表示設定は正しい。
> > 
> > ということで、何か問題があることだけは確かなようです。
>
> 自分で試していませんが、
> 
> tdiary/defaultio.rbの CommentIO#store_comment
> 
>   diaries.each do |date,diary|
>     diary.each_comment( diary.count_comments(true) ) do |com|  # << この行のtrueの部分を追加
>       fhc.puts( "Date: #{date}" )
>       fhc.puts( "Name: #{com.name}" )
>       fhc.puts( "Mail: #{com.mail}" )
> 
> で直るかもしれません。
> 
> CommentManager#count_comments は引数なしで実行すると、見えないコメントが
> カウントされないので、実際に存在するコメントより少ない値が返ります。
> そのため、コメント書き込み時に存在するコメントより少ない数のコメントしか
> 書き込まれません。これが原因でご指摘の 2 のようなことが起きます。

なるほど。上の修正をした上でテストしたら予想通りに動きました。

> 次に、3 ですが、これは私には分かりません。
> 修正前に3の条件でconvert2.rbを使えば、最後と最後から二つ目の
> コメントがtdcファイルに存在しないようになると思うのですが…。

ボクの書き方が悪かったです。ツッコミが三件入っている状態で
上の実験は行いました。だから、
「一件目のツッコミしか tdc ファイルにない」
　=「最後と最後から二つ目のコメントがtdcファイルに存在しない」
です。

ということで、みずほさんの修正で解決ということで良さそうです。
-- 
NISHIMURA Takashi <ni46rtk4@...>
  GnuPG: 0F7B 4211 F62A E46C 0F6C  7CBD 2861 2D77 6BDF DFDA
  Website: http://246ra.ath.cx/~takasi/

ただです。

On Sat, 24 Aug 2002 15:25:51 +0900 NISHIO Mizuho <gha@...> wrote:
>tdiary/defaultio.rbの CommentIO#store_comment
>
>  diaries.each do |date,diary|
>    diary.each_comment( diary.count_comments(true) ) do |com|  # << この行のtrueの部分を追加
>      fhc.puts( "Date: #{date}" )
>      fhc.puts( "Name: #{com.name}" )
>      fhc.puts( "Mail: #{com.mail}" )
>
>で直るかもしれません。

　その通りです。いやはや、隠したら最後、永久に消えてしまう作りになっ
ていたとは(笑)←笑いごとではない　うちの日記もいくつか消えちゃったか
なぁ。まぁいいか。

　commitしておきました。

>ところで、ただただしさん、CommentManager#each_visible_comment は作らないのですか?

　うーん、そうですねぇ。あってもいいけど、なくてもそれほど困らない
(たかだかifがひとつ減るくらい?)という、微妙な立場のメソッドですね。

━━━━━━━━━━━━━━━━━━━
ただただし <http://www.spc.gr.jp/sho/>
    ♪ツッコミは、短く鋭く、愛をこめて
━━━━━━━━━━━━━━━━━━━

どうもみずほです。

TADA Tadashi wrote:
> ただです。
> 
>>ところで、ただただしさん、CommentManager#each_visible_comment は作らないのですか?
> > 
> 　うーん、そうですねぇ。あってもいいけど、なくてもそれほど困らない
> (たかだかifがひとつ減るくらい?)という、微妙な立場のメソッドですね。
本体側はどうでも良いですが、
プラグイン側はこちらを使うように勧めるべきだと思います。

********
NISHIO Mizuho
gha@...

ただです。

NISHIO Mizuho <gha@...> wrote:
>>>ところで、ただただしさん、CommentManager#each_visible_comment は作らないのですか?
>> > 
>> 　うーん、そうですねぇ。あってもいいけど、なくてもそれほど困らない
>> (たかだかifがひとつ減るくらい?)という、微妙な立場のメソッドですね。
>本体側はどうでも良いですが、
>プラグイン側はこちらを使うように勧めるべきだと思います。

　そう言われてみればそうかも(←気弱)。じゃあ、追加してみました。
each_comment_tailもそうだけど、ブロックパラメタの2番目idxは、1 originな
ので注意されたし。

━━━━━━━━━━━━━━━━━━━
ただただし <http://www.spc.gr.jp/sho/>
    ♪ツッコミは、短く鋭く愛を込めて。
━━━━━━━━━━━━━━━━━━━