В чем смысл при путешествии фрейма от одного узла к другому (находятся не в одной сети) указывать первым мак адрес, если было бы логичнее указать IP, чтобы фрейм нашел нужную сети, а затем уже в этой сети по мак адресу нашел нужный узел.Если получатель не в нашей сети, то в L2 заголовке получателем будет роутер, а не конечный получатель.
if ((valVAT == 10) || (valVAT == 18) || (valVAT == 20)) {
...if(a) { много кода } стоит писать: if (!a) {
continue; // или return; если это в функции.
}
// много кода.valVAT == 0 и выйти из цикла через break в этом случае. Потом проверить, что valVat != 10 && valVAT != 18 && valVAT != 20 и вывести сообщение об ошибке и сделать continue. Дальше уже идет тело цикла с вычислениями.if(chng == 1) {} else if (chng == 2) {}... стоит использовать конструкцию switch (chng) {
case 0:
// код
break;
case 1:
// код
break;
case 2:
// код
break;
default:
// сообщение об ошибке
continue;
}sleep(1) после вывода сообщения об ошибке, на мой взгляд не нужен. Зачем это? Заставить пользователя прочитать сообщение об ошибке? 
std::cin, то это будет работать жутко медленно.cin читает в свой внутренний буфер сразу, скажем, 4096 байт, и потом отдаёт тебе из буфера по одному символу, которые ты у него запрашиваешь. Нетрудно заметить, что работать это будет (в моём примере) где-то в 4096 раз быстрее.flush, то, что ты вывел в поток, находится в буфере. cout ждёт, пока ты докинешь ему ещё байтиков, чтобы вывести всё потом большим куском. Если же ты хочешь, чтобы вывод произошёл прямо сейчас, то нужен flush. 
Почему выводит только первую строку а остальные что-то непонятное?
int print_arr(int str, int stlb, int matr[][stlb]){ ... } int mass[10][10]; ... print_arr(N, L, &mass[k][i]);
int main()
{
int L = 0, N = 0, k, i;
scanf("%d", &N);
L=N;
int mass[N][L];
for(k = 0; k < N; k++){
printf("[%d]\n", k);
for(i = 0; i < L; i++){
scanf("%d", &mass[k][i]);
}
}
printf("matrix NxN\n");
print_arr(N, L, mass);
return 0;
}