C# Sharp
- Автор темы Maxik
- Дата начала
- 41
- 5
>Хорошо, скинь мне пример, по твоему мнению, неговнокода.
Нормальный код не использует циклы, желательно вообще.
Вот пример того же, что у тебя:
Демка: http://codepen.io/anon/pen/oZXvBx
Непосредственно код генерации слова:
'bcdfghjklmnpqrstvwxzaeiouy'.split(/(.{20})/g).slice(1).map(i => i.split('')).map(i => i.sort(() => Math.random() > 0.5)).reduce((s, c) => s.push(c.join('').repeat(3).split('').sort(()=>Math.random() > 0.5).slice(0, 3)) && s, []).reduce((s, c, i) => c.forEach((c_, i_) => s[i_ * 2 + i] = c_) || s, []).join('')
Нормальный код не использует циклы, желательно вообще.
Вот пример того же, что у тебя:
Демка: http://codepen.io/anon/pen/oZXvBx
Непосредственно код генерации слова:
'bcdfghjklmnpqrstvwxzaeiouy'.split(/(.{20})/g).slice(1).map(i => i.split('')).map(i => i.sort(() => Math.random() > 0.5)).reduce((s, c) => s.push(c.join('').repeat(3).split('').sort(()=>Math.random() > 0.5).slice(0, 3)) && s, []).reduce((s, c, i) => c.forEach((c_, i_) => s[i_ * 2 + i] = c_) || s, []).join('')
- 2,505
- 81
- 398
Бегло прочитал и что? Ты хочешь, сказать, что я говнокод пишу? А ты попробуй обосновать? Судя по тому, что таких обоснованных претензий в мой адрес пока не поступало, я сделал вывод что мой код нормальный. Что нельзя сказать о тебеMaxik001 написал(а):
Rumia написал(а):
Абсолютно нечитабельно, больно и нерационально. Еще и внутри этих методов куча циклов.
- 5,149
- 49
- 899
Rumia написал(а):
>>Нормальный код не использует циклы, желательно вообще.
No comments. Давай просто закроем эту тему, потому что у всех понятие хорошего кода разное.
А вот это, извини меня, какая то полная жесть с вызовом херпоймичегохерпоймизачем
Или ты эту красоту(нет) в 3 строчки написал, думал смутить меня огромным количеством
символов?
Короче, это какая то дикая дичь, которая даже не компилируется. Или это ты специально так делаешь, чтобы никто не разобрал, что за каля-маля там написано? Короче все, закрыли тему. Никто не в обиде!
- 41
- 5
Бегло прочитал и что? Ты хочешь, сказать, что я говнокод пишу? А ты попробуй обосновать? Судя по тому, что таких обоснованных претензий в мой адрес пока не поступало, я сделал вывод что мой код нормальный. Что нельзя сказать о тебеMaxik001 Писал(а): написал(а):
Если ты до сих пор не используешь функциональную парадигму, то очевидно говнокод.
Императивное программирование придумали аутисты, не осилившие функциональщину.
- 5,149
- 49
- 899
Каждый пишет для себя так, как хочет, и никого не ебет как там у тебя написано . Но вот если ты @Rumia , будешь писать такой код для заказчика, то мне кажется, он швырнет тебе этот код обратно в лицо и скажет, переделывать. Для других надо писать, чтобы было понятно что, где, к чему и зачем. Я вот так считаю.
Я буду теперь тебя советовать,как авторитета в этой области, когда будет кто нибудь жесткое мясо выкладывать.
Я буду теперь тебя советовать,как авторитета в этой области, когда будет кто нибудь жесткое мясо выкладывать.
- 2,505
- 81
- 398
Заказчику код не интересен, для него важен рабочий продукт. А вот другие программисты, которые буду сопровождать продукт, сразу обрадуются, увидев это. Именно из-за этого сопровождение обычно обходится для заказчиков дороже, чем заказ продукта.
Если даже для такой простой задачи получился такой монстр, то что получится, если задача будет чуть сложнее. Легче будет переписать заново.
- 5,149
- 49
- 899
Ага, а когда он начнет работать плохо, например через год, тебе кинут предъяву, типо исправляй, наши программисты сломали глаза ,пытаясь найти ошибку. Ты такой открываешь свой код а там.
То есть, ты сам нихера не поймешь, и вместо того чтобы искать ошибку, будешь переписывать заново с документацией, чтобы в след. раз такого не было. Но если ты болванчик деревянный, то перепишешь во второй раз точно также, наступая на те же грабли.
P.S Я хз че тут, в хакертайпере поклацал, чисто для примера
Код:
struct group_info init_groups = { .usage = ATOMIC_INIT(2) };struct group_info *groups_alloc(int gidsetsize){struct group_info *group_info;int nblocks;int i;nblocks = (gidsetsize + NGROUPS_PER_BLOCK - 1) / NGROUPS_PER_BLOCK;
/* Make sure we always allocate at least one indirect block pointer */
nblocks = nblocks ? : 1;group_info = kmalloc(sizeof(*group_info) + nblocks*sizeof(gid_t *), GFP_USER);if (!group_info)return NULL;group_info->ngroups = gidsetsize;group_info->nblocks = nblocks;atomic_set(&group_info->usage, 1);if (gidsetsize <= NGROUPS_SMALL)group_info->blocks[0] = group_info->small_block;else {for (i = 0; i < nblocks; i++) {gid_t *b;b = (void *)__get_free_page(GFP_USER);if (!b)goto out_undo_partial_alloc;group_info->blocks[i] = b;}}
return group_info;out_undo_partial_alloc:while (--i >= 0) {free_page((unsigned long)group_info->blocks[i]);}kfree(group_info);return NULL;}EXPORT_SYMBOL(groups_alloc);void groups_free(struct group_info *group_info){if (group_info->blocks[0] != group_info->small_block) {int i;
for (i = 0; i < group_info->nblocks; i++)free_page((unsigned long)group_info->blocks[i]);
kfree(group_info);}
EXPORT_SТо есть, ты сам нихера не поймешь, и вместо того чтобы искать ошибку, будешь переписывать заново с документацией, чтобы в след. раз такого не было. Но если ты болванчик деревянный, то перепишешь во второй раз точно также, наступая на те же грабли.
P.S Я хз че тут, в хакертайпере поклацал, чисто для примера
- 41
- 5
Код:
using System;
namespace HelloWorld
{
public class Program
{
public static char[] randomWord = new char[20]{'b','c','d','f','g','h','j','k','l','m','n','p','q','r','s','t','v','w','x','z'};
public static char[] randomWordVowel = new char[6]{'a','e','i','o','u','y'};
public static String randWordFromChar;
public static void Main(){
Random random= new Random();
for(int j = 0; j < 9; ++j){
for(int i = 0;i < 3; ++i){
int z = random.Next(20);
int y = random.Next(6);
Console.Write(randomWordVowel[y]);
Console.Write(randomWord[z]);
}
Console.Write(" ");
for(int i = 0;i < 3; ++i){
int z = random.Next(20);
int y = random.Next(6);
Console.Write(randomWord[z]);
Console.Write(randomWordVowel[y]);
}
}
Console.Write(" ");
Console.Write("Press any key to continue . . . ");
Console.ReadKey(true);
}
}
}