public static MyClass {
private static volatile ProcessManager singleton = null;
public static ProcessManager getInstance() throws Exception {
if (singleton == null) {
synchronized (MyClass.class) {
if (singleton == null) {
singleton = new ProcessManager();
}
}
}
return singleton;
}
}The double-checking was invented prior to Java5.
The purpose is if the field isn't null, you don't have to synchronize. But since the java memory model specification was cleaned up and Synchronize/Volatile were given much better specification and semantics it is totally useless boilerplate that you should only write if you have to support Java4. There is a Google Tech Talk that covers this.
Long timestamp = Instant.now().toEpochMilli(); // fallback
if (oTimestamp instanceof Integer)
timestamp = (Integer) oTimestamp *1L;
else
if (oTimestamp instanceof Long)
timestamp = (Long) oTimestamp * 1L;bool b = connection.isConnected() == true ? true : false;My first internship presented me with this.
string month = DateTime.Today.Month.ToString();
if (DateTime.Today.Month < 10)
{
month = "0" + month;
}
string day = DateTime.Today.Day.ToString();
if (DateTime.Today.Day < 10)
{
day = "0" + day;
}
string dateCorrect = String.Format("{0}.{1}.{2}", DateTime.Today.Year, month, day);
string dateDue = "";
if (transferFields.DueDate.SelectedDate.HasValue)
{
var dateDuearr = transferFields.DueDate.Value.Split(' ')[0].Split('.');
dateDue = dateDuearr[2] + '.' + dateDuearr[1] + '.' + dateDuearr[0];
}
else {
dateDue = dateCorrect;
}Real developers don't use built in parsing and formatting methods.
switch (numberOfQuestionsToAnswer) {
case 0:
displayGrid1 = false;
displayGrid2 = false;
displayGrid3 = false;
displayGrid4 = false;
displayGrid5 = false;
displayGrid6 = false;
displayGrid7 = false;
displayGrid8 = false;
displayGrid9 = false;
displayGrid10 = false;
break;
case 1:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayGrid1 = true;
break;
case 2:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayAnsInputTextBox2 = displayPreselectedAnsBox();
displayGrid1 = true;
displayGrid2 = true;
break;
case 3:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayAnsInputTextBox2 = displayPreselectedAnsBox();
displayAnsInputTextBox3 = displayPreselectedAnsBox();
displayGrid1 = true;
displayGrid2 = true;
displayGrid3 = true;
break;
case 4:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayAnsInputTextBox2 = displayPreselectedAnsBox();
displayAnsInputTextBox3 = displayPreselectedAnsBox();
displayAnsInputTextBox4 = displayPreselectedAnsBox();
displayGrid1 = true;
displayGrid2 = true;
displayGrid3 = true;
displayGrid4 = true;
break;
case 5:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayAnsInputTextBox2 = displayPreselectedAnsBox();
displayAnsInputTextBox3 = displayPreselectedAnsBox();
displayAnsInputTextBox4 = displayPreselectedAnsBox();
displayAnsInputTextBox5 = displayPreselectedAnsBox();
displayGrid1 = true;
displayGrid2 = true;
displayGrid3 = true;
displayGrid4 = true;
displayGrid5 = true;
break;
case 6:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayAnsInputTextBox2 = displayPreselectedAnsBox();
displayAnsInputTextBox3 = displayPreselectedAnsBox();
displayAnsInputTextBox4 = displayPreselectedAnsBox();
displayAnsInputTextBox5 = displayPreselectedAnsBox();
displayAnsInputTextBox6 = displayPreselectedAnsBox();
displayGrid1 = true;
displayGrid2 = true;
displayGrid3 = true;
displayGrid4 = true;
displayGrid5 = true;
displayGrid6 = true;
break;
case 7:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayAnsInputTextBox2 = displayPreselectedAnsBox();
displayAnsInputTextBox3 = displayPreselectedAnsBox();
displayAnsInputTextBox4 = displayPreselectedAnsBox();
displayAnsInputTextBox5 = displayPreselectedAnsBox();
displayAnsInputTextBox6 = displayPreselectedAnsBox();
displayAnsInputTextBox7 = displayPreselectedAnsBox();
displayGrid1 = true;
displayGrid2 = true;
displayGrid3 = true;
displayGrid4 = true;
displayGrid5 = true;
displayGrid6 = true;
displayGrid7 = true;
break;
case 8:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayAnsInputTextBox2 = displayPreselectedAnsBox();
displayAnsInputTextBox3 = displayPreselectedAnsBox();
displayAnsInputTextBox4 = displayPreselectedAnsBox();
displayAnsInputTextBox5 = displayPreselectedAnsBox();
displayAnsInputTextBox6 = displayPreselectedAnsBox();
displayAnsInputTextBox7 = displayPreselectedAnsBox();
displayAnsInputTextBox8 = displayPreselectedAnsBox();
displayGrid1 = true;
displayGrid2 = true;
displayGrid3 = true;
displayGrid4 = true;
displayGrid5 = true;
displayGrid6 = true;
displayGrid7 = true;
displayGrid8 = true;
break;
case 9:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayAnsInputTextBox2 = displayPreselectedAnsBox();
displayAnsInputTextBox3 = displayPreselectedAnsBox();
displayAnsInputTextBox4 = displayPreselectedAnsBox();
displayAnsInputTextBox5 = displayPreselectedAnsBox();
displayAnsInputTextBox6 = displayPreselectedAnsBox();
displayAnsInputTextBox7 = displayPreselectedAnsBox();
displayAnsInputTextBox8 = displayPreselectedAnsBox();
displayAnsInputTextBox9 = displayPreselectedAnsBox();
displayGrid1 = true;
displayGrid2 = true;
displayGrid3 = true;
displayGrid4 = true;
displayGrid5 = true;
displayGrid6 = true;
displayGrid7 = true;
displayGrid8 = true;
displayGrid9 = true;
break;
case 10:
displayAnsInputTextBox1 = displayPreselectedAnsBox();
displayAnsInputTextBox2 = displayPreselectedAnsBox();
displayAnsInputTextBox3 = displayPreselectedAnsBox();
displayAnsInputTextBox4 = displayPreselectedAnsBox();
displayAnsInputTextBox5 = displayPreselectedAnsBox();
displayAnsInputTextBox6 = displayPreselectedAnsBox();
displayAnsInputTextBox7 = displayPreselectedAnsBox();
displayAnsInputTextBox8 = displayPreselectedAnsBox();
displayAnsInputTextBox9 = displayPreselectedAnsBox();
displayAnsInputTextBox10 = displayPreselectedAnsBox();
displayGrid1 = true;
displayGrid2 = true;
displayGrid3 = true;
displayGrid4 = true;
displayGrid5 = true;
displayGrid6 = true;
displayGrid7 = true;
displayGrid8 = true;
displayGrid9 = true;
displayGrid10 = true;
break;
}This was for a wizard type questionnaire where there could be up to 10 questions.
public static class DecimalHelpers
{
/// <summary>
/// Format a decimal XX.XX to XX,XX%.
/// </summary>
/// <param name="value"></param>
/// <returns></returns>
public static string FormatToStringPercentageValue(this decimal value)
{
return value.ToString().Replace(".", ",").FormatToPercentageValue();
}
}
public static class StringHelpers
{
/// <summary>
/// Add % at the end of the string.
/// </summary>
/// <param name="value"></param>
/// <returns></returns>
public static string FormatToPercentageValue(this string value)
{
return string.Concat(value, "%");
}
}
" ".split(" ");Very weird code found multiple times in old code base.
private HashMap<String, Tuple6<float[], String[], String[], String[], String[], String>> memberNameChangedToProtectTheInnocent = null;public function getProduct($product_id) {
$query = $this->db->query("SELECT DISTINCT *, pd.name AS name, p.image, m.name AS manufacturer, (SELECT price FROM " . DB_PREFIX . "product_discount pd2 WHERE pd2.product_id = p.product_id AND pd2.customer_group_id = '" . (int)$this->config->get('config_customer_group_id') . "' AND pd2.quantity = '1' AND ((pd2.date_start = '0000-00-00' OR pd2.date_start < NOW()) AND (pd2.date_end = '0000-00-00' OR pd2.date_end > NOW())) ORDER BY pd2.priority ASC, pd2.price ASC LIMIT 1) AS discount, (SELECT price FROM " . DB_PREFIX . "product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '" . (int)$this->config->get('config_customer_group_id') . "' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special, (SELECT points FROM " . DB_PREFIX . "product_reward pr WHERE pr.product_id = p.product_id AND pr.customer_group_id = '" . (int)$this->config->get('config_customer_group_id') . "') AS reward, (SELECT ss.name FROM " . DB_PREFIX . "stock_status ss WHERE ss.stock_status_id = p.stock_status_id AND ss.language_id = '" . (int)$this->config->get('config_language_id') . "') AS stock_status, (SELECT wcd.unit FROM " . DB_PREFIX . "weight_class_description wcd WHERE p.weight_class_id = wcd.weight_class_id AND wcd.language_id = '" . (int)$this->config->get('config_language_id') . "') AS weight_class, (SELECT lcd.unit FROM " . DB_PREFIX . "length_class_description lcd WHERE p.length_class_id = lcd.length_class_id AND lcd.language_id = '" . (int)$this->config->get('config_language_id') . "') AS length_class, (SELECT AVG(rating) AS total FROM " . DB_PREFIX . "review r1 WHERE r1.product_id = p.product_id AND r1.status = '1' GROUP BY r1.product_id) AS rating, (SELECT COUNT(*) AS total FROM " . DB_PREFIX . "review r2 WHERE r2.product_id = p.product_id AND r2.status = '1' GROUP BY r2.product_id) AS reviews, p.sort_order FROM " . DB_PREFIX . "product p LEFT JOIN " . DB_PREFIX . "product_description pd ON (p.product_id = pd.product_id) LEFT JOIN " . DB_PREFIX . "product_to_store p2s ON (p.product_id = p2s.product_id) LEFT JOIN " . DB_PREFIX . "manufacturer m ON (p.manufacturer_id = m.manufacturer_id) WHERE p.product_id = '" . (int)$product_id . "' AND pd.language_id = '" . (int)$this->config->get('config_language_id') . "' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '" . (int)$this->config->get('config_store_id') . "'");
/* ... */Opencart's Product model
if (type == cSTR_Integer) {
type = cSTR_Integer;
} else if (ctype == cSTR_String) {
type = cSTR_String;
}def copy( variable ):
return variable
Decade old code, used in 2 places to "copy" a float and a string. No one wants to touch the working code.
if(date.getFullYear() <= 2017) {
if(date.getMonth() <= 11) {
if(date.getDate() <= 26) {
if(date.getHours() <= 23) {
if(date.getMinutes() <= 59) {
if(date.getSeconds() <= 59) {
//Display popup before 2017-11-27
}
}
}
}
}
}
UIButton * btn = [UIButton new];
[btn addTarget:target action:action forControlEvents:UIControlEventTouchUpInside];
//May god have mercy of me
[[btn titleLabel] setFont:[UIFont systemFontOfSize:14 weight:UIFontWeightMedium]];
[btn setTitle:[NSString stringWithFormat:@" %@ ", btnTxt] forState:UIControlStateNormal];
[btn sizeToFit];
[btn setTitle:btnTxt forState:UIControlStateNormal];Why keep playing with label padding or insets when you can add spaces?
public override List<Value> Values
{
get
{
if (_values == null && shouldLoadLazily)
{
_values = _lazerLoader.LoadValues<Value>()
}
return _values;
}
set
{
throw new InvalidOperationException("Values are actually readonly- please set in constructor")
}
}How to create immutable fields in c#