Как найти эффективное решение задачи (Java)?

Question

[mk_qwerty]

Доброго времени суток, решал задачу, но решение не эффективно. Как можно сделать его эффективней и короче?

Условие задачи: Напишите программу, которая получает целое число в n как аргумент командной строки и использует вложенные циклы для вывода "шахматной доски" n*n элементов.

public class checkerBoard{
  public static void main(String[] args){
    int n = Integer.parseInt(args[0]);

    for (int i = 1; i <= n; i++){
        for (int j = 1; j <= n; j++){
          if (i % 2 == 0){
            if(j % n != 0){
              if (j % 2 == 0){
                System.out.print("*");
              }
              else System.out.print("-");
            }
            else{
              if (j % 2 == 0){
                System.out.println("*");
              }
              else System.out.println("-");
            }
          }
          else{
            if(j % n != 0){
              if (j % 2 == 0){
                System.out.print("-");
              }
              else System.out.print("*");
            }
            else{
              if (j % 2 == 0){
                System.out.println("-");
              }
              else System.out.println("*");
            }
          }

        }
    }
  }
}

Comments

[Виктор Мальков]

короче
for (int i = 1; i <= n; i++) {
for (int j = 1; j <= n; j++) {
System.out.print((i + j) % 2 == 0 ? '*' : '-');
}
System.out.println();
}

[Roman]

деление по модулю это не самая эффективная операция.

System.out.print( ((i + j) & 1) == 0 ? '*' : '-' );

для четных можно так

public static void main(String args[])
{
  int n = 8;
  char c1 = '*';
  char c2 = '-';
  for(int i = 0; i < n; ++i)
  {
    for(int j = 0; j < n; j += 2)
    {
      System.out.printf("%c%c", c1, c2);
    }
    System.out.print('\n');
    // swap(a, b)
    char tmp = c1;
    c1 = c2;
    c2 = tmp;
  }
}

Answer 1

[Дмитрий Александров]

Виктор Мальков,
еще в копилку, не короче но вызовет приступ у ненавистников лямбд и стримов

IntStream.rangeClosed(1, n).forEach(i -> {
            IntStream.rangeClosed(1, n).forEach(j -> 
                System.out.print((i + j) % 2 == 0 ? '*' : '-'));
        System.out.println();});

Еще изврат, правда будет работать только с нечетными числами но зато в один цикл! =).

IntStream.rangeClosed(1, n*n).forEach(i -> System.out.print(i % n != 0 ? (i % 2==0 ?'*':'-') : '\n'));

Answer 2

[krka92]

Джаву, увы, не знаю

procedure WriteCheckLine(Len: Integer; Offset: Byte);
var i: Integer;
begin
  for i := Offset to Len - 1 + Offset do
  begin
    if i and 1 > 0 then Write('*')
    else Write('-');
  end;
  WriteLn;
end;

procedure WriteCheckBoard(N: Integer);
var i: Integer;
begin
  for i := 0 to N - 1 do WriteCheckLine(N, i and 1);
end;

или без вызова

procedure WriteCheckBoard(N: Integer);
var i, j, Offset: Integer;
begin
  for i := 0 to N - 1 do
  begin
    Offset := i and 1;
    for j := Offset to N - 1 + Offset do
    begin
      if j and 1 > 0 then Write('*')
      else Write('-');
    end;
    WriteLn;
  end;
end;

Как удобнее будет