CareerCup

This is quite easy, Here is right implementation

public class SeqTest { 

	public static void main(String args[]) { 
		for(int i = 1; i <= 50; i++) {
			if ( (i%3 == 0) && (i%5 == 0) ) System.out.println("ATGC"); 
			else if(i % 3 == 0) System.out.println("DNA"); 
			else if(i % 5 == 0) System.out.println("RNA"); 
			else System.out.println(i); 
		} 
	}
	
}

public class Test{
public static void main(String args[]){
for(int i=1;i<=50;i++){
if(i%3==0){
if(i%5==0){
System.out.println("ATGC");
continue;
}
System.out.println("DNA");
}
else if(i%5==0){
System.out.println("RNA");
}
else{
System.out.println(i);
}
}
}
}

Bug 1: condition “else if(i%(3+5)==0)” should be replaced with “if(i%3==0&&i%5==0)”

Bug 2: condition if(i%3==0&&i%5==0) should be first line in the program.

Bug 3: Last line should be System.out.println(i); instead of System.out.println(“1 to 50”);

bug1: i%(3+5) should be i%15
bug2: i%15 should be before i%3 and i%5 because any number that is divisible by 15 is automatically divisible by 3 and 5. So the specialized case need to be first.
bug3:??

public class SeqTest {

public static void main(String args[]) {

for(int i = 1; i <= 50; i++) {
if(i % 3 == 0) System.out.println("DNA");
else if(i % 5 == 0) System.out.println("RNA");
else if(i % (3+5) == 0) System.out.println("ATGC");
else System.out.println(“1 to 50”);
}
}
}

public class SeqTest {

public static void main(String args[]) {

for(int i = 1; i <= 50; i++) {
if(i % 3 == 0) System.out.println("DNA");
else if(i % 5 == 0) System.out.println("RNA");
else if(i % (3+5) == 0) System.out.println("ATGC");
else System.out.println(“1 to 50”);
}
}
}

public class SeqTest {

public static void main(String args[]) {
for(int i = 1; i <= 50; i++) {
if ( (i%3 == 0) && (i%5 == 0) ) System.out.println("ATGC");
else if(i % 3 == 0) System.out.println("DNA");
else if(i % 5 == 0) System.out.println("RNA");
else System.out.println(i);
}