ответьте пожалуйста

[waleri]

for ( int column = 0; column <= 12; column++ )
{
    if (deck[mast][naminal]==card) 
    {
        coo=1;
        deck[mast][naminal]=0;
    }
}

И зачем здесь цикл по column?
Зачем здесь переменная coo, если после deck[mast][naminal]=0; можно выполнить return?
Зачем здесь два комплекта циклов, если они отличаются только начальным значением card?

[Человек_Борща]

Это студент.

[phomm]

http://www.gunsmoker.ru/2011/01/blog-post.html
язык не ваш, но по сути статьи это не имеет значения.

А к ооп надо прийти, я считаю. Сперва увидеть как код и задача в голове и на глазах дробится на куски, создавать для этого функции с атомарной логикой, комплектовать структуры данных, потом объединять функции и структуры в объекты, инкапсулировать логику, оставляя выверенные интерфейсы объектов. Потом уже более усложнять, наследование, полиморфизм, делегаты, свойства, обобщения.

[Smitt&Wesson]

Сообщение от Stilet

+100500 функций = 100500 файлов?О_о

Я имел ввиду несколько другую ситуацию. Делаем отдельный файл с наиболее часто применяемыми функциями и классами. Тогда не нужно будет в каждой программе их переписывать. Это типа библиотеки, только собственной.

[ds.Dante]

Сообщение от MihalNik

С.Макконнел "Совершенный код".

Сообщение от Rififi

Стив Макконнелл "Совершенный код"

+1. Alekopoko, сначала прочтите главу 34.1, а потом - всю книгу, от корки до корки.

Чтобы полностью охватить и понять сразу все детали даже средней программы, человек должен был бы обладать почти неограниченными возможностями. Способ использования интеллекта важнее, чем его уровень.

[Larboss]

ну вот мой код карточной игры(компьютер против компьютера)
оцените пожалуйста

Я, конечно, не профессионал, но у вас в коде вижу: "здесь комментирую, а здесь - нет". Где-то по телу функции комментарии стоят, а перед самой функцией комментарий оставить, что он делает нельзя? Т.е. если через n-ое время вы вернетесь к коду, вам понадобится каждую функцию разбирать от корки до корки... А еще бы я "ентерами" отделял переменные и функции.

Пример:

// Инициализация

// Переменные
int ses=29; // Чай
int deck[ 4 ][ 13 ]; // Варенье
int coo=0; // Плюшки

// Массивы
static const char*suit[4]={".B",".C",".T",".P"}; // Винегрет
static const char*face[13]={"2","3","4","5","6","7","8","9","10","V","D","K","T"}; // Салат 

// Действие. Ход игрока. Возвращает 1, если "это", возвращает 0, если "то"
int hod_igroka(int mast,int naminal,int i)

[alekopoko]

waleri после вашего сообщения я сам от себя в шоке
ну окей переменная coo там лишния, ладно, проглотил косяк
Но 4 лишних цикла в функции hod_igroka!!
ну видимо я думал чем больше запихаю стиля харви дейтела тем лутше и помпезней код будет) ну понравилось мне очень это фишка тасования карт с помощью функции rand ...я прям восхитился, поразился

"Это студент." - если это был намек что типо это студент и у него не может быть лучше ,этож студент-раздолбай,писал код на отвали,то это НЕ так!!.Я вообще-то старался.И душу вкладывал.Пускай и коряво и это все выглядит как быдло-код какашки но я писал не для того чтобы сдать и поскорей распрощаться с кодом а для себя я писал. Я хочу стать отличным программистом и это моя мечта.И она сбудется!

[Larboss]

Да тут даже может быть и школьник. Пишите как в вконтакте, пробелы не ставите и знаки препинания.

[Naive]

Сообщение от Larboss

Я, конечно, не профессионал, но у вас в коде вижу: "здесь комментирую, а здесь - нет". Где-то по телу функции комментарии стоят, а перед самой функцией комментарий оставить, что он делает нельзя? Т.е. если через n-ое время вы вернетесь к коду, вам понадобится каждую функцию разбирать от корки до корки... А еще бы я "ентерами" отделял переменные и функции.

В целом, конечно верно, зачем писать что-то вроде

// если в массиве из строк не нашлась строка qwerty, то выводим сообщение фубар
!~myMas.indexOf('qwerty')&&alert('foo bar');

ес-но, с достаточным уровнем подготовки люди, это прочтут и поймут без коммента и так, для более тупого было бы логичнее записать нормально и без коммента:

if ( myMas.indexOf('qwerty') === -1 ) 
  alert('foo bar');

но тут, наверное, если человек это не поймет, то наверно он и не нужен нахрен.

Комментировать нужно:
1. классы, для чего они предназначены, какие входные параметры для конструктора;
2. методы классов, для чего предназначены, что принимают (формат) при входе, и что выдают (если не свой объект);
3. функция (процедура), аналогино п.2.
4. замыкание (или модуль), что принимает, что выдает
5. пивное горлышко, когда пишешь

!~...&&... вместо if (...) {....

(код js), (везде есть утырканые синтаксисы, которые работают быстрее).

других моих личных примеров не вспомнил)
п.с. не всегда пишу ООП, всегда ищу баланс между удобством юзера и скоростью выполнения работы, так что пишу в ООП, в ФП, в процедурном, но никогда в говно-стайл.

[alekopoko]

Да нормальный я ) Блин)) Я это не отрицаю.) ты мне вот глаза открыл правда.Я об этом не знал да. У меня по-русскому в школе двойка была и в сочинениях 100 ошибок было и сейчас из-за этого плохая разговорная речь плохо складываю слова и составляю безграмотные предложения и иногда говорю чушь Но я перевелся на программиста и у меня академ разница(16 долгов) ноги болят(расхождение пальцев искревление стопы) еле-еле ползаю на таблетках вредных обезболивающих сижу мне все твердят академ отпуск а я пускай сдохну лучше чем академ отпуск.

Ну я думаю теперь понятно что мне сейчас как бы немножко не до запятых сейчас и да это тупая отмазка и я вообще как идиот себя сейчас виду , все я заткнулся.