The code is working for few cases not for allPls hep

Question about the task

Shuffled just in time

Java Syntax, Level 7, Lesson 12

Resolved

package com.codegym.task.task07.task0720; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; /* Shuffled just in time */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int N = Integer.parseInt(reader.readLine()); BufferedReader reader2 = new BufferedReader(new InputStreamReader(System.in)); int M = Integer.parseInt(reader2.readLine()); ArrayList<String> list = new ArrayList<>(); BufferedReader reader3 = new BufferedReader(new InputStreamReader(System.in)); for(int i = 0 ; i < N;i++) { String s = reader3.readLine(); list.add(s); } //for (int i = 0;i<N;i++) //{ int i =0; for(int j = M;j <N;j++) { String temp = list.get(j); list.set(j ,list.get(i) ); list.set(i,temp); i++; } for (int k=0;k<N;k++) { System.out.println(list.get(k)); } //} //write your code here } }

Comments (3)

Popular
New
Old

You must be signed in to leave a comment

sourav bamotra

Level 20 , jammu, India

29 October 2020, 10:05

public static void main(String[] args) throws Exception {
		BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
		int n = Integer.parseInt(reader.readLine());
		int m = Integer.parseInt(reader.readLine());
		ArrayList<String> list = new ArrayList<>();

		for (int i = 0; i < n; i++) {
			list.add(i, reader.readLine());
		}
		for (int i = 0; i < m; i++) {
			String s = list.remove(0);
			list.add(s);
		}
		for (String x : list) {
			System.out.println(x);
		}
	}

initializing the list. Removing 1st element and adding it to the last in same list for m times.

Dmitri

Level 22 , Seversk, Russia

29 October 2020, 05:59

You have got redundant readers. One is enough: int n = Integer.parseInt(reader.readLine()); int m = Integer.parseInt(reader.readLine()); for (int i = 0; i < n; i++) list.add(reader.readLine()); As for the algorithm, reflect on this: for (int i = 0; i < m; i++) list.add(list.size()-1, list.remove(0)); or may be just for (int i = 0; i < m; i++) list.add(list.remove(0)); is good enough

Nouser

Level 36 , Germany

28 October 2020, 13:43

If you open a stream you can use it for all inputs till you close the stream like:

BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
int N = Integer.parseInt(reader.readLine());
int M = Integer.parseInt(reader.readLine());

If you want to move now eg. 2 (or M) elements from the beginning to the end of the list, then you iterate from 0 to < M. In the loop you just read the first element and write it to the end. Then you remove the first element (so you again can read the first element in the next iteration)...

list.add(list.get(0));
list.remove(0);

The output should be OK (but I think it's better to use list.size() instead of N or a for-each loop)