Как написать попроще?

Question

[Luka Tarkhnishvili]

int summ = 0;
foreach (int r in new int[] { 5, -6, 2, 7, -5, 9, 1, -3 }) summ += (r < 0) ? r : 0;

как написать это выражение по проще(на низком уровне), для лектора которий любит "все по простому" .

Comments

[Rou1997]

Низкий уровень - это ассемблер, причем он здесь?

[Luka Tarkhnishvili]

вы неправильно поняли я просто хочу упростить код для чтения а не на ассемблере переписать

[Rou1997]

Да Ну: Тогда это не "на низком уровне", упростить можно так:
int summ = 0;
var arr = new int[] { 5, -6, 2, 7, -5, 9, 1, -3 };
foreach (int r in arr)
{
if (r < 0)
summ += r;
else
summ += 0;
}

Answer 1

[Kronic]

Может так? Расписать для легкочитаемости

int summ = 0;
int[] massive = new int[] {5, -6, 2, 7, -5, 9, 1, -3};
foreach (int r in massive)
{
	if (r < 0)
	{
		summ += r;
	}
}

Либо использовать Linq

int summ = (new int[] {5, -6, 2, 7, -5, 9, 1, -3}).Where(x => x < 0).Sum();

Answer 2

[Толстый Лорри]

int result = new int[] { 5, -6, 2, 7, -5, 9, 1, -3 }.Where(i => i<0).Sum();

Answer 3

[Алексей]

foreach (int r in new int[] { -6, -5, -3 }) summ += r;

или
summ = -14;
или я не понял что хочет лектор.

Answer 4

[AcidBat]

А что тут упрощать? Сам код в принципе нелогичен. Если этот массив создаётся в цикле (читай, по слабой ссылке) с заранее известными значениями, которые потом просто складываются по триггеру, то какой вообще в этом смысл?
Какой-то неадекватный у вас лектор.

Comments

[Толстый Лорри]

А почему ссылка-то слабая?

[Luka Tarkhnishvili]

лектор сам неадекватность, что вы посоветуйте сменить в коде? если не прямо так откуда брать цифри?

[AcidBat]

Free_ze: объект, созданный через оператор new и используемый как параметр, будет удалён сразу же после окончания процедуры, в которой он был создан. В данном случае он нигде кроме цикла будет недоступен и вы нигде его не сможете применить.

[AcidBat]

Да Ну: если без библиотек и для удобочитаемости, заменить тернарный оператор на if и занести массив в объект. Больше то и менять нечего.

[Luka Tarkhnishvili]

Спасибо болшое за совети, и если често я в C# не очен.может глупый вопрос но не подскажете как занести массив в объект?

[AcidBat]

Да Ну: В C# нет переменных, есть только объекты. В данном случае вам нужен экземпляр класса Array. Можно как в лоб

int[] mass = new int[] {5, -6, 2, 7, -5, 9, 1, -3};

Так и с автоматическим определением типа.

object[] mass = new object[] {5, -6, 2, 7, -5, 9, 1, -3};

Вам уже дали ответ с точно таким же содержанием.

[Luka Tarkhnishvili]

спасибо

[Толстый Лорри]

AcidBat: Вы не могли бы по поводу "слабости" этих ссылок сослаться на литературу?

[Толстый Лорри]

AcidBat: И чем вам примитивы не переменные?

[AcidBat]

Free_ze:
> И чем вам примитивы не переменные?
Примитивы - это синонимы классов. int - это синоним System.Int32 и так далее.
> Вы не могли бы по поводу "слабости" этих ссылок сослаться на литературу?
Окей. Получите значение массива, инициализированного как параметр цикла, где нибудь вне цикла.

[Толстый Лорри]

AcidBat:
> Примитивы - это синонимы классов. int - это синоним System.Int32 и так далее.
И что из этого следует?) Int32 хранится на стеке, в методы передается по значению - вполне-себе переменная. Подробнее можно почитать в [спецификации](https://www.microsoft.com/en-us/download/confirmat...

> Окей. Получите значение массива, инициализированного как параметр цикла, где нибудь вне цикла.
Ой, то есть это была ваша фантазия?) Сильные ссылки отличаются от слабых тем, что управляют временем жизни объектов. Оператор new в C# возвращяет сильные ссылки (с WeakReference все понятно, т.к. это враппер).