for (i=0; i<strlen(s); i++) {
  // and never notice that counting lentgth is on every iteration
}