【初心者歓迎】C/C++室 Ver.102【環境依存OK】

**デフォルトの名無しさん** · 2017/11/04(土) 16:33:35.07

エスケープシーケンスやWin32APIなどの環境依存なものもOK
そのような質問は必ず環境を書きましょう
半角空白やタブでのインデントはスレに貼ると無くなります

コードを貼れる所
http://codepad.org/
https://ideone.com/

前スレ
【初心者歓迎】C/C++室 Ver.101【環境依存OK】
https://mevius.5ch.net/test/read.cgi/tech/1500329247/

**デフォルトの名無しさん** · 2017/11/05(日) 17:44:45.44

>>1
乙

**デフォルトの名無しさん** · 2017/11/05(日) 20:58:13.61

unsigned long strtoulxn( char const* psz, size_t n )
{
　　　　unsigned long v;

　　　　for ( v = 0; *psz && n--; psz++ ) {
　　　　　　　　if ( ! isdigit( *psz ) ) {
　　　　　　　　　　　　continue;
　　　　　　　　}
　　　　　　　　v = 10 * v + ( *psz - '0' );
　　　　}
　　　　return v;
}

**デフォルトの名無しさん** · 2017/11/05(日) 21:07:57.69

これは酷いクソコード

**デフォルトの名無しさん** · 2017/11/05(日) 22:25:14.64

実行速度
見やすさ
汎用性
コードサイズ

どれかひとつ位は満たさないと

**デフォルトの名無しさん** · 2017/11/05(日) 22:42:14.76

>>5
まあそう言うごたくを言うのは一つぐらいは満たしたコードを書いてからでないとね

**デフォルトの名無しさん** · 2017/11/06(月) 08:01:53.31

>>3 の関数は前のスレッドの終わりに出た
"12 34 56"(NUL終端なし8byte) を数値の 123456 に変換、
てお題の答案かな。

…と思ったけど、NUL終端文字列が仮定されてるね。
どういう流れで投稿されたんだろう?

**デフォルトの名無しさん** · 2017/11/06(月) 08:45:24.00

size_t n で文字数を受けとるからNUL終端である必要は無いのでは

**デフォルトの名無しさん** · 2017/11/06(月) 09:40:14.23

if (strncmp(psz, "12 34 56", 8) == 0) return 123456;

**デフォルトの名無しさん** · 2017/11/06(月) 23:29:19.85

相変わらず片山はクソコードしか書かないな
""だと不定値返る数値変換なんてゴミ

**デフォルトの名無しさん** · 2017/11/06(月) 23:59:53.41

>>10
確かにクソコードだが不定値は返らんぞ

**デフォルトの名無しさん** · 2017/11/07(火) 04:37:45.33

>>9 は質問者に対する皮肉のつもりだったんだけど

**デフォルトの名無しさん** · 2017/11/07(火) 04:47:47.90

意図的に糞コードを書いた事もわからないクソがいるとは
こんな簡単なコードの動作も理解できてないようだし

**デフォルトの名無しさん** · 2017/11/07(火) 06:16:53.55

初心者歓迎をうたうスレッドで「意図的に『糞コード』を書いた」などと
しかも後付けで言われてもあんまり支持できないなぁ。
せめて最初から「お前らこれ改善してみろよ」とかで出してくれ。
「糞コード」とけなすだけで、具体的な問題点の指摘がないってのも寂しいわ。

>>3 のソース、引数のpszが""でも、nが0でも妥当な返り値 0 が返るけど、
実は最初に見たとき n==0 で呼ばれた場合に即時終了しないと騙されたわ。
後置デクリメント評価値、ね。