■スレッドリストへ戻る■ 全部 1- 101- 201- 301- 401- 501- 601- 701- 801- 901- 最新50
プログラマが2chを救う！？
- 18 ：これは？ ：2001/08/25(土) 20:52
- fprintf(fp,"(%15s)",zz_remote_addr);
- 19 ：デフォルトの名無しさん ：2001/08/25(土) 20:53
- >>18
>>13でがいしゅつ
- 20 ：デフォルトの名無しさん ：2001/08/25(土) 20:53
- >>18
でもさ、書き込みログじゃなくて、読み込みログだから
いいんじゃないの？
でも、なんか、書き込み時もログとってそう、、、
- 21 ：仕様書無し ：2001/08/25(土) 20:53
- >>18
アラ探しより先にする事があるだろう
- 22 ：どうでも良い事 ：2001/08/25(土) 20:55
- セミコロンの前にtab入れてるんだけどなんか意味あるの？
http://piza.2ch.net/read4.txt read.cgiのソース
- 23 ：デフォルトの名無しさん ：2001/08/25(土) 20:59
- みんながバラバラに最適化していくより、
関数毎にやっていったほうが効率よくない？
- 24 ：デフォルトの名無しさん ：2001/08/25(土) 20:59
- とりあえず関数ごとに分けて機能表つくろうよ。
---
intBadAccess()
機能：ブラウザが正当かをチェックする
引き値：なし
返戻値：USER_AGENT が正当であれば１、不正であれば０
---
こんな感じでいいかなぁ？
- 25 ：デフォルトの名無しさん ：2001/08/25(土) 21:00
- じゃ、まずdat_readからか？
- 26 ：どうでも良い事 ：2001/08/25(土) 21:00
- zz_http_refererに関する行を全削除する。
- 27 ：デフォルトの名無しさん ：2001/08/25(土) 21:01
- 改良版hrefStop
const char s_pHrefTable[] = {
1,0,1,0,0,0,0,0,1,1,0,0,0,0,0,0,
0,0,0,0,0,0,0,0,0,0,0,0,1,0,1,0,
0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,
0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,
1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,
0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1};
#define hrefStop(x) (isgraph(x) ? 1 : s_pHrefTable[(x)-0x20])
- 28 ：デフォルトの名無しさん ：2001/08/25(土) 21:02
- みなさん、応援しています。がんばって下さい。
- 29 ：どうでも良い事 ：2001/08/25(土) 21:02
- 最適化する前にラフな仕様書を作るか、きちんとコメント書くかした方が良いんでない？
- 30 ：27 ：2001/08/25(土) 21:02
- #define hrefStop(x) (isgraph(x) ? s_pHrefTable[(x)-0x20] : 0)
間違えた。
- 31 ：デフォルトの名無しさん ：2001/08/25(土) 21:02
- 問題は帯域幅だから、read.cgiが速くなっても別に意味ないんだよねえ..
まあ、いろいろと突っ込みたくなる codeだけどさ。
- 32 ：デフォルトの名無しさん ：2001/08/25(土) 21:03
- データ転送量が問題なんじゃなかった？
それだと高速化しても無意味じゃ…
- 33 ：デフォルトの名無しさん ：2001/08/25(土) 21:03
- intlogOut(char *txt)
機能：読み込みログの記録
引き値：用途不明
返戻値：ファイルエラーで０ 、それ以外１
呼び出す関数：BadAccess()
- 34 ：sage@sage.com ：2001/08/25(土) 21:03
- 確かに今のままじゃ流れがちょっと見えにくいし、
簡単なフローとか作ったほうがいいのかも。
- 35 ：27 ：2001/08/25(土) 21:04
- #define hrefStop(x) (isgraph(x) ? s_pHrefTable[(x)-0x20] : 1)
鬱死
- 36 ：デフォルトの名無しさん ：2001/08/25(土) 21:04
- >>31
そっちは
http://cocoa.2ch.net/test/read.cgi?bbs=unix&key=998695422
にお任せということで...
>>7 >>8 参照
- 37 ：デフォルトの名無しさん ：2001/08/25(土) 21:04
- >>31-32
既出です( >>7-9 )。
- 38 ：デフォルトの名無しさん ：2001/08/25(土) 21:05
- >29
んなもんコード見りゃわかる。
わからなきゃ調べて関数仕様ａｇｅたら。
最適化前->後の変化は初学者には結構勉強になるかもね。
- 39 ：デフォルトの名無しさん ：2001/08/25(土) 21:08
- linuxのstrstrって速いんか？
- 40 ：デフォルトの名無しさん ：2001/08/25(土) 21:08
- 名スレの予感
- 41 ：デフォルトの名無しさん ：2001/08/25(土) 21:08
- inthtml_bannerNew()
inthtml_banner()
機能：バナーを標準出力に
返戻値：なし （っていうかreturnしてないし)
---------
ほんと役に立たなくてすまないが残りは頼んだ。ちょっと落ちる。
- 42 ：デフォルトの名無しさん ：2001/08/25(土) 21:09
- >>39
つーか、使わないほうがいいやね。それ。
- 43 ：デフォルトの名無しさん ：2001/08/25(土) 21:09
- doReplace()も激遅だな
- 44 ：デフォルトの名無しさん ：2001/08/25(土) 21:11
- 81 名前：夜勤 ★ 投稿日：2001/08/25(土) 20:10
あっ へっだ これです、すんません。
----
/**/
/**/
/*
#define CM_BBSPINK
#define LOGLOGOUT 1
*/
#define TYPE_TERI
#define Katjusha_Beta_kisei
----
read.2ch.h のヘッダの中身です。
- 45 ：デフォルトの名無しさん ：2001/08/25(土) 21:11
- 改良版html_bannerNew
inthtml_bannerNew()
{
cosnt char *banner =
"<TABLE CELLSPACING=0 CELLPADDING=0 BORDER=1 WIDTH=100%%><TR><TD>\n"
"<TABLE CELLSPACING=0 CELLPADDING=0 BORDER=0 WIDTH=100%%>\n"
中略
"</TD></TR></TABLE><P>\n";
puts(banenr);
}
つーか、putsひとつだけだから関数にするまでもないな。
- 46 ：デフォルトの名無しさん ：2001/08/25(土) 21:13
- できるだけ最適化して高速化しておけば、
gzip の圧縮レベルを上げることができる。
http://cocoa.2ch.net/test/read.cgi?bbs=unix&key=998695422
では、圧縮レベルをどうするか迷ってたみたいだからね。
ところで、gzip 採用でボトルネックになるのは
cgi なのかな。それとも、gzip エンジン?
- 47 ：うーん ：2001/08/25(土) 21:13
- ストリームへの書き出し単位が小さいとパケットが分断化しやすいからね。
もっとも、printfストリーム関数使ってるからある程度まとまってるとは思うけど、ライブラリ依存だなぁ。
CGI内部でバッファ制御して吐き出せば多少変わるかもな。
ほとんど気休めな程度な気がするが・・・
- 48 ：デフォルトの名無しさん ：2001/08/25(土) 21:14
- >>45
return 忘れてるYO!!
- 49 ：デフォルトの名無しさん ：2001/08/25(土) 21:16
- >>27
isgraph()もテーブルに押し込めろ
- 50 ：地道にやろう ：2001/08/25(土) 21:17
- とりあえず printfの中の \n を全て取っ払う。
- 51 ：デフォルトの名無しさん ：2001/08/25(土) 21:22
- UNIX板の作業所に荒しが入った模様。
IRCへ移動するプランが提案されています。
- 52 ：名無 ：2001/08/25(土) 21:25
- プログラマーはＵＮＩＸ板へ集合して下さい。
対策を練ります。
- 53 ：デフォルトの名無しさん ：2001/08/25(土) 21:26
- 削除人が待機してるようだから大丈夫。
荒らしがしつこければ制限かけるだろう
- 54 ：デフォルトの名無しさん ：2001/08/25(土) 21:28
- つーか、いっそのことサーブレットで書き直すと、かなり
よいんじゃない？CGIじゃ限界あるでしょ。
- 55 ：デフォルトの名無しさん ：2001/08/25(土) 21:31
- 共通仕様の鯖しか使えないからダメ。つーか転送量に関係無いし
- 56 ：デフォルトの名無しさん ：2001/08/25(土) 21:32
- うーん、replace関係がやばそうだな。
doReplace って
*des のうち *str0がみつかったら消す。で、消したところの
始めのポインタを返す
でいいの？
*des ="abcdefg" *str0="cde"
なら
*des = "abfg"
戻り値： des + 2
ってこと？
でもstr1の意味って何？
- 57 ：デフォルトの名無しさん ：2001/08/25(土) 21:35
- あ、すまん >>56 >>17
かわりににstr1が追加されるのね。
*des ="abcdefg" *str0="cde" *str1="kkkk"
なら
*des = "abkkkkfg"
戻り値： des + 2
なのかな？
- 58 ：デフォルトの名無しさん ：2001/08/25(土) 21:37
- BadAccess()のuser agent検出ってstrstrの代わりにstrcmpの完全一致じゃ駄目なの？
- 59 ：デフォルトの名無しさん ：2001/08/25(土) 21:38
- なんせHTTPという超曖昧プロトコルだからねえ
- 60 ：デフォルトの名無しさん ：2001/08/25(土) 21:38
- 1000超えたら表示がおかしくなる、っていうバグもこの
中にあるのかNA...
- 61 ：デフォルトの名無しさん ：2001/08/25(土) 21:39
- >>58
strcmpのほうが速いの？
- 62 ：デフォルトの名無しさん ：2001/08/25(土) 21:39
- マジで書きなおしたくなってくる
- 63 ：58>61 ：2001/08/25(土) 21:42
- strcmpというか完全一致でいいなら
if(strstr(zz_http_user_agent,"DreamPassport"))return 0;
if(strstr(zz_http_user_agent,"DoCoMo"))return 0;
if(strstr(zz_http_user_agent,"J-PHONE"))return 0;
↓
strstr(zz_http_user_agent, "Dreamcast\nDoComo\nJ-PHONE...") return 0
みたいに書き換えられる。かも
- 64 ：デフォルトの名無しさん ：2001/08/25(土) 21:43
- >63
UAにはたいていバージョン番号が含まれてるし難しいと思われ
とりあえず仕様を変えないで最適化できる範囲で
- 65 ：デフォルトの名無しさん ：2001/08/25(土) 21:44
- >63じゃなくて>58
- 66 ：デフォルトの名無しさん ：2001/08/25(土) 21:44
- hlinkReplace、ExistHlink、ExistHlinkX、cutWordOff、someReplace、doReplaceが
協力してurlにリンクを張っているらしいね。
何度もバッファに書き出していて効率悪い。
全部総とっかえしたほうがよさそう。
- 67 ：デフォルトの名無しさん ：2001/08/25(土) 21:45
- >>66
その関数の依存関係を簡単でいいからおしえてくれるとたすかる。
- 68 ：デフォルトの名無しさん ：2001/08/25(土) 21:45
- つーかurlにリンク張る処理は全廃でいいんじゃない？
クライアントに任せろ。
- 69 ：68 ：2001/08/25(土) 21:47
- もちろん仕様は変わるが、IOは確実に減る。
- 70 ：デフォルトの名無しさん ：2001/08/25(土) 21:47
- かちゅとかだと勝手にリンク入れてくれるしね。
- 71 ：デフォルトの名無しさん ：2001/08/25(土) 21:47
- 最新レスの数を減らせば……(^^;
- 72 ：デフォルトの名無しさん ：2001/08/25(土) 21:48
- >69
I/O減らすだけが目的ならこのスレで話し合ってる最適化自体
ほとんど無意味です
- 73 ：69>72 ：2001/08/25(土) 21:50
- 処理だってかな〜り高速化されるよ。
２ｃｈブラウザとの連携を視野に入れつつ最適化。
- 74 ：デフォルトの名無しさん ：2001/08/25(土) 21:50
- リンク張りはhttp://探して見つかったら、その手前まで出力してタグを開く
URLの終端まで一文字ごとに出力、URLの終端に来たらタグを閉じて
次のURLを探す。
ってやった方が良いんでない？
- 75 ：デフォルトの名無しさん ：2001/08/25(土) 21:51
- >73
それならread.cgiは全面的に止めたほうがいいかと
- 76 ：66 ：2001/08/25(土) 21:52
- hlinkReplaceはリンクリプレースのメイン。
ExistHlinkを呼び出して、対象となるソースの中からurlを抜き出し、同時にリンク済みのタグつきurlを出力してもらい、
それをdoReplaceでhtmlに埋め込んでいる。
ExistHlinkは、http://とftp://とでExistHlinkXを呼び出していて、実際にurlを抜き出しているのはこちら。
ExistHlinkではhttp://があるかどうかを調べて、あるならそこからurlを抜き出し、
そこからcutWordOffで"""を省いて、タグつきurlをsprintfで出力している。
これを直すのはたいへんだな。
- 77 ：デフォルトの名無しさん ：2001/08/25(土) 21:53
- (いろんな意味でw)自主的に
ttp://〜
ってする板も多いので、いっそ廃止でも……。>http〜
- 78 ：67 ：2001/08/25(土) 21:54
- >>66 サンクス
- 79 ：デフォルトの名無しさん ：2001/08/25(土) 21:54
- もっといい方法ありそうだけどね。
int BadAccess()
{
char *agent_accept[] = {
"DreamPassport",
"DoCoMo",
"J-PHONE",
"ASTEL",
"[ja]",
"iCab",
"iBOX",
"WannaBe",
"Macintosh; I;",
"Mozilla/3.0N",
"CBBoard",
"PersonaWare",
"Sharp",
"95",
"NT 4.0",
"WinNT",
};
char *agent_kick[] = {
"WebFetch",
"origin",
"Nozilla",
"WWWD",
};
int i;
if(!*zz_http_user_agent && !*zz_http_language) {
return 1;
}
for (i=0; i<sizeof(agent_accept)/sizeof(char*); i++) {
if (strstr(zz_http_user_agent, agent_accept[i])) {
return 0;
}
}
for (i=0; i<sizeof(agent_kick)/sizeof(char*); i++) {
if (strstr(zz_http_user_agent, agent_kick[i])) {
return 1;
}
}
return 0;
}
- 80 ：デフォルトの名無しさん ：2001/08/25(土) 21:54
- >75
それだと通常のブラウザから一切見えなくなっちゃうじゃん。
- 81 ：ペイント聖矢 ：2001/08/25(土) 21:55
- ぷうぷぷぷぷぷぷぷぷプログラマー板がっつつつつつつｔ！！！！！！？？ｊｆだｊふぉ＠あえああひゃはやはやはやはあはやあはやごふっ
- 82 ：sage@sage.com ：2001/08/25(土) 21:57
- PGer板停止中？
今、全体のフローらしきものを作ってます。
ところで、dat_readはレス１つ指定でも必ずファイル全部読み出して
処理しているのかな？
- 83 ：デフォルトの名無しさん ：2001/08/25(土) 21:59
- >>79
すっきりしたね。
でもkickだけあればいいのではないかという気もする。
- 84 ：デフォルトの名無しさん ：2001/08/25(土) 22:01
- 改良版BadAccess
int BadAccess()
{
char *agent_kick[] = {
"WebFetch",
"origin",
"Nozilla",
"WWWD",
};
int i;
if(!*zz_http_user_agent && !*zz_http_language)
return 1;
for (i=0; i<sizeof(agent_kick)/sizeof(char*); i++) {
if (strstr(zz_http_user_agent, agent_kick[i]))
return 1;
return 0;
}
- 85 ：デフォルトの名無しさん ：2001/08/25(土) 22:02
- 抜本的に見直す為、クラサバから検討しするスレも要るのかも・・・？
- 86 ：79 ：2001/08/25(土) 22:04
- >>83
確かにそうでした、accept のチェックは要らないですね。
- 87 ：デフォルトの名無しさん ：2001/08/25(土) 22:04
- ここが止まった場合は↓に行くことにしとく？
http://www.shitaraba.com/bbs/programtech/index.html
どっかもっと良いとこある？
- 88 ：デフォルトの名無しさん ：2001/08/25(土) 22:05
- >>85
そういう構想だけは前からある。
「2chのような掲示板システムってP2Pで」
http://piza2.2ch.net/test/read.cgi?bbs=tech&key=990334284
- 89 ：デフォルトの名無しさん ：2001/08/25(土) 22:10
- >>86
改良版の問題は、ドリキャスがブラウザのシェアを独占したら
かえって重くなるということだな（笑）。
- 90 ：sage@sage.com ：2001/08/25(土) 22:10
- いまさらって感じはするけど流れを簡単にまとめてみました。
間違いとかあったら訂正してくださひ。
1.zz_GetEnv（HTTP各環境変数の取得）
2.logOut（アクセスログ保存）
3.dat_read（datファイルの読み込み）
ファイルサイズの取得
環境変数から読み出し位置等の解析
メモリの確保
ファイルの読み出し
最大表示可能レス数の解析？（getLineMax）
4.dat_out（HTTPD出力要求（表示））
・forループの中身
（st,ls,to等の表示ログ番号制御処理（zz_nfがよくわからない）
指定ログ番号データ表示（out_html））
最終ライン処理（out_html1）
書き込み用フォーム表示処理（html_foot）
5.メモリ開放
out_html関数辺りが結構重要かも、引き続き細かいところ解析していきます。
- 91 ：デフォルトの名無しさん ：2001/08/25(土) 22:20
- 2ch閉鎖の危機なんだと
http://cocoa.2ch.net/test/read.cgi?bbs=unix&key=998695422
の 291 も加えてください。
- 92 ：デフォルトの名無しさん ：2001/08/25(土) 22:22
- out_html と似た名前の関数があってわかりずらいですよね。
out_html1() だって、、、
コメントもコピペなのかあてにならんし
- 93 ：デフォルトの名無しさん ：2001/08/25(土) 22:34
- zz_GetStringが何をやりたい関数なのかわからん……
- 94 ：sage@sage.com ：2001/08/25(土) 22:39
- >>93
zz_GetStringはHTTPの環境変数を取得します。
ブラウザから要求をかけたときに付帯するプロトコル情報です。
あと、http://**.com/read.cgi?st=8%ls=5
のようなcgiに変数を引き渡すときに使用されます。
解析続き
各バッファの相関関係
・BigBuffer datファイルから読み出したデータ格納バッファ
・BigLine BigBufferの各レス先頭位置を示すポインタが格納されている配列
getLineMaxでBigBufferを全てスキャンして、
\nならそのポインタ位置を記憶、NULLに置き換える。
最大転送量・\ｎが見つからない場合（ファイルの最後？）で処理終了
取得したライン（ログ数）を返す
- 95 ：仕様無しさん ◆NwLv.g/w ：2001/08/25(土) 22:40
- ＞93 関数を呼んでるところを見て御覧。
- 96 ：sage@sage.com ：2001/08/25(土) 22:42
- >>94
ごめん read.cgi?st=3%ls=5
の?以下が環境変数となります。
HTTPDのAPIでこの値を取得できます。
ただ、折れの知っている情報って組み込み用HTTPDについてなので
汎用的なサーバのHTTPDだと違うかもしんない・・・。
- 97 ：93 ：2001/08/25(土) 22:45
- >>94-95
ありがとう。
- 98 ：sage@sage.com ：2001/08/25(土) 22:47
- 環境変数って表現はちょっと違ってたかもしれませんね。
どちらかっていうとget_envでとってくるのが環境変数ですね。
- 99 ：デフォルトの名無しさん ：2001/08/25(土) 23:10
- UNIX 板でも同時進行してるんだがmerge しなくていいのか？
- 100 ：デフォルトの名無しさん ：2001/08/25(土) 23:11
- http://cocoa.2ch.net/test/read.cgi?bbs=unix&key=998695422
でも対策スレが立っています。夜勤さんも来ています。
- 101 ：265 ：2001/08/25(土) 23:12
- >>7-9
ってことにしたんだけど.. あっちどうなってるんだろう..
- 102 ：デフォルトの名無しさん ：2001/08/25(土) 23:15
- これってうわさのリファクタリング??
- 103 ：デフォルトの名無しさん ：2001/08/25(土) 23:16
- >>99
当面問題なのは転送量だからUNIX板の方には
先に走ってもらって転送量削減のめどかついた時点で
圧縮により増えた負荷を下げるようにこちらの結果を
反映していけばよいのではないでしょうか
- 104 ：デフォルトの名無しさん ：2001/08/25(土) 23:16
- >>102
いいえ、Cプログラミング診断室です。
- 105 ：デフォルトの名無しさん ：2001/08/25(土) 23:18
- なんか向こうでどんどこ新しいの作っちゃってんだけど
こっちの意見も言っといた方がよくない？
- 106 ：265 ：2001/08/25(土) 23:20
- >>104
ワラタ
- 107 ：デフォルトの名無しさん ：2001/08/25(土) 23:20
- main()のdat_read(); dat_out(); は見直したほうが良くない?
全レスbufに溜める方式で良いのか？
- 108 ：デフォルトの名無しさん ：2001/08/25(土) 23:21
- とりあえず>>84あたりはmergeされてもいいと思うんだけど
- 109 ：デフォルトの名無しさん ：2001/08/25(土) 23:21
- >>105
とりあえず期日は今月末だから、結果が出るものを
先に作ったほうがよい。というか作り終えたみたいだが
(これからテスト)
- 110 ：デフォルトの名無しさん ：2001/08/25(土) 23:21
- >>107
そうそう、それ良くないよ。
- 111 ：デフォルトの名無しさん ：2001/08/25(土) 23:21
- >>103
それがいいと思う。
ただそうする場合、向こうにその方針伝えて置いた方がよさげ。
- 112 ：デフォルトの名無しさん ：2001/08/25(土) 23:21
- むこうで作ってるのはgzip対応版でしょ？
向こうはデータの入出力に手を入れている。
こっちはデータの解析・加工に手を入れている。
違うところを修正しているのだから問題ないと思うけど。
- 113 ：デフォルトの名無しさん ：2001/08/25(土) 23:22
- http://5ch.tanuq.net/
5chへ移動
- 114 ：デフォルトの名無しさん ：2001/08/25(土) 23:22
- むこうは間に合わせの quick hackだから、
こっちはちゃんと書き直すのがよいと思われ。
っていうか、mmapして行数数えたほうがよくねえか?
- 115 ：UNIX板より ：2001/08/25(土) 23:23
- read.cgi ver5
419 名前：夜勤 ★ 投稿日：2001/08/25(土) 23:20
piza2 に入りました。
私は 転送量と load average を観察中。。。
- 116 ：デフォルトの名無しさん ：2001/08/25(土) 23:25
- >>112
今必要とされてる課題は転送量の削減だから、
それ以外の改良を同時に施してもわけわからなく
なるか。
あっちがリリースされてからのほうがいいよ
- 117 ：デフォルトの名無しさん ：2001/08/25(土) 23:26
- とりあえず、サーバへの負荷よりも、トラフィックが問題らしい。
231KB
新着レスの表示
スレッドリストへ戻る 全部 前100 次100 最新50
0ch BBS 2004-10-30