Этот фрагмент кода:
int& func1()
{
int i;
i = 1;
return i;
}
не будет работать, потому что вы возвращаете псевдоним (ссылку) объекту со временем жизни, ограниченным областью вызова функции. Это означает, что когда func1()
возвращается, int i
умирает, делая ссылку, возвращенную из функции бесполезной, потому что теперь она относится к объекту, который не существует.
int main()
{
int& p = func1();
/* p is garbage */
}
Вторая версия работает, потому что переменная выделяется в свободном хранилище, которая не привязана к времени жизни вызова функции. Однако вы несете ответственность за delete
за выделение int
.
int* func2()
{
int* p;
p = new int;
*p = 1;
return p;
}
int main()
{
int* p = func2();
/* pointee still exists */
delete p; // get rid of it
}
Обычно вы переносите указатель в некоторый класс RAII и / или заводскую функцию, поэтому вам не нужно delete
это самостоятельно.
В любом случае вы можете просто вернуть само значение (хотя я понимаю, что приведенный вами пример был, вероятно, надуманным):
int func3()
{
return 1;
}
int main()
{
int v = func3();
// do whatever you want with the returned value
}
Обратите внимание, что отлично возвращать большие объекты таким же образом func3()
возвращает примитивные значения, потому что почти каждый компилятор в настоящее время реализует некоторую форму оптимизации возвратного значения :
class big_object
{
public:
big_object(/* constructor arguments */);
~big_object();
big_object(const big_object& rhs);
big_object& operator=(const big_object& rhs);
/* public methods */
private:
/* data members */
};
big_object func4()
{
return big_object(/* constructor arguments */);
}
int main()
{
// no copy is actually made, if your compiler supports RVO
big_object o = func4();
}
Интересно, что привязка временного к ссылке const является совершенно законным C ++ .
int main()
{
// This works! The returned temporary will last as long as the reference exists
const big_object& o = func4();
// This does *not* work! It's not legal C++ because reference is not const.
// big_object& o = func4();
}
Я поместил в код цикл Parallel.ForEach. Я также заметил, что я "должен был ??" поместите объект блокировки, так как приведенные ниже списки, похоже, полностью запутались, так как они делят переменные с другими потоками.
if (calc1 > 0) { plusLIST2.Add(str); }
else
{
minusLIST2.Add(str);
}
Кажется, что Parallel.ForEach не является потокобезопасным?
Когда этот код работал, он занимал: 62 секунды - 57 секунд с одним циклом.
Разве нельзя сделать это быстрее с 24 ядрами? Я знаю, что объект блокировки также блокирует код, который является «узким местом» для свободного выполнения кода?
public void calculationFunction(Dictionary<String, List<String>> tr1, Dictionary<String, List<ArrayList>> v1, Dictionary<String, bool> v5,
Dictionary<String, int> infoIndex, out List<String> minusLIST, out List<String> plusLIST)
{
System.Threading.Thread.CurrentThread.CurrentCulture = new System.Globalization.CultureInfo("en-US");
double num1 = 0; double num2 = 0; double num3 = 0; double number1 = 0; double number2 = 0; double number3 = 0; double thenum1 = 0; double thenum2 = 0; double thenum3 = 0;
double NUM1 = 0; double NUM2 = 0; double NUM3 = 0; double NUMBER1 = 0; double NUMBER2 = 0; double NUMBER3 = 0; String str = ""; String num11 = ""; String num22 = ""; String num33 = "";
String number11 = ""; String number22 = ""; String number33 = ""; double calc1 = 0;
minusLIST = new List<String>(); plusLIST = new List<String>();
List<String> minusLIST2 = new List<String>(); List<String> plusLIST2 = new List<String>(); object lockobj = new object();
Parallel.ForEach(tr1, entry =>
{
List<String> allparams = new List<String>(entry.Value);
if (allparams.Count == 10)
{
if (v5.ContainsKey(allparams[7]) && v5.ContainsKey(allparams[8]) && v5.ContainsKey(allparams[9]))
{
if (v5[allparams[7]] && v5[allparams[8]] && v5[allparams[9]])
{
//Calculate both calc scenarios!
num1 = (double)v1[allparams[0]][infoIndex[allparams[7]]][0]; number1 = (double)v1[allparams[0]][infoIndex[allparams[7]]][1]; thenum1 = (double)v1[allparams[0]][infoIndex[allparams[7]]][2];
num2 = (double)v1[allparams[1]][infoIndex[allparams[8]]][0]; number2 = (double)v1[allparams[1]][infoIndex[allparams[8]]][1]; thenum2 = (double)v1[allparams[1]][infoIndex[allparams[8]]][2];
num3 = (double)v1[allparams[2]][infoIndex[allparams[9]]][0]; number3 = (double)v1[allparams[2]][infoIndex[allparams[9]]][1]; thenum3 = (double)v1[allparams[2]][infoIndex[allparams[9]]][2];
NUM1 = num1; NUM2 = num2; NUM3 = num3; NUMBER1 = number1; NUMBER2 = number2; NUMBER3 = number3;
if (num1 <= 0 || number1 <= 0) { NUM1 = thenum1; NUMBER1 = thenum1; }
if (num2 <= 0 || number2 <= 0) { NUM2 = thenum2; NUMBER2 = thenum2; }
if (num3 <= 0 || number3 <= 0) { NUM3 = thenum3; NUMBER3 = thenum3; }
if (NUM1 > 0 && NUM2 > 0 && NUM3 > 0 && NUMBER1 > 0 && NUMBER2 > 0 && NUMBER3 > 0)
{
lock (lockobj)
{
str = ""; num11 = ""; num22 = ""; num33 = ""; number11 = ""; number22 = ""; number33 = "";
if (num1 > 0 && num2 > 0 && num3 > 0 && number1 > 0 && number2 > 0 && number3 > 0) { } else { str = string.Format("{0:F10}", thenum1) + " / " + string.Format("{0:F10}", thenum2) + " / " + string.Format("{0:F10}", thenum3); }
if (num1 <= 0) { num11 = "0"; num1 = thenum1; } else { num11 = string.Format("{0:F10}", num1); }
if (num2 <= 0) { num22 = "0"; num2 = thenum2; } else { num22 = string.Format("{0:F10}", num2); }
if (num3 <= 0) { num33 = "0"; num3 = thenum3; } else { num33 = string.Format("{0:F10}", num3); }
if (number1 <= 0) { number11 = "0"; number1 = thenum1; } else { number11 = string.Format("{0:F10}", number1); }
if (number2 <= 0) { number22 = "0"; number2 = thenum2; } else { number22 = string.Format("{0:F10}", number2); }
if (number3 <= 0) { number33 = "0"; number3 = thenum3; } else { number33 = string.Format("{0:F10}", number3); }
//Calculate
if (allparams[6] == "0")
{
calc1 = ((num1 * number2 * number3) - 45) / 10;
}
else
{
calc1 = (((num1 * number2) / number3) + 45) / 10;
}
//String
str = calc1 + "," + allparams[0] + " - " + allparams[1] + " - " + allparams[2] + "," +
allparams[3] + " - " + allparams[4] + " - " + allparams[5] + "," +
num11 + " / " + num22 + " / " + num33 + "," +
number11 + " / " + number22 + " / " + number33 + "," +
str + "," +
calc1 + "%";
if (calc1 > 0) { plusLIST2.Add(str); }
else
{
minusLIST2.Add(str);
}
}
}
}
}
}
});
plusLIST = new List<String>(plusLIST2);
minusLIST = new List<String>(minusLIST2);
}
Мне удалось запустить код в цикле: «Parallel.ForEach (tr1, entry =>». Когда я объявил все переменные ВНУТРИ цикла. Переменные теперь не перепутались в потоках. Также я необходимо добавить объект блокировки, когда .Добавить к 2 спискам.
Теперь это заняло 12 секунд (Parallel.ForEach) -VS- 57 секунд (Single for loop) Так что это улучшение в 4,75 раза.
Но, тем не менее, у меня есть 24 ядра, поэтому мне интересно, почему он не работает, например, как минимум в 15-20 раз быстрее? [ 116]
Что я могу сделать, чтобы использовать как можно больше / все ядра?
public void calculationFunction(Dictionary<String, List<String>> tr1, Dictionary<String, List<ArrayList>> v1, Dictionary<String, bool> v5,
Dictionary<String, int> infoIndex, out List<String> minusLIST, out List<String> plusLIST)
{
System.Threading.Thread.CurrentThread.CurrentCulture = new System.Globalization.CultureInfo("en-US");
minusLIST = new List<String>(); plusLIST = new List<String>();
List<String> minusLIST2 = new List<String>(); List<String> plusLIST2 = new List<String>(); object lockobj = new object();
Parallel.ForEach(tr1, entry =>
{
List<String> allparams = new List<String>(entry.Value);
if (allparams.Count == 10)
{
if (v5.ContainsKey(allparams[7]) && v5.ContainsKey(allparams[8]) && v5.ContainsKey(allparams[9]))
{
if (v5[allparams[7]] && v5[allparams[8]] && v5[allparams[9]])
{
double num1 = 0; double num2 = 0; double num3 = 0; double number1 = 0; double number2 = 0; double number3 = 0; double thenum1 = 0; double thenum2 = 0; double thenum3 = 0;
double NUM1 = 0; double NUM2 = 0; double NUM3 = 0; double NUMBER1 = 0; double NUMBER2 = 0; double NUMBER3 = 0; String str = ""; String num11 = ""; String num22 = ""; String num33 = "";
String number11 = ""; String number22 = ""; String number33 = ""; double calc1 = 0;
//Calculate both calc scenarios!
num1 = (double)v1[allparams[0]][infoIndex[allparams[7]]][0]; number1 = (double)v1[allparams[0]][infoIndex[allparams[7]]][1]; thenum1 = (double)v1[allparams[0]][infoIndex[allparams[7]]][2];
num2 = (double)v1[allparams[1]][infoIndex[allparams[8]]][0]; number2 = (double)v1[allparams[1]][infoIndex[allparams[8]]][1]; thenum2 = (double)v1[allparams[1]][infoIndex[allparams[8]]][2];
num3 = (double)v1[allparams[2]][infoIndex[allparams[9]]][0]; number3 = (double)v1[allparams[2]][infoIndex[allparams[9]]][1]; thenum3 = (double)v1[allparams[2]][infoIndex[allparams[9]]][2];
NUM1 = num1; NUM2 = num2; NUM3 = num3; NUMBER1 = number1; NUMBER2 = number2; NUMBER3 = number3;
if (num1 <= 0 || number1 <= 0) { NUM1 = thenum1; NUMBER1 = thenum1; }
if (num2 <= 0 || number2 <= 0) { NUM2 = thenum2; NUMBER2 = thenum2; }
if (num3 <= 0 || number3 <= 0) { NUM3 = thenum3; NUMBER3 = thenum3; }
if (NUM1 > 0 && NUM2 > 0 && NUM3 > 0 && NUMBER1 > 0 && NUMBER2 > 0 && NUMBER3 > 0)
{
str = ""; num11 = ""; num22 = ""; num33 = ""; number11 = ""; number22 = ""; number33 = "";
if (num1 > 0 && num2 > 0 && num3 > 0 && number1 > 0 && number2 > 0 && number3 > 0) { } else { str = string.Format("{0:F10}", thenum1) + " / " + string.Format("{0:F10}", thenum2) + " / " + string.Format("{0:F10}", thenum3); }
if (num1 <= 0) { num11 = "0"; num1 = thenum1; } else { num11 = string.Format("{0:F10}", num1); }
if (num2 <= 0) { num22 = "0"; num2 = thenum2; } else { num22 = string.Format("{0:F10}", num2); }
if (num3 <= 0) { num33 = "0"; num3 = thenum3; } else { num33 = string.Format("{0:F10}", num3); }
if (number1 <= 0) { number11 = "0"; number1 = thenum1; } else { number11 = string.Format("{0:F10}", number1); }
if (number2 <= 0) { number22 = "0"; number2 = thenum2; } else { number22 = string.Format("{0:F10}", number2); }
if (number3 <= 0) { number33 = "0"; number3 = thenum3; } else { number33 = string.Format("{0:F10}", number3); }
//Calculate
if (allparams[6] == "0")
{
calc1 = ((num1 * number2 * number3) - 45) / 10;
}
else
{
calc1 = (((num1 * number2) / number3) + 45) / 10;
}
//String
str = calc1 + "," + allparams[0] + " - " + allparams[1] + " - " + allparams[2] + "," +
allparams[3] + " - " + allparams[4] + " - " + allparams[5] + "," +
num11 + " / " + num22 + " / " + num33 + "," +
number11 + " / " + number22 + " / " + number33 + "," +
str + "," +
calc1 + "%";
if (calc1 > 0)
{
lock (lockobj) { plusLIST2.Add(str); }
}
else
{
lock (lockobj) { minusLIST2.Add(str); }
}
}
}
}
}
});
plusLIST = new List<String>(plusLIST2);
minusLIST = new List<String>(minusLIST2);
}