Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+134
// Как вы думаете какой вариант кода более правильный?
using System;
using System.Linq;
public class Program {
public static int Puzzle(string s) {
return s.Where(x=>x=='a').Count();
}
}
// Или вот этот?
using System;
using System.Linq;
public class Program {
public static int Puzzle(string s) {
return s.Count(x=>x=='a');
}
}
//С точки зрения Майкрософт 1-ый вариант лучше
Майкрософт запустила игру головоломку, каждая из головоломок решается определнным кодом. После - элегантность решения оценивается, весьма неоднозначным способом.
Подробнее - https://www.codehunt.com/ и статейка на хабре http://habrahabr.ru/post/223173/ . Данная особенность замечена не мной, задачи 2.06 и 2.07
Zuzik,
18 Мая 2014
+132
public void Check()
{
try
{
this.Function = "check";
this.Params = HttpUtility.UrlEncode((this.Params), Encoding.GetEncoding(1251));
if ((this.Function.Length > 0) &&
(this.PaymExtId.Length > 0) &&
(this.PaymSubjTp.ToString().Length > 0) &&
(this.Amount.ToString().Length > 0) &&
(this.Params.Length > 0) &&
(this.TermType.Length > 0) &&
(this.TermId.Length > 0) &&
(this.FeeSum.ToString().Length > 0))
{
this.Request = string.Format("function={0}&PaymExtId={1}&PaymSubjTp={2}&Amount={3}&Params={4}&TermType={5}&TermID={6}&FeeSum={7}",
this.Function, this.PaymExtId, this.PaymSubjTp.ToString(), this.Amount.ToString(), this.Params, this.TermType, this.TermId, this.FeeSum.ToString());
SendRequest();
}
else
{
GetError("-1");
}
}
catch
{
GetError("-9");
}
}
Оттуда же. Тип полей PaymSubjTp, Amount, FeeSum - int, всех остальных - string.
Amount.ToString().Length > 0 и т.п. всегда true, а вот ноль или отрицательное значение спокойно пропустят.
Интересно, разработчики хоть знают, что функции умеют принимать и возвращать данные? :)
yamamoto,
18 Мая 2014
+135
private void SendRequest()
{
try
{
HttpWebRequest HTTPRequest;
try
{
HTTPRequest = (HttpWebRequest)WebRequest.Create(this.Host + this.Request);
HTTPRequest.ClientCertificates.Add(this.certificate);
}
catch
{
throw new Exception("-2");
}
try
{
HttpWebResponse response = (HttpWebResponse)HTTPRequest.GetResponse();
Stream stream = response.GetResponseStream();
TextReader xml = new StreamReader(stream, Encoding.GetEncoding(1251));
this.Response = xml.ReadToEnd();
stream.Close();
response.Close();
}
catch
{
throw new Exception("-3");
}
GetError("0");
}
catch (Exception ex)
{
GetError(ex.Message);
this.Response = "ОШИБКА!";
}
}Оттуда же. Круговорот исключений в природе.
yamamoto,
17 Мая 2014
+133
public void ExtIdGen()
{
try
{
Random rnd = new Random();
string uin = "";
for (int i = 0; i < 20; i++)
{
uin += rnd.Next(10).ToString();
}
this.ExtId = uin;
GetError("0");
}
catch
{
GetError("-6");
}
}
private void GetError(string error_code)
{
switch (error_code)
{
case "0":
this.LastErrorCode = 0;
this.LastErrorDescription = "Нет ошибок";
break;
// -1 .. -5
case "-6":
this.LastErrorCode = -6;
this.LastErrorDescription = "Ошибка при получении идентификатора запроса";
break;
}
}
LastErrorCode, LastErrorDescription, ExtId - public поля.
Там весь класс написан в таком стиле, с вызовами GetError, принимающими номер ошибки в виде строки, молчаливым catch-ем всех исключений и т.п.
yamamoto,
17 Мая 2014
+134
private byte Summ(byte a, byte b, byte p)
{
return Or(Or(Or(And(p, Not(a)), And(p, Not(b))), Or(And(b, Not(a)), And(b, Not(a))))Or(And(a, Not(b)),And(a, Not(p))), ()) ;//тут я и сломался
}
Shadowy1,
17 Мая 2014
+123
namespace Рунге_Кутты_1._4
{
public partial class Form1 : Form
{
double x0 = 1, y0 = 2, z0 = 4, t0 = 0, T = 1, h = 0.001, Eps = 0.000001, fx, fy, fz, xx, yy, zz, Ex, Ey, Ez, Eps1;
double[] x = new double[4]; double[] y = new double[4]; double[] z = new double[4];
double[] Kx = new double[4]; double[] Ky = new double[4]; double[] Kz = new double[4];
bool a;
public void Calculates()
{
System.IO.File.Delete(@"C:\Results\results.dat");
x[0] = x0; y[0] = y0; z[0] = z0;
while (t0 <= T)
{
for (int i = 0; i <= 3; i++)
{
fx = -y[i] - z[i];
fy = x[i] + 0.2 * y[i];
fz = 0.2 + (x[i] - 5.7) * z[i];
Kx[i] = h * fx;
Ky[i] = h * fy;
Kz[i] = h * fz;
x[1] = x[0] + 0.5 * Kx[0];
x[2] = x[0] + 0.5 * Kx[1];
x[3] = x[0] + Kx[2];
y[1] = y[0] + 0.5 * Ky[0];
y[2] = y[0] + 0.5 * Ky[1];
y[3] = y[0] + Ky[2];
z[1] = z[0] + 0.5 * Kz[0];
z[2] = z[0] + 0.5 * Kz[1];
z[3] = z[0] + Kz[2];
}
Ex = Math.Abs(2 * (Kx[0] - Kx[1] - Kx[2] + Kx[3]) / 3);
Ey = Math.Abs(2 * (Ky[0] - Ky[1] - Ky[2] + Ky[3]) / 3);
Ez = Math.Abs(2 * (Kz[0] - Kz[1] - Kz[2] + Kz[3]) / 3);
Eps1 = Eps / 8;
a = Check();
if (a == false)
{
xx = x[0] + (Kx[0] + 2 * Kx[1] + 2 * Kx[2] + Kx[3]) / 6;
yy = y[0] + (Ky[0] + 2 * Ky[1] + 2 * Ky[2] + Ky[3]) / 6;
zz = z[0] + (Kz[0] + 2 * Kz[1] + 2 * Kz[2] + Kz[3]) / 6;
x[0] = xx;
y[0] = yy;
z[0] = zz;
t0 += h;
string f1 = Convert.ToString(xx);
string f2 = Convert.ToString(yy);
string f3 = Convert.ToString(zz);
string[] f = new string[] { f1, f2, f3 };
string path = @"C:\Results\results.dat";
string appendtext = xx + " " + yy + " " + zz + Environment.NewLine;
System.IO.File.AppendAllText(path, appendtext);
}
}
}
public bool Check()
{
bool a;
if (Ex > Eps | Ey > Eps | Ez > Eps)
{
a = true;
h /= 2;
Calculates();
}
else a = false;
return a;
}
public Form1()
{
InitializeComponent();
Calculates();
DrawGraph();
}
public void DrawGraph()
{
GraphPane pane = zedGraph.GraphPane;
pane.CurveList.Clear();
PointPairList list = new PointPairList();
for (t0 = 0; t0 <= T; t0 += h)
{
list.Add(t0, x[0]);
}
LineItem MyCurve = pane.AddCurve("x(t)", list, Color.Blue, SymbolType.None);
zedGraph.AxisChange();
zedGraph.Invalidate();
}
}
}пытаюсь вывести решения в виде графика. выводится только последнее решение из цикла
mr07th,
16 Мая 2014
+142
try
{
int.TryParse(splitString[j], out I[i, j]);
}
catch (Exception)
{
Console.WriteLine("...");
break;
}
sys2712,
13 Мая 2014
+143
public static string CapitalizeFirstLetter(this string s)
{
if (String.IsNullOrEmpty(s))
throw new ArgumentException("Stirng is empty");
return s.First().ToString().ToUpper() + String.Join("", s.Skip(1));
}не ищем легких путей
baks,
07 Мая 2014
+133
public static bool In<T>(this T t, params T[] values)
{
return t == null ? values.Any(v => v == null) : values.Contains(t);
}
public static bool NotIn<T>(this T t, params T[] values)
{
return !t.In(values);
}замечательный экстэншн к object
baks,
07 Мая 2014
+124
private int MsUntilNextRefresh(DayOfWeek refreshDay, uint refreshHour)
{
int days = 0;
if (DateTime.Now.DayOfWeek > refreshDay)
days = refreshDay + 7 - DateTime.Now.DayOfWeek;
else if (DateTime.Now.DayOfWeek < refreshDay)
days = refreshDay - DateTime.Now.DayOfWeek;
else
days = 7;
int hours = 0;
if (DateTime.Now.Hour > refreshHour)
{
days--;
hours = (int)refreshHour + 24 - DateTime.Now.Hour;
}
else if (DateTime.Now.Hour < refreshHour)
{
hours = (int)refreshHour - DateTime.Now.Hour;
}
return days * 86400000 + hours * 3600000 - DateTime.Now.Minute * 60000 - DateTime.Now.Second * 1000 - DateTime.Now.Millisecond;
}
классический индусский код, вместо:
private int MsUntilNextRefresh(DayOfWeek refreshDay, uint refreshHour)
{
var dtnow = DateTime.UtcNow;
var nextRefreshDate =
dtnow.Date.AddDays(dtnow.DayOfWeek >= refreshDay ? dtnow.DayOfWeek + 7 - refreshDay : refreshDay - dtnow.DayOfWeek).AddHours(refreshHour);
return (nextRefreshDate - dtnow).Milliseconds;
}
valery_chistyakov,
07 Мая 2014
Follow us!