user-0 提问: 求问下这段代码哪里错了 我各种调试了一上午也没发现错误

回复:
user: user-1
content: 有时不将“调用函数名字＋各参数值，进入函数后各参数值，中间变量值，退出函数前准备返回的值，返回函数到调用处后函数名字＋各参数值＋返回值”这些信息写日志到文件中是无论如何也发现不了问题在哪里的，包括捕获各种异常、写日志到屏幕、单步或设断点或生成core文件、……这些方法都不行！ 写日志到文件参考下面： #include <stdio.h> #include <stdlib.h> #include <string.h> #ifdef WIN32     #include <windows.h>     #include <io.h> #else     #include <unistd.h>     #include <sys/time.h>     #include <pthread.h>     #define  CRITICAL_SECTION   pthread_mutex_t     #define  _vsnprintf         vsnprintf #endif //Log{ #define MAXLOGSIZE 20000000 #define MAXLINSIZE 16000 #include <time.h> #include <sys/timeb.h> #include <stdarg.h> char logfilename1[]="MyLog1.log"; char logfilename2[]="MyLog2.log"; static char logstr[MAXLINSIZE+1]; char datestr[16]; char timestr[16]; char mss[4]; CRITICAL_SECTION cs_log; FILE *flog; #ifdef WIN32 void Lock(CRITICAL_SECTION *l) {     EnterCriticalSection(l); } void Unlock(CRITICAL_SECTION *l) {     LeaveCriticalSection(l); } #else void Lock(CRITICAL_SECTION *l) {     pthread_mutex_lock(l); } void Unlock(CRITICAL_SECTION *l) {     pthread_mutex_unlock(l); } #endif void LogV(const char *pszFmt,va_list argp) {     struct tm *now;     struct timeb tb;      if (NULL==pszFmt||0==pszFmt[0]) return;     _vsnprintf(logstr,MAXLINSIZE,pszFmt,argp);     ftime(&tb);     now=localtime(&tb.time);     sprintf(datestr,"%04d-%02d-%02d",now->tm_year+1900,now->tm_mon+1,now->tm_mday);     sprintf(timestr,"%02d:%02d:%02d",now->tm_hour     ,now->tm_min  ,now->tm_sec );     sprintf(mss,"%03d",tb.millitm);     printf("%s %s.%s %s",datestr,timestr,mss,logstr);     flog=fopen(logfilename1,"a");     if (NULL!=flog) {         fprintf(flog,"%s %s.%s %s",datestr,timestr,mss,logstr);         if (ftell(flog)>MAXLOGSIZE) {             fclose(flog);             if (rename(logfilename1,logfilename2)) {                 remove(logfilename2);                 rename(logfilename1,logfilename2);             }         } else {             fclose(flog);         }     } } void Log(const char *pszFmt,...) {     va_list argp;      Lock(&cs_log);     va_start(argp,pszFmt);     LogV(pszFmt,argp);     va_end(argp);     Unlock(&cs_log); } //Log} int main(int argc,char * argv[]) {     int i; #ifdef WIN32     InitializeCriticalSection(&cs_log); #else     pthread_mutex_init(&cs_log,NULL); #endif     for (i=0;i<10000;i++) {         Log("This is a Log %04d from FILE:%s LINE:%d\n",i, __FILE__, __LINE__);     } #ifdef WIN32     DeleteCriticalSection(&cs_log); #else     pthread_mutex_destroy(&cs_log); #endif     return 0; } //1-78行添加到你带main的.c或.cpp的那个文件的最前面 //81-85行添加到你的main函数开头 //89-93行添加到你的main函数结束前 //在要写LOG的地方仿照第87行的写法写LOG到文件MyLog1.log中

user: user-1
content: 代码功能归根结底不是别人帮自己看或讲解或注释出来的；而是被自己静下心来花足够长的时间和精力亲自动手单步或设断点或对执行到某步获得的中间结果显示或写到日志文件中一步一步分析出来的。 提醒：再牛×的老师也无法代替学生自己领悟和上厕所！ 单步调试和设断点调试（VS IDE中编译连接通过以后，按F10或F11键单步执行，按Shift+F11退出当前函数；在某行按F9设断点后按F5执行停在该断点处。）是程序员必须掌握的技能之一。

user: user-0
content: 引用 5 楼 enlinux 的回复:x是随机数，可能超过2010，可能是字符串的结尾位置，可能是负数，。。。。 没有啊 x在前面已经初始化过了

user: user-0
content: 引用 7 楼 anshiny 的回复:lz的代码在逻辑上有诸多漏洞。 试一下这个： #include<stdio.h> #include<string.h>  void takeOff1(char *str); void takeOffKs(char *str, int *k); void doPreZeros(char *str);   int main() { 	char s[1001]; 	int t, k, i; 	scanf("%d", &t); 	for(i=1; i<=t; i++) { 		scanf("%s", s); 		scanf("%d", &k); 		takeOffKs(s, &k); 		printf("Case #%d: %s\n", i, s); 	} 	return 0; }  void doPreZeros(char *str) { 	int i, mark = -1, start; 	int lengthOfStr; 	i = 0; 	lengthOfStr = strlen(str); 	 	while(i < lengthOfStr) { 		if(str[i] != '0') break; 		mark = i; 		++i; 	} 	if(mark != -1) { 		i = mark + 1; 		start = 0; 		while(i < lengthOfStr) { 			str[start] = str[i]; 			start++; 			i++; 		} 		str[start] = '\0'; 	} 	return; }   void takeOff1(char *str) { 	int i = 0; 	int lengthOfStr = strlen(str); 	while(i+1 < lengthOfStr && str[i] <= str[i+1]) { 		++i; 	} 	if(i+1 == lengthOfStr) { 		str[i] = '\0'; 	} else { 		while(str[i+1] != '\0') { 			str[i] = str[i + 1]; 			i++; 		} 		str[i] = '\0'; 	} }   void takeOffKs(char *str, int *k) { 	while(*k) { 		takeOff1(str); 		doPreZeros(str); 		(*k)--;		 	} } 这道题我用别的方法做出来了 但是我就想不懂我这个代码哪里错了 逻辑上没什么问题啊

