for文を使いドットでエックスを描くプログラム

Question

前提・実現したいこと

16×16のドットピクセルを扱い、ウィンドウいっぱいにＸの文字をfor文でプログラムをしたいです。
中心から左上と右下には伸びたのですが、右上と左下が反映されません。
どこが間違えているのでしょうか。

発生している問題・エラーメッセージ

エラーメッセージはありません。

該当のソースコード

void setup() 
{
   size(512,512);
   background(0);
   noStroke();
}

void draw( ) 
{
  v_pixel(32,8,8,255,255,255);
  for(int i=8;i<width;i++){
   v_pixel(32,i,i,255,255,255);
  }
  for(int i=8;i<width;i--){
   v_pixel(32,i,i,255,255,255);
  }
  for(int i=8 ;i<height;i++ ){
    for(int z=8;z<height;z--){
      v_pixel(32,i,z,255,255,255);
    }
  }
  for(int i=8 ;i<height;i++ ){
    for(int z=8;z<height;z--){
      v_pixel(32,z,i,255,255,255);
    }
  }

}


void keyPressed(){
  if( key== 's' ) {
    saveFrame("Result.jpg");  // Save File Name
  }
}

void v_pixel(int n, int x, int y, int r,int m, int b) 
{
   if(n==1)
       { stroke(color(r,m,b));  point(x,y); }
     else
       { fill(r,m,b);  rect(x*n,y*n,n,n);  }
}

試したこと

補足情報（FW/ツールのバージョンなど）

Answer 1

-  for(int i=8;i<width;i--){
+  for(int i=8;i<16;i--){
   v_pixel(32,i,i,255,255,255);
  }
-  for(int i=8;i<width;i--){
+  for(int i=8;i>=0;i--){
   v_pixel(32,i,i,255,255,255);
  }
-  for(int i=8 ;i<height;i++ ){
+  for(int i=8 ;i<16;i++ ){
-    for(int z=8;z<height;z--){
-      v_pixel(32,i,z,255,255,255);
+      v_pixel(32,i,16-i-1,255,255,255);
-    }
  }
-  for(int i=8 ;i<height;i++ ){
+  for(int i=8 ;i>=0;i-- ){
-    for(int z=8;z<height;z--){
-      v_pixel(32,z,i,255,255,255);
+      v_pixel(32,i,16-i-1,255,255,255);
-    }
  }

zの条件式が間違っていますし、widthやheightは16ではなく512です。
16と書きたくないならwidth/32やheight/32でしょうか。
とはいえ8が即値ですが…

---

上記は元のコードに合わせましたが、Xが描きたいだけならfor1つでも描けます。

  for(int i=0;i<16;i++){
    v_pixel(32,i,i,255,255,255);
    v_pixel(32,i,16-i-1,255,255,255);
  }

---

おまけ
ピクセルサイズ32以外での描画。

void draw( ) 
{
  int pixel_size = 8;
  int count = min(width, height)/pixel_size;
  for(int i=0;i<count;i++){
    v_pixel(pixel_size,i,i,255,255,255);
    v_pixel(pixel_size,i,count-i-1,255,255,255);
  }
}

Comments

[likuson]

編集したのですが結果は変わりませんでした。
void draw( )
{
v_pixel(32,8,8,255,255,255);
for(int i=8;i<width;i++){
v_pixel(32,i,i,255,255,255);
}
for(int i=8;i<width;i--){
v_pixel(32,i,i,255,255,255);
}
for(int i=8 ;i<height;i++ ){
v_pixel(32,i,height-i,255,255,255);
}
for(int i=8 ;i<height;i++ ){
v_pixel(32,height-i,i,255,255,255);
}

}

[SHOMI]

書きかけのものが送信されていたようなので修正中です

[likuson]

ありがとうございます。待ちます。

[SHOMI]

for2つで書けますが、理由があって(真ん中からか四方に描きたくて？)4つに分けたのでしょうか。

[likuson]

出来れば簡単に書きたいのでfor二つの書き方も教えてください。

[SHOMI]

あ、1つで書けますね…

[likuson]

ありがとうございます。いくつものプログラムを紹介して頂いてありがとうございます。