Bitmap манипуляции метания System.ArgumentOutOfRangeException

Я пытаюсь написать кусок кода, чтобы создать черный растровое изображениеBitmap манипуляции метания System.ArgumentOutOfRangeException

Bitmap rectangle = new Bitmap(100, 100); 
for (int i = 1; i <= rectangle.Width - 1; i++) 
    for (int j = 1; i <= rectangle.Height - 1; j++) 
     rectangle.SetPixel(i, j, Color.Black);

Но он продолжает бросать System.ArgumentOutOfRangeException.

источник

2016-02-14 T.Brown

Вы пробовали вот так? 'i <100' вместо' i <= rectangle.Width - 1' –

C# использует индексирование на основе нуля, поэтому ваши циклы должны начинаться с нуля, а не с нуля. Обратите внимание, что это не причина, по которой она выбрасывает исключение. Я просто говорю, что вы пропускаете первые значения i и j. – Trauer

Я пробовал оба, т. Е. (Int i = 0; i <100; i ++), но это не решило проблему. –

Bitmap rectangle = new Bitmap(100, 100); 
for (int i = 1; i <= rectangle.Width - 1; i++) 
    for (int j = 1; i <= rectangle.Height - 1; j++) 
     rectangle.SetPixel(i, j, Color.Black);

Проблема в состоянии останавливая вашей второй петли в:

i <= rectangle.Height - 1

должен быть

j <= rectangle.Height - 1

я теперь это не «просмотр кода» вопрос, но вот предложение : избегать использования magic numbers Мы можем сделать это, просто удалив -1 и используя < вместо <=.

Bitmap rectangle = new Bitmap(100, 100); 
for (int i = 0; i < rectangle.Width; i++) 
    for (int j = 0; j < rectangle.Height; j++) 
     rectangle.SetPixel(i, j, Color.Black);

источник

2016-02-14 01:35:53 Trauer

Спасибо, это была глупая ошибка –

Не глупо. Это просто отсутствие внимания. Я делаю это все время, когда устаю и пытаюсь закодировать. – Trauer

Bitmap манипуляции метания System.ArgumentOutOfRangeException

ответ

Смежные вопросы