Eu não vou nem tentar resolver o problema nesse código porque ele é confuso demais e talvez o problema tenha nascido da confusão. Quanto mais simples puder fazer, melhor. Mesmo que queira aprender algum conceito específico com este tipo de código, é melhor aprender em um caso onde ele seria necessário. Se alguém pediu para fazer desta forma, a pessoa fez um desserviço para você. Seria um requisito inútil e improdutivo. e eu argumentaria que essa forma é desvantajosa.
Não declare variáveis fora da função sem necessidade. Declare sempre o mais próximo possível de onde será usada. Tem gente que não vê o valor disto, mas cria uma capacidade cognitiva maior para fazer coisas mais simples. Se precisa da variável apenas dentro do laço, crie ela dentro do laço.
Se você tem dois vetores a serem considerados, crie dois laços, é a forma natural de se fazer isto. Está é a forma estruturada de fazer este código. O original dá nó na cabeça. Inclusive um laço for
é muito mais natural para o caso.
Havia uma variável da coluna do tipo char
. Ou faz a linha também ser char
e limita o número de asteriscos em 127 ou deixa int
em ambos. Eu deixei int
porque não há ganho real em usar char
aí, mesmo que só queira valores baixos, o que na verdade nem é validado, até porque o scanf()
nem é muito adequado para isto. Seja consistente.
Dei uma organizada. É muito difícil entender o que o código faz na forma escrita.
Feito isso o código fica simples demais.
#include<stdio.h>
int main(void) {
printf("TELL ME THE NUMBER YOU WANT: \n");
int n;
scanf("%d", &n);
for (int row = 0; row < n; row++) {
for (int column = 0; column < n; column++) printf("*");
printf("\n");
}
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.