LINUX.ORG.RU

Как вам такой Coding Standard?

 


0

0
if (!$result)
    return false;
  
for ($i = 0; $i < 17; $i++)
    if ($myArray[$i] == $value)
    {
        $result[] = $myArray[$i];
        return $result;
    }
    else
        $failed++;

Это я не предлагаю) Это требования заказчика.

★★

Последнее исправление: webmak (всего исправлений: 2)
Ответ на: комментарий от EXL

web

Не бузи на web-программистов, мы настоящие!

Alve ★★★★★
()
Ответ на: комментарий от EXL

)) еще чучуть)

Вот что я счас делаю https://www.youtube.com/watch?v=FzHSLOXi3Z0

Это только начал) работаю недельку.

Да пусть я безграмотный, да пусть я дерьмовый программист, да пусть мне не место на Земле.

Но нифига веб не хуже и не проще чем классическое программирование для обычных ПК

webmak ★★
() автор топика

$

Когда я вижу зашкаливающее кол-во этих символов, мне вспоминаются негры со своими золотыми цепями со знаками доллара, золотыми зубами и т.п. побрякушками :3

Deleted
()

На GNU Coding Style похоже

anonymous
()
Ответ на: комментарий от post-factum

То есть того, что там и так почти пустая строка, торчащая налево, тебе мало? Иными словами, вполне достаточно того, что они поделены на два блока отступами.

Или ты успел перейти на мелкие и квадратные шрифты?

thriller ★★
()
Ответ на: комментарий от I-Love-Microsoft

Так писать, как мне кажется, можно если уверен, что ты пишеш идеальный код, а представь его потом дебажить. Как задалбывает добавление скобок, чтоб в нутрь поставить var_dump, а потом их убирать, чтоб соответствовать кодинг стандарту.

webmak ★★
() автор топика
Ответ на: комментарий от Deleted

Когда я вижу зашкаливающее кол-во этих символов, мне вспоминаются негры

А мне творчество Айн Рэнд.

no-such-file ★★★★★
()

Ужасный Code Style, так лучше:

if(!$result)
{
  return false;
}  
for ($i = 0; $i < 17; $i++) {
    if ($myArray[$i]==$value)
    {
      $result[] = $myArray[$i];
      return $result;
    }
    else
    {
      $failed++;
    }
}

После блока в if можно и TAB ставить (просто, чтоб быстрее набирать код было), пусть будут 4 пробела.

Как вариант на любителя, можно так:

if(!$result){
    return false;
}  
for ($i = 0; $i < 17; $i++) {
    if ($myArray[$i]==$value) {
        $result[] = $myArray[$i];
        return $result;
    } else {
        $failed++;
    }
}

peregrine ★★★★★
()
Ответ на: комментарий от MrClon

} else { нормально парсится глазами.

И вообще, K&R рекомендуют.

no-such-file ★★★★★
()
Ответ на: комментарий от peregrine
if(!$result){
  ^
...

if(!$result){
            ^
...

if ($myArray[$i]==$value) {
  ^
...
if ($myArray[$i]==$value) {
                         ^

Вы либо трусы наденьте, либо пробелы нормально расставьте.

EXL ★★★★★
()

Для пхп вообще PSR-2 придумали, и он вполне ничего.

risenshnobel ★★★
()
if (!$result) return false;
  
for ($i = 0; $i < 17; $i++)
    if ($myArray[$i] == $value) {
        $result[] = $myArray[$i];
        return $result;
    }
    else $failed++;
olibjerd ★★★★★
()

==

У этого оператора в этом языке есть одно назначение — обеспечивать сохранность твоего рабочего места.

Про нечитабельность твоего отсутствия скобок тут уже всё сказали. Если думаешь, что вайтспейса достаточно — выбери язык, где это принято.

x3al ★★★★★
()
Ответ на: комментарий от x3al

спасибо, это не мой вариант стандарта)

webmak ★★
() автор топика

тут все еще кто-то не понимает, что писать надо вот так:

if (!$result)
    return false;
  
for ($i = 0; $i < 17; $i++) {
    if ($myArray[$i] == $value) {
        $result[] = $myArray[$i];
        return $result;
    }
    else {
        $failed++;
    }
}

?

unt1tled ★★★★
()
Последнее исправление: unt1tled (всего исправлений: 1)
Ответ на: комментарий от unt1tled

В приличном обществе ногами не бьют, так что вы всё врёте.

olibjerd ★★★★★
()
Ответ на: комментарий от post-factum

эскобар.jpg

if (<condition>) 
{
    statement
}
else
{
    statement
}

Цель всегда была в визуальном отбое стэйтмента от условия и прочих. А то млять, накуролесят «ну типа по-логике» пустые строчки вставляют, для наглядности, а потом половина комитов - они блин, пустые строчки туда-сюда гоняли...

ioway
()
Ответ на: комментарий от post-factum

Не, ну это, наверное, слишком. Хотя всяко лучше того, что тут остальные предлагают.

Может быть. Но тут при длинных условиях, без пустой строки читать нижеследующий код трудновато, а выставлять межстрочный промежуток побольше, так начинаются такие пляски на тачпаде - прям узбекский диджей Затрудодыр. И еще вспомнил, почему такой код у себя договорились держать - фолдинг. Некоторые редакторы сворачивают его через одно место и если скобка стоит на одной строке с условием, а условие в несколько строк получается что-то вроде: +---if(someconst>....100 lines more..-----------------------

ioway
()
Ответ на: комментарий от aol
if(!$result)
{
    return false;
}
for($i=0;$i<17;$i++)
{
    if($myArray[$i] == $value)
    {
        $result[] = $myArray[$i];
        return $result;
    }
    else
    {
        $failed++;
    }
}
anonymous
()

ты про отсутствие управляющих символов? ну тащемта да, я тоже не понимаю и не одобряю

Ford_Focus ★★★★★
()

Эмм, говно если честно, надо что то типа:

public fun test(result: Array<Any>?, value: Any): Any? = result?.firstOrNull { it == value }
Nagwal ★★★★
()
Ответ на: комментарий от user_id_68054

# P.S.: ну ещё не понятно — откуда у людей такая любовь к ``$i++`` (читаю «Ай — Увеличить!») .. ведь ОЧЕВИДНО что более приятно видеть глазу запись ``++$i`` (читаю «Увеличить Ай!»)

Думаю, вам полезно будет изучить спецификации языков программирования и тогда может поймёте, чем отличается «i++» от «++i»

Это относится не только к ПЫХу.

Может, тогда больше будет понимания, почему иногда ошибки лезут.

Serg_HIS
()
Ответ на: комментарий от aol

Исходя из увиденного — ему всеравно что возвращает ф-ция false или null, отсюда можно сплясать и так:

if ($result) {
    if (false !== $key = array_search($value, $myArray)) {
        $failed  += $key;
        $result[] = $myArray[$key];
        return $result;
    }
    $failed += 17 % sizeof($myArray) - 1;
}

deep-purple ★★★★★
()
Последнее исправление: deep-purple (всего исправлений: 1)

а что собственно, не так?

AndreyKl ★★★★★
()
Ответ на: комментарий от deep-purple

Вот так корректнее:

if ($result && false !== $key = array_search($value, $myArray)) {
    $failed  += $key;
    $result[] = $myArray[$key];
    return $result;
}
$failed += 17 % sizeof($myArray) - 1;

deep-purple ★★★★★
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.