字符串比较的问题

[qq7825591]

int comparewith(String s1,String s2,int num){
s1 = new String();
s2 = new String();
int number;
for(int i=0;i<num;i++)
{
if(s1.charAt(i)>s2.charAt(i))
{
number = 1;
break;
}
else if(s1.charAt(i)<s2.charAt(i))
{
number = -1;
break;

}
else
{
number = 0;
break;
}
return number;
}
}


why it registers me :the method must be return a result of type int ?

[Hao粑粑爱生活]

int comparewith(String s1,String s2,int num){
s1 = new String();
s2 = new String();
int number=0;
for(int i=0;i <num;i++)
{
if(s1.charAt(i)>s2.charAt(i))
{
number = 1;
break;
}
else if(s1.charAt(i) <s2.charAt(i))
{
number = -1;
break;

}
else
{
number = 0;
break;
}
}
return number;
}

[anqini]

//这么改虽然没出编译出错,但是你这程序有问题!
static int comparewith(String s1,String s2,int num){ 
		s1 = new String(); //这个地方有问题!
		s2 = new String(); 
		int number=0; 
		for(int i=0;i <num;i++) { 		
		if(s1.charAt(i)>s2.charAt(i)) { //这里也有问题!
		
			number = 1; 
			break; 
		} else if(s1.charAt(i) <s2.charAt(i)) { 		
			number = -1; 
			break; 
		} 
		else 
		{ 
			number = 0; 
			break; 
		} 
		
		} 
		return number; //改了这里
	}

[KKK2007]

把return number; 这句放到for循环的外面。

[Alawn2005]

新手马克一下，关注中

[xueshenzhou]

public static int comparewith(String s1,String s2,int num){
/**
*String 是一个类，当对象s1传过来以后，你又new了一个对象导致他的内容是空的啊！报出
*StringIndexOutOfBoundsException异常。简单说就是s1 和s2永远是空值。
*/
//s1 = new String();
//s2 = new String();
int number=0;
for(int i=0;i < num -1;i++) {
if(s1.charAt(i)>s2.charAt(i)) {

number = 1;
break;
} else if(s1.charAt(i) <s2.charAt(i)) {
number = -1;
break;
}
else
{
number = 0;
break;
}

}
return number; //返回要在最后啊
}

[action_520_12]

这个程序还真看不太懂。。。
漏洞太多。。。

[wensheng_zh2007]

错误改过来，这个函数有什么作用？

[action_520_12]

而且，重复实现String类已经实现的功能的意义？

[loujianchengdd]

在循环中不能直接放置return,那个错误意思是说你的方法没有返回值，如果要在循环中加return的话，在循环中应该加一个判断，if()(return x)else(){retrun x} ,这种形式是可以的。

[zhengpeiyong]

晕倒，这个，不至于连基础知识都不了解吧？？

[zhengpeiyong]

int comparewith(String s1,String s2,int num){ 
//s1 = new String(); 
//s2 = new String(); 
int number; 
for(int i=0;i <num;i++) 
{ 
if(s1.charAt(i)>s2.charAt(i)) 
{ 
number = 1; 
break; 
} 
else if(s1.charAt(i) <s2.charAt(i)) 
{ 
number = -1; 
break; 

} 
else 
{ 
number = 0; 
break; 
} 
} 
return number; 
} 

[zh_heng]

每一个break前面都要有一个return number;

[zh_heng]

如果只是把return number;改放到循环外面的话 返回的只是循环最后一次的结果，显然不符合lz的需求，所以要在每一个break前面加一个return number;就对了。

[hujing101]

[Quote=引用 2 楼 anqini 的回复:]
Java code
//这么改虽然没出编译出错,但是你这程序有问题!
static int comparewith(String s1,String s2,int num){
s1 = new String(); //这个地方有问题!
s2 = new String();
int number=0;
for(int i=0;i <num;i++) {
if(s1.charAt(i)>s2.charAt(i)) { //这里也有问题!

number = 1;
break;
} else if(s1.charAt(i…
[/Quote]
顶一个 学习了~~

[Li_Fangbin]

把retrun放在最后，不要重新new String 就可以了

[IT小岛]

return 应放在for语句外层！ 但是你着代码只比较第一个字符就会结束！没有比较整个字符串！

[zlss_17]

return number 位置不对撒

[blliy117]

算法且不说！！！

这些人方法都写错
5楼的正确！
好好学习一下基础！！

[cyazhang]

public class StringCompare {
		public static int compareWith(String s1,String s2,int num){
				int value = 0;
				for(int i=0;i<num;i++){
						if(s1.charAt(i)>s2.charAt(i)){
							value = 1;
							return value;
						}
						else if(s1.charAt(i)<s2.charAt(i)){
							value = -1;
							return value;
						}
						else
							return value;
					}
				return s1.compareTo(s2);
			}
		public static void main(String[] args){
				System.out.println(compareWith("cheng","csdn",4));
			}
	}