user: user-2
content: lz的代码在逻辑上有诸多漏洞。 试一下这个： #include<stdio.h> #include<string.h>  void takeOff1(char *str); void takeOffKs(char *str, int *k); void doPreZeros(char *str);   int main() { 	char s[1001]; 	int t, k, i; 	scanf("%d", &t); 	for(i=1; i<=t; i++) { 		scanf("%s", s); 		scanf("%d", &k); 		takeOffKs(s, &k); 		printf("Case #%d: %s\n", i, s); 	} 	return 0; }  void doPreZeros(char *str) { 	int i, mark = -1, start; 	int lengthOfStr; 	i = 0; 	lengthOfStr = strlen(str); 	 	while(i < lengthOfStr) { 		if(str[i] != '0') break; 		mark = i; 		++i; 	} 	if(mark != -1) { 		i = mark + 1; 		start = 0; 		while(i < lengthOfStr) { 			str[start] = str[i]; 			start++; 			i++; 		} 		str[start] = '\0'; 	} 	return; }   void takeOff1(char *str) { 	int i = 0; 	int lengthOfStr = strlen(str); 	while(i+1 < lengthOfStr && str[i] <= str[i+1]) { 		++i; 	} 	if(i+1 == lengthOfStr) { 		str[i] = '\0'; 	} else { 		while(str[i+1] != '\0') { 			str[i] = str[i + 1]; 			i++; 		} 		str[i] = '\0'; 	} }   void takeOffKs(char *str, int *k) { 	while(*k) { 		takeOff1(str); 		doPreZeros(str); 		(*k)--;		 	} }

user: user-3
content: x是随机数，可能超过2010，可能是字符串的结尾位置，可能是负数，。。。。

user: user-0
content: 引用 3 楼 user-4 的回复:看了半天,不明白变量x的意义... 没什么含义 一开始写的 后来没用上

user: user-4
content: 看了半天,不明白变量x的意义...

user: user-0
content: #include<stdio.h> #include<string.h> #include<stdlib.h> void Delete(char s[2010],int k,int x) {     int i,j;     for(i=x+1;s[i]!='\0';i++){         if(s[i]<s[i-1]){             for(j=i-1;j>=x;j--){                 if((s[i]<s[j])&&(s[j]!='0')&&(s[j]!='.')){                     s[j]='.';                     k--;                     if(k==0)                         return;                 }                 else{                     if((s[i]>s[j])&&(s[j]!='0')&&(s[j]!='.'))                         break;             }         }     }     }     i--;     for(;i>=x;i--){         if(s[i]!='.'){             s[i]='.';             k--;             if(k==0){                 return;             }         }     } }   int main(void) {     int T,k,i,j,x,p;     char s[2010];     scanf("%d",&T);     for(i=0;i<T;i++){         x=0;p=0;         scanf("%s",s);         scanf("%d",&k);         Delete(s,k,x);         printf("Case #%d: ",i+1);         for(j=x;s[j]!='\0';j++){             if((s[j]!='.')){                 if((s[j]=='0')){                     if(p==0){                         continue;                         }                     else                         printf("0");                 }                 else{                     printf("%c",s[j]);                     p=1;                 }             }         }         if(p==0){             printf("0");         }         printf("\n");     }     return 0; }

user: user-0
content: #include<stdio.h> #include<string.h> #include<stdlib.h> void Delete(char s[2010],int k,int x) {     int i,j;     for(i=x+1;s[i]!='\0';i++){         if(s[i]<s[i-1]){             for(j=i-1;j>=x;j--){                 if((s[i]<s[j])&&(s[j]!='0')&&(s[j]!='.')){                     s[j]='.';                     k--;                     if(k==0)                         return;                 }                 else{                     if((s[i]>s[j])&&(s[j]!='0')&&(s[j]!='.'))                         break;             }         }     }     }     i--;     for(;i>=x;i--){         if(s[i]!='.'){             s[i]='.';             k--;             if(k==0){                 return;             }         }     } }   int main(void) {     int T,k,i,j,x,p;     char s[2010];     scanf("%d",&T);     for(i=0;i<T;i++){         x=0;p=0;         scanf("%s",s);         scanf("%d",&k);         Delete(s,k,x);         printf("Case #%d: ",i+1);         for(j=x;s[j]!='\0';j++){             if((s[j]!='.')){                 if((s[j]=='0')){                     if(p==0){                         continue;                         }                     else                         printf("0");                 }                 else{                     printf("%c",s[j]);                     p=1;                 }             }         }         if(p==0){             printf("0");         }         printf("\n");     }     return 0; }